review

Identify correctness, security, and spec bugs in code diffs with file-level evidence.

88|6|Updated Mar 25, 2026
One-click install
npx skills add https://github.com/heliohq/ship --skill review-heliohq
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/heliohq/ship/tree/main/skills/review
Command: npx skills add https://github.com/heliohq/ship --skill review-heliohq

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill helps you inspect a code change for real correctness, security, and spec-compliance issues before the change ships. It is designed to catch bugs that survive tests, regressions that break callers, and trust-boundary problems that need evidence-backed review.

Core Features & Use Cases

  • Active diff review: Focuses on staged, unstaged, and branch-level changes rather than the whole repository.
  • Evidence-based findings: Reports only concrete bugs with file and line references, trigger conditions, impact, and fix direction.
  • Release-quality triage: Useful for PR review, pre-merge verification, and checking whether a change can safely move forward.
  • Example use case: A developer submits a feature update that alters shared interfaces, and this Skill helps verify whether the change introduces broken behavior, data integrity issues, or security regressions.

Quick Start

Ask the review skill to examine the current code changes and return only concrete findings with file:line evidence.

Frequently Asked Questions about review

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

FAQPage Schema
How do I catch bugs in staged and unstaged diffs before merging?

To catch bugs in active diffs before merging, inspect staged, unstaged, and branch-level changes for correctness, security, and spec compliance using evidence-backed findings with file and line references.

How does cross-file tracing work for detecting regressions in code review?

Cross-file tracing detects regressions by tracking how modified shared interfaces impact callers across the repository, validating actual bugs through file-level evidence and severity ranking before code approval.

Can I use automated diff analysis for release gating and pre-merge verification?

Yes, automated diff analysis supports release gating and pre-merge verification by inspecting active code changes for security regressions and data integrity issues, returning concrete findings to determine if a change safely moves forward.

What is the best way to perform a security audit on a branch diff?

The best way to perform a security audit on a branch diff is to apply evidence-based review targeting trust-boundary problems, requiring concrete file and line references, trigger conditions, and impact assessments to validate security regressions.

Does diff analysis work for spec compliance checking during bug triage?

Yes, diff analysis works for spec compliance checking during bug triage by identifying whether active code changes violate specifications. It applies to staged, unstaged, and branch diffs, requiring file-level evidence to validate real issues before approval.

Why does code review require file-level evidence for bug detection?

Code review requires file-level evidence for bug detection to ensure only concrete issues are reported. By demanding file and line references, trigger conditions, and impact analysis, it prevents false positives and validates real regressions before a change ships.