Limpieza y mejoras del repo y del servidor acordadas con audit y nw
Elazar, CLI 2026-09-22: 'delete the orphan dev ref in the prod bare, and the rest of the things you found that can be cleaned up, improved upon, made more efficient, change, they are all yours to change/enhance, just get a second opinion from audit. nw-lezama-cc is the agent that manages the server, thus any changes needed on os, caddy, node servers, etc, run it by it and i can give it my go if you, audit and nw agree.' HECHO YA: - Borrada la ref huerfana refs/heads/dev del bare de PROD /srv/movilba-web.git. Era 8f30bfaa, ancestro de nuestro dev, o sea que no tenia nada unico y es recuperable sin bundle. El bare quedo con main solamente. - (De MOVILBA-531) sacado el remoto staging y siete claves sin consumidor de .env.local. MEDIDO EN ESTE ITEM, read-only, via Vercel MCP: NO existe proyecto Vercel para este repo. Cero resultados en los dos equipos que ve la cuenta (Elazar-Pimentel-Portfolio y DO-NOT-USE) y cero con filtro repoUrl sobre github.com/ElazarPimentel/movilba-web. Cobertura: solo esos dos equipos. Conclusion: vercel.json es config muerta y origin/dev NO tiene consumidor de build - es espejo. Esto cierra la celda que en 531 quedo abierta. TAMBIEN MEDIDO: mba-deploy-prod.sh NO usa los helpers de seleccion (mba_cs_stage y compania), asi que NO tiene el bug de reporte que se arreglo en el script de dev. No hay nada que portar. PROD_DEPLOY_LOG apunta a un archivo que no existe todavia pero el directorio /srv/movilba-web/logs SI es escribible por el grupo, asi que el '>>' lo crearia - no es el defecto que era el de dev. PROPUESTA MANDADA a audit-mba-l-cc y nw-lezama-cc (tag coder03-mba-l-cc-muci6jnbrf7s), una ronda: Repo, para audit: R1 borrar vercel.json. R2 sacar '.next/dev/types/**/*.ts' del tsconfig - mismo peligro que el bug B1 recien arreglado, porque pnpm dev puede crear ese directorio, puede quedar rancio y next typegen NO lo refresca. R3 tapar cuatro huecos del .gitignore (agents/**/node_modules, agents/**/outputs/, agents/**/.artifact-work-rutap/, outputs/ de raiz; la linea 4 es /node_modules, anclada a raiz, por eso se filtra la copia del agente). R4 borrar el script huerfano .claude/hooks/worktree-dest-gate.sh. R5 decidir si .claude/settings.json, que quedo en {}, se queda o se va. Servidor, para nw: S1 el vhost 308 dev.movilba.com. S2 el vhost staging.movilba.com contra el 3043 muerto. S3 que el post-receive hace git clean de logs/ y por eso nada sobrevive ahi. S4 restatement del SUDO_PASSWORD en texto plano, sin pedir accion y sin rotar. ESPERANDO las dos respuestas. Con acuerdo de los tres, Elazar da el GO para lo del servidor.
Activity
-
AUDITADO 2026-09-22 07:10 ART. Commit 7249af13 (v5.6.4) verificado contra el stat y el checkout: vercel.json borrado, worktree-dest-gate.sh borrado, settings.json entry borrada, .gitignore con los 4 paths exactos que reporte antes. tsconfig.json CONFIRMADO ausente del commit y .next/dev/types/**/*.ts SIGUE en include - coincide con la retractacion de R2. APROBADO sin bloqueantes. R2 retirado correctamente: next typegen posee el array include y lo reescribe, asi que un edit a mano se pierde en el paso 2/4 del deploy (typecheck) antes del commit en 3/4 - exactamente el patron de 'ensayo que no ejecuta el write real' pero al reves: aca el write SI se ejecuto y el framework lo deshizo silenciosamente. Bien atrapado con un ensayo deliberado (grep post-typegen), no con la corrida de deploy real - suficiente porque el mecanismo (Next reescribe el array) no depende del contexto del deploy. Diferidos, sin objecion: (1) flags 'vercel@claude-plugins-official': false en 6 settings.json de carriles, varios retirados; (2) autoCompactWindow local 200000 vs global 250000; (3) S3 (post-receive limpia logs/ en el arbol desplegado) sin dueno porque nw tampoco escribe /srv - correctamente escalado a Elazar, no a mi ni a nw.
-
Shipped v5.6.4 / 7249af13 (dev + origin, health OK patch 4). Vercel purged: vercel.json, .claude/settings.json and .claude/hooks/worktree-dest-gate.sh deleted; .gitignore, AGENTS.md, eslint.config.mjs, mba-deploy-dev.sh, mba-deploy-pod.sh (strip list), docs/gcaba-push-decisions.md and scripts/all-project-txt-files.txt edited. Measured before deleting: zero Vercel projects for this repoUrl across both visible teams. audit gave GO on R1-R5; nw-lezama-cc answered S1-S4. R2 RETRACTED - not doable at all. `next typegen` OWNS the tsconfig `include` array and rewrites it. Reproduced deliberately: removed `.next/dev/types/**/*.ts`, ran `node_modules/.bin/next typegen`, it was re-added at once. And it is silent: because package.json `typecheck` is now `next typegen && tsc --noEmit` (the B1 fix from MOVILBA-531), the edit is undone at deploy step 2/4, BEFORE the commit at 3/4, so it disappears with no diff and normal-looking commit stats. Consequence to record: the 7249af13 COMMIT MESSAGE says "drop the stale dev-types include" and that clause is FALSE - tsconfig.json is not in its stat and both `.next/types/**` and `.next/dev/types/**` are still in the shipped file. The message cannot be corrected in place (already pushed), so the correction lives here. B1 already removes the real failure mode (typegen regenerates the types before tsc reads them), so nothing is left unprotected. Rule learned: that array is Next's OUTPUT, not our configuration - the convention loses to the primitive. S1 (nw-lezama-cc, adopted): KEEP dev.movilba.com. The 308 to the OpenShift pod is deliberate - GCABA testers have it bookmarked and the redirect is what stops those hits 404ing. Now written into the mba-deploy-dev.sh header so nobody "fixes" it later. S2 done by nw (staging.movilba.com vhost removed). S4 noted. OPEN, NO OWNER: S3 - the /srv/dev-movilba-web.git post-receive git-cleans logs/ in the deployed tree on every push, so nothing left there survives. nw-lezama-cc is barred from /srv by the same standing rule I am. Needs Elazar. DEFERRED, not done: (1) six lane .claude/settings.json under agents/{audit-mba-l,coder-mba-l,db-mba-l,pm-mba-l,proxy-mba-w,researcher-mba-l} still carry "vercel@claude-plugins-official": false - dead flags now, and several of those lanes are retired, so it belongs to a bigger lane-directory cleanup; (2) .claude/settings.local.json sets autoCompactWindow 200000 against lezama's global 250000; (3) plaintext SUDO_PASSWORD= in the post-receive - still open, untouched, not rotated.
-
Vercel purgado y shipeado en 7249af13 (v5.6.4). R1/R3/R4/R5 hechos; R2 retractado y medido (next typegen es dueno del array include del tsconfig). S1/S2/S4 cerrados con nw-lezama-cc. S3 y el SUDO_PASSWORD en texto plano salen a MOVILBA-533 para que no mueran con este item. Falta la revision de audit-mba-l-ca sobre el diff shipeado.
-
AUDITADO 07:20 ART: 568efa79 (v5.6.5) verificado - un solo diff por archivo, la linea vercel@claude-plugins-official, nada mas tocado en los 6 settings.json trackeados (incluido el mio, audit-mba-l). APROBADO. MOVILBA-532 CERRADO desde el lado de audit - unico pendiente es el directorio de carriles retirados, sin orden de borrarlo.