review-and-ship

Review a Git branch for bugs, intent fit, and test coverage before merging.

Updated May 18, 2025
One-click install
npx skills add https://github.com/nthpaul/dotfiles --skill review-and-ship-nthpaul
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review-and-ship
Source: https://github.com/nthpaul/dotfiles/tree/main/cursor/.cursor/plugins/cache/cursor-public/cursor-team-kit/d1cdb88a9eb33cf392395c87e3fd76419fc1010e/skills/review-and-ship
Command: npx skills add https://github.com/nthpaul/dotfiles --skill review-and-ship-nthpaul

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It helps you confidently review a branch before merging by checking intent fit, correctness, regressions, and test coverage, then turning that review into a focused commit and a ready PR.

Core Features & Use Cases

  • Context-aware review: Uses diff vs base branch, recent commits, changed files, and relevant intent to ensure the work matches what the user actually needs.
  • Targeted testing and gap handling: Runs focused tests for the changed behavior and decides whether to add tests or document missing coverage.
  • Practical readiness workflow: Fixes critical issues, commits selectively, pushes, and opens or updates a PR with concrete check evidence (including gh pr checks).

Quick Start

Review your current branch against origin/main, run targeted tests for the changed areas, fix critical issues, then open or update a PR using the suggested checks.

Frequently Asked Questions about review-and-ship

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

FAQPage Schema
How do I validate a Git branch for bugs and test coverage before opening a pull request?

To validate a Git branch before a pull request, analyze the diff against the base branch to check for bugs, regressions, and test coverage gaps. This pre-PR workflow ensures changed files match the intended goals and pass targeted tests.

How can I run targeted tests only for changed files instead of the whole test suite?

Run targeted tests for changed files by analyzing the branch diff to identify modified behavior. The workflow determines which specific tests to execute for those areas and decides whether to add missing tests or document the coverage gaps.

Can I automatically fix critical issues and commit changes selectively before pushing?

Yes, you can fix critical issues found during the diff analysis and commit changes selectively. This practical readiness workflow ensures only focused fixes are pushed, preventing unrelated modifications from contaminating the final pull request.

How do I assess CI readiness and check PR status using GitHub CLI?

Assess CI readiness by executing `gh pr checks` after opening or updating the pull request. This validates that the continuous integration pipelines have passed, providing concrete evidence that the branch is ready for merging.

What is the best way to ensure a code review checks both intent fit and security regressions?

The best way to ensure a code review checks intent fit and security regressions is to combine diff analysis with recent commit context. This approach prioritizes correctness and security by validating whether the implementation matches the actual requirements.

When should I not use an automated pre-PR validation workflow for my pull request?

You should avoid using an automated pre-PR validation workflow when your branch requires complex architectural discussions or manual subjective review that cannot be captured by diff analysis, targeted testing, or selective commit workflows.