mirror of
https://github.com/mattpocock/skills.git
synced 2026-09-16 14:43:25 +07:00
retro currently treats every reviewer-agent gap the same way: write a new CODING_STANDARDS.md line. That's fine for judgement calls, but for fixed syntactic patterns it just asks the reviewer agent to re-derive the same call on every future diff, forever, instead of paying once for a check. Adds two things to the Automated checks / Coding standards categories: - Automated checks now treats an un-linted repo (no pre-commit hook, no CI job running lint/typecheck/test) as a finding in its own right, not just a consequence of a specific mistake. - Coding standards now classifies a violation as mechanical (deterministic check) vs. judgement call (CODING_STANDARDS.md) before writing a finding, and defaults to building the check. Kept language-agnostic (no ESLint/ts-specific naming) since retro runs across repos in different languages. No em-dashes, per this repo's prose rule in CLAUDE.md. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>