feat(review): over-engineering auditor

[dagoaie]

feat(review): over-engineering auditor

Adds a dedicated over-engineering-auditor agent to Convoy's review family: an audit-only reviewer that hunts speculative generality, premature abstraction, unnecessary indirection, excessive configurability, and out-of-scope gold-plating in the scoped change. It becomes the fourth audit track in the review, review-lite, refine, and ultra-refine pipelines, alongside clean-code-auditor, security-reviewer, and bug-auditor, feeding the same review-report / review-adversary synthesis.

This is PR 2 of a 4-PR program inspired by Ponytail — Convoy dogfooding itself through its own implement pipeline.

What changed

• prompts/over-engineering-auditor.md (new): audit-only prompt structurally mirroring clean-code-auditor.md, with an explicit N1–N7 level / V1–V7 variant taxonomy and seven tags (yagni, premature-abstraction, configurability, indirection, complexity, over-processing, scope-creep). The canonical YAGNI case is N1/V1.
• src/built-in-prompts.ts: registers the new prompt (sync test in test/agents.test.ts covers it).
• src/pipeline.ts: registers over-engineering-auditor in builtInAgents (readOnly: true, temperature: 0.1, between clean-code-auditor and security-reviewer) and wires an over-engineering step into review, review-lite, refine, and ultra-refine, with reports/over-engineering.md fed to review-report and to review-adversary triage.
• prompts/review-report.md / prompts/review-adversary.md: synthesis and triage now read the over-engineering audit report.
• Tests: registry list and pipeline-shape asserts updated; new coverage for prompt loading, refine triage reports, and ultra-refine fan-out.

Verification

• bun typecheck — pass
• bun test — 757 tests, 0 fail
• bun run build — pass

Out of scope (follow-ups)

• review-cc, hunter/hunter-max, ultra-implement untouched — a later PR can fan the audit into those pipelines.

[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.