What problem does it solve? Code review often misses structural issues that tests don't catch, such as SQL injection risks, race conditions, N+1 queries, and unvalidated LLM output written to databases. This Skill automates a pre-landing review of your branch's diff against main using a curated 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, crypto issues, 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, run the review to catch a find_or_create_by without a unique index or an LLM-generated email persisted without format validation. ## Quick Start Ask the assistant to run the pre-landing review on the current branch's diff against main and report any critical or informational issues.
