pactkit-reviewlisted

# PactKit Review Structured PR code review with severity-ranked findings. ## When Invoked - **Check Phase 4** (PR variant): When `/project-check` is given a PR number/URL. - **Sprint Stage B**: As part of automated QA in Sprint orchestration. ## Severity Levels | Level | Name | Action | |-------|------|--------| | **P0** | Critical | Must block merge | | **P1** | High | Should fix before merge | | **P2** | Medium | Fix in PR or follow-up | | **P3** | Low | Optional improvement | ## Protocol ### 1. PR Information - Fetch PR metadata: `gh pr view $ARG --json title,body,author,baseRefName,headRefName,files` - Fetch PR diff: `gh pr diff $ARG` - Extract STORY-ID from title/body if present. ### 2. Review Checklists - **SOLID**: SRP, OCP, LSP, ISP, DIP analysis on changed files. - **Security**: OWASP baseline (injection, auth, secrets, XSS, SSRF). - **Quality**: Error handling, performance, boundary conditions, logic correctness. ### 3. Report ``` ## Code Review: PR $ARG **Result**: APPROVE / REQUEST_CHANGES ### Issues - [P0] [file:line] Description - [P1] [file:line] Description ### Spec Alignment - [x] R1: Implemented - [ ] R2: Missing ``` > **CONSTRAINT**: This skill is read-only. Do not modify code files.