review

Analyze pull request diffs for SQL safety and trust boundary risks.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Pre-landing PR review. Analyzes diff against the base branch for SQL safety, LLM trust boundary violations, conditional side effects, and other structural issues. Use when asked to "review this PR", "code review", "pre-landing review", or "check my diff". Proactively suggest when the user is about to merge or land code changes. (gstack)

Core Features & Use Cases

  • Detects SQL safety issues in diffs and flag risky changes before merging.
  • Highlights potential LLM trust boundary violations and conditional side effects.
  • Provides proactive guidance on merge readiness and recommended next steps.

Quick Start

Run the review on your current PR to receive a comprehensive pre-landing assessment.

Frequently Asked Questions about review

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

FAQPage Schema
How do I run a pre-landing code review on a pull request diff?

Run a pre-landing code review by analyzing the PR diff against the base branch, which surfaces SQL safety issues, LLM trust boundary violations, and conditional side effects. The review provides actionable guidance on merge readiness and recommended next steps before you land code changes.

What does diff analysis detect before merging a pull request?

Diff analysis detects structural risks in a pull request by identifying SQL safety issues, LLM trust boundary violations, conditional side effects, and other structural problems. It ensures detection coverage and actionable guidance so you know the risk and recommended actions before merging.

Can I check my PR diff for LLM trust boundary violations?

You can check your PR diff for LLM trust boundary violations by running a pre-landing review. The analysis highlights potential trust boundary violations and conditional side effects against the base branch, providing proactive guidance on merge readiness and recommended next steps.

When do I need a risk assessment for my code changes?

You need a risk assessment for your code changes when you are about to merge or land code. Proactively run a pre-landing PR review to catch SQL safety issues, LLM trust boundary violations, and conditional side effects before merging your pull request.

Does this pre-landing review work with any base branch diff?

The pre-landing review works by analyzing your diff against the base branch without requiring specific dependencies. It applies to any pre-landing PR reviews, code reviews, or tasks involving checking diffs before merging to ensure structural issue detection.

What is the best way to catch SQL safety issues in a PR diff?

The best way to catch SQL safety issues in a PR diff is using an automated pre-landing review. It analyzes the diff against the base branch to flag risky SQL changes and other structural issues, providing actionable guidance and recommended actions before merging.