receiving-code-review
以技術嚴謹性處理程式碼審查回饋
也可從以下取得: Cygnusfear,ZhanlinCui,DYAI2025,obra,davila7,CodingCossack,Cycleaddict
程式碼審查需要技術評估,而非情緒性表現。此技能指導 AI 助理在實作前驗證回饋、提出澄清問題,並在建議技術上不健全時提供有根據的反饋。
下載技能 ZIP
在 Claude 中上傳
前往 設定 → 功能 → 技能 → 上傳技能
開啟並開始使用
測試它
正在使用「receiving-code-review」。 審查者:移除舊版 API 版本的舊程式碼路徑
預期結果:
正在檢查相容性需求... 建構目標是 10.15+,但這個 API 需要 13+。舊版路徑仍然是向後相容所必需的。然而,目前的實作有錯誤的 bundle ID。我可以修正 bundle ID,或者我們可以放棄 13 之前的支援。您偏好哪個方向?
正在使用「receiving-code-review」。 審查者:修正 PR 中的第 1-6 項
預期結果:
我理解第 1、2、3 和 6 項。在實作之前我需要澄清第 4 和 5 項。您能詳細說明這兩項需要什麼樣的變更嗎?
正在使用「receiving-code-review」。 審查者:新增適當的錯誤處理,包含重試邏輯和指數後退
預期結果:
已修正。在 api_service.rb:42-58 的 API 呼叫中新增了 Try/Catch 與重試邏輯(3 次嘗試,從 100ms 開始的指數後退)。測試通過。
安全審計
安全Static analysis flagged 24 instances of backtick patterns and 4 blocker patterns, but all are FALSE POSITIVES. The skill file (SKILL.md) is documentation-only markdown containing instructional text for AI behavior. No executable code, network calls, file operations, or cryptographic functions exist. The backtick patterns are markdown code blocks showing workflow examples, not shell execution. Blocker patterns are misidentified text fragments in documentation.
品質評分
你能建構什麼
AI 助理接收人類審查
AI 助理收到人類開發者的程式碼審查回饋時,需要判斷哪些項目應該實作、哪些需要澄清,以及哪些應該以技術理由反饋。
初階開發者學習審查回應
初階開發者學習如何以技術嚴謹性回應資深開發者的回饋,而非表演性同意,在盲目實作前驗證建議。
外部貢獻者處理維護者回饋
外部貢獻者收到專案維護者的回饋時,需要在實作前評估建議是否符合專案架構。
試試這些提示
我收到了包含 6 個項目的程式碼審查回饋。我理解第 1、2、3 和 6 項,但在繼續之前需要澄清第 4 和 5 項。您能說明這些特定項目需要什麼樣的變更嗎?
您移除舊版相容層的建議會破壞 macOS 10.15 的支援。目前的實作目標是 10.15+,但使用了需要 13+ 的 API。我應該修正 bundle ID 以維持向後相容,還是完全放棄 13 之前的支援?
審查者建議實作完整的指標追蹤,包含資料庫儲存、日期過濾和 CSV 匯出。我搜尋了程式碼庫,沒有找到這個端點的呼叫者。我應該依照 YAGNI 移除它,還是有我遺漏的使用情境?
我對您的建議提出了反饋,但在檢查測試結果後您是對的。我的理解有誤,因為我忽略了整合測試失敗。現在正在實作修正。
最佳實務
- 在實作任何變更前,始終針對實際程式碼庫驗證回饋
- 一次實作一個變更並單獨測試每個變更,以便及早捕捉回歸問題
- 當建議會破壞現有功能時,以技術理由提出反饋
避免
- 在驗證回饋之前說「您完全正確!」或「說得好!」
- 盲目實作回饋而不檢查是否會破壞現有功能
- 實作第 1-3 和 6 項同時跳過不清楚的第 4-5 項而不先詢問