review

Analyze repository diffs to identify defects, security risks, and structural problems.

Updated May 5, 2026
One-click install
npx skills add https://github.com/yanochka11/harness_bro --skill review-yanochka11
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/yanochka11/harness_bro/tree/main/.claude/skills/curated/gstack/review
Command: npx skills add https://github.com/yanochka11/harness_bro --skill review-yanochka11

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill helps engineers detect structural issues, security risks, and correctness problems in code changes before merging or landing a pull request.

Core Features & Use Cases

  • Pre-Landing Code Review: Analyze diffs against a base branch and identify SQL safety issues, trust boundary problems, side effects, and maintainability concerns.
  • Specialized Review Checks: Evaluate changes across security, performance, API contracts, migrations, testing, and design-related concerns.
  • Use Case: A developer preparing a pull request can use this Skill to inspect the diff, find actionable issues, and apply fixes before reviewers or users encounter defects.

Quick Start

Use the review skill to check my current diff for issues before I merge this pull request.

Frequently Asked Questions about review

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

FAQPage Schema
How do I analyze a git diff for security risks before merging a pull request?

To analyze a git diff for security risks, use pre-landing code review to inspect changes against a base branch. This process identifies SQL safety issues, trust boundary problems, and side effects to ensure implementation safety before deployment.

What is pre-landing code review and when do I need it?

Pre-landing code review is the process of evaluating code changes for defects and structural problems before merging. You need it when preparing a pull request to find actionable issues and apply fixes before reviewers or users encounter defects.

How do I check my current diff for correctness and maintainability issues?

Check your current diff by applying specialized review checks across security, performance, API contracts, migrations, testing, and design. This validates implementation correctness and maintainability by inspecting the repository diff before deployment.

Can I detect SQL safety and trust boundary problems in my code changes automatically?

You can detect SQL safety and trust boundary problems by analyzing code changes before landing a pull request. The review process evaluates diffs against a base branch to identify these specific security risks and structural issues.

What types of structural problems can be found during pull request reviews?

During pull request reviews, structural problems found include side effects, maintainability concerns, API contract violations, and design-related issues. The review evaluates changes across security, performance, migrations, and testing to validate implementation safety.

Do I need access to repository diffs to perform a pre-landing code review?

Yes, you need access to repository diffs and code analysis tools to perform a pre-landing code review. This access is required to validate implementation safety, maintainability, and correctness before deployment or merging.