actions/calculate-version
2
0
Fork 0
Code Pull Requests 1 Actions Releases 24 Activity

feat(CI 工作流程): 僅在 beta 啟用 AI 審查 #7

Merged
admin merged 5 commits from ai-review-resolve/develop-20260711-150530 into develop 2026-07-11 10:05:54 +00:00
Conversation 2 Commits 5 Files Changed 13
+641 -66
jiantw83 commented 2026-07-11 06:30:39 +00:00
Member
Copy Link
Copy Source

變更摘要

  • 將 .gitea/workflows/ci.yaml 中的 LLM CLI 設定步驟限制為 beta 流程才執行。
  • 將 AI Code Review 步驟同樣限制為 beta 流程，避免正式版 CI 額外觸發審查。

影響範圍

  • .gitea/workflows/ci.yaml

注意事項

  • 這是 CI 流程行為調整，會影響 beta 與非 beta 的工作流程路徑。
## 變更摘要 - 將 `.gitea/workflows/ci.yaml` 中的 LLM CLI 設定步驟限制為 beta 流程才執行。 - 將 AI Code Review 步驟同樣限制為 beta 流程，避免正式版 CI 額外觸發審查。 ## 影響範圍 - `.gitea/workflows/ci.yaml` ## 注意事項 - 這是 CI 流程行為調整，會影響 beta 與非 beta 的工作流程路徑。
jiantw83 added 1 commit 2026-07-11 06:30:40 +00:00
feat(CI 工作流程): 僅在 beta 啟用 AI 審查
CI / 2. TEST (pull_request) Successful in 45s
Details
CI / 3. RESULT (pull_request) Has been skipped
Details
CI / 1. BUILD (pull_request) Successful in 2s
Details
5883abd306
jiantw83 added 3 commits 2026-07-11 10:00:44 +00:00
fix(calculate-version): 統一輸出與錯誤格式 88e3e3d184
chore(action 設定): 整理用途標頭與排版 1ed40a6b04
docs(README 與文件草稿): 重整說明與草稿內容
CI / 1. BUILD (pull_request) Successful in 1s
Details
CI / 2. TEST (pull_request) Successful in 3m32s
Details
CI / 3. RESULT (pull_request) Has been skipped
Details
24ed2ab164
admin commented 2026-07-11 10:01:21 +00:00
Owner
Copy Link
Copy Source

🤖 AI Code Review 團隊

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

🔍 服務：codex 模型：gpt-5.4-mini

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

AI Code Review 統計

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

🤖 AI 助理使用量

本次審查（codex / gpt-5.4-mini，共 13 次呼叫）

提示 token 回應 token 合計
0 0 0

剩餘可用

剩餘可用：無法計算百分比（未支援 codex 額度查詢）

## AI Code Review 統計 | 類型 | 🔴 嚴重 | 🟡 警告 | 🔵 建議 | ⚪ 無法標示 | | --- | --- | --- | --- | --- | | 新問題 | 0 筆 | 2 筆 | 0 筆 | 0 筆 | | 舊問題 | 0 筆 | 0 筆 | 0 筆 | 0 筆 | ## 🤖 AI 助理使用量 **本次審查**（codex / gpt-5.4-mini，共 13 次呼叫） | 提示 token | 回應 token | 合計 | | --- | --- | --- | | 0 | 0 | 0 | **剩餘可用** 剩餘可用：無法計算百分比（未支援 codex 額度查詢）
src/index.js
@@ -269,3 +399,3 @@
if (require.main === module) {
main().catch((error) => {
process.stderr.write(`[error] ${error.message}\n`);
const timestamp = new Date()
admin commented 2026-07-11 10:04:19 +00:00
Owner
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：這裡在 catch 裡重新手寫一次時間戳與前綴格式，和前面的 info() 形成兩套彼此獨立的輸出格式。未來只要調整 log 格式、時區或錯誤前綴，就必須記得同步兩處，半年後很容易出現訊息格式不一致的問題。
建議：把時間戳與 prefix 組裝抽成共用的 formatter，例如 formatLogLine() 或 formatTimestamp()，讓 info() 與錯誤處理共用同一套輸出規則。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：這裡在 `catch` 裡重新手寫一次時間戳與前綴格式，和前面的 `info()` 形成兩套彼此獨立的輸出格式。未來只要調整 log 格式、時區或錯誤前綴，就必須記得同步兩處，半年後很容易出現訊息格式不一致的問題。 **建議**：把時間戳與 prefix 組裝抽成共用的 formatter，例如 `formatLogLine()` 或 `formatTimestamp()`，讓 `info()` 與錯誤處理共用同一套輸出規則。
admin commented 2026-07-11 10:04:19 +00:00
Owner
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Maya
問題：這次把錯誤輸出從單純的 [error] ... 改成包含 currentStage 與台灣時區時間戳的格式，但沒有任何測試覆蓋失敗路徑。只要之後格式字串、stage 前綴或 process.exit(1) 的行為回歸，CI 很容易看不出來。
建議：補一組針對 main().catch() 的單元測試：模擬 main() 拋錯或回傳 rejected promise，先設定 currentStage，再斷言 process.stderr.write 寫出的內容包含 [階段][ERR][YYYY/MM/DD ...]: 錯誤訊息，同時驗證 process.exit(1) 有被呼叫。建議再加一個 currentStage === null 時的案例，確認前綴會正確省略。

**嚴重等級**：🟡 警告 **審查員**：Maya **問題**：這次把錯誤輸出從單純的 `[error] ...` 改成包含 `currentStage` 與台灣時區時間戳的格式，但沒有任何測試覆蓋失敗路徑。只要之後格式字串、stage 前綴或 `process.exit(1)` 的行為回歸，CI 很容易看不出來。 **建議**：補一組針對 `main().catch()` 的單元測試：模擬 `main()` 拋錯或回傳 rejected promise，先設定 `currentStage`，再斷言 `process.stderr.write` 寫出的內容包含 `[階段][ERR][YYYY/MM/DD ...]: 錯誤訊息`，同時驗證 `process.exit(1)` 有被呼叫。建議再加一個 `currentStage === null` 時的案例，確認前綴會正確省略。
admin added 1 commit 2026-07-11 10:04:21 +00:00
chore: update ai-review findings [ai-review-bot][success]
CI / 1. BUILD (pull_request) Successful in 1s
Details
CI / 2. TEST (pull_request) Successful in 32s
Details
CI / 3. RESULT (pull_request) Has been skipped
Details
96cb99c136
admin merged commit c60b71e987 into develop 2026-07-11 10:05:54 +00:00
admin deleted branch ai-review-resolve/develop-20260711-150530 2026-07-11 10:05:54 +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: actions/calculate-version#7