fix(deploy): cache path correcto

[Cheewye]

Ajuste de cache en setup-node para usar react-app/package-lock.json.

[Cheewye]

⚠️ Review: NO MERGEAR (requiere ajustes)

Problemas Críticos:

1️⃣ Inconsistencia de Estructura:

❌ PR usa: react-app/ (Create React App)
✅ Repo actual: frontend/ (Vite)

Resultado: Dos frontends diferentes, deploy roto

2️⃣ Duplicación de Código:

❌ PR crea:
   - react-app/src/services/fishermenGeo.ts
   - src/components/FishermenDashboard.tsx

✅ Ya existen:
   - src/services/fishermen.ts
   - src/components/Fishermen.tsx

Resultado: Imports rotos, confusión de dependencias

3️⃣ Deploy Roto:

❌ PR deploy: react-app/build/ (CRA)
✅ CI actual: frontend/dist/ (Vite)
✅ Nginx prod: /var/www/iuri-frontend/ (espera Vite dist/)

Resultado: rsync copiará archivos incorrectos, sitio no cargará

4️⃣ Checks no corrieron:

❌ Sin checks en esta rama
⚠️ No sabemos si compila o pasa tests

---

✅ Soluciones:

Opción A (Recomendada): Migrar a frontend/ Vite

# 1. Mover cambios a la estructura actual:
mv react-app/src/services/fishermenGeo.ts frontend/src/services/
mv src/components/FishermenDashboard.tsx frontend/src/components/

# 2. Consolidar con archivos existentes:
# Mergear fishermenGeo.ts → fishermen.ts
# Mergear FishermenDashboard.tsx → Fishermen.tsx

# 3. Actualizar deploy.yml:
cache-dependency-path: 'frontend/package-lock.json'
working-directory: frontend/
rsync frontend/dist/ ...

# 4. Eliminar react-app/ completo

Opción B: Unificar en react-app/ (NO recomendada)

⚠️ Requiere:
- Actualizar CI (.github/workflows/ci.yml)
- Actualizar todos los scripts de deploy
- Migrar TODO de frontend/ a react-app/
- Archivar/eliminar frontend/

Costo: Alto, muchos cambios

---

🎯 Recomendación:

NO MERGEAR hasta que:

1. ✅ Se consolide en frontend/ Vite (Opción A)
2. ✅ Se eliminen duplicados de servicios/componentes
3. ✅ Se actualice deploy.yml para usar frontend/dist/
4. ✅ Pasen los checks CI

---

¿Necesitás ayuda para migrar los cambios a frontend/ Vite? Puedo hacerlo en 10-15 minutos. 🔧

[Cheewye]

Gracias por el PR. Dejo feedback para poder mergearlo de forma segura:

🚨 Problemas Críticos

1. Estructura inconsistente entre apps

• Actualmente el repo usa `frontend/` (Vite, `dist/`)
• Este PR apunta a `react-app/` (CRA, `build/`)
• Solución: Unificar o CI/CD se rompe

2. Deploy workflow

• `deploy.yml` hace rsync de `react-app/build/`
• Scripts y CI actuales trabajan con `frontend/dist/`
• Solución: Ajustar `deploy.yml` para publicar `frontend/dist/`

3. Duplicación de código

• Ya existe `src/services/fishermen.ts` y `src/components/Fishermen.tsx`
• El PR añade `react-app/src/services/fishermenGeo.ts` y duplicados
• Solución: Consolidar en una sola fuente

4. Conflictos con main

• El PR está en estado CONFLICTING
• Solución: Rebase/merge con `main`

✅ Sugerencias

Opción 1: Mantener `frontend/` (RECOMENDADO)

• Mover cambios de UI/servicios a `frontend/` y `src/` existentes
• Ajustar `deploy.yml` para publicar `frontend/dist/`
• Eliminar `react-app/` del PR

Opción 2: Migrar a `react-app/`

• Actualizar `.github/workflows/ci.yml` y `scripts/deploy.sh`
• Archivar o remover `frontend/`

📋 Checklist

• Resolver conflictos con `main`
• Unificar app objetivo (frontend/ o react-app/)
• Eliminar duplicados (consolidar servicios y UI)
• Asegurar que los checks corran y pasen

Cuando lo ajustes, avisa y lo reviso de nuevo. ¡Gracias! 🙌

[Cheewye]

✅ Quick review del PR #5

Resumen

• Corrige ruta de cache en .github/workflows/deploy.yml.
• Agrega docs (docs/FISHERMEN_API_FULL.md).
• Suma fishermenGeo service y FishermenDashboard.

Observación importante
Hay mezcla de raíces: react-app/src/** y src/**. Esto suele causar imports/builds inconsistentes. Propongo unificar:

Opción A (recomendada)

• Mover src/components/FishermenDashboard.tsx → react-app/src/components/
• Mover src/App.tsx → react-app/src/App.tsx
• Ajustar imports y rutas (ej. App.tsx con /fishermen)
• Mantener el proxy de Vite y VITE_API_BASE=/api

O, Opción B

• Mover react-app/src/services/fishermenGeo.ts → src/services/
• Simplificar el workflow para trabajar solo con src/**

Workflow cache

• Verificar que el key use el lockfile correspondiente (npm: react-app/package-lock.json | pnpm: pnpm-lock.yaml | yarn: yarn.lock)
• Path de cache acorde (npm: ~/.npm o react-app/node_modules; pnpm: ~/.pnpm-store; yarn: ~/.cache/yarn)

Siguientes pasos sugeridos

1. Re-run checks para actualizar mergeable.
2. Unificar raíz (A o B) y ajustar imports.
3. Enlazar docs/FISHERMEN_API_FULL.md desde el README.
4. Confirmar que el service usa /api/... (proxy Vite) o VITE_API_BASE.

Si les sirve, puedo enviar un patch que:

• Unifica en react-app/src/**
• Ajusta App.tsx con la ruta /fishermen
• Alinea el workflow de deploy al lockfile y gestor actual