old-actions/code-review
2
0
Fork 0
Code Pull Requests Actions Releases 2 Activity

refactor(pipeline 日誌): pipeline 日誌可讀性優化（步驟連號、輸入輸出、成敗明確） #43

Merged
jiantw83 merged 4 commits from ai-review-resolve/20260623-151524 into ai-review-resolve/20260623-110950 2026-06-23 07:18:10 +00:00
Conversation 2 Commits 4 Files Changed 9
+144 -159
jiantw83 commented 2026-06-23 07:16:07 +00:00
Member
Copy Link
Copy Source

變更摘要

優化 AI Code Review pipeline 的 log 輸出可讀性，讓每個階段任務單一、輸入與輸出明確、檢查成敗一目了然。

影響範圍與重點檔案

  • app/log.js：新增 input()（← 輸入）、output()（→ 輸出）、result(passed, msg)（✅ 成功／❌ 失敗）三個日誌 helper。
  • app/main.js：
    • 修掉原本兩個 Step2 撞號問題，步驟重新連號為 Step1–Step11，每步單一任務。
    • 每個階段開頭以 ← 輸入 標示吃進什麼、結尾以 → 輸出 標示產出什麼。
    • 把關步驟（前置驗證、JSON 驗證、嚴重問題把關）改用 ✅/❌ 明確標示成功或失敗。
    • 移除 Step4/Step6 冗餘的多行統計，各保留一行「輸入 → 輸出」。
  • app/gitea.js：清除 bot-check commit api: keys=…、bot-check start/sha/branch … 等大量 debug log；shouldSkipBotCommit 改為安靜判斷，由 Step3 統一回報結果。
  • app/preflight.js：前置驗證結尾改用 ✅ 成功：前置驗證通過。
  • 測試：app/log.test.js 補 input/output/result 格式測試；另補 filterFalsePositivesWithAI 惡劣輸入保守保留、extractUsage 異常 payload 不崩潰、parseBotReviewComment 多行注入只取第一行等健壯性測試。

