merge-ready

Reviews, fixes, and prepares a git branch for final human review with a clean PR.

1.1k|103|Updated Nov 28, 2024
One-click install
npx skills add https://github.com/VladSez/easy-invoice-pdf --skill merge-ready
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: merge-ready
Source: https://github.com/VladSez/easy-invoice-pdf/tree/main/.agents/skills/merge-ready
Command: npx skills add https://github.com/VladSez/easy-invoice-pdf --skill merge-ready

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Getting a branch from "code exists" to "ready for review" requires tedious manual work: syncing with main, auditing changes across multiple risk axes, running CI checks, and writing a clear PR description. This Skill automates that entire pre-merge workflow while keeping the final merge decision with a human reviewer.

Core Features & Use Cases

  • Multi-axis subagent review: Spawns parallel reviewers covering unnecessary complexity, security, billing/metering risks, library idioms, and leftover cruft, each returning findings with file:line and severity.
  • Adversarial verification of findings: Every blocker and should-fix finding is independently verified before being applied, so false positives are rejected instead of patched.
  • Checkpoint commits and CI gating: Commits merges and fixes in logical groups, then runs pnpm ci:check (prettier, knip, tsc, oxlint) in a loop until clean.
  • Use Case: After building a feature, say "make this branch merge-ready" and receive a pushed branch with verified fixes, a passing CI check, and an updated PR containing a high-level summary, review guide, and unresolved nitpicks.

Quick Start

Tell the agent to make the current branch merge-ready so it reviews all changes against origin/main, fixes verified findings, and opens an updated pull request.

Frequently Asked Questions about merge-ready

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

FAQPage Schema
How do I get a branch ready for code review automatically?

Invoke the merge-ready workflow on your branch. It syncs with origin/main, runs parallel review subagents across five risk axes, verifies each finding adversarially, applies fixes in checkpoint commits, and opens or updates a PR with a review guide.

What does the multi-axis code review check for?

Five parallel reviewers examine unnecessary complexity, security issues like authz and SSRF, billing and metering leaks, adherence to TanStack and project idioms, and vibe-coded cruft such as dead code and stale comments. Each finding includes file:line, severity, and rationale.

Does this skill merge the pull request automatically?

No. The workflow explicitly never merges the PR. It pushes the branch, updates the PR description with a summary and review notes, and reports back so the human reviewer always makes the final merge decision.

How are review findings validated before fixes are applied?

Every blocker and should-fix finding is sent to a separate verification subagent that adversarially checks it against the actual code and returns APPLY, APPLY-MODIFIED, or REJECT. Rejected findings are dropped, and nitpicks are reported without mandatory fixes.

What CI checks must pass before the PR is updated?

The branch must pass pnpm ci:check, which runs prettier, knip, tsc, and oxlint. The workflow fixes failures and re-runs the check in a loop until clean, with a quick re-review for any substantial fix.

Can it review uncommitted or untracked files on the branch?

Yes. Review scope includes the committed diff against origin/main, staged and unstaged changes, and every in-scope untracked file, so newly created files cannot escape review.