diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-22 14:07:44 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-22 14:07:44 -0500 |
| commit | 5ff5088b0a3978f35b49e35c71add4764697c257 (patch) | |
| tree | f514beee250bcaddb6e4dc4f0025b65630516315 /debug | |
| parent | 6c91a4e898595e41717d72398f8a63290aac9a92 (diff) | |
| download | rulesets-5ff5088b0a3978f35b49e35c71add4764697c257.tar.gz rulesets-5ff5088b0a3978f35b49e35c71add4764697c257.zip | |
docs(skills): scope review-code's CI-trust and CLAUDE.md-citation rules
Two clarifications to review-code where it appeared to contradict other rules.
The "trust CI, don't run builds" rule read as a blanket license to skip verification. I scoped it to reviewing a diff, not shipping one. A pre-commit or pre-push flow still owes the local verification verification.md requires. Reading a PR doesn't duplicate CI. Producing one doesn't get to skip it.
The CLAUDE.md-adherence audit could put a CLAUDE.md citation into a team-visible PR comment, which commits.md says not to do. I added two modes. A private review cites CLAUDE.md directly. A public review translates the rule into the engineering reason and doesn't name the file, since a teammate can act on the reason but not on a file they can't reach.
Diffstat (limited to 'debug')
0 files changed, 0 insertions, 0 deletions