SKILL·21E1E0

review-pull-request

pjt222
Aktualisiert 3 months ago
13 Ansichten
31
3
31
Auf GitHub ansehen
Andereai

Über

Diese Claude Skill führt umfassende Pull-Request-Reviews unter Verwendung der GitHub CLI durch, analysiert Diffs, Commit-Verläufe und CI/CD-Prüfungen. Sie bietet Feedback mit Schweregraden (blockierend/Empfehlung/Kleinigkeit/Lob) und übermittelt Reviews via `gh pr review`. Nutzen Sie sie, wenn Sie zur Überprüfung eines PR zugewiesen sind, zur Selbstprüfung vor Feedback-Einholung, für Folge-Reviews nach Änderungen oder für Qualitätsaudits nach dem Merge.

Schnellinstallation

Claude Code

Empfohlen
Primär
npx skills add pjt222/agent-almanac -a claude-code
Plugin-BefehlAlternativ
/plugin add https://github.com/pjt222/agent-almanac
Git CloneAlternativ
git clone https://github.com/pjt222/agent-almanac.git ~/.claude/skills/review-pull-request

Kopieren Sie diesen Befehl und fügen Sie ihn in Claude Code ein, um diese Fähigkeit zu installieren

Dokumentation

審拉請

由 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 Repository

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

Häufig gestellte Fragen

Was ist der Skill review-pull-request?

review-pull-request ist ein Claude Skill von pjt222. Skills bündeln Anweisungen und Ressourcen, die Claude bei Bedarf lädt, um Aufgaben rund um review-pull-request ohne zusätzliche Eingaben auszuführen.

Wie installiere ich review-pull-request?

Verwende die Installationsbefehle auf dieser Seite: Füge review-pull-request als Plugin zu Claude Code hinzu oder klone das Repository in dein Skills-Verzeichnis. Starte Claude danach neu, damit der Skill geladen wird.

Zu welcher Kategorie gehört review-pull-request?

review-pull-request gehört zur Kategorie Andere.

Kann ich review-pull-request kostenlos nutzen?

Ja. review-pull-request ist auf AIMCP gelistet und kann kostenlos installiert werden.

Verwandte Skills

llamaguard
Andere

LlamaGuard ist Metas 7-8B-Parameter-Modell zur Moderation von LLM-Eingaben und -Ausgaben in sechs Sicherheitskategorien wie Gewalt und Hassrede. Es bietet eine Genauigkeit von 94-95 % und kann mit vLLM, Hugging Face oder Amazon SageMaker eingesetzt werden. Nutzen Sie diese Skill, um Inhaltsfilterung und Sicherheitsguardrails einfach in Ihre KI-Anwendungen zu integrieren.

Skill ansehen
cost-optimization
Andere

Diese Claude Skill unterstützt Entwickler bei der Optimierung von Cloud-Kosten durch Ressourcen-Dimensionierung, Tagging-Strategien und Ausgabenanalysen. Sie bietet einen Rahmen zur Senkung von Cloud-Ausgaben und zur Implementierung von Kosten-Governance für AWS, Azure und GCP. Nutzen Sie sie, wenn Sie Infrastrukturkosten analysieren, Ressourcen richtig dimensionieren oder Budgetvorgaben einhalten müssen.

Skill ansehen
sports-betting-analyzer
Andere

Diese Claude Skill analysiert Sportwettenmärkte inklusive Handicaps, Over/Unders und Spezialwetten, indem sie historische Trends und situative Statistiken untersucht, um Wertwetten zu identifizieren. Sie liefert strukturierte Markdown-Ausgaben mit umsetzbaren Empfehlungen zu Bildungszwecken. Entwickler sollten dies für Sportwetten-Analysetools nutzen, wobei zu beachten ist, dass es nur zur Unterhaltung/Bildung konzipiert wurde.

Skill ansehen
quantizing-models-bitsandbytes
Andere

Diese Fähigkeit quantisiert LLMs auf 8-Bit- oder 4-Bit-Präzision mittels bitsandbytes und erreicht dabei eine Speicherreduzierung von 50–75 % bei minimalem Genauigkeitsverlust. Sie ist ideal für den Betrieb größerer Modelle mit begrenztem GPU-Speicher oder zur Beschleunigung von Inferenzvorgängen und unterstützt Formate wie INT8, NF4 und FP4. Die Fähigkeit integriert sich in HuggingFace Transformers und ermöglicht QLoRA-Training sowie 8-Bit-Optimierer.

Skill ansehen