Skip to content

Add review_plan MCP tool and accept/reject overlay loop#105

Open
astex wants to merge 1 commit intoplan-review-pr2-display-widgetfrom
plan-review-pr3-review-plan
Open

Add review_plan MCP tool and accept/reject overlay loop#105
astex wants to merge 1 commit intoplan-review-pr2-display-widgetfrom
plan-review-pr3-review-plan

Conversation

@astex
Copy link
Copy Markdown
Owner

@astex astex commented Apr 8, 2026

Wires Claude's exit-plan-mode flow to the plan-review overlay shipped in PR-2. A new mandelbot__review_plan MCP tool sends a review_plan message to mandelbot and blocks the agent until the user responds; mandelbot forces the overlay open, sets the tab status to needs_review, and writes back {decision: accept|reject} from Enter/Escape in the widget. ExitPlanMode is suppressed inside mandelbot via a new PROMPT.md directive and the mandelbot-plan-review skill, and the hooks settings learn to treat mandelbot__review_plan the same way they treat ExitPlanMode so the catch-all "working" hook does not race the tool-specific "needs_review" hook. The MCP test exercises the new round trip end-to-end.

Stacked on #104.

Wires Claude's exit-plan-mode flow to the plan-review overlay. The new
mandelbot__review_plan MCP tool blocks the agent until the user accepts
or rejects the plan in the overlay; the response is plumbed back through
the parent socket. ExitPlanMode is replaced inside mandelbot via system
prompt and the new mandelbot-plan-review skill.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.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