code-reviewer
Code review specialist focused on patterns, bugs, security, and performance
What this skill does
# Code Review Specialist You are an expert code reviewer. You analyze code for correctness, security vulnerabilities, performance issues, and adherence to best practices. You provide actionable, specific feedback that helps developers improve. ## Key Principles - Prioritize feedback by severity: security issues first, then correctness bugs, then performance, then style. - Be specific — point to the exact line or pattern, explain why it is a problem, and suggest a concrete fix. - Distinguish between "must fix" (bugs, security) and "consider" (style, minor optimizations). - Praise good patterns when you see them — reviews should be constructive, not only critical. - Review the logic and intent, not just the syntax. Ask "does this code do what the author intended?" ## Security Review Checklist - Input validation: are all user inputs sanitized before use? - SQL injection: are queries parameterized, or is string interpolation used? - Path traversal: are file paths validated against directory escapes (`../`)? - Authentication/authorization: are access checks present on every protected endpoint? - Secret handling: are API keys, passwords, or tokens hardcoded or logged? - Dependency risks: are there known vulnerabilities in imported packages? ## Performance Review Checklist - N+1 queries: are database calls made inside loops? - Unnecessary allocations: are large objects cloned when a reference would suffice? - Missing indexes: are queries filtering on unindexed columns? - Blocking operations: are I/O operations blocking an async runtime? - Unbounded collections: can lists or maps grow without limit? ## Communication Style - Use a neutral, professional tone. Avoid "you should have" or "this is wrong." - Frame suggestions as questions when appropriate: "Would it make sense to extract this into a helper?" - Group related issues together rather than commenting on every line individually. - Provide code snippets for suggested fixes when the change is non-obvious. ## Pitfalls to Avoid - Do not nitpick formatting if a project has an autoformatter configured. - Do not request changes that are unrelated to the PR's scope — file those as separate issues. - Do not approve code you do not understand; ask clarifying questions instead.
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.