pr-review

Review pull requests against a multi-dimensional readiness checklist before merging.

6|1|Updated May 13, 2026
One-click install
npx skills add https://github.com/Xipher-Labs/walter-os --skill pr-review-xipher-labs
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: pr-review
Source: https://github.com/Xipher-Labs/walter-os/tree/main/skills/pr-review
Command: npx skills add https://github.com/Xipher-Labs/walter-os --skill pr-review-xipher-labs

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It eliminates risky PR merges by ensuring every pull request is checked against a rigorous, multi-dimensional quality and readiness rubric before it enters the mainline.

Core Features & Use Cases

  • Pre-opening and pre-merge PR readiness: validates PR title conventions, checks spec and plan alignment, verifies tests/lint/typechecks, and confirms no debug artifacts or secrets are present.
  • Multi-dimensional code review scoring: evaluates correctness, security, performance, testing rigor, readability, backward compatibility, and operational readiness with explicit blocking criteria.
  • Definition-of-Done coverage enforcement: confirms acceptance criteria are mapped to tests and that review iterations stay within defined limits.

Quick Start

Use pr-review to evaluate the pull request diff for merge readiness by asking the AI: "Review this PR for correctness, security, performance, testing rigor, readability, backward compatibility, and operational risks, and highlight blocking findings."

Frequently Asked Questions about pr-review

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

FAQPage Schema
How do I automate pull request reviews to catch security and correctness issues before merging?

Automated pull request reviews use a checklist-based rubric to evaluate correctness, security, performance, and testing rigor before merging. This validates PR title conventions, spec alignment, and backward compatibility while explicitly blocking merges when critical findings are detected.

What is the best way to enforce CI quality gates and test rigor across branch promotion scenarios?

CI quality gates and test rigor are enforced by applying a multi-dimensional readiness rubric across branch promotion scenarios. This verifies that acceptance criteria map to tests, checks lint and typecheck status, confirms no debug artifacts exist, and ensures review iterations stay within defined limits.

How does checklist-driven code review prevent unsafe merges into the mainline?

Checklist-driven code review prevents unsafe merges by validating PR readiness against explicit blocking criteria before code enters the mainline. It checks for secrets, enforces title and plan alignment, and scores operational readiness to ensure no risky code bypasses the review process.

Can I validate PR readiness before opening a pull request?

PR readiness can be validated before opening a pull request by running a pre-opening review. This checks spec and plan alignment, verifies tests and typechecks pass, and confirms no debug artifacts or secrets are present to ensure the PR is ready for review.

What should a code review checklist include to ensure backward compatibility and operational readiness?

A code review checklist for backward compatibility and operational readiness should validate spec alignment, test rigor, and performance. It must confirm acceptance criteria map to tests, check for debug artifacts, and use an explicit blocking findings template to flag critical issues.

Why do pull requests fail merge readiness checks despite passing tests?

Pull requests fail merge readiness checks despite passing tests when they violate non-functional requirements. The review catches missing spec alignment, insufficient test rigor for acceptance criteria, backward compatibility risks, or operational readiness gaps that basic test suites do not cover.