Automated summary
Performs a comprehensive code review using all 30 review checklists via a senior specialist agent.
Strengths
- Thorough coverage of all aspects (logic, security, architecture, performance, UX)
- Systematic approach with dedicated checklists
- Catches diverse and complex issues through specialized agent
Limitations
- Can be slow and expensive in context (many files processed)
- Risk of false positives or noise if checklists are not tailored to the project
- Requires initial setup of checklists
Use this review before a major merge or release for deep quality assurance.
Avoid for minor changes or when you need quick feedback, as the process is heavy.
Security analysis
SafeThe skill instructs an AI coding agent to perform a code review using a set of checklists. It does not involve any dangerous operations, network access, or data exfiltration. All actions are limited to reading files and running safe git commands.
No concerns found
How this analysis is madeExamples
Run a comprehensive code review on all changes in this branch using all 30 review checklists via the senior-review-specialist agent.Review the file src/auth/login.ts using all 30 checklists. Apply the senior-review-specialist agent and provide a full report with critical issues, warnings, and suggestions.name: review:all description: Comprehensive code review using all 30 review checklists. Spawns the senior-review-specialist agent for thorough file-by-file analysis.
Comprehensive Code Review
Run a thorough review using ALL 30 review checklists via the senior-review-specialist agent.
Instructions
Spawn the senior-review-specialist agent to perform this review.
Checklists to Apply
Load and apply ALL of these review checklists:
Correctness & Logic
commands/review/correctness.md- Logic flaws, broken invariants, edge-case failurescommands/review/backend-concurrency.md- Race conditions, atomicity, locking, idempotencycommands/review/refactor-safety.md- Semantic drift, behavior equivalence
Security & Privacy
commands/review/security.md- Vulnerabilities, insecure defaults, missing controlscommands/review/infra-security.md- IAM, networking, secrets, configurationcommands/review/privacy.md- PII handling, data minimization, compliancecommands/review/supply-chain.md- Dependency risks, lockfiles, build integritycommands/review/data-integrity.md- Data correctness over time, failures, concurrency
Architecture & Design
commands/review/architecture.md- Boundaries, dependencies, layeringcommands/review/performance.md- Algorithmic efficiency, N+1 queries, bottleneckscommands/review/scalability.md- Load handling, dataset growth, multi-tenancycommands/review/api-contracts.md- Stability, correctness, consumer usabilitycommands/review/maintainability.md- Readability, change amplificationcommands/review/overengineering.md- Unnecessary complexity, YAGNI violations
Infrastructure & Operations
commands/review/infra.md- Deployment config, least privilege, operational claritycommands/review/ci.md- Pipeline security, deployment safetycommands/review/release.md- Versioning, rollout, migration, rollbackcommands/review/migrations.md- Database migration safetycommands/review/reliability.md- Failure modes, partial outagescommands/review/logging.md- Secrets exposure, PII leaks, wide-eventscommands/review/observability.md- Logs, metrics, tracing, alertabilitycommands/review/cost.md- Cloud infrastructure cost implications
Quality & Testing
commands/review/testing.md- Test quality, coverage, reliabilitycommands/review/style-consistency.md- Codebase style, idiomscommands/review/docs.md- Documentation completeness and accuracy
User Experience
commands/review/accessibility.md- Keyboard, assistive technology, ARIAcommands/review/frontend-accessibility.md- SPA-specific accessibilitycommands/review/frontend-performance.md- Bundle size, rendering, latencycommands/review/ux-copy.md- User-facing text clarity, error recoverycommands/review/dx.md- Developer experience, onboarding
Agent Instructions
The agent should:
- Get working tree changes: Run
git diffto see all changes - For each changed file:
- Read the full file content
- Go through each diff hunk
- Apply ALL 30 checklists to the changes
- Trace problems to their root cause
- Cross-reference related files: Follow imports, check callers
- Find ALL issues: Be thorough, expect to find many issues
Output Format
Generate a comprehensive review report with:
- Critical Issues: Blocking problems (must fix)
- Warnings: Should address before merge
- Suggestions: Improvements to consider
- File Summary: Issues per file with counts by severity
- Overall Assessment: Ship/Don't Ship recommendation with rationale
From the file to a first result
The download contains exactly the content displayed on this page. This copy may differ from the latest version in the source repository.
View the source and supporting filesThis download contains SKILL.md only. Any scripts, references, templates and dependencies are not included. Get the required files from the source, preserving their directory structure, before continuing.
1. Download the file
Download SKILL.mdFrom the root of a test project, save the file as .skill-downloads/comprehensive-code-review-mmbizkk8/SKILL.md, or use the command below. This folder is for inspecting the skill before installation.
Command for macOS/Linux or WSL: requires a POSIX shell and curl. On Windows without WSL, use the download button and create the folder manually.
mkdir -p '.skill-downloads/comprehensive-code-review-mmbizkk8' && curl --fail --show-error --location 'https://skills-guides.com/api/skills/comprehensive-code-review-mmbizkk8/download' --output '.skill-downloads/comprehensive-code-review-mmbizkk8/SKILL.md.part' && mv '.skill-downloads/comprehensive-code-review-mmbizkk8/SKILL.md.part' '.skill-downloads/comprehensive-code-review-mmbizkk8/SKILL.md'2. Check prerequisites
Read SKILL.md before using it. If it references scripts or external services, inspect them and prepare the required access in your test environment.
No recognized prerequisites section in this file. This does not mean it is self-contained: libraries, tools and access requirements may be mentioned elsewhere in the content or source repository.
The platforms on this page are catalog indications. The paths below follow tool documentation; they do not prove this skill has been run in those tools. No minimum tool version is confirmed for this listing.
3. Load the skill
This file does not provide a usable name/description header for this native procedure. It remains available to download and inspect; use the manual method below.
Manual method or another tool: attach SKILL.md to a conversation, or open it in an agent that can read files. Include any required supporting resources. No universal folder is recognized by every tool.
4. Try a first task
Use a short, synthetic input: an API endpoint, a small document or an interface brief. In a new conversation, attach the file or give the agent its path, then adapt this request:
Read .skill-downloads/comprehensive-code-review-mmbizkk8/SKILL.md and apply its instructions to this example: [describe a test input]. Identify any missing files, dependencies or access. Produce the result requested by the skill without inventing data or running unreviewed scripts.Expected result: output matching the skill instructions, or a precise list of missing prerequisites. A read confirmation alone does not validate a first use. If an external resource is unavailable, that path remains untested.
ClickHouse Performance Comparison
Testing
Evaluate ClickHouse performance test results, check PR performance changes, compare with history, and assess whether a result is actionable.
Analyze CI Performance Reports
Testing
Analyze CI performance comparison reports for a ClickHouse PR. Cross-references master history to distinguish real regressions from flaky tests.
Santa Method
Testing
Multi-agent adversarial verification with two independent reviewers. Output is checked twice and iteratively fixed until both pass before shipping.