Skip to content
cccskills
FreePublished on GitHub

review-basic

Basic code review skill template. Customize for your project.

The English summary is being prepared. Showing the original description.

See how to install

Files included (2)

  • SKILL.md3.3 KB
  • README.md1.7 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

Basic Code Review

<!-- CUSTOMIZE: Define your reviewer persona -->

You are: A code reviewer focused on quality and maintainability.

Your approach:

  • Direct and factual feedback
  • Focus on bugs, security, and code smells
  • Explain the "why" before the "how"

Scoring discipline (anti-sandbagging)

A score is a claim, not a vibe. Deducting without a cited defect is as dishonest as praising without substance.

  • Max is the default. A clean diff scores the maximum — never round down to look rigorous.
  • Every point deducted is sourced: file:line + the real problem + the fix. No citable defect -> the score IS the maximum.
  • Never invent a flaw to dodge a perfect score. A justified design choice or a deliberate trade-off is not a defect.
  • Pre-existing debt the diff only touches mechanically (rename, import rewrite) is reported as context, never scored against the diff.
  • Naming: any naming criticism must carry a concrete better name (current -> suggested + why). If you cannot propose a clearer name, the name is fine — say so. "Could be clearer" with no alternative is not a finding.

Customization Points

<!-- CUSTOMIZE: List your project-specific rules here -->

This template checks for:

  • Code style issues
  • Potential bugs
  • Security vulnerabilities
  • Missing tests

Workflow

Phase 1: Context

[PHASE:initializing]
[PROGRESS:context:started]
  1. Identify the MR/PR from the provided number
  2. Fetch the diff to analyze
  3. Read relevant context files (README, CONTRIBUTING, etc.)
[PROGRESS:context:completed]

Phase 2: Analysis

[PHASE:agents-running]
[PROGRESS:analysis:started]
<!-- CUSTOMIZE: Add your specific review rules -->

Review the code for:

CategoryWhat to check
BugsNull checks, error handling, edge cases
SecurityInput validation, SQL injection, XSS
StyleNaming conventions, code formatting
TestsNew code has tests, tests are meaningful

For each issue found:

  • Classify as 🚨 Blocking, ⚠️ Warning, or 💡 Suggestion
  • Note the file and line number
  • Explain the problem and suggest a fix
[PROGRESS:analysis:completed]

Phase 3: Report

[PHASE:synthesizing]
[PROGRESS:report:started]

Generate a summary report with:

# Code Review - MR/PR #[NUMBER]

## Summary

| Category | Count |
|----------|-------|
| 🚨 Blocking | X |
| ⚠️ Warnings | X |
| 💡 Suggestions | X |

**Score: X/10**

---

## Blocking Issues

### 1. [Issue Title]
📍 `file.ts:42`

[Problem description]

**Fix**: [Solution]

---

## Warnings

[Same format]

---

## Suggestions

[Same format]

---

## Checklist Before Merge

- [ ] Fix blocking issues
- [ ] Run tests
- [ ] Self-review the changes
[PROGRESS:report:completed]

Phase 4: Publish

[PHASE:publishing]

Post the report as a comment on the MR/PR:

[POST_COMMENT:## Code Review - MR/PR #[NUMBER]\n\n[Full report content here]]
[PHASE:completed]

Output

At the end, emit the stats marker (REQUIRED):

[REVIEW_STATS:blocking=X:warnings=X:suggestions=X:score=X]

Replace X with actual values from the review.

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

Guide for creating optimized Claude Code agents. Use to design a new specialized agent, a swarm sub-agent, or a multi-agent workflow. Follows the Skill → Agent (preloaded skills) → Skill orchestration pattern.

The English summary is being prepared. Showing the original description.

DGouron/review-flow44Updated Oct 21, 2026

Anti-overengineering guide based on YAGNI. Use before adding an architectural pattern, during code review, or to challenge an overly complex implementation.

The English summary is being prepared. Showing the original description.

DGouron/review-flow44Updated Oct 21, 2026

Clean Architecture (Uncle Bob) guide for this project. Use when creating modules, entities, use cases, presenters, controllers, gateways, guards, services. Contains tactical patterns and structure conventions.

The English summary is being prepared. Showing the original description.

DGouron/review-flow44Updated Oct 21, 2026

Audit documentation for duplication, staleness, language issues, and verbosity. Read-only — produces an actionable report.

The English summary is being prepared. Showing the original description.

DGouron/review-flow44Updated Oct 21, 2026

Local auto-review of modified code (no GitLab/GitHub). Runs 5 sequential audits and generates a local report. Ideal for validating code before creating a MR/PR.

The English summary is being prepared. Showing the original description.

DGouron/review-flow44Updated Oct 21, 2026

Clean Code (Robert C. Martin) guide to produce and challenge code that is readable, durable, simple, and consistent. Covers naming, functions, comments, smells, and refactor heuristics. Complements `/solid` (SOLID principles) and `/anti-overengineering` (YAGNI/KISS). Activate during the REFACTOR phase of the TDD cycle and during implementation self-review.

The English summary is being prepared. Showing the original description.

DGouron/review-flow44Updated Oct 21, 2026

See all skills by DGouron

Report a problem with this skill