Community程式設計與開發github.com

sandeshp1995/claude-code-team-kit

審查程式碼變更的正確性、可讀性、測試與基本安全性。在使用者要求審查其變更、比對差異 (diff) 或分支,或是在開啟合併請求 (merge request) 前使用此技能。

claude-code-team-kit 是什麼?

此技能專為程式碼審查設計,能自動分析使用者所指明的變更,或若未指定時自動比對目前分支與其基礎分支(依序嘗試 `origin/main`、`origin/master`、`origin/develop`),若無基礎分支則審查未提交的變更 (`git diff HEAD`)。它會完整讀取 diff,並對難以僅從 diff 判斷的變更,開啟周圍程式碼進行輔助判斷。 技能提供一份結構化檢查清單,涵蓋正確性(邊界情況、錯誤處理)、測試(行為測試而非實作細節測試)、可讀性(命名準確性、無殘留的除錯或註解程式碼)、基本安全性(無機密資訊、輸入驗證與參數化查詢)以及一致性(遵循 `CLAUDE.md` 與週遭程式碼的慣例)。 輸出結果按嚴重程度分組:「必須修正」(錯誤、安全問題、遺漏危險邏輯的測試)、「建議修正」(未來會耗時的可維護性問題)、「微調」(選擇性的潤飾)。每個發現附帶 `file:line`、問題說明與具體修改建議,最終給出一行結論:*可合併*、*在必須修正後可合併* 或 *需要再次審查*。若某一類別無發現則省略該類別,不會憑空捏造問題。 Pro 版本額外提供完整的安全、效能、無障礙與 API 設計檢查清單,以及 `/review` 命令,能自動為 diff 挑選適當的檢查項目。此技能支援任何能執行 git 操作的平台(如 Claude Code、Cursor、Codex 等 CLI 工具)。

相容平台✓Claude Code~Codex CLI~Cursor
npx skills add https://github.com/sandeshp1995/claude-code-team-kit/tree/HEAD/.claude/skills/code-review-basics

Installed? Explore more 程式設計與開發 skills: steipete/bluebubbles, steipete/eightctl, steipete/blucli · View all 6 →

在你喜歡的 AI 中提問

開啟一個已預先載入此 Agent Skill 的新對話。

說明文件

程式碼審查基礎

審查使用者指出的變更。若使用者未指定,則審查目前分支相對於其基礎分支的變更:

  1. 找出基礎分支:依序嘗試 git merge-base HEAD origin/main、origin/master、origin/develop。若都不存在,則審查未提交的變更 (git diff HEAD)。
  2. 讀取完整的 diff,然後對任何無法僅從 diff 判斷的變更,開啟周圍的程式碼。

檢查清單

逐項檢查。僅回報有實際發現的項目。

正確性

  • 程式碼是否達成變更預期的目的?請檢查差一錯誤、錯誤的條件、未處理的 None/null,以及被吞沒的例外。
  • 是否處理了邊界情況:空輸入、極大輸入、並發呼叫、重試?

測試

  • 每個行為變更是否都有測試覆蓋,且測試在缺少該變更時會失敗?
  • 測試是否檢查行為,而非實作細節?

可讀性

  • 名稱是否準確?新加入的同事能否一次讀懂?
  • 是否遺留了死碼、除錯輸出或註解掉的程式碼?

安全性(基本)

  • 程式碼、設定檔或記錄中無機密資訊、權杖或憑證。
  • 使用者輸入已驗證,查詢已參數化,絕不透過字串串接來建構。

一致性

  • 是否遵循 CLAUDE.md 及周圍程式碼中的慣例?

輸出

依嚴重程度分組發現:

  • 必須修正:錯誤、安全問題、危險邏輯的遺漏測試。
  • 建議修正:未來會耗時的可維護性問題。
  • 微調:選擇性的潤飾。請保持在少數。

對每個發現,提供 file:line、問題說明以及具體的修改建議。最後以一行結論結尾:可合併、必須修正後可合併 或 需要再次審查。若某一類別無任何發現,則省略該類別。請勿憑空捏造問題。

Pro 版本額外增加了完整的安全、效能、無障礙及 API 設計檢查清單,以及 /review 命令,能為 diff 自動挑選適當的檢查項目。

Individual skills in this repo

This repo contains 3 individual skills — each has its own dedicated page.

相關技能