Skip to content

fix(plan): remove deprecated /technical_review references#258

Merged
tmchow merged 1 commit intoEveryInc:mainfrom
mvanhorn:fix/remove-technical-review-refs
Mar 13, 2026
Merged

fix(plan): remove deprecated /technical_review references#258
tmchow merged 1 commit intoEveryInc:mainfrom
mvanhorn:fix/remove-technical-review-refs

Conversation

@mvanhorn
Copy link
Contributor

Summary

  • Remove all references to the deprecated /technical_review command from ce:plan and deepen-plan skill files
  • Renumber menu options after removal to keep numbering sequential
  • The /technical_review command was removed in v2.32.0 but 6 references remained in the post-generation menus and selection handlers

Why removal instead of replacement

The closed PR #247 took the wrong approach by replacing /technical_review with /ce:review. That's incorrect because /ce:review is a code review tool (for reviewing PRs/diffs), not a plan review tool. The plan menu already has the correct tool for plan review - "Review and refine" (option 4, now option 3), which loads the document-review skill. There's nothing to replace these references with - they just need to be removed.

Supersedes #247.
Fixes #244

Changes

plugins/compound-engineering/skills/ce-plan/SKILL.md (4 removals):

  • Removed option 3 (/technical_review) from post-generation menu, renumbered remaining options
  • Removed /technical_review selection handler
  • Updated loop-back text to remove /technical_review mention
  • Removed /technical_review from issue creation follow-up prompt

plugins/compound-engineering/skills/deepen-plan/SKILL.md (2 removals):

  • Removed option 2 (/technical_review) from post-enhancement menu, renumbered remaining options
  • Removed /technical_review selection handler

Test plan

  • Run /ce:plan and verify the post-generation menu shows correct options without /technical_review
  • Run /deepen-plan and verify the post-enhancement menu shows correct options without /technical_review
  • Verify "Review and refine" (document-review) still works as the plan review option

🤖 Generated with Claude Code

The /technical_review command was removed in v2.32.0 but references
remained in ce:plan and deepen-plan menus. Remove them entirely -
the existing 'Review and refine' option (document-review) already
covers plan review.

Fixes EveryInc#244

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@tmchow tmchow merged commit a8f4909 into EveryInc:main Mar 13, 2026
1 check passed
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.

Deprecated /technical_review command still referenced in /ce:plan prompt

2 participants