review

Analyze branch diffs for SQL safety, LLM trust boundaries, and side effects.

Updated Apr 20, 2026
One-click install
npx skills add https://github.com/sennett-lau/alice --skill review-sennett-lau
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/sennett-lau/alice/tree/main/framework/skills/review
Command: npx skills add https://github.com/sennett-lau/alice --skill review-sennett-lau

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

An automated pre-landing PR review that analyzes the current branch diff against the base to surface structural issues, SQL safety concerns, and LLM trust boundary violations before merging.

Core Features & Use Cases

  • Diff-scoped analysis: compares the branch against origin/main to identify risky changes and potential scope drift.
  • Plan-driven audit: optionally leverages an active plan to extract actionable items and map them to diff changes.
  • Actionable output: reports DONE/PARTIAL/NOT DONE/CHANGED statuses and provides concrete follow-ups for risk mitigation.
  • Output synthesis: combines scope drift insights with plan-completion context to inform stakeholders before landing.

Quick Start

Run the review workflow on your branch to generate a pre-landing risk audit.

Frequently Asked Questions about review

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

FAQPage Schema
How do I automate a pre-landing PR review for SQL safety and scope drift?

Automated pre-landing PR review analyzes your branch diff against the base branch to surface SQL safety concerns, LLM trust boundary violations, and scope creep before merging. It maps active plan items to structural changes, marking them as DONE, PARTIAL, NOT DONE, or CHANGED.

What is diff analysis in the context of automated branch reviews?

Diff analysis in automated branch reviews compares the current branch against origin/main to identify risky structural changes and potential scope drift. It maps actionable plan items to specific code modifications, tracking completion status and highlighting side effects before the code lands.

How do I check if my branch changes violate LLM trust boundaries before merging?

To check LLM trust boundary violations before merging, run a pre-landing review workflow that analyzes your diff against the base branch. The audit identifies risky changes and side effects, providing a structured summary that reviewers can use to assess LLM trust and safety.

Can I map active plan items to diff changes to track scope creep?

Yes, you can map active plan items to diff changes to track scope creep. The review workflow optionally leverages an active plan to extract actionable items, auditing them against the branch diff and reporting completion statuses to inform stakeholders of any scope drift.

Does the pre-landing risk audit work without a predefined plan?

Yes, the pre-landing risk audit works without a predefined plan by relying on diff-scoped analysis. While it optionally leverages an active plan to map actionable items, it independently compares the branch against origin/main to identify risky SQL changes and structural side effects.