Conversation
…fier separation - verify-test: replace surface-level checklist with 3 mandatory steps (artifact inspection, artifact execution via exec, instruction semantic review against plan.md domain knowledge). Structural checks (delegates, pick, modelTier) moved to supplementary within semantic review. - test-expert: strip all evaluation logic, making it a pure executor that reports facts only (query, produced artifacts, errors). Evaluation is verify-test's job. - design-roles: remove prescriptive architecture principles (cohesion/coupling/ information hiding lecture) and role summary output. Keep Built-in Verification with explicit executor/verifier separation rationale (context contamination prevention). Keep Trust the LLM. - write-definition: consolidate structural self-checks (delegates array, pick list, defaultModelTier) into pre-write self-check, preventing defects at write time rather than catching them in verification. - Remove persona-style role labels (Product Manager, Technical Architect) from section headers and instruction openings. Co-Authored-By: Claude Opus 4.6 <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.
Summary
exec(build/test/lint), (3) instruction semantic review against plan.md domain knowledgeTest plan
bun run buildpassesbun run testpassesbun run format-and-lintpasses (warnings are pre-existing)bun test --timeout 300000 e2e/(requires API keys)🤖 Generated with Claude Code