review

Analyze diffs against the base branch for SQL safety and structural risks.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Pre-landing PR reviews are manual, error-prone and slow. This Skill automates the analysis of diffs against the base branch to surface issues such as SQL safety risks, LLM trust boundary violations, conditional side effects, and other structural problems before code lands.

Core Features & Use Cases

  • Automatically analyze diffs for SQL safety, LLM trust boundaries, and side effects before merge.
  • Flag potential merge blockers and suggest concrete remediation steps for engineering teams.
  • Support proactive guidance during reviews and PR discussions to improve ship confidence.

Quick Start

Describe the PR to review and run the /review workflow to start the pre-landing review.

Frequently Asked Questions about review

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

FAQPage Schema
How do I automate pre-landing PR reviews for SQL safety and side effects?

Automated pre-landing PR reviews analyze your diff against the base branch to surface SQL safety issues, LLM trust boundary violations, and conditional side effects before code lands. This process flags potential merge blockers and suggests concrete remediation steps.

What is an LLM trust boundary violation in a diff check?

An LLM trust boundary violation in a diff check occurs when untrusted input crosses into LLM processing without proper safeguards. Automated diff analysis proactively surfaces these structural risks along with SQL safety issues before merge.

Does automated PR review require access to the repository diff?

Yes, automated PR review requires access to the repository diff and the gstack toolkit to function. These inputs allow the analysis to compare changes against the base branch and surface actionable insights for engineering teams.

How do I start a code review for potential merge blockers?

To start a code review for merge blockers, describe the PR and run the review workflow. This initiates the pre-landing analysis to proactively flag structural risks and suggest fixes to improve ship confidence.

What's the best way to check diffs for conditional side effects before merge?

The best way to check diffs for conditional side effects is applying automated pre-landing analysis against the base branch. This proactively identifies structural problems and flags potential merge blockers with suggested remediation steps.

Can I use diff analysis to proactively guide PR discussions?

Yes, you can use diff analysis to support proactive guidance during PR discussions. By surfacing SQL safety risks and structural problems before code lands, engineering teams improve ship confidence and resolve issues faster.