review

Detects SQL injection, LLM trust boundary violations, race conditions, and enum gaps in pull request diffs.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill solves the problem of shipping code with critical safety issues, SQL injection vulnerabilities, LLM trust boundary violations, and race conditions that are easy to miss in manual review. It automates the detection of structural issues in pull request diffs before code lands on the main branch.

Core Features & Use Cases

  • SQL & Data Safety Analysis: Detects string interpolation in SQL queries, N+1 queries, and TOCTOU race conditions.
  • LLM Trust Boundary Checks: Identifies unsanitized LLM output written to databases or passed to mailers, and SSRF risks from LLM-generated URLs.
  • Enum & Value Completeness: Traces new enum values through all consumers to catch missing handlers.
  • Use Case: A developer about to merge a feature that adds a new payment status can run this Skill to ensure every branch in the codebase handles the new status correctly, preventing runtime crashes in production.

Quick Start

Use the review skill to analyze the current git diff against the base branch and report any critical SQL safety, LLM trust boundary, or race condition issues found in the changed files.

Frequently Asked Questions about review

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

FAQPage Schema
How do I detect SQL injection vulnerabilities in a pull request diff?

To detect SQL injection vulnerabilities in a pull request diff, analyze the changed files for string interpolation in SQL queries and N+1 query patterns before merge. This prevents unsafe data access code from landing on the main branch.

How do I check for missing enum handlers when adding a new value in a code review?

To check for missing enum handlers during code review, trace any new enum values through all consumers across the codebase. This cross-file tracing ensures every branch handles the new status correctly, preventing runtime crashes.

What is an LLM trust boundary and how do I validate it before merging?

An LLM trust boundary validation checks unsanitized LLM outputs written to databases or passed to mailers. Validating it before merge identifies SSRF risks from LLM-generated URLs and prevents malicious content from executing.

How can I automate race condition detection for pre-landing review workflows?

You can automate race condition detection for pre-landing review by analyzing pull request diffs against the base branch. This process specifically targets TOCTOU race conditions to catch critical safety issues before code lands.

Does pre-landing code review work without setting up external dependencies?

Pre-landing code review works without external dependencies by parsing git diffs directly and applying pattern matching for SQL, LLM, and shell injection vectors. It requires only the pull request diff to function.

What is the best way to prevent unsafe code from shipping to the main branch?

The best way to prevent unsafe code from shipping is automating structural issue detection in pull request diffs. This catches critical safety issues, injection vectors, and race conditions that are easy to miss in manual review.