reviewing-code

Produces detailed code reviews highlighting risks, gaps, and improvements for pull requests and diffs.

22|5|Updated Jan 9, 2018
One-click install
npx skills add https://github.com/tdhopper/dotfiles2.0 --skill reviewing-code-tdhopper
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: reviewing-code
Source: https://github.com/tdhopper/dotfiles2.0/tree/main/.claude/skills/reviewing-code
Command: npx skills add https://github.com/tdhopper/dotfiles2.0 --skill reviewing-code-tdhopper

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Reviewing code changes is time-consuming and error-prone; this skill helps identify gaps in tests, potential edge cases, performance concerns, and architectural issues to ensure PRs meet quality standards beyond linting.

Core Features & Use Cases

  • Focus on substantive quality signals such as test coverage, edge cases, performance, and maintainability, while ignoring formatting and style linting.
  • Provide a structured review checklist for PRs, branch changes, and diffs, covering completeness, tests, complexity, performance, duplication, and side effects.
  • Support optional multi-model augmentation by leveraging external analysis tools when available, with results clearly integrated into the final review.

Quick Start

Review the latest pull request diff to produce a comprehensive quality review with identified gaps and recommended fixes.

Frequently Asked Questions about reviewing-code

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review a pull request for code quality and potential side effects?

To review a pull request for code quality, you assess code changes for completeness, risk, tests, performance, and side effects. This process identifies gaps in test coverage, complexity, and architectural issues to ensure changes meet quality standards beyond basic linting.

What is the best way to automate code reviews for branch diffs?

The best way to automate code reviews for branch diffs is to apply a structured review checklist that evaluates tests, complexity, performance, and duplication. This approach focuses on substantive quality signals while ignoring formatting and style linting to produce a comprehensive review.

Does this code review approach work across different programming languages and frameworks?

Yes, this code review approach works across different programming languages and frameworks. It assesses unmerged changes, branch diffs, and pull requests by focusing on universal quality signals like edge cases, maintainability, and potential side effects regardless of the specific tech stack.

Can I use multi-model synthesis to augment code reviews?

Yes, you can use optional multi-model synthesis to augment code reviews. This leverages external analysis tools when available, clearly integrating their results into the final review to identify gaps in tests and adherence to project guidelines.

When should I not use automated code review for a pull request?

You should not use automated code review if your goal is to enforce formatting and style linting, as this approach explicitly ignores those aspects. It is designed strictly for identifying substantive issues like performance concerns, edge cases, and architectural risks.

How do I identify gaps in tests during a code review?

To identify gaps in tests during a code review, you apply a structured checklist to the pull request diff that specifically targets test coverage and edge cases. This highlights missing test scenarios and ensures the changes satisfy requirements for a thorough review.