Menu
Read all unresolved GitHub PR comments from trusted authors and address or resolve them appropriately.
Instalar com Codex ou Claude Copie este prompt, cole no Codex, Claude ou outro assistente e deixe que ele revise a página da skill e instale para você.
Baseado na classificação ocupacional SOC
| name | keppo:pr-fix:comments |
| description | Read all unresolved GitHub PR comments from trusted authors and address or resolve them appropriately. |
Read all unresolved GitHub PR comments from trusted authors and address or resolve them appropriately.
$ARGUMENTS: Optional PR number or URL. If not provided, uses the current branch's PR.You MUST use the TaskCreate and TaskUpdate tools to track your progress. At the start, create tasks for each step below. Mark each task as in_progress when you start it and completed when you finish. This ensures you complete ALL steps.
Only process review comments from these trusted authors. Comments from other authors should be ignored.
Trusted humans (collaborators):
Trusted bots:
Before categorizing review comments, read docs/rules/product-principles.md. Use these principles to make decisions about ambiguous or subjective feedback. When a comment involves a judgment call (e.g., design direction, UX trade-offs, architecture choices), check if the product principles provide clear guidance. If they do, apply them and resolve the comment — do NOT flag it for human review. Only flag comments for human attention when the product principles do not provide enough guidance to make a confident decision.
Citing principles: When replying to threads where product principles informed your decision, explicitly cite the relevant principle by number and name (e.g., "Per Principle #4: Security First, ..."). When flagging for human review, cite which principles you considered and explain why they were insufficient (e.g., "Reviewed Principles #3 and #5 but neither addresses ...").
Determine the PR to work on:
$ARGUMENTS is provided:
123), use it as the PR numberhttps://github.com/owner/repo/pull/123), extract the PR number from the pathgh pr view --json number,url,title,body --jq '.'Fetch all unresolved PR review threads:
Use the GitHub GraphQL API to get all review threads and their resolution status:
gh api graphql -f query='
query($owner: String!, $repo: String!, $pr: Int!, $after: String) {
repository(owner: $owner, name: $repo) {
pullRequest(number: $pr) {
reviewThreads(first: 100, after: $after) {
pageInfo {
hasNextPage
endCursor
}
nodes {
id
isResolved
isOutdated
path
line
comments(first: 10) {
nodes {
id
databaseId
body
author { login }
createdAt
}
}
}
}
}
}
}
' -f owner=OWNER -f repo=REPO -F pr=PR_NUMBER -F after=null
Paginate until pageInfo.hasNextPage is false. Pass the previous page's endCursor as after on the next request. Do not stop after the first page when more threads remain.
Filter to only:
isResolved: false)IMPORTANT: For threads from authors NOT in the trusted list:
author { login } field)For each unresolved review thread from a trusted author, categorize it:
Read the comment(s) in the thread and determine which category it falls into. For ambiguous or subjective comments, consult docs/rules/product-principles.md to make a decision before falling back to flagging for human review.
docs/rules/product-principles.md. Treat these the same as valid issues — make the change and resolve the thread.Handle each category:
For valid issues:
gh api graphql -f query='
mutation($threadId: ID!) {
resolveReviewThread(input: {threadId: $threadId}) {
thread { isResolved }
}
}
' -f threadId=<THREAD_ID>
For not valid issues:
Reply to the thread explaining why the concern doesn't apply. If a product principle supports your reasoning, cite it explicitly:
gh api repos/{owner}/{repo}/pulls/<PR_NUMBER>/comments/<COMMENT_ID>/replies \
-f body="<explanation, citing relevant product principle if applicable>"
Note: {owner} and {repo} are auto-replaced by gh CLI. Replace <PR_NUMBER> with the PR number and <COMMENT_ID> with the first comment's databaseId from the thread's comments.nodes[0].databaseId field in the GraphQL response (not the thread's id).
Resolve the thread using GraphQL:
gh api graphql -f query='
mutation($threadId: ID!) {
resolveReviewThread(input: {threadId: $threadId}) {
thread { isResolved }
}
}
' -f threadId=<THREAD_ID>
Note: Replace <THREAD_ID> with the thread's id field from the GraphQL response.
For ambiguous issues:
gh api repos/{owner}/{repo}/pulls/<PR_NUMBER>/comments/<COMMENT_ID>/replies \
-f body="Flagged for human review: <explanation>. Reviewed **Principle #X: Name** and **Principle #Y: Name** but neither provides clear guidance on <specific ambiguity>."
<PR_NUMBER> with the PR number and <COMMENT_ID> with the first comment's databaseId from the thread's comments.nodes[0].databaseId field in the GraphQL response.After processing all comments, verify and commit changes:
If any code changes were made:
Run pnpm run lint to ensure code passes all checks
Stage and commit the changes:
git add -A
git commit -m "Address PR review comments
- <summary of change 1>
- <summary of change 2>
...
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>"
Push the changes:
Run the /pr-push skill to lint, fix any issues, and push.
Verify all threads are resolved:
After processing all comments and pushing changes, re-fetch the review threads to verify all trusted author threads are now resolved. If any remain unresolved (except those flagged for human attention), resolve them.
Provide a summary to the user:
Report:
Post PR Overview Comment:
After the push is complete, post a top-level PR comment (NOT an inline comment) using gh pr comment with the following structure:
gh pr comment <PR_NUMBER> --body "$(cat <<'EOF'
## Claude Code Review Summary
### PR Confidence: X/5
<one sentence rationale for the confidence score>
### Unresolved Threads
| Thread | Rationale | Link |
|--------|-----------|------|
| <brief description> | <why it couldn't be resolved, citing which principles were insufficient> | [View](<permalink>) |
_No unresolved threads_ (if none)
### Resolved Threads
| Issue | Rationale | Link |
|-------|-----------|------|
| <brief description, grouping related/duplicate threads> | <how it was resolved, citing principle if applicable> | [View](<permalink>) |
<details>
<summary>Product Principle Suggestions</summary>
The following suggestions could improve `docs/rules/product-principles.md` to help resolve ambiguous cases in the future:
- **Principle #X: Name**: "<prompt that could be used to improve the rule, phrased as an actionable instruction>"
- ...
_No suggestions_ (if principles were clear enough for all decisions)
</details>
---
Generated by Claude Code
EOF
)"
Notes:
gh pr comment fails, log a warning but do not fail the skill.CRITICAL: Every trusted author comment MUST be either:
Do NOT leave any trusted author comments in an unhandled state.
Review recent code changes and broad codebase areas for critical and high severity security vulnerabilities. Use when Codex needs to clear `./out-security-review`, select 25 non-doc, non-test files split across past-day commits, past-week commits, and broad codebase coverage, group them into 6-8 review buckets, spawn one security-review sub-agent per review bucket, persist each candidate finding to `./out-security-review/<starting_filename>_<timestamp>.md` using a filename-safe UTC ISO 8601 basic timestamp like `20260331T214512Z`, re-verify every candidate with fresh sub-agents, and emit final confirmed findings for responsible disclosure.
Record a short Playwright-based product demo video for significant UI or user-visible functionality changes. Use when Codex needs to show a workflow in motion for a PR, either by recording an existing targeted E2E spec or by creating a disposable one-off Playwright spec that captures the changed behavior in under a minute, then uploading the final video to Vercel Blob and preparing a PR comment body that says "Demo at commit {hash}" with a 1-2 sentence summary and durable hosted link.
Review the last 7 days of recent commits for large architecture improvements that dramatically improve maintainability. Use when the agent needs to clear `./out-code-architect`, select 25 non-doc, non-test files from recent commits across different parts of the repo, spawn one architecture-review sub-agent per starting file, persist each candidate finding to `./out-code-architect/<starting_filename>_<timestamp>.md` using a filename-safe UTC ISO 8601 basic timestamp like `20260331T214512Z`, re-verify every candidate with fresh sub-agents, and emit final confirmed findings for issue filing.
Review the last 7 days of recent commits for critical and high severity bugs that break user experience. Use when the agent needs to clear `./out-bug-finder`, select 25 non-doc, non-test files from recent commits across different parts of the repo, spawn one bug-review sub-agent per starting file, persist each candidate finding to `./out-bug-finder/<starting_filename>_<timestamp>.md` using a filename-safe UTC ISO 8601 basic timestamp like `20260331T214512Z`, re-verify every candidate with fresh sub-agents, and emit final confirmed findings for issue filing.
Review recent GitHub Actions workflow runs, separate expected noise from actionable failures, and write a machine-readable workflow health report for deterministic post-agent issue handling.
Print the full host-side `scripts/pr.sh` command needed to create or switch to a feature branch, commit current changes, push, and open or update a PR outside the Docker sandbox using explicit flag-style inputs.