review

Analyze PR diffs for SQL safety, trust boundary, and side-effect issues.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Pre-landing PR review automates analysis of diffs against the base branch to surface structural issues, potential SQL safety problems, LLM trust boundary violations, and unintended side effects before merging.

Core Features & Use Cases

  • Analyzes diffs between a feature branch and base branch to flag risky changes and potential issues.
  • Proposes concrete, actionable fixes or mitigations and proactive review suggestions.
  • Useful for code reviews, PR checks, and merge readiness in CI or manual review workflows.

Quick Start

Perform a pre-landing PR review by analyzing the diff against the base branch and reporting issues with fixes.

Frequently Asked Questions about review

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

FAQPage Schema
How do I automate code review for SQL safety and side effects before merging?

Automated pre-landing code review analyzes branch diffs to detect SQL safety issues, LLM trust boundary violations, and conditional side effects. It generates a structured report with actionable fixes and clear guidance to approve, modify, or reject changes.

What is a pre-landing PR review and how does it identify risky changes?

Pre-landing PR review is an automated analysis of diffs against a base branch to surface structural issues. It identifies risky changes by flagging potential SQL safety problems, LLM trust boundary violations, and unintended side effects before code merges.

How do I check pull request merge readiness for LLM trust boundary violations?

Check pull request merge readiness by analyzing the diff against the base branch. This surfaces LLM trust boundary violations alongside SQL safety issues, providing concrete fixes and proactive suggestions to improve PR quality before merging.

Can I use automated diff analysis to block merges in a CI workflow?

Yes, automated diff analysis supports CI workflows by evaluating merge readiness. It provides proactive guidance for pre-landing reviews, suggesting when to block merges based on detected SQL issues, side effects, and trust boundary violations.

What is the best way to find actionable fixes for unintended side effects in a feature branch?

The best way to find actionable fixes is by analyzing diffs between the feature branch and base branch. This process detects unintended conditional side effects and generates a structured report highlighting specific findings and concrete mitigations.

When should I not rely on automated diff analysis for code review?

Automated diff analysis focuses specifically on SQL safety, LLM trust boundaries, and conditional side effects. You should not rely on it exclusively for business logic validation or stylistic linting, as it targets structural risks and merge readiness rather than comprehensive logic verification.