rhdh-pr-review

Files (7)

• SKILL.md
• openai.yaml · agents/
• review-perspectives.md · references/
• fetch_pr_context.py · scripts/
• fetch-github.md · workflows/
• post-to-github.md · workflows/
• review-code.md · workflows/

SKILL.md (first lines)

# RHDH Pull Request Review

Keep forge I/O at the edges: fetch produces the PR context, analysis works from
that context and checked-out code alone, and posting sends only findings already
verified against the head SHA.

## Route by outcome

| Outcome | Workflow sequence |
|---|---|
| Code review and post | `workflows/fetch-github.md` → `workflows/review-code.md` → `workflows/post-to-github.md` |
| Analysis only | `workflows/fetch-github.md` → `workflows/review-code.md`; stop after the edited draft |

A bare PR URL or number defaults to code review and post.

## Review invariants

- `/code-review` is required on every draft-review path, including
  analysis-only. If it is missing, stop, say that `code-review` is missing,
  name `/setup-rhdh-skills install`, and do not substitute a local two-axis
  review.
- Every `/code-review` run also dispatches Adversarial. Team, worktree, and
  draft steps live in `workflows/review-code.md`.
- Present the complete edited draft before stating any post operation. An
  explicit request to post is intent, not approval of the exact write.

## Write gate

Fetch and analysis are read-only. Posting a GitHub review is an external write:
invoke the named skill `mutation-gate` and follow the gate it owns rather than
restating it here. Creating or removing a local git worktree is not that gate.

A review operation's target pins the head SHA. An earlier confirmation of
findings approves no write. Report each outcome with the review URL, the
verification done, and any recovery still owed.

## What each stage carries forward

Every stage passes its result to the next in conversation. The field names are
defined once, where they are produced:

| Stage | Result | Defined in |
|---|---|---|
| Fetch | PR context: repository, changeRequest, files, diff, linkedIssues, jiraKeys, existingComments, existingReviews, ciStatus, specSource | `workflows/fetch-github.md` |
| Analysis | Review draft: changeRequest, summary, verdict, findings, edited, worktreePath | `workflows/review-code.md` |

## Completion

Complete when the report names the head SHA reviewed, has presented the
`/code-review` Standards and Spec reports, and presents the edited draft.
On the post route, also give the outcome of every approved write with its
target.

From redhat-developer/rhdh-skills, Apache-2.0.

More from redhat-developer/rhdh-skills