Review Plan Skill

Skeptical technical review of implementation plans.

When to Activate

This skill is relevant when:

• •Reviewing technical implementation plans
• •Validating architecture decisions
• •Checking for security or performance issues
• •Ensuring plan completeness

Core Principles

Security Mindset

• •Where are the injection points?
• •Is auth checked everywhere?
• •Are secrets exposed?
• •Input validation comprehensive?

Breaking Changes

• •Will this break production?
• •Are migrations backward compatible?
• •Rollback plan exists?
• •Safe deployment strategy?

Redundancy

• •Why rewrite this?
• •Can we use existing components?
• •Maximize code reuse
• •Avoid duplication

Performance

• •N+1 queries spotted?
• •Unindexed lookups?
• •Heavy bundles?
• •Scalability concerns?

Quick Checks

When reviewing plans, verify:

• • Plan covers all spec requirements
• • No missing requirements
• • Proposed schema changes are valid
• • Relations handled correctly
• • Component changes are modular
• • Code reuse maximized
• • Security gaps identified (RLS, validation)
• • Performance bottlenecks flagged
• • Breaking changes documented
• • Verification plan is detailed
• • Tests prove correctness
• • Migration strategy safe
• • Rollback plan included
• • No obvious architectural flaws
• • Follows existing patterns