fix(agent-core): fix plan mode auto review bug#1151
fix(agent-core): fix plan mode auto review bug#1151hewlett-packard-lovecraft wants to merge 1 commit into
Conversation
🦋 Changeset detectedLatest commit: 32747c4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 32747c4c52
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
Related Issue
Resolve #(1150)
Problem
See linked issue.
If in Auto mode before entering Plan mode, ExitPlanMode automatically approves the plan and begins executing. In contrast, YOLO mode still shows the Plan Review prompt, so only Auto mode auto-execs plan
This is a bug - the docs for Plan mode mention that exiting Plan mode will always require user approval
What changed
packages/agent-core/src/agent/permission/policies/exit-plan-mode-review-ask.ts, remove an early-return if in auto modepackages/agent-core/src/agent/permission/policies/index.ts, make sure that exit-plan-mode-review-ask runs before auto-mode-approveupdate various tests:
test/agent/permission.test.ts:test/agent/plan.test.tspackages/agent-core/test/tools/planning/exit-plan-mode-telemetry.test.tsChecklist
gen-changesetsskill, or this PR needs no changeset.gen-docsskill, or this PR needs no doc update.