Repository navigation
feat: review does less, aimed better (MUSTs the range wrote, agents.skip, wider fold) - #56
Merged
Merged
Conversation
…, agents.skip, wider fold start writes into the brief the MUST, SHALL and REQUIRED lines the range added or reworded, the guard command's form and the two severity floors external reviewers otherwise never read. Paths agents.skip names go to no reviewer, listed ten at a time. add folds a suggestion into a blocking line on its anchor, whatever its words. A refusal that names another worktree prints the command to run there, --root before the command. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…w rules follow the tool The conformance reviewer proves a guard for each binding sentence the brief lists, and for at most three older ones whose guard or test changed. review.md gives behaviour a caller can trigger an important floor, makes a keyword missing from a sentence the code meets no lead, and names agents.skip. sdd-scaffold suggests a Go reviewer map of *.go and go.mod, so data and fixtures reach no Go reviewer. sdd-review --panel names the flags scope takes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… fold keeps the suggestion binding_lines diffs the specifications folder whole (the constitution on the informative profile), so a renamed spec is no list of new MUSTs and a changelog or README line is none; a content line that reads like a file header is read as content. A MUST written with no code still plans the conformance reviewer. add folds only another reviewer's suggestion into a blocking line on its anchor and keeps it as a resolved line, its words intact; a batch counts each input line once. The doc reviewer no longer raises a missing keyword, and the examples and methodology §13 state the guard rule the conformance reviewer now follows. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 19, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A review pass does less, aimed where defects were found. The conformance reviewer guards only the MUSTs the range wrote or reworded (plus at most three older ones), paths in
agents.skipreach no reviewer,addfolds another reviewer's suggestion into a blocking line on the same anchor, and the brief carries the two severity floors external reviewers never read.Docs
references/review.md§ Severity, § Scope, § The tool;traceability-schema.md(agents.skip); methodology §13;agents/sdd-spec-conformance-reviewer.mdstep 4;agents/sdd-doc-reviewer.md(no missing-keyword suggestion);sdd-scaffold(Go reviewer block map);sdd-review --panel(scope's flags);docs/examples.md.Verification
./scripts/validate.sh: OK, every word budget held (no raise).sdd-check selftest: 26 OK;claude plugin validate .: passed.Notes for review
binding_linesdiffs the specifications folder whole with-M, so a renamed spec lists no MUSTs.- [~] … deferred: merged into the blocking line on its anchor. It is never posted or harvested, and its words are kept.Checklist
🤖 Generated with Claude Code