review

Analyze diffs against the base branch for SQL safety and trust boundary issues.

Updated Mar 30, 2026
One-click install
npx skills add https://github.com/nhattran998/personal-athlete-agent --skill review-nhattran998
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/nhattran998/personal-athlete-agent/tree/main/.agents/skills/gstack-review
Command: npx skills add https://github.com/nhattran998/personal-athlete-agent --skill review-nhattran998

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

  • SQL safety analysis on diffs
  • LLM trust boundary checks
  • Detection of conditional side effects and structural issues
  • Proactive merge-safety recommendations

Quick Start

Provide a pre-merge review by analyzing the diff against the base branch for SQL safety, LLM trust boundaries, and structural issues, and propose proactive merge-safe actions.

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 PR review for SQL safety and LLM trust boundary issues?

A pre-landing PR review analyzes your diff against the base branch to identify SQL safety issues, LLM trust boundary violations, and conditional side effects. Provide the repository base and diffs to get an accurate merge readiness assessment.

What is the best way to check my diff for potential conditional side effects before merging?

Checking your diff for conditional side effects involves analyzing structural changes against the base branch. This review process detects potential side effects and LLM trust boundary violations, proposing proactive merge-safe actions before you land code changes.

How does diff analysis detect SQL safety issues in a code review?

Diff analysis detects SQL safety issues by comparing your changes against the repository base. It evaluates SQL modifications within the diff to ensure merge readiness and prevent unsafe database operations from landing.

Do I need repository base access to validate merge readiness with a diff review?

Yes, you need repository base and diffs access to validate merge readiness. Accurate assessment of LLM trust boundaries and SQL safety requires comparing the proposed changes directly against the base branch.

Can I use this code review approach for prerelease checks and diff validations?

You can use this code review approach for prerelease checks, merge readiness evaluations, and diff validations. It proactively suggests merge-safety recommendations when you are about to land code changes.

What are the limitations of automated diff analysis for LLM trust boundary violations?

Automated diff analysis for LLM trust boundary violations is limited to analyzing the provided diff against the base branch. It requires complete repository access for accurate assessment and focuses on structural issues rather than runtime behavior.