風險與注意事項

  • 純屬 log 輸出與測試調整，不更動審查流程的功能行為。
  • node --test app/*.test.js 全數通過（241 passed）。
  • 本 PR 以 ai-review-resolve/20260623-110950 為 base（stacked 子 PR），合併後再隨母分支進入 develop。
## 變更摘要 優化 AI Code Review pipeline 的 log 輸出可讀性，讓每個階段任務單一、輸入與輸出明確、檢查成敗一目了然。 ## 影響範圍與重點檔案 - `app/log.js`：新增 `input()`（`← 輸入`）、`output()`（`→ 輸出`）、`result(passed, msg)`（`✅ 成功`／`❌ 失敗`）三個日誌 helper。 - `app/main.js`： - 修掉原本兩個 `Step2` 撞號問題，步驟重新**連號為 Step1–Step11**，每步單一任務。 - 每個階段開頭以 `← 輸入` 標示吃進什麼、結尾以 `→ 輸出` 標示產出什麼。 - 把關步驟（前置驗證、JSON 驗證、嚴重問題把關）改用 `✅/❌` 明確標示成功或失敗。 - 移除 Step4/Step6 冗餘的多行統計，各保留一行「輸入 → 輸出」。 - `app/gitea.js`：清除 `bot-check commit api: keys=…`、`bot-check start/sha/branch …` 等大量 debug log；`shouldSkipBotCommit` 改為安靜判斷，由 Step3 統一回報結果。 - `app/preflight.js`：前置驗證結尾改用 `✅ 成功：前置驗證通過`。 - 測試：`app/log.test.js` 補 `input/output/result` 格式測試；另補 `filterFalsePositivesWithAI` 惡劣輸入保守保留、`extractUsage` 異常 payload 不崩潰、`parseBotReviewComment` 多行注入只取第一行等健壯性測試。 ## 風險與注意事項 - 純屬 log 輸出與測試調整，**不更動審查流程的功能行為**。 - `node --test app/*.test.js` 全數通過（241 passed）。 - 本 PR 以 `ai-review-resolve/20260623-110950` 為 base（stacked 子 PR），合併後再隨母分支進入 develop。
jiantw83 added 3 commits 2026-06-23 07:16:07 +00:00
refactor(pipeline 日誌): 步驟連號單一任務、輸入輸出與成敗訊息更明確、清除 bot-check 雜訊 374bb4ec75
test(ai-review): 補日誌 input/output/result helper 與誤報裁決、usage 解析、parseBot 健壯性測試 756b2cd4ef
chore(ai-review 狀態): findings 已由測試涵蓋，清空 findings.json
AI / 計算版本號 (pull_request) Successful in 2s
Details
AI / Code Review (pull_request) Successful in 38s
Details
ccd830234e
admin commented 2026-06-23 07:16:15 +00:00
Owner
Copy Link
Copy Source

🤖 AI Code Review 團隊

👤 角色 🎯 面向 🧠 個性
🗡️ Assassin security 多疑偏執、以攻擊者視角看世界，假設每筆輸入都是惡意的，每個信任都會被濫用
🎼 Bard style 唯美龜毛、追求優雅，把可讀性與一致性當作旋律，最受不了走調的命名與排版
🧰 Leo maintainability 有遠見、重視長期維護成本，凡事先問「六個月後的自己還看得懂嗎？」，討厭把債留給未來
🔮 Mage logic 嚴謹冷靜、滴水不漏，凡事推演到最壞情況，深信「沒驗證過的假設都是 bug」
🧪 Maya testing 對測試覆蓋率有執念，深信「沒有測試的程式碼等於沒寫完」，溫和但堅持，最在意邊界與失敗路徑
⚡ Rogue efficiency 急性子、講求速度，最痛恨被浪費的 CPU 週期與記憶體，凡事先問「這能不能更快、更省」

🔍 服務：opencode 模型：gemini-2.5-flash

## 🤖 AI Code Review 團隊 | 👤 角色 | 🎯 面向 | 🧠 個性 | |--------|--------|--------| | **🗡️ Assassin** | security | 多疑偏執、以攻擊者視角看世界，假設每筆輸入都是惡意的，每個信任都會被濫用 | | **🎼 Bard** | style | 唯美龜毛、追求優雅，把可讀性與一致性當作旋律，最受不了走調的命名與排版 | | **🧰 Leo** | maintainability | 有遠見、重視長期維護成本，凡事先問「六個月後的自己還看得懂嗎？」，討厭把債留給未來 | | **🔮 Mage** | logic | 嚴謹冷靜、滴水不漏，凡事推演到最壞情況，深信「沒驗證過的假設都是 bug」 | | **🧪 Maya** | testing | 對測試覆蓋率有執念，深信「沒有測試的程式碼等於沒寫完」，溫和但堅持，最在意邊界與失敗路徑 | | **⚡ Rogue** | efficiency | 急性子、講求速度，最痛恨被浪費的 CPU 週期與記憶體，凡事先問「這能不能更快、更省」 | > 🔍 服務：opencode 模型：gemini-2.5-flash
admin reviewed 2026-06-23 07:16:49 +00:00
admin left a comment
Owner
Copy Link
Copy Source

AI Code Review 統計

類型 🔴 嚴重 🟡 警告 🔵 建議 ⚪ 無法標示
新問題 0 筆 1 筆 0 筆 0 筆
舊問題 0 筆 0 筆 0 筆 0 筆

🤖 AI 助理使用量

本次審查（opencode / gemini-2.5-flash，共 9 次呼叫）

提示 token 回應 token 合計
87,752 3,290 146,720

剩餘可用

剩餘可用：無法計算百分比（自架服務，無帳號額度概念）

## AI Code Review 統計 | 類型 | 🔴 嚴重 | 🟡 警告 | 🔵 建議 | ⚪ 無法標示 | | --- | --- | --- | --- | --- | | 新問題 | 0 筆 | 1 筆 | 0 筆 | 0 筆 | | 舊問題 | 0 筆 | 0 筆 | 0 筆 | 0 筆 | ## 🤖 AI 助理使用量 **本次審查**（opencode / gemini-2.5-flash，共 9 次呼叫） | 提示 token | 回應 token | 合計 | | --- | --- | --- | | 87,752 | 3,290 | 146,720 | **剩餘可用** 剩餘可用：無法計算百分比（自架服務，無帳號額度概念）
admin added 1 commit 2026-06-23 07:16:52 +00:00
chore: update ai-review findings [ai-review-bot][success]
AI / 計算版本號 (pull_request) Successful in 2s
Details
AI / Code Review (pull_request) Successful in 3s
Details
51585345ae
jiantw83 merged commit 43d7bc9911 into ai-review-resolve/20260623-110950 2026-06-23 07:18:10 +00:00
jiantw83 deleted branch ai-review-resolve/20260623-151524 2026-06-23 07:18:10 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
admin
Labels
No items
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
admin (系統管理員) jiantw83 (Jeffery)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: old-actions/code-review#43