What problem does it solve? Code review often misses structural issues that tests don't catch, such as SQL injection risks, race conditions, and unvalidated LLM output written to databases. This Skill automates a pre-landing review of your branch's diff against main using a structured two-pass checklist. ## Core Features & Use Cases - Two-Pass Review: Runs a CRITICAL pass (SQL & data safety, race conditions, LLM output trust boundary) followed by an INFORMATIONAL pass (dead code, test gaps, magic numbers, view performance). - Structured Findings: Outputs every issue with file:line citations and one-line fixes, classifying them as blocking or non-blocking. - Interactive Fix Workflow: For each critical issue, asks whether to fix it now, acknowledge it, or dismiss it as a false positive, then applies approved fixes. - Use Case: Before merging a feature branch that adds an LLM-generated email field, run the review to catch missing format validation and an N+1 query in the new view code. ## Quick Start Ask the assistant to run the pre-landing review on the current branch's diff against main.
