返回技能列表

review-pull-request

pjt222
更新于 2 days ago
9 次查看
17
2
17
在 GitHub 上查看
其他ai

关于

This Claude Skill performs comprehensive pull request reviews using GitHub CLI, analyzing diffs, commit history, and CI/CD checks. It provides severity-leveled feedback (blocking/suggestion/nit/praise) and submits reviews via `gh pr review`. Use it when assigned to review a PR, for self-review before seeking feedback, for follow-up reviews after changes, or for post-merge quality audits.

快速安装

Claude Code

推荐
主要方式
npx skills add pjt222/agent-almanac -a claude-code
插件命令备选方式
/plugin add https://github.com/pjt222/agent-almanac
Git 克隆备选方式
git clone https://github.com/pjt222/agent-almanac.git ~/.claude/skills/review-pull-request

在 Claude Code 中复制并粘贴此命令以安装该技能

技能文档

審拉請

由 GitHub CLI 全程審拉請——自解變至遞構之反。用 gh CLI 行諸 GitHub 之動,生重之等審注。

用時

  • PR 備審且授汝乃用
  • 著者處反後行二審乃用
  • 請他審前自審己之 PR 乃用
  • 為合後質察審已合之 PR 乃用
  • 欲構之審程而非臨之掃乃用

  • 必要:PR 之識(號、URL、或 owner/repo#number
  • 可選:審之專(安、性、正、格)
  • 可選:碼庫之熟(熟、稍、生)
  • 可選:審之時算(速掃、標、深)

第一步:解其境

讀 PR 之述而解此變欲成何。

  1. 取 PR 之屬:
    gh pr view <number> --json title,body,author,baseRefName,headRefName,labels,additions,deletions,changedFiles,reviewDecision
    
  2. 讀 PR 之題與述:
    • 此 PR 解何患?
    • 著者用何徑?
    • 著者欲審何特區乎?
  3. 察 PR 之大而估所需之時:
PR Size Guide:
+--------+-----------+---------+-------------------------------------+
| Size   | Files     | Lines   | Review Approach                     |
+--------+-----------+---------+-------------------------------------+
| Small  | 1-5       | <100    | Read every line, quick review       |
| Medium | 5-15      | 100-500 | Focus on logic changes, skim config |
| Large  | 15-30     | 500-    | Review by commit, focus on critical  |
|        |           | 1000    | files, flag if should be split       |
| XL     | 30+       | 1000+   | Flag for splitting. Review only the  |
|        |           |         | most critical files.                 |
+--------+-----------+---------+-------------------------------------+
  1. 審提交之史:
    gh pr view <number> --json commits --jq '.commits[].messageHeadline'
    
    • 提交為邏且善構乎?
    • 史敘故乎(各提交為合之步)?
  2. 察 CI/CD 之狀:
    gh pr checks <number>
    
    • 諸察皆過乎?
    • 若察敗,記其敗者——影審之徑

得:明解 PR 為何、為何存、何大、CI 綠乎。此境形審之徑。

敗則:若 PR 述空或不清,標此為首反。無境之 PR 為審之反形。若 gh 命敗,驗已認證(gh auth status)且有庫之訪。

第二步:析其差

系讀實之碼變。

  1. 取全差:
    gh pr diff <number>
    
  2. 小/中之 PR,序讀全差
  3. 大 PR,依提交審:
    gh pr diff <number> --patch  # full patch format
    
  4. 各變文評:
    • :碼行 PR 所述乎?
    • 邊例:界處乎?
    • 誤處:誤捕與宜處乎?
    • :注入、認、數露之險乎?
    • :明 O(n^2) 環、缺索、存患乎?
    • :新變/函/類命清乎?
    • :新行為試覆乎?
  5. 讀時記之,依重分各察

得:諸有意變之察集,覆正、安、性、質。各察有重等。

敗則:若差過大不能效審,標之:「此 PR 變 {N} 文 {M} 行。吾議分為小 PR 以效審。」仍審最險之文。

第三步:分反

組諸察為重之等。

  1. 各察分之:
Feedback Severity Levels:
+-----------+------+----------------------------------------------------+
| Level     | Icon | Description                                        |
+-----------+------+----------------------------------------------------+
| Blocking  | [B]  | Must fix before merge. Bugs, security issues,      |
|           |      | data loss risks, broken functionality.             |
| Suggest   | [S]  | Should fix, but won't block merge. Better           |
|           |      | approaches, missing edge cases, style issues that   |
|           |      | affect maintainability.                            |
| Nit       | [N]  | Optional improvement. Style preferences, minor      |
|           |      | naming suggestions, formatting.                    |
| Praise    | [P]  | Good work worth calling out. Clever solutions,      |
|           |      | thorough testing, clean abstractions.              |
+-----------+------+----------------------------------------------------+
  1. 各 Blocking 之入釋:
    • 何誤(具體之患)
    • 何要(其影)
    • 如何修(具體議)
  2. 各 Suggest 之入釋替與何以勝
  3. Nit 簡——一句即足
  4. 若有陽事,至少一 Praise

得:排序之反列附明重等。Blocking 之入有修議。比常宜為:少 Blocking、某 Suggest、微 Nit、至少一 Praise。

敗則:若皆似 Blocking,PR 或宜重作而非補。考於 PR 等請變而非行行注。若無事似誤,述之——「LGTM」乃碼善時之效反。

第四步:書審注

撰附構、可行反之審。

  1. 審摘(頂注):
    • 一句:PR 行何(確解)
    • 整評:批准、請變、或注
    • 要入:列 Blocking 患(若有)與首 Suggest
    • Praise:標善勞
  2. 行注於具體碼所:
    # Post inline comments via gh API
    gh api repos/{owner}/{repo}/pulls/{number}/comments \
      -f body="[B] This SQL query is vulnerable to injection. Use parameterized queries instead.\n\n\`\`\`suggestion\ndb.query('SELECT * FROM users WHERE id = $1', [userId])\n\`\`\`" \
      -f commit_id="<sha>" \
      -f path="src/users.js" \
      -F line=42 \
      -f side="RIGHT"
    
  3. 反式恆:
    • 各注始以重標:[B][S][N][P]
    • 用 GitHub 之議塊行具修
    • 鏈至文檔以行格/形之議
  4. 遞其審:
    # Approve
    gh pr review <number> --approve --body "Review summary here"
    
    # Request changes (when blocking issues exist)
    gh pr review <number> --request-changes --body "Review summary here"
    
    # Comment only (when unsure or providing FYI feedback)
    gh pr review <number> --comment --body "Review summary here"
    

得:已遞之審附明、可行之反。著者明知何修(Blocking)、何慮(Suggest)、何善(Praise)。

敗則:若 gh pr review 敗,察權。汝需庫之書權或為所請審者。若行注敗,退以諸反置於審體附 file:line 引。

第五步:續

追審之解。

  1. 著者應或推更後:
    gh pr view <number> --json reviewDecision,reviews
    
  2. 獨再審處汝反之變:
    gh pr diff <number>  # check new commits
    
  3. 批准前驗 Blocking 之入已解
  4. 患解時解其注串
  5. Blocking 入皆修後批准:
    gh pr review <number> --approve --body "All blocking issues resolved. LGTM."
    

得:Blocking 患驗為修。審談已解。PR 已批准或續請變附具餘入。

敗則:若著者異反,於 PR 串議之。專於影(何要)而非權。若異於非阻入,優雅讓——著者擁其碼。

  • PR 境已解(用、大、CI 狀)
  • 諸變文已審(或 XL PR 之最險文)
  • 反依重分(Blocking/Suggest/Nit/Praise)
  • Blocking 入有具修議
  • 至少一 Praise 為陽面
  • 審決合反(獨無 Blocking 入時批准)
  • 行注引具行附重標
  • CI/CD 察已驗(綠後批准)
  • 著者改後續已畢

  • 橡章:未實讀差而批准。各批准乃質之斷
  • Nit 之雪崩:以格之喜淹著者。Nit 留為教境;緊審時略之
  • 失林:行行審而不解整設。先讀 PR 述與提交史
  • 格之 Blocking:式與命幾不為 Blocking。Blocking 留為訛、安、數整
  • 無 Praise:獨指患令意沮。善碼宜認
  • 審範漂:注 PR 未變之碼。若先存患擾汝,立別問題

  • review-software-architecture — 系等構審(補 PR 等審)
  • security-audit-codebase — 為安敏變 PR 行深安析
  • create-pull-request — 程之他面:立易審之 PR
  • commit-changes — 淨之提交史使 PR 審甚易

GitHub 仓库

pjt222/agent-almanac
路径: i18n/wenyan/skills/review-pull-request
0
agentsagentskillsai-assisted-developmentclaude-codeskillsteams

相关推荐技能

llamaguard

其他

LlamaGuard是Meta推出的7-8B参数内容审核模型,专门用于过滤LLM的输入和输出内容。它能检测六大安全风险类别(暴力/仇恨、性内容、武器、违禁品、自残、犯罪计划),准确率达94-95%。开发者可通过HuggingFace、vLLM或Sagemaker快速部署,并能与NeMo Guardrails集成实现自动化安全防护。

查看技能

cost-optimization

其他

这个Claude Skill帮助开发者优化云成本,通过资源调整、标记策略和预留实例来降低AWS、Azure和GCP的开支。它适用于减少云支出、分析基础设施成本或实施成本治理策略的场景。关键功能包括提供成本可视化、资源规模调整指导和定价模型优化建议。

查看技能

quantizing-models-bitsandbytes

其他

这个Skill使用bitsandbytes库量化大语言模型,能在GPU内存有限时通过8位或4位量化减少50-75%内存占用,同时保持精度损失最小。它支持INT8、NF4、FP4等多种量化格式,可与HuggingFace Transformers无缝集成,适用于需要部署更大模型或加速推理的场景。还提供QLoRA训练和8位优化器支持,让开发者能轻松实现高效模型压缩。

查看技能

dispatching-parallel-agents

其他

该Skill用于并行处理3个以上无依赖关系的独立故障,可为每个问题域分派专属Claude代理同时执行调查修复。它通过并发处理多个独立问题显著提升故障排查效率,特别适用于测试文件、子系统等无共享状态的场景。

查看技能