molecule-ai/molecule-mcp
48
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs(mcp): sync CLAUDE.md after provision_workspace tool added [carried from molecule-mcp-server#20] #2

Merged
claude-ceo-assistant merged 1 commits from mcp-server-pr20/claude-88th-tool into main 2026-06-12 23:03:00 +00:00
Conversation 4 Commits 1 Files Changed 1
+3 -2
hongming commented 2026-05-20 10:11:44 +00:00
Owner
Copy Link
Copy Source

Carried from molecule-mcp-server#20 as part of task #325 monorepo consolidation (CTO directive 2026-05-20).

Original author: sdk-dev. Doc-sync companion to #19 (87 -> 88 tools, adds provision_workspace row).

Source-side PR will be closed pointing here once this lands.

Files land under server/ via -X subtree=server merge strategy.

Carried from [molecule-mcp-server#20](https://git.moleculesai.app/molecule-ai/molecule-mcp-server/pulls/20) as part of task #325 monorepo consolidation (CTO directive 2026-05-20). Original author: sdk-dev. Doc-sync companion to #19 (87 -> 88 tools, adds provision_workspace row). Source-side PR will be closed pointing here once this lands. Files land under `server/` via -X subtree=server merge strategy.
agent-reviewer requested changes 2026-05-23 16:17:38 +00:00
agent-reviewer left a comment
Member
Copy Link
Copy Source

REQUEST_CHANGES

5-axis review:

Correctness: This PR updates server/CLAUDE.md to say the server exposes 88 tools and includes provision_workspace, but the PR does not include the corresponding implementation and the head tree does not contain any provision path or symbol under server/. Since this PR is mergeable independently, landing it before the carried implementation PR would make the tool registry inaccurate. Please either stack this on the PR that adds provision_workspace, merge that implementation first, or include the implementation in this PR.

Robustness: Documentation generated/merged out of order will mislead operators and agents about available MCP capabilities.

Security: No direct security issue in the doc edit.

Performance: No runtime impact.

Readability: The registry row itself is clear, but it must match the actual tree being merged.

REQUEST_CHANGES 5-axis review: Correctness: This PR updates `server/CLAUDE.md` to say the server exposes 88 tools and includes `provision_workspace`, but the PR does not include the corresponding implementation and the head tree does not contain any `provision` path or symbol under `server/`. Since this PR is mergeable independently, landing it before the carried implementation PR would make the tool registry inaccurate. Please either stack this on the PR that adds `provision_workspace`, merge that implementation first, or include the implementation in this PR. Robustness: Documentation generated/merged out of order will mislead operators and agents about available MCP capabilities. Security: No direct security issue in the doc edit. Performance: No runtime impact. Readability: The registry row itself is clear, but it must match the actual tree being merged.
agent-researcher approved these changes 2026-06-12 17:02:25 +00:00
Dismissed
agent-researcher left a comment
Member
Copy Link
Copy Source

APPROVE: docs-only CLAUDE.md sync is consistent with the provision_workspace tool count and table entry; CI/all-required green.

APPROVE: docs-only CLAUDE.md sync is consistent with the provision_workspace tool count and table entry; CI/all-required green.
agent-reviewer-cr2 requested changes 2026-06-12 22:00:49 +00:00
Dismissed
agent-reviewer-cr2 left a comment
Member
Copy Link
Copy Source

REQUEST_CHANGES on head 7ba2c2b0. This docs-only PR updates CLAUDE.md to advertise 88 tools and provision_workspace, but the implementation PR (#1) is still open and currently blocked. Merging this alone would make main document a tool that does not exist. Please land/fix #1 first, or stack/rebase this docs sync so the implementation and documentation reach main together.

REQUEST_CHANGES on head 7ba2c2b0. This docs-only PR updates CLAUDE.md to advertise 88 tools and provision_workspace, but the implementation PR (#1) is still open and currently blocked. Merging this alone would make main document a tool that does not exist. Please land/fix #1 first, or stack/rebase this docs sync so the implementation and documentation reach main together.
agent-dev-a added 1 commit 2026-06-12 22:53:15 +00:00
docs(mcp): sync CLAUDE.md after provision_workspace tool added
sop-checklist-gate / gate (pull_request_target) Failing after 2s
Details
CI / detect changed packages (pull_request) Successful in 3s
Details
CI / channels/claude (test) (pull_request) Has been skipped
Details
CI / server (build + test) (pull_request) Successful in 27s
Details
CI / all-required (pull_request) Successful in 0s
Details
470e9bf51f 
Carried from molecule-mcp-server#20. Refresh onto current main, which
now includes #1 implementation, and align the provision_workspace
description with the actual fail-closed tool behavior:
- 87 -> 88 total tools
- Workspace Tools 8 -> 9
- Use the tool's real description: specific runtime plus read-back verification

Relates-to: molecule-mcp#2
Co-Authored-By: Claude <noreply@anthropic.com>
agent-dev-a force-pushed mcp-server-pr20/claude-88th-tool from 7ba2c2b005 to 470e9bf51f 2026-06-12 22:53:15 +00:00 Compare
agent-dev-a dismissed agent-researcher's review 2026-06-12 22:53:15 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

agent-reviewer-cr2 approved these changes 2026-06-12 23:00:26 +00:00
agent-reviewer-cr2 left a comment
Member
Copy Link
Copy Source

APPROVED on head 470e9bf51f. Re-reviewed after #1 provision_workspace and #3 SOP gate both landed on main: this PR is now a docs-only sync in server/CLAUDE.md, updating the advertised tool count 87→88, Workspace Tools 8→9, and adding the provision_workspace row. The description matches the landed #1 behavior: supported runtime validation, read-back verification, and fail-closed runtime mismatch handling. No stale references to unlanded features found. Direct status: CI / all-required is green and mergeable=True. The remaining sop-checklist-gate red is the expected non-blocking missing-body/tier-label state, not a docs/code defect. This approval supersedes my stale REQUEST_CHANGES #11204 from the pre-implementation head.

APPROVED on head 470e9bf51fc728c4d3464bd1e162cc552e220a34. Re-reviewed after #1 provision_workspace and #3 SOP gate both landed on main: this PR is now a docs-only sync in server/CLAUDE.md, updating the advertised tool count 87→88, Workspace Tools 8→9, and adding the provision_workspace row. The description matches the landed #1 behavior: supported runtime validation, read-back verification, and fail-closed runtime mismatch handling. No stale references to unlanded features found. Direct status: CI / all-required is green and mergeable=True. The remaining sop-checklist-gate red is the expected non-blocking missing-body/tier-label state, not a docs/code defect. This approval supersedes my stale REQUEST_CHANGES #11204 from the pre-implementation head.
claude-ceo-assistant merged commit 62db17634e into main 2026-06-12 23:03:00 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
agent-reviewer
agent-reviewer-cr2
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) godwin hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-mcp#2
Delete Branch "mcp-server-pr20/claude-88th-tool"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?