merge-review-pr

Read changed files and cross-reference Jira acceptance criteria for open PRs.

Updated May 16, 2026
One-click install
npx skills add https://github.com/cedricfarinazzo/agile-skills --skill merge-review-pr
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: merge-review-pr
Source: https://github.com/cedricfarinazzo/agile-skills/tree/main/agile-merge-review/skills/merge-review-pr
Command: npx skills add https://github.com/cedricfarinazzo/agile-skills --skill merge-review-pr

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Independent PR review by someone other than the author ensures the code meets the spec, ADR, and acceptance criteria before merging. The implementer has already self-reviewed, and this stage serves as the second pair of eyes before landing on main.

Core Features & Use Cases

  • Thorough review of every changed file: read full files, not just the diff, to catch context, consistency, and hidden issues.
  • Spec and ticket alignment: cross-reference Jira ACs and ADR against the PR to verify traceability.
  • Divergence signaling: flag any gaps between PR body ACs and Jira ticket ACs and ensure alignment before merge.
  • Global sprint governance: this gate complements the per-PR checks within agile-11-merge-train workflow.

Quick Start

To start, invoke the PR review on an open PR to trigger the depth-based review.

Frequently Asked Questions about merge-review-pr

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

FAQPage Schema
How do I automate Jira acceptance criteria checks during a PR review?

To automate Jira acceptance criteria checks during a PR review, the system cross-references Jira ticket ACs against the full PR diff. It extracts the Jira key, reads changed files in full, and flags any divergence from the ticket before merging.

Why should a code review read the full file instead of just the diff?

A code review should read the full file instead of just the diff to verify surrounding context, consistency, and hidden issues. Reading full files ensures the changes align properly with project guidelines, naming conventions, security standards, and documentation requirements.

What's the best way to ensure PR security and naming conventions are verified before merge?

The best way to ensure PR security and naming conventions are verified is applying a depth-based review across Jira-integrated workflows. This process reads every changed file, cross-references project guidelines, and acts as a governance gate before landing on main.

Can I use this PR review process to flag divergence between PR body and Jira ticket ACs?

Yes, you can use this PR review process to flag divergence between PR body ACs and Jira ticket ACs. It explicitly cross-references both sets of acceptance criteria, signaling gaps and ensuring strict alignment before allowing a merge.

How does a global sprint governance gate work with per-PR checks?

A global sprint governance gate works with per-PR checks by complementing agile merge train workflows. While per-PR checks validate individual changes, the governance gate independently verifies overall correctness, security, tests, and documentation before final merge.

Do I need a Jira-integrated workflow to trigger this depth-based PR review?

Yes, you need a Jira-integrated workflow to trigger this depth-based PR review. The process requires extracting a Jira key from the open PR to cross-reference the specific ticket's acceptance criteria and project ADRs during the review.