Agent-Skills.md

Agent Skills: GitHub PR Review Fix

Fetch unresolved GitHub PR review threads, normalize them, fix them end-to-end, verify the results, and re-check until the PR is clean or blocked. Use when the user wants GitHub PR comments resolved with minimal verified fixes. Do not use for local review files, Codex reviews, or Zen reviews; review-remediation owns those.

UncategorizedID: bjornmelin/dev-skills/gh-pr-review-fix

Repository

bjornmelin/dev-skills
BjornMelin
2

Install this agent skill to your local

pnpm dlx add-skill https://github.com/BjornMelin/dev-skills/tree/HEAD/skills/gh-pr-review-fix

Skill Files

Browse the full folder contents for gh-pr-review-fix.

Download Skill

Loading file tree…

skills/gh-pr-review-fix/SKILL.md

Skill Metadata

Name
gh-pr-review-fix
Description
"Fix unresolved GitHub PR review threads with codex-dev: fetch hosted state, verify, patch, validate, commit, push, and close out."

GitHub PR Review Fix

Use this as the sole active GitHub PR review remediation workflow. The canonical implementation is codex-dev; this skill is the operating procedure.

Default Behavior

When the user invokes $gh-pr-review-fix, run the full closeout loop unless blocked:

  1. Read repo AGENTS.md and inspect git status --short.
  2. Capture fresh hosted review work:
    • codex-dev --json pr review start --repo <owner/repo> --number <pr> --fresh
  3. If the worklist reports zero actionable unresolved items, stop with a no-op summary.
  4. Verify each item against current code; skip stale findings with a brief reason.
  5. Apply minimal fixes, using exact suggestion fences only when codex-dev pr review apply-suggestions reports an exact hunk match.
  6. Run focused repo-native validation, then broad required gates once before publishing.
  7. Create scoped, semantically grouped, reviewable Conventional Commits following references/closeout-and-commits.md.
  8. Push the PR branch once after all intended semantic commits pass validation.
  9. Re-capture fresh PR head/thread state and resolve every matching fixed hosted thread:
    • codex-dev --json pr review closeout --repo <owner/repo> --number <pr> --worklist <json> --expected-head-sha <pushed-head> --commit <sha> --validation-command <cmd> --apply
  10. Re-run codex-dev --json pr review start --fresh --repo <owner/repo> --number <pr> and continue until zero actionable unresolved items remain or a real blocker appears.

Hosted Closeout Policy

  • Default is fix, commit, push, and resolve fixed hosted threads.
  • Do not auto-reply by default. Reply only when a finding cannot be fixed/resolved cleanly or the user asks for comments.
  • Resolve only current hosted thread IDs that map to verified fixes or already-fixed current head state.
  • Never resolve when the PR head changed unexpectedly, validation failed, the finding was skipped, or the thread cannot be matched to closeout evidence.

Commit Policy

  • Read references/closeout-and-commits.md before committing.
  • Use codex-dev --json commit plan to inspect semantic groups in a mixed tree.
  • Use codex-dev --json commit validate --subject "<subject>" before committing.
  • codex-dev commit validate owns forbidden process-wording checks; do not maintain a second local phrase list here.

Route Away

  • Local review file, Codex review, Zen review, or manual notes -> $review-remediation.
  • Passive post-push monitoring -> codex-dev --json pr readiness plus GitHub status checks.
  • CI-only failure with no review-thread context -> GitHub plugin or gh workflow-log remediation.

Outputs

  • codex-dev.pr-review-worklist.v1 worklist
  • verified fix/skipped-item summary
  • validation commands and outcomes
  • semantic commit SHAs
  • codex-dev.pr-review-closeout.v1 thread-to-commit closeout evidence
  • terminal status: completed, blocked, or no-op

Resources

  • references/closeout-and-commits.md