集中 AI Review 留言並補齊 TLS 測試 #26

2026-06-22T09:06:18Z

jiantw83 commented

2026-06-22 09:06:18 +00:00

變更摘要

將 AI Review 的 PR 留言集中到單一主 comment，後續舊問題、新問題與嚴重問題摘要都透過更新同一則 comment 呈現。
新增 Gitea issue comment 更新 API 封裝，讓 pipeline 可 patch 既有 comment。
補齊 comment 聚合模式與 OpenCode TLS skip 設定的測試覆蓋。
更新 README 流程說明，並清空已解決的 ai-review findings。

影響範圍

app/main.js：建立主 review comment 後，後續 finding 區塊 append 並更新同一則 comment。
app/comments.js：comment helper 支援注入更新函式，critical 在集中模式下不再強制發行內 comment。
app/gitea.js：新增 updateComment 對應 Gitea issue comment patch API。
app/*test.js：補上 update comment、集中 critical comment、OpenCode TLS skip 的測試。
README.md 與 .gitea/ai-review/findings.json：同步流程文件與 review 狀態。

驗證

npm test：152 個測試通過。

風險與注意事項

方案 1 會把 critical findings 也集中到主 comment，不再建立行內 review comment；這符合本次需求，但會少掉 diff 行上的視覺標註。
Gitea comment 更新依賴 PATCH /repos/{owner}/{repo}/issues/comments/{id}，若目標 Gitea 版本權限或 API 行為不同，可能需要回退成新增 comment。

## 變更摘要 - 將 AI Review 的 PR 留言集中到單一主 comment，後續舊問題、新問題與嚴重問題摘要都透過更新同一則 comment 呈現。 - 新增 Gitea issue comment 更新 API 封裝，讓 pipeline 可 patch 既有 comment。 - 補齊 comment 聚合模式與 OpenCode TLS skip 設定的測試覆蓋。 - 更新 README 流程說明，並清空已解決的 ai-review findings。 ## 影響範圍 - `app/main.js`：建立主 review comment 後，後續 finding 區塊 append 並更新同一則 comment。 - `app/comments.js`：comment helper 支援注入更新函式，critical 在集中模式下不再強制發行內 comment。 - `app/gitea.js`：新增 `updateComment` 對應 Gitea issue comment patch API。 - `app/*test.js`：補上 update comment、集中 critical comment、OpenCode TLS skip 的測試。 - `README.md` 與 `.gitea/ai-review/findings.json`：同步流程文件與 review 狀態。 ## 驗證 - `npm test`：152 個測試通過。 ## 風險與注意事項 - 方案 1 會把 critical findings 也集中到主 comment，不再建立行內 review comment；這符合本次需求，但會少掉 diff 行上的視覺標註。 - Gitea comment 更新依賴 `PATCH /repos/{owner}/{repo}/issues/comments/{id}`，若目標 Gitea 版本權限或 API 行為不同，可能需要回退成新增 comment。

jiantw83 added 4 commits 2026-06-22 09:06:18 +00:00

feat(ai-review comment): 集中更新單一審查留言 8e4ea97dac

test(ai-review comment): 補齊留言更新與 TLS 設定覆蓋 f0544bb758

docs(README): 更新 AI Review 留言流程說明 3b4ea2dad0

chore(ai-review 狀態): 移除已解決 findings

AI / 計算版本號 (pull_request) Successful in 2s

Details

AI / Code Review (pull_request) Successful in 1m30s

Details

ff9413e48b

admin commented

2026-06-22 09:06:43 +00:00

🤖 AI Code Review 團隊

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

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

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

jiantw83 closed this pull request

2026-06-22 09:15:44 +00:00

jiantw83 deleted branch ai-review-resolve/20260622090525

2026-06-22 09:16:05 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: old-actions/code-review#26