Skip to content

fix(dev): make local Firebase and LAN startup deterministic#23

Draft
WILLIAMMDN wants to merge 1 commit into
mainfrom
codex/local-network-development
Draft

fix(dev): make local Firebase and LAN startup deterministic#23
WILLIAMMDN wants to merge 1 commit into
mainfrom
codex/local-network-development

Conversation

@WILLIAMMDN

Copy link
Copy Markdown
Owner

Qué cambia

  • usa una URL relativa /api/v1 en desarrollo y un proxy de Angular hacia Laravel local;
  • añade npm run start:lan para pruebas desde teléfonos en una red privada;
  • mantiene Laravel enlazado a loopback y evita abrir CORS con comodines;
  • detiene el arranque cuando un worktree no tiene .env o FIREBASE_PROJECT_ID;
  • permite copiar conscientemente el .env local con -EnvSource sin versionarlo;
  • expone en /salud una señal booleana, no secreta, del estado de Firebase;
  • documenta causa, operación, seguridad y evidencia del diagnóstico.

Causa raíz

Los worktrees no heredan archivos ignorados como backend-laravel/.env. Además, el frontend de desarrollo usaba http://localhost:8000, que desde un teléfono apunta al propio teléfono, mientras Laravel y Angular escuchaban solo en loopback y CORS no autorizaba la IP privada. Los datos no estaban eliminados.

Impacto

El mismo frontend funciona desde localhost o desde la IP privada mediante un origen único. Los errores de configuración de Firebase aparecen antes de abrir los servidores y ya no se confunden con cursos o estudiantes ausentes.

Validación

  • Laravel: 134 pruebas, 477 aserciones
  • Angular Jest: 11 suites, 39 pruebas
  • arquitectura frontend: OK
  • build Angular: 919.89 kB iniciales, bajo 1 MB
  • salud por proxy en localhost: 200
  • salud por proxy en 10.0.0.9: 200
  • parser y guard del script PowerShell: OK
  • git diff --check: OK

Seguridad

No contiene .env, claves ni credenciales. Producción, Firebase Hosting y la configuración cloud no cambian. La modalidad LAN debe usarse solo en una red privada de confianza.

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