fix: el diff pasa por el fusible de frases prohibidas y deja fuera credenciales #13

Merged

julio.caicedo merged 1 commit from fix/diff-pasa-por-el-fusible into main

2026-10-23 14:07:13 +00:00

julio.caicedo commented

2026-10-23 14:06:27 +00:00

Owner

Sigue a #12, que dejó de filtrar el diff por extensión.

Fusible en el diff. forbidden (en balanzio: precios, márgenes y costo por comercio de la §8h) se revisaba en los documentos y en la tarea, pero no en el diff. Con docs/ ahora dentro del diff, un PR que agregue esa sección se la mandaba a OpenRouter. El diff pasa por VerifyForbidden igual que los documentos: si aparece una frase, el revisor no arranca.
Fuera del diff .env*, *.env, *.pem y *.key, en cualquier carpeta. gitleaks avisa en el CI, pero no frena que el revisor mande el archivo a un tercero.

Probado: las exclusiones en un repo de prueba dejan main.go, Makefile y docs.md, y sacan .env, .env.local, sub/.env, prod.env, sub/cert.pem e id.key. go test ./cmd/reviewer pasa.

🤖 Generated with Claude Code

Sigue a #12, que dejó de filtrar el diff por extensión. - **Fusible en el diff.** `forbidden` (en balanzio: precios, márgenes y costo por comercio de la §8h) se revisaba en los documentos y en la tarea, pero no en el diff. Con `docs/` ahora dentro del diff, un PR que agregue esa sección se la mandaba a OpenRouter. El diff pasa por `VerifyForbidden` igual que los documentos: si aparece una frase, el revisor no arranca. - **Fuera del diff** `.env*`, `*.env`, `*.pem` y `*.key`, en cualquier carpeta. gitleaks avisa en el CI, pero no frena que el revisor mande el archivo a un tercero. Probado: las exclusiones en un repo de prueba dejan `main.go`, `Makefile` y `docs.md`, y sacan `.env`, `.env.local`, `sub/.env`, `prod.env`, `sub/cert.pem` e `id.key`. `go test ./cmd/reviewer` pasa. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

julio.caicedo added 1 commit

2026-10-23 14:06:27 +00:00

fix: el diff pasa por el fusible de frases prohibidas y deja fuera credenciales

Go / Test (pull_request) Successful in 43s

Details

57a4be0d1e

Desde que el diff trae todos los archivos, docs/ incluido, una sección
que no debe salir podía llegar a OpenRouter por el diff y no por los
documentos. Pasa por VerifyForbidden como ellos. Los .env, .pem y .key
quedan fuera del diff.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

julio.caicedo merged commit 11e262de2b into main

2026-10-23 14:07:13 +00:00

julio.caicedo deleted branch fix/diff-pasa-por-el-fusible

2026-10-23 14:07:13 +00:00

julio.caicedo referenced this pull request from a commit

2026-10-23 14:07:14 +00:00

Merge pull request 'fix: el diff pasa por el fusible de frases prohibidas y deja fuera credenciales' (#13) from fix/diff-pasa-por-el-fusible into main