review

Reviews pull request diffs against a base branch for SQL safety, race conditions, and structural issues.

Updated Mar 26, 2026
One-click install
npx skills add https://github.com/LGJ-Jonathan/Jonathan_Global_Claude_Skills --skill review-lgj-jonathan
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/LGJ-Jonathan/Jonathan_Global_Claude_Skills/tree/main/gstack/review
Command: npx skills add https://github.com/LGJ-Jonathan/Jonathan_Global_Claude_Skills --skill review-lgj-jonathan

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve? Code review before merging is where bugs like SQL injection, race conditions, and unvalidated LLM output slip through when reviewers are rushed. This Skill automates a structured pre-landing review of your branch's diff so structural problems get caught before the code lands. ## Core Features & Use Cases - Two-Pass Checklist Review: Runs a critical pass (SQL & data safety, race conditions, LLM output trust boundaries, enum completeness) followed by an informational pass (test gaps, dead code, performance, CI/CD issues). - Scope Drift & Plan Completion Audit: Compares the diff against TODOS.md, PR descriptions, and plan files to detect scope creep and missing requirements. - Greptile Comment Triage: Fetches, classifies, and replies to Greptile bot review comments on GitHub PRs with evidence-based responses. - Fix-First Workflow: Auto-applies mechanical fixes (dead code, N+1 queries, stale comments) and batches judgment calls into a single user question. - Use Case: Before merging a feature branch, run the review to get a report like "Pre-Landing Review: 5 issues (2 critical, 3 informational)" with file:line citations and suggested fixes. ## Quick Start Ask the assistant to review this PR or check my diff against the base branch before merging.

Frequently Asked Questions about review

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

FAQPage Schema
How do I review a PR diff before merging?

Run the review workflow on your feature branch and it fetches the base branch, generates the full diff, and applies a checklist covering SQL safety, race conditions, and test gaps. Findings are reported with file:line citations and suggested fixes.

What does automated pre-landing code review check for?

It checks SQL injection and N+1 queries, race conditions, unvalidated LLM output, enum completeness, conditional side effects, dead code, test gaps, crypto weaknesses, and CI/CD pipeline issues. Critical findings are separated from informational ones.

Does this code review work with GitLab merge requests?

Yes, it detects the platform from the git remote URL and uses glab for GitLab or gh for GitHub to determine the target branch. If neither CLI is available, it falls back to git-native commands like symbolic-ref to find the default branch.

Can the review auto-fix issues it finds?

Yes, mechanical fixes like dead code removal, missing eager loading, stale comments, and magic numbers are applied automatically. Riskier changes involving security, race conditions, or user-visible behavior are batched into a question for your decision.

How does it handle Greptile bot comments on pull requests?

It fetches Greptile line-level and top-level comments via the GitHub API, classifies each as valid, already fixed, false positive, or suppressed, then posts evidence-based replies. Known false positives are suppressed using a per-project history file.

When should I not rely on automated diff review?

Automated review catches structural and pattern-based issues but cannot judge product intent, architecture fit, or visual design quality. Use it alongside human judgment for design decisions and anything requiring business context.