Agent skill

review-readiness

PR readiness dashboard — tracks which reviews have been completed per branch and gates merge decisions. Shows code review, tests, security, QA, and linting status.

Stars 11,668
Forks 1,338

Install this agent skill to your Project

npx add-skill https://github.com/nearai/ironclaw/tree/staging/skills/review-readiness

SKILL.md

Review Readiness Dashboard

Track process completeness per PR branch. Gate merges on evidence, not gut feel.

How it works

For each active PR branch, maintain a readiness state in projects/<owner>-<repo>/readiness/<branch-slug>.md:

---
type: review-readiness
repo: owner/repo
branch: feature-branch-name
pr_number: 123
updated_at: YYYY-MM-DD
---
# Review Readiness — PR #123

| Check | Status | Last Run | Score | Notes |
|-------|--------|----------|-------|-------|
| Code review | completed | 2026-03-28 | — | Approved by @alice |
| Tests | passing | 2026-03-28 | — | CI green, 3 new tests |
| Security review | completed | 2026-03-28 | 85/100 | 1 P3 finding (accepted) |
| QA review | pending | — | — | Not yet run |
| Linting | passing | 2026-03-28 | — | Zero warnings |

## Verdict: NOT READY
Missing: QA review

## Findings log
- [2026-03-28] Security: P3 — missing rate limit on new endpoint (accepted risk)
- [2026-03-28] Code review: approved, 2 nits addressed

When to use

Checking readiness: User asks "is this PR ready?" or "can I merge?"

  1. Read the readiness file for the branch
  2. If no file exists, create one with all checks as pending
  3. Present the dashboard
  4. If all checks passed: "Ready to merge."
  5. If checks missing: "Not ready. Missing: . Run /security-review and /qa-review to complete."

After running a review skill: When /security-review or /qa-review completes, update the readiness file with the result and score.

In the morning brief: For PRs that are "READY TO MERGE" (approved + CI green), also check review readiness. If security or QA is missing, note it: "READY TO MERGE (code + CI), but security review not run."

Automated updates

The readiness file is updated by:

  • Code review: when PR gets GitHub approval or changes-requested
  • Tests: from CI status (green/red/pending)
  • Security review: when /security-review runs on the branch
  • QA review: when /qa-review runs on the branch
  • Linting: from CI or manual cargo clippy / eslint output

Readiness verdict logic

  • READY: All checks completed, no unresolved P1/P2 findings, CI green
  • ALMOST READY: All checks completed but has accepted findings or pending CI
  • NOT READY: One or more checks not yet run
  • BLOCKED: P1 finding unresolved, or CI failing

Effort estimates

When presenting readiness, show dual-time estimates for remaining work:

  • "Missing: QA review (~15min AI-assisted, ~2h manual)"
  • "Missing: Security review (~10min AI-assisted, ~1h manual)"

This reframes the cost — completeness is cheap with AI assistance.

Expand your agent's capabilities with these related and highly-rated skills.

Didn't find tool you were looking for?

Be as detailed as possible for better results