MCP HubMCP Hub
Retour aux compétences

review-pull-request

pjt222
Mis à jour 2 days ago
8 vues
17
2
17
Voir sur GitHub
Autreai

À propos

Cette compétence Claude effectue des revues exhaustives de demandes de tirage (pull requests) en utilisant GitHub CLI, en analysant les différences, l'historique des commits et les vérifications CI/CD. Elle fournit des retours hiérarchisés par niveau de sévérité (bloquant/suggestion/détail mineur/éloge) et soumet les revues via `gh pr review`. Utilisez-la lorsqu'elle est assignée pour revoir une PR, pour une auto-revue avant de solliciter des retours, pour des revues de suivi après modifications, ou pour des audits qualité post-fusion.

Installation rapide

Claude Code

Recommandé
Principal
npx skills add pjt222/agent-almanac -a claude-code
Commande PluginAlternatif
/plugin add https://github.com/pjt222/agent-almanac
Git CloneAlternatif
git clone https://github.com/pjt222/agent-almanac.git ~/.claude/skills/review-pull-request

Copiez et collez cette commande dans Claude Code pour installer cette compétence

Documentation

審拉請

由 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 審甚易

Dépôt GitHub

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

Compétences associées

llamaguard

Autre

LlamaGuard est le modèle de Meta, doté de 7 à 8 milliards de paramètres, conçu pour modérer les entrées et sorties des LLM selon six catégories de sécurité comme la violence et les discours haineux. Il offre une précision de 94 à 95 % et peut être déployé avec vLLM, Hugging Face ou Amazon SageMaker. Utilisez cette compétence pour intégrer facilement le filtrage de contenu et des garde-fous de sécurité dans vos applications d'IA.

Voir la compétence

cost-optimization

Autre

Cette compétence de Claude aide les développeurs à optimiser les coûts du cloud grâce au redimensionnement des ressources, aux stratégies d'étiquetage et à l'analyse des dépenses. Elle fournit un cadre pour réduire les dépenses cloud et mettre en œuvre une gouvernance des coûts sur AWS, Azure et GCP. Utilisez-la lorsque vous devez analyser les coûts d'infrastructure, redimensionner les ressources ou respecter des contraintes budgétaires.

Voir la compétence

quantizing-models-bitsandbytes

Autre

Cette compétence quantifie les LLMs en précision 8 bits ou 4 bits à l'aide de bitsandbytes, permettant une réduction de 50 à 75 % de la mémoire utilisée avec une perte de précision minime. Elle est idéale pour exécuter des modèles plus volumineux sur une mémoire GPU limitée ou pour accélérer l'inférence, prenant en charge des formats comme INT8, NF4 et FP4. La compétence s'intègre à HuggingFace Transformers et permet l'entraînement QLoRA ainsi que l'utilisation d'optimiseurs en 8 bits.

Voir la compétence

dispatching-parallel-agents

Autre

Cette compétence Claude déploie plusieurs agents pour enquêter et résoudre simultanément 3 problèmes indépendants ou plus. Elle est conçue pour des scénarios impliquant des défaillances non liées qui peuvent être résolues sans état partagé ni dépendances. La capacité fondamentale est la résolution de problèmes en parallèle, en assignant un agent par domaine problématique indépendant afin de maximiser l'efficacité.

Voir la compétence