review-pr

# review-pr 以 reviewer 角色審查別人的單一 PR，依 repo rules / handbook / diff context 留 inline comments，並提交 GitHub review。 ## Contract 此 skill 只處理單一 PR review。多 PR discovery 與 batch orchestration 交給 `review-inbox`；修自己的 PR review comments 交給 `engineering` revision mode。 Reviewer stance：prioritize bugs、behavior regressions、security、type safety、project rule violations、missing tests。不要用 personal style preference 擋 merge。 reviewer-side 屬 read-only lane，但對 `changes_requested`、`active unresolved comments`、 `awaiting_re_review`、`mergeable_ready` 的語義必須與 author-side mutable lane 一致；不得自行重寫。 本 skill 可輸出 reviewer 結論（`APPROVE` / `COMMENT` / `REQUEST_CHANGES`），但不得把 reviewer 結論寫成 author-side stage authority；「可 merge / 可 release / 已完成」仍必須沿用 shared PR / workflow state，而不是 reviewer prose。 ## Reference Loading | Situation | Load | |---|---| | Any run | `review-pr-entry-fetch-flow.md`, `pr-input-resolver.md`, `workspace-config-reader.md`, `shared-defaults.md` | | Analysis | `review-pr-analysis-flow.md`, `repo-handbook.md`, `library-change-protocol.md` as needed | | Submit and notify | `review-pr-submit-flow.md`, `workspace-language-policy.md`, `external-write-gate.md`, `github-slack-user-mapping.md` | | Re-review | `review-pr-rereview-learning-flow.md`, `review-lesson-extraction.md`, `repo-handbook.md` | Large PR 分批 review 可派 sub-agent；所有 dispatch 必須注入 `sub-agent-roles.md` 的 Completion Envelope。Sub-agent 只做 analysis，不提交 review、不改檔。 ## Flow 1. 從使用者輸入或 Slack context 解析 PR URL；找不到單一 PR 時停止或轉 `review-i...