Repository navigation
PRD: model capability by job, not named-model review - #178
Merged
Merged
Conversation
Drop the requirement that review run on a different named model. Independence is a fresh worker. Judgment work defaults to the most capable model; the only cheap slot is mechanical runtime-verify. Open assumptions & waivers: - review independence is worker/session not model-name — resolved: independent worker remains a flip requirement; same named model is allowed for write and review - cheap-model slot is FR-52 mechanical verify only — resolved: parent sequencer stays most capable - authoring sessions warn when not most capable — resolved: /prd-author, /tdd-author, and /build-tdds warn and ask continue/stop when the parent session is below the latest top-tier binding or the session model cannot be read - cheaper write/review pin is warned not refused — resolved: honor THROUGHLINE_BUILD_MODEL / THROUGHLINE_REVIEW_MODEL with a warning; run continues - most capable is the latest top-tier binding — resolved: one binding site; rebinding on a new generation is an implementation change - design-reviewer is most capable independent worker — resolved: FR-10 is a fresh worker on the most-capable default; same-name as the TDD author is allowed - session model is always observable on supported harnesses — resolved: Claude Code and Grok Build expose it; unreadable is a defect with the same warn-and-ask shape - /build-tdds parent session is also warn+ask — resolved: same check as authoring Cascade: - ADR 0009 (accepted) — supersede. Its decision (review = prior-gen top tier to keep author≠reviewer by name) is no longer the requirement. - ADR 0008 (accepted) — refine or supersede remaining author≠reviewer-by-name consequence. Rework authoring on the build model is already vacated by ADR 0013; the diversity rationale is what this PRD change removes. - ADR 0010 (accepted) — refine the consequence "Review flip authority is one independent different-model artifact" to independent worker, not different-model. - TDD 0057 (implemented) — supersede. It bound review to prior-gen top tier. - TDD 0062 (implemented) — supersede the models.sh / NFR-3 pairing (review = prior-gen). New TDD(s) must rebind review (and design-reviewer, judgment verify) to the latest top-tier default, add the cheaper-pin warning (FR-87), and add the parent-session warn-and-ask (FR-86) on /prd-author, /tdd-author, and /build-tdds. - Skills and README still say "different model" / prior-gen review (skills/implement, skills/tdd-author, agents/design-reviewer.md, README.md). Those are implementation follow-through for /tdd-author, not this PR.
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
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.
Drop the requirement that review run on a different named model.
Independence is a fresh worker. Judgment work defaults to the most
capable model; the only cheap slot is mechanical runtime-verify.
Open assumptions & waivers:
Cascade: