review

Review pull request diffs for SQL safety, concurrency, and LLM trust-boundary risks.

1|Updated Mar 27, 2026
One-click install
npx skills add https://github.com/yibeichan/schist --skill review-yibeichan
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/yibeichan/schist/tree/main/.claude/skills/review
Command: npx skills add https://github.com/yibeichan/schist --skill review-yibeichan

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

This Skill helps you catch risky issues before merging by analyzing a pull request diff for SQL/data safety, race/concurrency hazards, and LLM output trust-boundary problems.

Core Features & Use Cases

  • Pre-landing diff analysis: reviews the current branch against the base branch to surface structural problems.
  • Two-pass safety-first review: focuses first on SQL & data safety, concurrency, LLM trust boundary, and enum/value completeness, then checks secondary issues like conditional side effects and prompt problems.
  • Scope and intent sanity checks: detects scope creep and missing requirements by comparing stated intent (PR body/TODOs/commits) to what the diff actually changes.
  • Optional Greptile triage: if Greptile comments exist, classifies them into valid/actionable, already-fixed, false positives, or suppressed.
  • Test coverage gap mapping: traces changed code and user flows, identifies missing test paths, and enforces the regression-test rule.

Quick Start

Ask your AI to review this PR diff before merging and explicitly include SQL safety, concurrency, and LLM trust-boundary checks.

Frequently Asked Questions about review

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

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

To check a pull request diff for SQL safety, you can run a pre-landing review that analyzes branch differences to detect data risks, race conditions, and LLM output trust-boundary violations. It applies a two-pass safety-first checklist to surface structural problems before merge.

What is an LLM trust boundary violation in code review?

An LLM trust boundary violation in code review occurs when unstructured model outputs directly trigger side effects without validation. Pre-landing diff analysis detects these issues by checking if structured intent matching and conditional side effects properly isolate model-generated content.

How do I detect scope drift in a pull request?

You detect scope drift in a pull request by comparing stated intent from PR bodies, TODOs, and commits against the actual diff changes. This sanity check identifies scope creep and missing requirements by mapping what the code actually modifies versus what was intended.

Can I triage Greptile comments on a pull request automatically?

Yes, you can triage Greptile comments on a pull request automatically if they exist. The review process classifies Greptile comments into valid and actionable issues, already-fixed items, false positives, or suppressed comments to streamline pre-landing checks.

How do I map test coverage gaps for changed code and user flows?

You map test coverage gaps by tracing changed code paths and user flows within the diff to identify missing test paths. This enforces regression-test requirements by linking structural modifications directly to necessary test coverage before landing.

Does pre-landing code review work for enum completeness and conditional side effects?

Pre-landing code review works for enum completeness and conditional side effects by applying a two-pass safety-first checklist. The initial pass focuses on value completeness and safety, while the secondary pass evaluates conditional side effects and prompt problems.