Skip to content

feat(skills): add deep-plan + super-plan planning skills#214

Closed
Number531 wants to merge 1 commit into
mainfrom
feat/deep-plan-super-plan-skills
Closed

feat(skills): add deep-plan + super-plan planning skills#214
Number531 wants to merge 1 commit into
mainfrom
feat/deep-plan-super-plan-skills

Conversation

@Number531

Copy link
Copy Markdown
Owner

Summary

Tracks two planning skills that were previously local-only / untracked in the working tree, bringing them onto main alongside the rest of the skills catalog.

  • deep-plan (.claude/skills/deep-plan/) — a dynamic Workflow harness: 5 parallel Explore mappers (edit-sites, blast-radius, smoke-surface, integration-surface, conventions-arch) → plan synthesis → 5-lens adversarial audit with per-finding verification, cross-lens dedup, keep-best scoring, and stop-on-no-progress. Returns the best-scoring plan + an honest audit trail. Read-only.
  • super-plan (.agents/skills/super-plan/) — prose planning skill that writes a phased implementation-*.md spec into a pending-updates/ folder.

Review fixes included (deep-plan.workflow.js)

This PR also folds in three corrections from a code review of the workflow:

  1. Null-synthesis guard — if plan synthesis dies, fail loud instead of serializing "null", burning a full audit loop, and returning plan: null as a real artifact. Mirrors the existing map-phase degraded handling.
  2. Verifier drop-safety — a finding is rejected only on an explicit real === false verdict, never on an omitted one, so a partial verifier response can't silently suppress a genuine critical and surface the plan as falsely converged/sound.
  3. Grounding accuracy — removed the phantom src/utils/sdkRouting.js (no such file) → real files; pinned src/schemas/toolEnvelopes.js.

Notes

  • deep-plan.workflow.js REPO_ROOT defaults to a machine-absolute path (/Users/ej/Super-Legal/...) but is overridable via args.repoRoot — documented in the SKILL.md.
  • node --check passes on the workflow script.

🤖 Generated with Claude Code

Two previously local-only planning skills, now tracked:

- deep-plan (.claude/skills/deep-plan/): a dynamic Workflow harness —
  5 parallel Explore mappers (edit-sites, blast-radius, smoke-surface,
  integration-surface, conventions-arch) → plan synthesis → 5-lens
  adversarial audit with finding-verification, cross-lens dedup,
  keep-best scoring, and stop-on-no-progress. Returns the best-scoring
  plan plus an honest audit trail. Read-only.

- super-plan (.agents/skills/super-plan/): prose planning skill that
  writes a phased implementation-*.md spec to a pending-updates folder.

Includes three review fixes to deep-plan.workflow.js:
- guard against null plan synthesis (fail loud instead of auditing "null")
- verifier only drops a finding on an explicit real=false verdict, never
  on an omitted one (prevents falsely "converged"/sound plans)
- corrected stale REPO_CONTEXT grounding (sdkRouting.js -> real files;
  pinned schemas/toolEnvelopes.js)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Number531

Copy link
Copy Markdown
Owner Author

Closing — keeping deep-plan / super-plan skills local-only for now. Will revisit.

@Number531 Number531 closed this Jun 7, 2026
@Number531 Number531 deleted the feat/deep-plan-super-plan-skills branch June 7, 2026 06:33
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