review
Reviews code for security vulnerabilities, performance issues, correctness bugs, and production reliability concerns. Use after writing or modifying code, when reviewing PRs, when the user asks "review this", "check this code", mentions code quality, security review, or performance audit. Covers all languages and frameworks.
What this skill does
# Code Review Security, correctness, and production reliability — in that priority order. ## Priority Order 1. **Security** — SQL injection, XSS, CSRF, auth bypass, secrets in code, input validation 2. **Correctness** — Logic errors, data loss risks, broken error handling, edge cases 3. **Performance** — N+1 queries, memory leaks, blocking operations, missing indexes 4. **Maintainability** — Complexity, duplication, unclear abstractions 5. **Style** — Last, and only if it deviates from team conventions ## Review Checklist **Security (block on any of these):** - [ ] No SQL injection vectors (parameterized queries used) - [ ] No XSS — user input sanitized before rendering - [ ] Auth checks on all sensitive operations - [ ] No secrets or credentials in code - [ ] Input validation at system boundaries **Correctness:** - [ ] Error paths handled — no silent failures - [ ] Edge cases covered (empty, null, zero, large values) - [ ] Async operations properly awaited - [ ] No data mutation where immutability expected **Performance:** - [ ] No N+1 queries in loops - [ ] Database queries have appropriate indexes - [ ] No memory leaks (event listeners, timers cleaned up) - [ ] Expensive operations cached where appropriate **Tests:** - [ ] Tests exist for new code - [ ] Tests cover error paths, not just happy path - [ ] No over-mocking that makes tests pass while real code breaks ## Output Format ``` ## Summary [1-sentence overall assessment] Issues: N critical · N high · N medium · N low ## 🔴 Critical (block) 1. `file:line` — [issue] **Problem:** ... **Fix:** ... ## 🟠 High (should fix) ... ## 🟡 Medium (suggestions) ... ## ✅ What's good ... ## Verdict [Approve / Request Changes / Block] ``` ## What Not To Do - Don't block on style preferences — that's what linters are for - Don't give vague feedback like "this could be better" - Don't approve without understanding the changes - Don't ignore config changes — they cause most production incidents ## References | Priority | Load when | Reference | |----------|-----------|-----------| | 1 — High | Reviewing UI code, component APIs, accessibility, or web interface patterns | `references/web-interface-guidelines.md` |
Related in Security
mac-ops
IncludedComprehensive macOS workstation operations — diagnose kernel panics, identify failing drives, audit launchd startup items, decode wake reasons, triage TCC permission denials, manage APFS snapshots, recover from no-boot. Use for: Mac is slow, slow bootup, won't boot, kernel panic, kernel_task hot, mds_stores CPU, photoanalysisd, cloudd, login loop, gray screen, sleep wake failure, drive failing, IO errors, APFS snapshots eating space, Time Machine local snapshots, Spotlight indexing, launchd, LaunchAgent, LaunchDaemon, login items, TCC permissions, Full Disk Access, Screen Recording denied, Gatekeeper, quarantine, com.apple.quarantine, app is damaged, helper tool, /Library/PrivilegedHelperTools, pmset, wake reasons, dark wake, sysdiagnose, panic.ips, DiagnosticReports, configuration profile, MDM profile, remote diagnostics over SSH.
a11y-audit
IncludedRun accessibility audits on web projects combining automated scanning (axe-core, Lighthouse) with WCAG 2.1 AA compliance mapping, manual check guidance, and structured reporting. Output is configurable: markdown report only, markdown plus machine-readable JSON, or markdown plus issue tracker integration. Use this skill whenever the user mentions "accessibility audit", "a11y audit", "WCAG audit", "accessibility check", "compliance scan", or asks to check a web project for accessibility issues. Also trigger when the user wants to verify WCAG conformance or map findings to a specific standard (CAN-ASC-6.2, EN 301 549, ADA/AODA).
erpclaw
IncludedAI-native ERP system with self-extending OS. Full accounting, invoicing, inventory, purchasing, tax, billing, HR, payroll, advanced accounting (ASC 606/842, intercompany, consolidation), and financial reporting. 413 actions across 14 domains, 43 expansion modules. Constitutional guardrails, adversarial audit, schema migration. Double-entry GL, immutable audit trail, US GAAP.
assess
IncludedAssesses and rates quality 0-10 across multiple dimensions (correctness, maintainability, security, performance, testability, simplicity) with pros/cons analysis. Compares against project conventions and prior decisions from memory. Produces structured evaluation reports with actionable improvement suggestions. Use when evaluating code, designs, architectures, or comparing alternative approaches.
spring-boot-security-jwt
IncludedProvides JWT authentication and authorization patterns for Spring Boot 3.5.x covering token generation with JJWT, Bearer/cookie authentication, database/OAuth2 integration, and RBAC/permission-based access control using Spring Security 6.x. Use when implementing authentication or authorization in Spring Boot applications.
code-hardcode-audit
IncludedDetect hardcoded values, magic numbers, and leaked secrets. TRIGGERS - hardcode audit, magic numbers, PLR2004, secret scanning.