Skip to content

ci(sprint-19b): scope api port env for e2e workflow#78

Merged
David-LS-Bilbao merged 1 commit into
devfrom
fix/sprint-19b-e2e-web-port
Jul 8, 2026
Merged

ci(sprint-19b): scope api port env for e2e workflow#78
David-LS-Bilbao merged 1 commit into
devfrom
fix/sprint-19b-e2e-web-port

Conversation

@David-LS-Bilbao

Copy link
Copy Markdown
Owner

Sprint 19B.4 — Fix del puerto de la Web en el workflow E2E manual

Resumen

Corrige el workflow manual JobIT E2E (manual) para que PORT=4000 solo aplique al
arranque de la API y no sea heredado por next start.

Causa raíz

La primera ejecución manual del workflow (run 28959177676) falló en "Esperar web": la Web
heredó PORT=4000 del entorno global del job, next start respetó esa variable e intentó
escuchar en :4000, ya ocupado por la API (EADDRINUSE). En local nunca ocurrió porque
PORT vive solo en apps/api/.env.

Cambio

  • Quita PORT: 4000 del env global del job.
  • Añade PORT: 4000 solo al step Arrancar API en background (junto a
    NODE_ENV: production, scopeada por el mismo motivo).
  • Actualiza el comentario del bloque env para documentar ambas exclusiones.

Verificaciones

  • YAML parseado correctamente; trigger sigue siendo único workflow_dispatch.
  • PORT: 4000 aparece exactamente una vez, en el step de la API.
  • git diff --check → OK. Un solo archivo, +7/−4.
  • La PR debe pasar los required checks actuales de JobIT CI.

Fuera de alcance respetado

  • Sin cambios en tests, Playwright config, package.json/lockfile, branch protection,
    triggers ni deploy. Sin E2E como required check.

Revisión

PR lista para revisión humana. No mergear por CLI. Tras el merge: relanzar
gh workflow run e2e.yml --ref dev para la validación completa del stack.

@David-LS-Bilbao David-LS-Bilbao merged commit 5b9418f into dev Jul 8, 2026
2 checks passed
@David-LS-Bilbao David-LS-Bilbao deleted the fix/sprint-19b-e2e-web-port branch July 8, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant