Skip to content
cccskills
FreePublished on GitHub

code-review

Perform structured code reviews focusing on correctness, readability, security, and maintainability. Use this skill when reviewing pull requests, evaluating code changes, or establishing review standards for a team.

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

See how to install

Files included (1)

  • SKILL.md2.8 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

Code Review

Perform structured, actionable code reviews.

When to Use

  • Reviewing a pull request or set of changes
  • Evaluating code quality before merging
  • Auditing a module for technical debt
  • Establishing review standards or checklists

Review Dimensions

Review code across these dimensions, in priority order:

1. Correctness

  • Does the code do what it claims?
  • Are edge cases handled? (null, empty, overflow, concurrency)
  • Are error paths tested?
  • Do tests cover the changed behavior?

2. Security

  • Input validation at system boundaries
  • No secrets in code (API keys, passwords, tokens)
  • SQL/command injection prevention
  • Proper authentication and authorization checks
  • See security-audit skill for deeper analysis

3. Readability

  • Clear naming (variables, functions, classes)
  • Functions do one thing
  • No deep nesting (max 3 levels)
  • Comments explain "why", not "what"
  • Consistent style with the surrounding codebase

4. Maintainability

  • No unnecessary abstractions
  • DRY without over-abstraction (rule of three)
  • Dependencies are justified
  • Breaking changes are flagged

5. Performance

  • Only flag when there is a real concern (hot path, large data, N+1 queries)
  • Do not micro-optimize unless the context demands it

Review Output Format

Structure feedback as:

## Review: <PR title or file>

### Must Fix
- [ ] **file.py:42** — [Correctness] Description of the issue and suggested fix

### Should Fix
- [ ] **file.py:78** — [Readability] Description and suggestion

### Consider
- [ ] **file.py:100** — [Performance] Optional improvement

### Positive
- file.py:15 — Good use of context manager for resource cleanup

Severity levels:

LevelMeaningMerge?
Must FixBug, security issue, or broken contractBlock
Should FixSignificant readability/maintainability concernRequest changes
ConsiderOptional improvement, style preferenceApprove with comment
PositiveGood patterns worth highlighting-

Guidelines

  1. Be specific - Point to exact lines, suggest concrete alternatives
  2. Explain why - "This could cause X because Y", not just "change this"
  3. Separate style from substance - Automate style (linters); review logic manually
  4. Limit scope - Review what changed, not the entire file (unless asked)
  5. Acknowledge good work - Include at least one positive observation
  6. Propose, don't impose - "Consider using X" not "You must use X" (unless it's a Must Fix)

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

Design, review, and improve Claude/Codex skills based on Anthropic's "The Complete Guide to Building Skills for Claude". Use when creating a new skill, rewriting SKILL.md frontmatter and workflows, fixing under-triggering or over-triggering, designing scripts/references/assets, building test cases, or preparing a skill for upload/distribution.

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

s-hiraoku/vscode-sidebar-terminal22Updated Oct 17, 2026

This skill enables web searching using Brave Search API directly via curl. Use when searching for current information, news, articles, or web content. MCP server not required - calls API directly.

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

s-hiraoku/vscode-sidebar-terminal22Updated Oct 17, 2026

Run code quality checks (ruff, mypy, pytest) and optionally simplify code. This skill should be used when the user wants to check code quality, run linters, run tests, or simplify recently modified code. Triggered by /lint, /check, or /code-quality commands.

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

s-hiraoku/vscode-sidebar-terminal22Updated Oct 17, 2026

Simplifies and refines code for clarity, consistency, and maintainability while preserving all functionality. This skill should be used after implementing features or bug fixes to clean up code, when refactoring for readability, or when reducing unnecessary complexity. It launches the code-simplifier subagent provided by the code-simplifier@claude-plugins-official plugin.

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

s-hiraoku/vscode-sidebar-terminal22Updated Oct 17, 2026

Simplifies and refines code for clarity, consistency, and maintainability while preserving all functionality. Focuses on recently modified code unless instructed otherwise. This skill should be used when code-quality checks pass but the code would benefit from structural cleanup — deduplication, branching simplification, naming improvements, or dead-code removal. Invoked as a subagent from /code-quality or directly via the Task tool.

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

s-hiraoku/vscode-sidebar-terminal22Updated Oct 17, 2026

commit

Free

Create a well-formed git commit from current changes using session history for rationale and summary; use when asked to commit, prepare a commit message, or finalize staged work.

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

s-hiraoku/vscode-sidebar-terminal22Updated Oct 17, 2026

See all skills by s-hiraoku

Report a problem with this skill