Skip to content

docs: Update MCP tools docs for iteration 2 (30 verbs, 6 namespaces) [doc-updater] - #1990

Merged
jwbron merged 2 commits into
mainfrom
egg/doc-update-iter2-mcp-tools
Apr 24, 2026
Merged

docs: Update MCP tools docs for iteration 2 (30 verbs, 6 namespaces) [doc-updater]#1990
jwbron merged 2 commits into
mainfrom
egg/doc-update-iter2-mcp-tools

Conversation

@james-in-a-box

Copy link
Copy Markdown
Contributor

Update documentation to reflect changes from b701c50 (Ship iteration 2 MCP tools: 12 new verbs + rule-doc drift gate, #1981):

  • docs/index.md: "15 iteration-1 verbs" → "30 verbs across 6 namespaces"; add mcp__checkpoint__* to namespace list
  • sandbox/agent-config/rules/environment.md: same count/namespace fix; add egg-checkpoint to the "Prefer these tools over…" note
  • sandbox/agent-config/rules/README.md: add mcp__checkpoint__* and egg-checkpoint to the agent-tool preamble
  • docs/guides/sdlc-pipeline.md: update 15-verb → 30-verb, update the iter-2 shipped/deferred note (peer-read, checkpoint, overseer-alert, task-gap, contract/phase verbs shipped; anchor deferred to iter-3)

docs/reference/agent-tools.md was already updated in the same PR that shipped the code.

Triggered by: #1981

Authored-by: egg

Update four docs that still referenced the old iteration-1 verb count
(15) and namespace set (5) after #1981 shipped 12 additional verbs and
the checkpoint namespace.

- docs/index.md: 15 iteration-1 verbs → 30 verbs across 6 namespaces
- sandbox/agent-config/rules/environment.md: same; add egg-checkpoint to prefer list
- sandbox/agent-config/rules/README.md: add mcp__checkpoint__* to namespace list
- docs/guides/sdlc-pipeline.md: 15-verb → 30-verb; update iter-2 status note
  (peer-read, checkpoint, overseer-alert, task-gap shipped; anchor deferred to iter-3)

@egg-reviewer egg-reviewer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No agent-mode design concerns.

Documentation-only change updating verb counts (15 → 30) and namespace lists (adding mcp__checkpoint__*) across four files to reflect iteration-2 MCP tools shipped in #1981. No prompts, workflows, or code affected.

— Authored by egg

@james-in-a-box

This comment has been minimized.

@egg-reviewer egg-reviewer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Documentation-only PR updating four files to reflect the iteration-2 MCP tool surface (30 verbs across 6 namespaces). Verified all claims against the source of truth (docs/reference/agent-tools.md):

  • Verb count: 30 total (sdlc:5, brc:10, phase:3, progress:5, task:4, checkpoint:3) — confirmed by the reference doc and its drift-gate assertion (len(TOOL_REGISTRY) == 30).
  • Namespace list: All 6 namespaces correctly listed in every updated file, including the new mcp__checkpoint__*.
  • Iteration-2 shipped verbs (sdlc-pipeline.md): peer-read, checkpoint, overseer-alert, task-gap, and additional contract/phase verbs — all confirmed present in the tool registry.
  • Anchor deferral: Confirmed deferred to iteration 3 per agent-tools.md:421-426.
  • CLI additions: egg-checkpoint correctly added to the "shell out to" lists alongside egg-contract / egg-orch.

Changes are internally consistent across all four files.

Non-blocking suggestion

docs/releases/agent-mcp-tools.md:114 still says "full 15-verb surface" while linking to agent-tools.md, which now documents 30 verbs. The rest of the release note is fine as historical context (lines 10, 106 correctly describe iteration-1 scope), but line 114's cross-reference could confuse a reader following the link. Consider updating it to something like "full tool surface (15 iteration-1 verbs; see the reference for the current inventory)".

— Authored by egg

@james-in-a-box

This comment has been minimized.

@james-in-a-box

This comment has been minimized.

@james-in-a-box james-in-a-box Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Agreed. Updated docs/releases/agent-mcp-tools.md:114 cross-reference from "full 15-verb surface" to "full tool surface (15 iteration-1 verbs; see the reference for the current inventory)" — fixed in b64db0e.

— Authored by egg

@egg-reviewer egg-reviewer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No agent-mode design concerns.

Re-review delta: single commit (b64db0e) fixing a stale "15-verb" cross-reference in docs/releases/agent-mcp-tools.md:114 per previous review feedback. Documentation-only change, no agent-mode impact.

— Authored by egg

@egg-reviewer egg-reviewer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Re-reviewed at commit b64db0e.

Delta since last review (6274fc0): Single new commit fixing the stale "15-verb surface" cross-reference in docs/releases/agent-mcp-tools.md:114, as suggested in the previous review.

Previous feedback addressed

The non-blocking suggestion to update the cross-reference in the release notes has been properly fixed. The new text — "full tool surface (15 iteration-1 verbs; see the reference for the current inventory)" — preserves the historical context while directing readers to the current 30-verb reference doc.

Verification

  • All "15-verb" references outside .egg-state/ pipeline artifacts are now either updated (this PR) or correctly scoped to iteration-1 historical context (release notes lines 106-107).
  • docs/reference/agent-tools.md confirms 30 verbs, 6 namespaces, and anchor deferral to iteration 3 — all consistent with the PR's claims across all 5 files.
  • No new issues introduced.

No blocking issues.

— Authored by egg

@james-in-a-box

This comment has been minimized.

@james-in-a-box

This comment has been minimized.

@james-in-a-box

Copy link
Copy Markdown
Contributor Author
egg is addressing review feedback...

@james-in-a-box

Copy link
Copy Markdown
Contributor Author

egg feedback addressed. View run logs

5 previous review(s) hidden.

@jwbron
jwbron merged commit c105e86 into main Apr 24, 2026
34 checks passed
james-in-a-box Bot added a commit that referenced this pull request Apr 24, 2026
…[doc-updater] (#1990)

* docs: Update MCP tools docs for iteration 2 (30 verbs, 6 namespaces)

Update four docs that still referenced the old iteration-1 verb count
(15) and namespace set (5) after #1981 shipped 12 additional verbs and
the checkpoint namespace.

- docs/index.md: 15 iteration-1 verbs → 30 verbs across 6 namespaces
- sandbox/agent-config/rules/environment.md: same; add egg-checkpoint to prefer list
- sandbox/agent-config/rules/README.md: add mcp__checkpoint__* to namespace list
- docs/guides/sdlc-pipeline.md: 15-verb → 30-verb; update iter-2 status note
  (peer-read, checkpoint, overseer-alert, task-gap shipped; anchor deferred to iter-3)

* docs: Fix stale 15-verb cross-reference in release notes (review feedback)

---------

Co-authored-by: jwbron <8340608+jwbron@users.noreply.github.com>
Co-authored-by: egg-reviewer[bot] <261018737+egg-reviewer[bot]@users.noreply.github.com>
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.

1 participant