pr-review-agent

$npx mdskill add wednesday-solutions/ai-agent-skills/pr-review-agent

Orchestrates full PR review with blast radius and drift analysis.

  • Reviews pull requests and determines merge readiness.
  • Depends on brownfield-fix, brownfield-drift, and pr-review skills.
  • Runs blast radius and drift checks on changed files.
  • Presents categorized comments, risks, and violations for approval.

SKILL.md

.github/skills/pr-review-agentView on GitHub ↗
---
name: pr-review-agent
description: Full PR review orchestrator. Use when asked to review a PR or check if it is ready to merge. Runs blast radius + drift check on changed files, then hands off to pr-review for comment triage and fixes. Do NOT use pr-review directly for full reviews — use this.
license: MIT
metadata:
  author: wednesday-solutions
  version: "1.0"
requires:
  - brownfield-fix
  - brownfield-drift
  - pr-review
permissions:
  allow:
    - Bash(git *)
    - Bash(gh *)
    - Bash(wednesday-skills drift *)
    - Bash(wednesday-skills blast *)
    - Bash(wednesday-skills score *)
---

# PR Review Agent

## When to use
- "Review this PR"
- "Is this PR ready to merge?"
- "Check this PR for issues"
- PR is open and review comments have been posted

## What to do

1. **triage-read** — Fetch all review comments via `gh pr view --comments`. Sort by impact: security → breaking → logic → performance → naming → style. Present the full categorised list to the dev.

2. **In parallel:**
   - **brownfield-fix** — Run `wednesday-skills score` and `wednesday-skills blast` on every file changed in the PR. Flag any file with risk > 60.
   - **brownfield-drift** — Run `wednesday-skills drift --since <base-sha>`. Report any architecture violations introduced by this PR.

3. Present a unified report:
   - Review comments (categorised)
   - High-risk changed files (score + dependent count)
   - Architecture violations (if any)

4. **triage-fix** — Wait for dev approval: "@agent fix #N" or "@agent fix all". Apply fixes in priority order. Read git-os skill before committing. One commit per fix item.

## Never
- Fix without explicit dev approval
- Skip blast radius check — always run even for small PRs
- Skip drift check — always run even if no violations are expected
- Bundle multiple file fixes in one commit
- Force-push or silently skip a failed fix

More from wednesday-solutions/ai-agent-skills

SkillDescription
brownfield-driftEnforces architecture boundaries defined in PLAN.md. Use when a PR crosses module/service boundaries, when the dev asks "are we following the architecture?", or as a scheduled architecture health check. Not for querying what a module does — use brownfield-chat for that.
codebase-intelUnified codebase intelligence. Handles all questions about structure, logic, risk, and dependencies. Combines natural-language Q&A with deterministic lookups and pre-edit blast radius checks.
deploy-checklistPre-deploy and post-deploy checklist skill. Ensures env vars, migrations, CI, rollback plan, smoke tests, and monitoring are verified before and after every deployment.
greenfieldParallel persona planning for new projects. Research agent runs first to build domain context, then Architect, PM, and Security agents run in parallel. Synthesis agent combines all perspectives into a detailed GSD-style PLAN.md with Tensions section.
module-audit-agentUse when asked to audit a module, check its health, or assess whether it should be refactored. Runs structural query, risk check, and test generation automatically.
onboard-dev-agentUse when a dev is new to the project, asks for an overview, or wants to get oriented quickly. Fills graph gaps then runs a guided codebase interview.
standards-kitUnified development and design standards. Enforces code quality (complexity < 8), strict naming conventions, and the mandatory use of approved UI component libraries.
wednesday-gitUnified Git workflow. Manages the entire task lifecycle: branch creation (sprint), atomic commits (git-os), and PR opening (pr-create).