Skip to content

feat: add /review-plan Claude Code slash command#1

Closed
vineus wants to merge 1 commit intotomasz-tomczyk:mainfrom
vineus:vdl/feat-add-review-plan-claude-code
Closed

feat: add /review-plan Claude Code slash command#1
vineus wants to merge 1 commit intotomasz-tomczyk:mainfrom
vineus:vdl/feat-add-review-plan-claude-code

Conversation

@vineus
Copy link
Copy Markdown

@vineus vineus commented Feb 18, 2026

Summary

  • Adds a /review-plan Claude Code slash command (.claude/commands/review-plan.md) that automates the crit review loop: finds the plan file, launches crit in the background, reads review comments, and addresses them by editing the plan
  • Adds a "Claude Code Integration" section to the README documenting the command

Details

The slash command supports multiple ways to find the plan file:

  1. Current session context (plan mode)
  2. User argument (/review-plan my-plan.md)
  3. Most recent file in ~/.claude/plans/
  4. Markdown files in the current directory

This turns plan review into a tight loop: comment in the browser, Claude addresses feedback, file change triggers live reload, review again.

Test plan

  • Clone the repo and verify /review-plan appears as a slash command in Claude Code
  • Run /review-plan with a plan file and confirm the full workflow works (crit launches, comments are read, plan is updated)

@vineus vineus marked this pull request as ready for review February 18, 2026 11:19
@tomasz-tomczyk tomasz-tomczyk force-pushed the main branch 2 times, most recently from 86c7b81 to 54a470c Compare February 22, 2026 13:19
@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Thanks so much for contributing! I've taken what you shared as a starting point and edited it to be up to date with latest changes (mainly crit go which notifies the browser the agent is done)

Implemented in 54a470c so I'll close this PR but if you have further improvements for it I'd love to see it! ❤️ 💚 💛 🧡

@vineus
Copy link
Copy Markdown
Author

vineus commented Feb 25, 2026

Thank you so much for including this!

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.

2 participants