🤖 Define architecture, planning, and implementation roles - #398
Merged
Merged
Conversation
PR #398: 🤖 Define architecture, planning, and implementation roles4 files, +275 / -11 Scope🟡 Large PR with no linked issue. Structural✅ No structural bloat detected. Slop✅ Slop indicators look low. Static Analysis✅ Oxlint found no issues. CorrectnessNo extraneous code patterns detected. |
taras
marked this pull request as ready for review
August 9, 2026 01:18
taras
enabled auto-merge (squash)
August 9, 2026 01:18
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.
Why
The repository assigns model families to agent roles, but only describes the
Implementor in one sentence and gives the Planner one narrow review instruction.
Architectural review, decision-complete planning and cross-agent continuity are
therefore carried in conversation instead of repository instructions.
What changes
Before:
Agents could identify a default role but could not reconstruct its authority,
workflow, evidence requirements or handoff contract from the repository.
After:
AGENTS.mdroutes Architect, Planner and Implementor work to separate rolecontracts. The contracts define decision ownership, review verdicts, planning
completeness, implementation escalation and a common continuity record.
How it works
Explicit task assignment wins. GPT remains the default Planner and acts as the
Architect for architecture, stack and milestone work; Opus remains the default
Implementor. The existing Fabel Problem solver mapping is unchanged.
Review guide
Start with:
AGENTS.mdunder Agent RolesThen review:
.agents/architect.mdfor product-decision ownership and architecture verdicts.agents/planner.mdfor decision completeness and Implementor handoffs.agents/implementor.mdfor implementation authority, escalation and evidenceLook carefully at:
role authority to silently make product decisions
What must stay true
AGENTS.mdandapply to every role.
How to verify it
git diff --checkpasses.AGENTS.mdresolves to a tracked file.an Plannertext are absent.Scope
Included
Intentionally unchanged
Risks and limitations
Scope confirmation