✂️ chore(claude.md): drop Code style section — bro doesn't write code#125
Merged
Conversation
Per user review: bro is a planner/router, not a code writer. The 'Self-documenting code' rule applies to SWE, not bro. The commit-message rule lives in the git-conventions skill (frontmatter `agent: bro, swe, pr-reviewer` auto-loads it for all three roles when needed). CLAUDE.md 149 → 144 lines.
Merged
3 tasks
ZaxShen
added a commit
that referenced
this pull request
Apr 26, 2026
…rotocol to skills/docs (#126) User feedback: 'audit every single word' — CLAUDE.md is loaded EVERY turn, must contain only what bro needs every turn (trigger, identity, routing, reactive skills lookup). Detail and reference moved out. Extracted: - 'Subagent prompt precedence' (4 lines) — DELETED. Subagent prompts declare themselves; CC handles dispatch automatically. - 'Two-layer agent model' (5 lines) → new docs/AGENTS.md. - 'Forbidden tools' detail (5 lines) → expanded existing tmb_mcp-error-handling skill (now also covers policy-key writes). - First-action chain rationale paragraph (3 lines) → compressed to the section header itself ('no exceptions, no shortcuts'). - 'Bro verification' + 'Tool-call batching' detail (5 lines) → already documented in planning skills; CLAUDE.md now points there. - 'Direct ops' verbose section (3 lines) → folded into routing table as one row. - 'Concerns + second opinions' (8 lines) → new tmb_concerns-protocol skill (procedural, loads on demand when bro disagrees). - 'Reference' pointer block (7 lines) → new docs/REFERENCE.md. Net: 268 → 99 lines (63% reduction over 3 PRs: #124, #125, this one). What's left in CLAUDE.md: - Trigger rule (when bro activates) — must be every turn - Identity (compressed Role) — must be every turn - MCP caller-id rule (every-turn rule) - First-action chain (mandatory routine on every triggered message) - Routing table (decision tree) - Code-touching ask chain (the planning workflow diagram) - Reactive skills lookup (so bro knows what to load when) - Catchphrase + Communication style (identity) - Reference pointer (2 lines pointing at docs/) Approaching the 60-line aspirational target — further compression would gut essential routing. Co-authored-by: Zax Shen <ZaxShen@users.noreply.github.com>
ZaxShen
added a commit
that referenced
this pull request
May 20, 2026
📝 docs(architecture): add UI.md — AskUserQuestion modes inventory (#125) See merge request trustmybot/plugin!43
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.
Per user review on PR #124: the
# Code stylesection in CLAUDE.md duplicated content already in thecode-qualityandgit-conventionsskills.git-conventionshasagent: bro, swe, pr-reviewerin its frontmatter so it auto-loads for whoever needs it.code-qualityis similarly shared. Bro almost never writes code (Direct Mode is the narrow exception, ≤3 lines), so 'Self-documenting code' was misplaced.CLAUDE.md: 149 → 144 lines.