feat(ce-plan): approach-altitude plan-for-a-plan with ce-work non-code carve-out#905
Conversation
…ude method-uncertainty
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6e0edcd443
ℹ️ 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".
Regression validation (existing behavior)Ran a fresh-session regression eval covering the existing flows this change touches — 19/19 match pre-change behavior, 0 regressions:
Combined with the earlier proactive-trigger eval — which caught and fixed an over-fire (N4, a costly-but-method-obvious migration, fired 1/12; tightened wording; re-validated 0/8) — the gate is validated for both new behavior and existing-behavior regression. |
What
Adds an approach altitude to
ce-plan: for a hard problem, answer one level up — produce a grounded plan for how the deliverable will be made and hold at a checkpoint before committing, instead of zero-shotting a fragile result. Non-code deliverables flow to a new lightweightce-workcarve-out.How it works
ce-planSKILL.md Phase 0.1a — sits between the resume/deepen fast paths and the domain split, so it's domain-general): the explicit path ("plan for a plan", "don't write it yet — plan the approach") is always honored; a proactive offer fires only when method-uncertainty AND cost-of-getting-it-wrong are both high — a single dismissible line, never a nag.references/approach-altitude.md): light recon (skim, not deep-read) → chat-first approach-plan (file-optional, deepenable) → checkpoint (do it now / save for later).ce-work's normal path; a non-code deliverable is markedexecution: knowledge-workand runs throughce-work's carve-out (references/non-code-execution.md) — or any agent, since the plan stays portable.ce-plannever executes.Validation
bun test tests/frontmatter.test.ts→ 331 pass;bun run release:validate→ in sync.Deferred (see plan)
ce-workcarve-out save/commit behavior beyond write + report.Surfaces changed
ce-plan(SKILL.md gate +approach-altitude.md+executionmarker field + R16 guards),ce-workandce-work-beta(non-code carve-out, kept in parity), and thece-plan/ce-workskill docs.