feat(review): debt-auditor

[dagoaie]

feat(review): debt-auditor

Adds a debt-auditor agent that consolidates the deferred / non-blocking findings from the other audit tracks (clean-code, over-engineering, security, bugs) into a single tracked debt ledger, reports/debt.md. Each entry carries a re-trigger (the concrete event that would make the finding worth re-evaluating); findings with no nameable trigger are tagged no-trigger so they surface as explicit debt instead of silently disappearing.

This is PR 3 of a 4-PR program inspired by Ponytail — Convoy dogfooding itself through its own implement pipeline. Stacked on top of PR 2 (feat/feat-review-over-engineering).

What changed

• prompts/debt-auditor.md (new): audit-only prompt that reads the other four audit reports, extracts deferred findings, and writes the consolidated ledger with triggers and the no-trigger tag.
• src/built-in-prompts.ts: registers the new prompt.
• src/pipeline.ts: registers debt-auditor in builtInAgents (readOnly: true, after security-reviewer) and wires a debt step into review, review-lite, refine, and ultra-refine; review-report and review-adversary triage now consume reports/debt.md.
• prompts/review-report.md / prompts/review-adversary.md: synthesis and triage read the debt ledger.
• Tests: registry, pipeline-shape, and triage coverage updated.

Verification

• bun typecheck — pass
• bun test — 0 fail
• bun run scripts/build.ts — pass
• node_modules symlink confirmed untracked

[dagoaie]

Hola, @ixjosemi:
te he invitado como colaborador (write) de dagocareer/convoy para poder añadirte como revisor oficial de este PR.

Cuando aceptes la invitación, te añado como requested reviewer.
Mientras tanto, tu revisión es bienvenida cuando quieras.

Un abrazo

[dagoaie]

@ixjosemi te comparto esto por si te apetece echarle un vistazo, sin ningún compromiso — solo era para compartirlo 😄

Es un fork de Convoy en mi GitHub (dagocareer/convoy). Me pareció una herramienta muy útil para el día a día y le he añadido cuatro features que me faltaban. Lo curioso es que toda la implementación se ha hecho usando el propio Convoy (dogfooding): cada feature se escribió con su pipeline implement, con su PRD, sus auditorías y su revisión adversarial — Convoy revisándose a sí mismo. Y además, cada PR se fue apilando encima de la anterior.

Un resumen de cada una, en una frase:

• PR feat(summary): diffstat scoreboard per fase en SUMMARY.md #1 — diffstat scoreboard: añade un marcador por fase en SUMMARY.md que muestra cuántas líneas/archivos toca cada paso del pipeline, para ver de un vistazo cuánto trabajo real hace cada fase.
• PR feat(review): over-engineering auditor #2 — over-engineering auditor: Convoy ahora revisa también sobre-ingeniería en cada PR — abstracciones prematuras, flags que nadie usa, capas innecesarias — etiquetadas y con niveles, igual que ya revisaba código limpio, seguridad y bugs.
• PR feat(review): debt-auditor #3 — debt-auditor: cuando una revisión difiere un hallazgo sin resolver, ya no se pierde: entra en un ledger de deuda técnica con un disparador para reabrirla; y si no hay forma honesta de justificar el aplazamiento, queda marcada como deuda «sin disparador».
• PR feat(review): hunter-over-engineering — 7º track repo-wide de sobre-ingeniería #4 — hunter-over-engineering: Convoy revisa el repositorio entero — no solo lo que cambia en cada PR — buscando código inflado que podría recortarse, ordenado por cuánto se puede ahorrar en líneas y dependencias.

Están todas abiertas y mergeables por si te apetece revisarlas, pero vamos, sin compromiso.