fix(tests): corregir fallos de tests pre-existentes en entornos Windows

[atarico]

---

🔗 Linked Issue

Closes #103

---

🏷️ PR Type

• type:bug — Bug fix (non-breaking change that fixes an issue)
• type:feature — New feature (non-breaking change that adds functionality)
• type:docs — Documentation only
• type:refactor — Code refactoring (no functional changes)
• type:chore — Build, CI, or tooling changes
• type:breaking-change — Breaking change (fix or feature that changes existing behavior)

---

📝 Summary

Al correr go test ./... en Windows, existen 3 categorías de fallos
pre-existentes causados por incompatibilidades del entorno, no por
errores de lógica del proyecto.

Este PR corrige las 3 categorías:

• Golden files con CRLF por la conversión automática de Git en Windows
• Separadores de ruta hardcodeados con / en TestSkillPathForAgent
• Tests de OpenCode que fallan cuando no hay npm/bun en el entorno

---

📂 Changes

File / Area	What Changed
.gitattributes	Agrega * text eol=lf para forzar LF en todos los sistemas
internal/components/golden_test.go	Regenera golden files con terminaciones LF consistentes
internal/components/skills/inject_test.go	Usa filepath.FromSlash() en TestSkillPathForAgent
internal/components/sdd/inject_test.go	Agrega skipIfNoPkgManager() y disablePluginInstall()
testdata/golden/sdd-vscode-instructions.golden	Golden file regenerado con LF

---

🧪 Test Plan

Unit Tests

  go test ./...

E2E Tests (Docker required)

  cd e2e && ./docker-test.sh

• Unit tests pass (go test ./...)
• E2E tests pass (cd e2e && ./docker-test.sh)
• Manually tested locally on Windows 11

---

✅ Contributor Checklist

• PR is linked to an issue with status:approved
• I have added the appropriate type:* label to this PR
• Unit tests pass (go test ./...)
• E2E tests pass (cd e2e && ./docker-test.sh)
• I have updated documentation if necessary
• My commits follow Conventional Commits format
• My commits do not include Co-Authored-By trailers

---

💬 Notes for Reviewers

[Alan-TheGentleman]

Gracias por trabajar en los tests de Windows — es un área que necesita amor.

Cierro porque:

1. El campo Closes # en el body del PR está vacío — no hay issue linkeada
2. La issue fix(tests): fallos de tests pre-existentes en entornos Windows #103 todavía no tiene status:approved

Próximos pasos:

1. Voy a revisar y aprobar fix(tests): fallos de tests pre-existentes en entornos Windows #103
2. Una vez aprobada, podés reabrir este PR (o abrir uno nuevo) linkeando Closes #103

Los fixes que tenés son valiosos — solo necesitamos seguir el proceso.

[Basparin]

Hola @atarico — tu fix técnico en este PR era sólido y el issue #103 ya quedó con status:approved desde que Alan lo aprobó. Como pasaron varias semanas sin movimiento, me tomo el trabajo de reabrirlo con tu crédito completo:

• Voy a cherry-pickear tu commit c765bef a una branch nueva sobre origin/main fresco
• Git preserva tu autoría automáticamente (Author: atarico), yo quedo como committer
• En el PR body voy a citar este PR fix(tests): corregir fallos de tests pre-existentes en entornos Windows #104 y acreditarte explícitamente como origen del fix
• Linkeo Closes #103 para que el proceso pase

Si preferís reabrirlo vos, avisame y lo cierro de mi lado — tu trabajo, tu PR, tu merge. Si no tenés bandwidth, todo tranquilo, yo lo empujo.

Gracias por el trabajo original, era lo que bloqueaba a varios contributores Windows (yo incluido — lo sufrí al preparar el PR #371 hoy).

[atarico]

Hola @Basparin !!! Muchas gracias por avisarme y por tomarte el tiempo de revisar la PR... y encima de eso, por la buena onda de querer preservar mi autoría en el cherry-pick. Se aprecia un montón, de verdad!!!

Lo que pasó de mi lado: vi que el issue estaba aprobado pero no me di cuenta que el PR no había sido mergeado. Para colmo, justo en ese tiempo reinstalé y migré a Linux, así que se me pasó completamente hacer el seguimiento.

Metele tranqui, dale curso vos. Por mi está todo OK. Espero que el fix les solucione el problema a los que contribuyen desde Windows.

¡Gracias de nuevo!