Skip to content
cccskills
FreePublished on GitHub

review-plan

Review an implementation plan before coding across architecture, code quality, tests, and performance. Compare concrete options for each issue, explain effort and risks, and choose a direction through an interactive review.

OriginalShow the original description

Structured plan review across 4 axes before writing any code (inspired by Garry Tan's workflow)

See how to install

Good for

  • Review a specification before implementation
  • Check component boundaries and dependencies
  • Find missing tests and failure scenarios
  • Compare fixes by effort, risk, and maintenance

Files included (1)

  • SKILL.md3.5 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

Review plan before implementation

Review the current plan thoroughly before making any code changes. For every issue or recommendation, explain the concrete tradeoffs, give an opinionated recommendation, and ask for user input before assuming a direction.

Engineering preferences

Use these to guide your recommendations (override with project-specific CLAUDE.md preferences if they exist):

  • DRY is important: flag repetition aggressively
  • Well-tested code is non-negotiable: prefer too many tests over too few
  • Code should be "engineered enough": not under-engineered (fragile, hacky) and not over-engineered (premature abstraction, unnecessary complexity)
  • Err on the side of handling more edge cases, not fewer
  • Bias toward explicit over clever; thoughtfulness over speed

Review pipeline

Work through each section sequentially. After each section, pause and ask for feedback before moving on.

1. Architecture review

Evaluate:

  • Overall system design and component boundaries
  • Dependency graph and coupling concerns
  • Data flow patterns and potential bottlenecks
  • Scaling characteristics and single points of failure
  • Security architecture (auth, data access, API boundaries)

2. Code quality review

Evaluate:

  • Code organization and module structure
  • DRY violations (be aggressive here)
  • Error handling patterns and missing edge cases (call these out explicitly)
  • Technical debt hotspots
  • Areas that are over-engineered or under-engineered relative to engineering preferences

3. Test review

Evaluate:

  • Test coverage gaps (unit, integration, e2e)
  • Test quality and assertion strength
  • Missing edge case coverage (be thorough)
  • Untested failure modes and error paths

4. Performance review

Evaluate:

  • N+1 queries and database access patterns
  • Memory-usage concerns
  • Caching opportunities
  • Slow or high-complexity code paths

Issue reporting format

For every specific issue found (bug, smell, design concern, or risk):

  1. Describe the problem concretely, with file and line references
  2. Present 2-3 options, including "do nothing" where that's reasonable
  3. For each option, specify: implementation effort, risk, impact on other code, and maintenance burden
  4. Give your recommended option and why, mapped to engineering preferences above
  5. Ask explicitly whether the user agrees or wants to choose a different direction before proceeding

Workflow

  • Do not assume priorities on timeline or scale
  • After each section, pause and ask for feedback before moving on
  • Use AskUserQuestion for structured option selection

Before starting

Ask if the user wants one of two options:

  1. BIG CHANGE: Work through this interactively, one section at a time (Architecture → Code Quality → Tests → Performance) with at most 4 top issues in each section
  2. SMALL CHANGE: Work through interactively ONE question per review section

Tips

  • Combine with .claude/rules/ files for project-specific review criteria
  • Engineering preferences above can be overridden by your project's CLAUDE.md
  • For deeper analysis, use this command with Opus model

Sources

$ARGUMENTS

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

Evaluate Claude Code agents, skills, and commands with weighted quality criteria. Get scores, readiness grades, template comparisons, and practical improvement suggestions in Markdown or JSON reports to guide reviews before release.

  • Review skills before release
  • Compare agent definitions with reference templates
  • Quickly check critical quality criteria
FlorianBruniaux/claude-code-ultimate-guide6.2KUpdated Oct 18, 2026

audit-codebase

FreeSummary

Assess a codebase across seven areas, including exposed credentials, security, dependencies, structure, and tests. Receive category scores, findings tied to file locations, and a prioritized improvement plan with smaller tasks that can be completed quickly.

  • Assessing the overall health of an inherited project
  • Reviewing exposed credentials and security risks
  • Identifying dependency and test coverage gaps
FlorianBruniaux/claude-code-ultimate-guide6.2KUpdated Oct 18, 2026

autoresearch

FreeSummary

Measure code patterns such as casts to any or disabled lint rules, then run repeated, targeted edits. Keep changes that improve the chosen metric, revert unsuccessful changes, and track results for each improvement loop.

  • Finding code patterns to prioritize for cleanup
  • Preparing measurement files for an improvement loop
  • Repeating targeted edits to reduce casts to any
FlorianBruniaux/claude-code-ultimate-guide6.2KUpdated Oct 18, 2026

best-of-n

FreeSummary

Generate several separate solutions, compare them against criteria fixed in advance, and verify the selected result through executable checks or an independent reviewer. Keep a proof log documenting candidates, evidence, failures, and remaining limits.

  • Comparing implementation options before choosing one
  • Verifying a solution when a wrong choice would be costly
  • Keeping evaluation evidence for delivery or audit
FlorianBruniaux/claude-code-ultimate-guide6.2KUpdated Oct 18, 2026

canary

FreeSummary

Monitor a live application after deployment and compare it with a saved baseline, a record of its earlier state. Check page responses, loading times, and new errors, flag persistent changes, and produce a per-page health report.

  • Recording a healthy baseline before deployment
  • Monitoring production immediately after deployment
  • Checking key pages for slower responses or failures
FlorianBruniaux/claude-code-ultimate-guide6.2KUpdated Oct 18, 2026

catchup

FreeSummary

Rebuild project context after clearing a conversation or returning to work. Review recent Git history, uncommitted changes, and TODO markers to get a structured summary of project status and suggested next steps.

  • Restoring project context after /clear
  • Reviewing recent changes after a long break
  • Checking unfinished work before choosing the next task
FlorianBruniaux/claude-code-ultimate-guide6.2KUpdated Oct 18, 2026

See all skills by FlorianBruniaux

Report a problem with this skill