Skip to content

Add coordination-patterns vocabulary and cross-model standardization#8

Merged
jeffgreendesign merged 2 commits intomainfrom
claude/review-agent-guidelines-2uhir
Apr 13, 2026
Merged

Add coordination-patterns vocabulary and cross-model standardization#8
jeffgreendesign merged 2 commits intomainfrom
claude/review-agent-guidelines-2uhir

Conversation

@jeffgreendesign
Copy link
Copy Markdown
Owner

Summary

Standardizes the plan-first Orchestrator-Subagent workflow across Claude Code, Codex (GPT-5.4+), and Gemini CLI/Code Assist by adopting Anthropic's published multi-agent coordination patterns vocabulary and extending guidance to all three platforms.

Key Changes

Documentation

  • Added docs/COORDINATION-PATTERNS.md: comprehensive reference mapping Anthropic's five coordination patterns to scaffold primitives, with platform-specific delivery mechanisms and evolution guidance
  • Updated docs/DECISION-TREES.md: broadened AGENTS.md decision tree to include Gemini; added new GEMINI.md decision tree
  • Updated docs/TIERS.md: added GEMINI.md to Tier 3 file list
  • Updated docs/DECISIONS.md: added 2026-04-12 decision entry documenting vocabulary adoption and cross-model standardization rationale
  • Updated docs/AX_UPGRADE_REPORT.md: documented all changes with rationale and official references

Scaffold Templates

  • Created scaffold/GEMINI.md.template: minimal context file for Gemini CLI / Code Assist with commands, working rules (including plan-first rule), and pointers to CLAUDE.md and AGENTS.md
  • Updated scaffold/AGENTS.md.template:
    • Added working rule 5 implementing Orchestrator-Subagent pattern (research → propose → approve → implement)
    • Updated Codex version reference from 5.3+ to GPT-5.4+
    • Added Gemini CLI as a reader alongside Codex
  • Updated scaffold/.claude/commands/design.md:
    • Added pattern attribution comment linking to COORDINATION-PATTERNS.md
    • Added sentence naming the Orchestrator-Subagent pattern and explaining the human approval step as the verifier

Implementation Details

The changes preserve the existing Claude Code workflow while extending the same plan-first guidance to Codex and Gemini through their respective instruction files. No behavioral changes to existing functionality—this is purely documentation and template standardization. The Orchestrator-Subagent pattern remains the default for all three platforms, with clear guidance on when to evolve toward other patterns (Agent Teams, Message Bus, Shared State loops) only when specific symptoms appear.

https://claude.ai/code/session_01CcZUjLWr77EdZ5mxYdNdCk

…workflow across Claude, Codex, and Gemini

Map existing scaffold primitives to Anthropic's five multi-agent
coordination patterns (published 2026-04-10). The scaffold already
shipped Orchestrator-Subagent (/design), Generator-Verifier (gates),
and Shared State (NOW.md) — this upgrade names them and extends the
plan-first workflow to Codex (GPT-5.4) and Gemini (3.0/3.1) via
AGENTS.md rule 5 and a new GEMINI.md template.

- Add docs/COORDINATION-PATTERNS.md (model-agnostic pattern mapping)
- Add scaffold/GEMINI.md.template (Gemini CLI context file)
- Add plan-first working rule to scaffold/AGENTS.md.template
- Annotate design.md with Orchestrator-Subagent attribution
- Add coordination anti-pattern row to docs/ANTI-PATTERNS.md
- Add GEMINI.md to Tier 3 and decision trees
- Update DECISIONS.md and AX_UPGRADE_REPORT.md

https://claude.ai/code/session_01CcZUjLWr77EdZ5mxYdNdCk
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bed5fdcd-dfb3-477a-9f3c-ba8a01512f59

📥 Commits

Reviewing files that changed from the base of the PR and between 8683755 and e09d0ac.

📒 Files selected for processing (3)
  • scaffold/.claude/commands/design.md
  • scaffold/GEMINI.md.template
  • scripts/init.sh
✅ Files skipped from review due to trivial changes (2)
  • scaffold/.claude/commands/design.md
  • scaffold/GEMINI.md.template
🚧 Files skipped from review as they are similar to previous changes (1)
  • scripts/init.sh

Walkthrough

This pull request introduces standardized multi-agent coordination patterns vocabulary across the repository scaffold. It adds comprehensive documentation of Anthropic's coordination patterns (Orchestrator-Subagent, Generator-Verifier, Agent Teams, Shared State, and Message Bus) with mappings to existing scaffold primitives. New platform-specific templates are created: scaffold/GEMINI.md.template for Gemini CLI/Code Assist context and updates to scaffold/AGENTS.md.template for Codex/Gemini. Existing instruction files (design.md, ANTI-PATTERNS.md) are annotated with pattern references. Working rules now require multi-file changes or new patterns to follow a research-propose-approve-implement flow before implementation.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Human as Researcher (Human)
  participant Orch as Orchestrator Agent
  participant Sub as Subagent Agent
  participant Ver as Verifier (Human/Tool)
  participant Repo as Repository

  Human->>Orch: request/brief (research need)
  Orch->>Sub: assign research task
  Sub-->>Orch: deliver proposal (changes/plan)
  Orch->>Ver: submit proposal for approval
  Ver-->>Orch: approve / reject
  alt approved
    Orch->>Repo: implement changes (commit/PR)
    Repo-->>Human: notify implementation
  else rejected
    Orch-->>Sub: request revision
  end
Loading
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adopting coordination-patterns vocabulary and standardizing the plan-first workflow across Claude, Codex, and Gemini platforms.
Description check ✅ Passed The description is comprehensive and directly related to the changeset, detailing documentation additions, template updates, and the coordination-patterns standardization across three platforms.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/review-agent-guidelines-2uhir

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (1)
scripts/init.sh (1)

