shipshapelisted

# Skill: /shipshape ## Purpose Simulate a rigorous plan review where two senior engineers examine an implementation plan. This skill assumes the plan was written by a junior developer (or an AI agent) and provides constructive, thorough feedback to catch issues BEFORE implementation begins. ### Review Personas 1. **10x Staff Developer** — Focuses on implementation elegance, code patterns, duplication avoidance, and practical execution 2. **10x Principal Developer** — Focuses on architecture, schema flows, logic correctness, and system-wide implications ## When to Use - Before starting implementation of any plan - After writing a new implementation plan (self-review) - When inheriting a plan from another developer - User explicitly requests a plan review ## Workflow ### Step 0: Mandatory Plan Validation (FIRST) **Before any review, validate the plan against baseline requirements.** Run this checklist on the plan: ``` PLAN VALIDATION CHECKLIST ═════════════════════════ □ Git Workflow section exists? → Branch name specified? → Checkpoint strategy defined? □ Implementation Waves section exists? → Agents / owners assigned? → File paths specified? → PARALLEL / SEQUENTIAL marked? □ Documentation Updates section exists? → Changelog / release-notes checkbox? □ Success Criteria section exists? → Type check requirement? → Build requirement? → Test requirement? ═════════════════════════ ``` **If plan is INCOMPLETE (missing required sections):** 1. Silen