Separate phase passage from authority over technical means - #181
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
marconildo
pushed a commit
to marconildo/claude-code-workflows
that referenced
this pull request
Sep 18, 2026
Reinstates the all-empty check, not the per-category one removed in shinpr#181: a unit with routes but no tests proceeds, a unit with no routes, tests, or public exports does not. `unit_inventory` is the completeness baseline for code-verifier, and three empty categories balance at `inputCount: 0`, so such a unit passed verification with no evidence examined and no limitation reported. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.
What changed
Requirement provenance. An adjacent capability the agent noticed stays out of
requirements[]and out of the exclusion hearing, instead of becoming a question for the user.Phase passage no longer fixes the means. ADR
Acceptedrecords the currently selected means rather than an obligation to retain it, and the same applies to UI Spec decomposition. Removing a technical choice that works but is no longer necessary is not a requirement change. The ADR qualifying condition now covers replacing a decision with a different durable choice, so a pure reduction updates the existing record instead of opening a new ADR cycle.Stops approve an action, not an artifact. At each stop the user approves completing the phase or authorizing implementation, rather than ratifying the artifact's technical content. Authority over required outcomes and explicit constraints comes from the user's own wording in the convergence record.
Review can select a reduction. Review Resolution weighs no change, removal, and reuse before a fix that keeps or adds a mechanism. The design-side route covers a Design Doc that is excessive or incorrect, not only stale, and a reduction takes both routes so the design statement and the implementation it authorized are removed together.
code-reviewerno longer treats a Design Doc selection as support for a mechanism, and the technical designers can remove or narrow an accepted decision during update mode.Reviewer output is
pass. The five reviewer and checker agents returnpassinstead ofapproved, matchingcode-reviewer. A reviewerpassrecords phase passage and grants no user-held execution authority.approveis kept for the user-facing stops, where the object is an action.Removed work-generating leftovers. The fixed ten-source discovery table and its
sourceAccountingoutput, the deadverboseinput on two agents, and the rule that re-ran discovery whenever a unit inventory category was empty.README. Documented the reduction route, and replaced a Work Plan coverage table that no longer exists in the template with the task entry the template actually defines. All six languages.
Version 0.26.1 to 0.27.0.
🤖 Generated with Claude Code