954-963: Use package-manager-neutral command examples in the new template.

The new GEMINI.md command block currently shows pnpm-only placeholders, which makes scaffold output less portable for npm/yarn users.

Proposed refactor
 ```bash
 # Quality gates (run before every commit)
-[pnpm gates]   # alias: verify
+[npm run gates] / [pnpm gates] / [yarn gates]   # alias: verify
 
 # Individual checks
-[pnpm lint]
-[pnpm typecheck]
-[pnpm test]
-[pnpm build]
+[npm run lint] / [pnpm lint] / [yarn lint]
+[npm run typecheck] / [pnpm typecheck] / [yarn typecheck]
+[npm run test] / [pnpm test] / [yarn test]
+[npm run build] / [pnpm build] / [yarn build]
</details>

Based on learnings: Applies to scaffold/**/*.md : Non-shell templates (`.md`) in `scaffold/` should use portable phrasing like "`npm run gates` / `pnpm gates` / `yarn gates`" instead of hardcoding a single package manager

<details>
<summary>🤖 Prompt for AI Agents</summary>

Verify each finding against the current code and only fix it if needed.

In @scripts/init.sh around lines 954 - 963, Replace the pnpm-only command
placeholders in the new template with package-manager-neutral alternatives:
change "[pnpm gates]" to "npm run gates / pnpm gates / yarn gates" and likewise
replace "[pnpm lint]", "[pnpm typecheck]", "[pnpm test]", "[pnpm build]" with
"npm run lint / pnpm lint / yarn lint", "npm run typecheck / pnpm typecheck /
yarn typecheck", "npm run test / pnpm test / yarn test", and "npm run build /
pnpm build / yarn build" respectively; apply this change to the non-shell
markdown templates under scaffold/ (the GEMINI.md block that contains the "[pnpm
gates]" and individual checks lines) so the scaffolded docs are package-manager
neutral.


</details>

</blockquote></details>

</blockquote></details>

<details>
<summary>🤖 Prompt for all review comments with AI agents</summary>

Verify each finding against the current code and only fix it if needed.

Inline comments:
In @scripts/init.sh:

  • Line 76: The generated .claude/commands/design.md references
    docs/COORDINATION-PATTERNS.md which init.sh does not create, producing a broken
    local link; update the init.sh script to either scaffold
    docs/COORDINATION-PATTERNS.md with appropriate content when creating the project
    or change the generation of .claude/commands/design.md so it does not reference
    that path (e.g., embed the coordination pattern text directly or point to a
    stable external URL). Locate the code that writes .claude/commands/design.md and
    the scaffolding logic in init.sh and implement one of these fixes so the
    generated project contains a valid reference.

Nitpick comments:
In @scripts/init.sh:

  • Around line 954-963: Replace the pnpm-only command placeholders in the new
    template with package-manager-neutral alternatives: change "[pnpm gates]" to
    "npm run gates / pnpm gates / yarn gates" and likewise replace "[pnpm lint]",
    "[pnpm typecheck]", "[pnpm test]", "[pnpm build]" with "npm run lint / pnpm lint
    / yarn lint", "npm run typecheck / pnpm typecheck / yarn typecheck", "npm run
    test / pnpm test / yarn test", and "npm run build / pnpm build / yarn build"
    respectively; apply this change to the non-shell markdown templates under
    scaffold/ (the GEMINI.md block that contains the "[pnpm gates]" and individual
    checks lines) so the scaffolded docs are package-manager neutral.

</details>

<details>
<summary>🪄 Autofix (Beta)</summary>

Fix all unresolved CodeRabbit comments on this PR:

- [ ] <!-- {"checkboxId": "4b0d0e0a-96d7-4f10-b296-3a18ea78f0b9"} --> Push a commit to this branch (recommended)
- [ ] <!-- {"checkboxId": "ff5b1114-7d8c-49e6-8ac1-43f82af23a33"} --> Create a new PR with the fixes

</details>

---

<details>
<summary>ℹ️ Review info</summary>

<details>
<summary>⚙️ Run configuration</summary>

**Configuration used**: Organization UI

**Review profile**: CHILL

**Plan**: Pro

**Run ID**: `62f22f46-531c-47a7-b0a7-b59ad58f8a9a`

</details>

<details>
<summary>📥 Commits</summary>

Reviewing files that changed from the base of the PR and between 11863a523879d7ecfc66e766e9112f359d840aac and 868375583acbfcfae6e908d9bab0b5101a30f895.

</details>

<details>
<summary>📒 Files selected for processing (10)</summary>

* `docs/ANTI-PATTERNS.md`
* `docs/AX_UPGRADE_REPORT.md`
* `docs/COORDINATION-PATTERNS.md`
* `docs/DECISION-TREES.md`
* `docs/DECISIONS.md`
* `docs/TIERS.md`
* `scaffold/.claude/commands/design.md`
* `scaffold/AGENTS.md.template`
* `scaffold/GEMINI.md.template`
* `scripts/init.sh`

</details>

</details>

<!-- This is an auto-generated comment by CodeRabbit for review status -->

Comment thread scripts/init.sh Outdated
…lity in GEMINI.md

- design.md: replace local docs/COORDINATION-PATTERNS.md reference (which
  won't exist in downstream projects) with the stable blog URL
- GEMINI.md.template: show all three package manager options in commands
  section to match existing multi-PM conventions in AGENTS.md.template

https://claude.ai/code/session_01CcZUjLWr77EdZ5mxYdNdCk
@jeffgreendesign jeffgreendesign merged commit 4f685ec into main Apr 13, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants