review

Analyze PR diffs for SQL safety, LLM trust boundary violations, and conditional side effects.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Pre-landing PR review analyzes the diff against the base branch for SQL safety, LLM trust boundary violations, conditional side effects, and other structural issues. It proactively suggests when the user is about to merge or land code changes.

Core Features & Use Cases

  • Analyzes pull request diffs for SQL safety, LLM trust boundary issues, and potential side effects.
  • Proactively surfaces risks and offers actionable guidance before merging.
  • Supports prompts like "review this PR", "code review", or "pre-landing review" to guide the analysis.

Quick Start

Provide the PR diff to the skill and let it run its checks to surface issues and recommendations.

Frequently Asked Questions about review

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

FAQPage Schema
How do I analyze a pull request diff for SQL safety issues before merging?

To analyze a pull request diff for SQL safety issues, you provide the PR diff to a pre-landing review process that surfaces injection risks and structural issues before merging. It proactively detects vulnerabilities and offers actionable fix suggestions.

What is a pre-landing code review for LLM trust boundary violations?

A pre-landing code review for LLM trust boundary violations analyzes your PR diff to detect unauthorized model permissions and conditional side effects. It proactively surfaces security risks before code changes are merged into the base branch.

Can I use automated diff analysis to detect conditional side effects in a pull request?

Yes, you can use automated diff analysis to detect conditional side effects in a pull request. By evaluating the PR diff against the base branch, the review process identifies potential side effects and enforces proactive risk detection.

Does pre-landing code review work without external dependencies or components?

Pre-landing code review works without external dependencies or components. You simply provide the PR diff, and the standalone analysis mechanism runs its checks to surface SQL safety issues, LLM trust boundary violations, and structural risks.

Why should I run a pre-landing review instead of a standard post-merge code review?

You should run a pre-landing review instead of a standard post-merge code review because it analyzes PR diffs against the base branch to surface risks and offer actionable guidance before merging. This prevents vulnerable code from landing in the main repository.

What is the best way to perform a pre-landing PR review for merge readiness?

The best way to perform a pre-landing PR review for merge readiness is to supply the PR diff for automated analysis. This process surfaces SQL safety issues, LLM trust boundary violations, and conditional side effects to ensure safer code integration.