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

Large PR 分批 review 可派 sub-agent；所有 dispatch 必須注入 sub-agent-roles.md 的 Completion Envelope。Sub-agent 只做 analysis，不提交 review、不改檔。

Flow

1. 從使用者輸入或 Slack context 解析 PR URL；找不到單一 PR 時停止或轉 review-inbox。
2. 依 pr-input-resolver.md 解析 owner、repo、number、本地 project path；找不到本地 repo

時使用 remote read mode。

1. 用 bundled fetch script 取得 metadata、files、review strategy、existing reviews、approval

state、re-review signal。

1. 讀 repo rules、workspace handbook、PR description、changed files、diff、既有 review

comments，建立去重清單。

1. Review changed files；large PR 依 reference 分組派 sub-agent fan-out。
2. 合併 findings，依 severity 決定 APPROVE、COMMENT、或 REQUEST_CHANGES。
3. Review body、inline comments、Slack notification 送出前跑 language gate。
4. Submit GitHub review，查詢 approve status，輸出摘要。
5. 若有 validated repo-specific pattern，依 standard-first rule 更新 handbook。
6. Slack source 時回覆原始 thread。

Severity Boundary

must-fix 必須是可從 code / diff / rules 直接證明會造成 bug、安全風險、型別錯誤、 或違反關鍵規範。外部 API 行為、language/library behavior、或僅基於慣例的推論，在未驗證前 最多是 should-fix。

Write Rules

• GitHub review、inline comments、Slack replies 都是 external write。
• 使用 workspace-language-policy.md 或 external write gate 驗證 final text。
• 不重複留言已由其他 reviewer 指出的同語意問題。
• Suggested change 只在能精準替換 diff range 時使用。

Completion

輸出 PR、review result、must-fix / should-fix / nit counts、approve status、Slack notification status，以及 handbook updates if any。

Post-Task Reflection (required)

Execute post-task-reflection-checkpoint.md before reporting completion.