MCP HubMCP Hub
SKILL·21E1E0

review-pull-request

pjt222
Actualizado 1 month ago
10 vistas
24
3
24
Ver en GitHub
Otroai

Acerca de

Esta Skill de Claude realiza revisiones exhaustivas de pull requests utilizando GitHub CLI, analizando diffs, historial de commits y verificaciones de CI/CD. Proporciona retroalimentación con niveles de severidad (bloqueante/sugerencia/observación menor/elogio) y envía las revisiones mediante `gh pr review`. Úsela cuando se le asigne revisar un PR, para autoevaluación antes de buscar comentarios, para revisiones de seguimiento después de cambios, o para auditorías de calidad posteriores a la fusión.

Instalación rápida

Claude Code

Recomendado
Principal
npx skills add pjt222/agent-almanac -a claude-code
Comando PluginAlternativo
/plugin add https://github.com/pjt222/agent-almanac
Git CloneAlternativo
git clone https://github.com/pjt222/agent-almanac.git ~/.claude/skills/review-pull-request

Copia y pega este comando en Claude Code para instalar esta habilidad

Documentación

審拉請

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

Repositorio GitHub

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

Frequently asked questions

What is the review-pull-request skill?

review-pull-request is a Claude Skill by pjt222. Skills package instructions and resources that Claude loads on demand, so Claude can perform review-pull-request-related tasks without extra prompting.

How do I install review-pull-request?

Use the install commands on this page: add review-pull-request to Claude Code as a plugin, or clone its repository into your skills directory, then restart Claude so it picks up the skill.

What category does review-pull-request belong to?

review-pull-request is in the Other category, tagged ai.

Is review-pull-request free to use?

Yes. review-pull-request is listed on AIMCP and free to install. It runs inside Claude, so no separate service account is required to use the skill itself.

Habilidades relacionadas

llamaguard
Otro

LlamaGuard es el modelo de Meta de 7-8B parámetros para moderar las entradas y salidas de LLM en seis categorías de seguridad como violencia y discurso de odio. Ofrece una precisión del 94-95% y puede implementarse usando vLLM, Hugging Face o Amazon SageMaker. Utiliza esta skill para integrar fácilmente filtrado de contenido y barreras de seguridad en tus aplicaciones de IA.

Ver habilidad
cost-optimization
Otro

Esta Skill de Claude ayuda a los desarrolladores a optimizar los costes en la nube mediante el ajuste de tamaño de recursos, estrategias de etiquetado y análisis de gastos. Proporciona un marco para reducir los gastos en la nube e implementar una gobernanza de costes en AWS, Azure y GCP. Úsala cuando necesites analizar los costes de infraestructura, ajustar el tamaño de los recursos o cumplir con restricciones presupuestarias.

Ver habilidad
sports-betting-analyzer
Otro

Esta habilidad de Claude analiza los mercados de apuestas deportivas, incluyendo spreads, over/unders y apuestas de propuestas, mediante el examen de tendencias históricas y estadísticas situacionales para identificar apuestas de valor. Proporciona una salida en markdown estructurado con recomendaciones accionables con fines educativos. Los desarrolladores deben utilizar esto para herramientas de análisis de apuestas deportivas, teniendo en cuenta que está diseñado únicamente para entretenimiento/educación.

Ver habilidad
quantizing-models-bitsandbytes
Otro

Esta habilidad cuantiza LLMs a precisión de 8 o 4 bits utilizando bitsandbytes, logrando una reducción de memoria del 50-75% con pérdida mínima de precisión. Es ideal para ejecutar modelos más grandes en memoria GPU limitada o para acelerar la inferencia, admitiendo formatos como INT8, NF4 y FP4. La habilidad se integra con HuggingFace Transformers y permite entrenamiento QLoRA y optimizadores de 8 bits.

Ver habilidad