Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(ce-plan): add interactive deepening mode for on-demand plan strengthening#443

Merged
tmchow merged 3 commits into
mainfrom
feat/plan-deepen-review
Mar 29, 2026
Merged

feat(ce-plan): add interactive deepening mode for on-demand plan strengthening#443
tmchow merged 3 commits into
mainfrom
feat/plan-deepen-review

Conversation

@tmchow
Copy link
Copy Markdown
Collaborator

@tmchow tmchow commented Mar 29, 2026

Summary

When the deepen-plan command was folded into ce: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

  • Skill description and argument-hint updated so ce:plan auto-triggers for deepening requests
  • Phase 0.1 — deepen intent now targets plan docs (not requirements docs), with light-touch discovery that confirms with the user when the match isn't obvious
  • Phase 5.3 — two deepening modes: auto (default during generation, unchanged) and interactive (on-demand, per-agent accept/reject/discuss review)
  • New Phase 5.3.6b — interactive finding review gate between agent dispatch and synthesis. Findings presented one agent at a time; only accepted findings are integrated
  • Phase 5.3.7 — interactive mode only synthesizes accepted findings
  • Pipeline mode — always auto, no behavioral change for automated workflows
  • Signal word tightening — only "deepen"/"deepening" auto-enter the fast path. Generic words like "strengthen", "confidence", "gaps" require user confirmation when targeting the whole plan, and route to normal editing when they name a specific section. Prevents false-positive deepening triggers from requests like "strengthen the risk section"

Design decisions

  • One skill, one entry point — behavior adapts to invocation intent rather than adding a separate /deepen-plan command back
  • "Discuss" leads to one exchange then a binary accept/reject re-ask — no automatic fallback that modifies the plan without explicit consent
  • When all findings are rejected, document-review is skipped (plan unchanged) and flow proceeds to post-generation options
  • When findings are accepted, document-review runs as a quality gate on the modifications
  • Three-tier signal word hierarchy: "deepen" (auto-trigger) > "strengthen"/"confidence"/"gaps" on whole plan (confirm first) > same words on a specific section (normal edit). This came from eval testing that showed generic words could false-positive into a full deepening pass when the user just wanted a targeted edit

Eval results

Ran 8 test cases across 2 iterations using skill-creator evals:

Eval Category Result
Plan creation (new feature) No regression 8/8
Explicit "deepen the plan" Deepening 6/6
"Strengthen" (whole plan) Signal words 3/3 (confirms first)
Normal edit ("update test scenarios") Disambiguation 4/4
Indirect intent ("confidence gaps") Signal words 4/4 (confirms first)
"Strengthen the risk section" Disambiguation 3/3 (normal edit)

Test plan

  • /ce:plan on a new feature — auto-deepening runs as before, no interactive prompts
  • /ce:plan deepen with a plan in docs/plans/ — enters interactive mode
  • "deepen the plan" without slash command — skill auto-triggers
  • "strengthen the plan" — confirms deepening intent before entering interactive mode
  • "strengthen the risk section" — routes to normal editing, not deepening
  • Interactive mode: accept some findings, reject others — only accepted ones integrated
  • Interactive mode: reject all findings — plan unchanged, skips to 5.4
  • Pipeline/headless mode — always auto, no interaction

Compound Engineering v2.59.0
🤖 Generated with Claude Opus 4.6 (1M context, extended thinking) via Claude Code

…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]>
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 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".

Comment thread plugins/compound-engineering/skills/ce-plan/SKILL.md Outdated
tmchow added 2 commits March 29, 2026 13:31
…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.
@tmchow
Copy link
Copy Markdown
Collaborator Author

tmchow commented Mar 29, 2026

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ 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".

@tmchow tmchow merged commit ca78057 into main Mar 29, 2026
2 checks passed
@github-actions github-actions Bot mentioned this pull request Mar 29, 2026
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.

Bring back the standalone deepen command

1 participant