feat(ce-plan): add interactive deepening mode for on-demand plan strengthening#443
Conversation
…ngthening When users explicitly ask to deepen an existing plan, sub-agent findings are now presented individually for accept/reject/discuss review before integration. Auto-deepening during plan generation remains unchanged. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1fd30e1991
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…e triggers Generic words like "strengthen", "confidence", "gaps" appeared in normal editing requests and could incorrectly trigger a full deepening pass. Now only "deepen"/"deepening" auto-enter the fast path; secondary signal words require user confirmation when they target the plan holistically, and route to normal editing when they name a specific section.
The reject-all branch in 5.3.6b jumped directly to 5.4, skipping 5.3.9 which handles cleanup of artifact-backed mode scratch files. Added explicit cleanup to the reject-all path so stale .context/ artifacts don't accumulate.
|
@codex review |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
When the
deepen-plancommand was folded intoce:plan, the ability to review sub-agent findings individually before they modified the plan was lost. Users working on complex multi-phase projects want to accept or reject each reviewer's suggestions rather than having them auto-integrated.This adds an interactive deepening mode that activates when a user explicitly asks to deepen an existing plan (e.g., "deepen the plan",
/ce:plan deepen). Auto-deepening during plan generation is unchanged.Fixes #429
What changed
Design decisions
/deepen-plancommand backEval results
Ran 8 test cases across 2 iterations using skill-creator evals:
Test plan
/ce:planon a new feature — auto-deepening runs as before, no interactive prompts/ce:plan deepenwith a plan indocs/plans/— enters interactive mode🤖 Generated with Claude Opus 4.6 (1M context, extended thinking) via Claude Code