fix(ai-review 同步): 限制自動提交只包含問題檔 #18

2026-06-18T08:08:08Z

jiantw83 commented

2026-06-18 08:08:08 +00:00

變更摘要

將 ai-review 的持久化流程收斂為只提交 .gitea/ai-review/findings.json 與 .gitea/ai-review/exclusions.json，不再同步 action 內建資產或入口檔到目標專案。
移除各平台的 triage skill 資產、入口文件與 Docker image 打包設定。
更新 README/TODO 與單元測試，使文件與測試都對齊新的問題檔提交行為。
清理 .gitea/ai-review/exclusions.json 中已過時的 skill 同步相關排除紀錄。

影響範圍

app/git.js：移除 skill tree、入口檔同步與 instruction merge 邏輯，只複製並提交 review 問題檔。
app/gitea.js：移除已不存在的 skill/入口路徑排除清單。
Dockerfile：不再 COPY skill 與平台入口目錄。
app/git.test.js、app/gitea.test.js：改驗證只提交問題檔、不覆寫 action source 檔，以及新的 diff filter 行為。
專案文件與 ai-review 狀態檔：移除過時的 skill 同步說明與排除紀錄。

驗證

npm test：142 tests passed。
已驗證 .gitea/ai-review/findings.json 與 .gitea/ai-review/exclusions.json 為合法 JSON。
已用 hidden 檔掃描確認沒有 skill/triage-findings 相關文字殘留。

Commits

fix(ai-review 同步): 限制自動提交只包含問題檔
test(ai-review 同步): 更新只提交問題檔的測試
docs(專案說明): 移除 skill 同步相關文件
chore(ai-review 狀態): 移除過時 skill 排除紀錄

## 變更摘要 - 將 ai-review 的持久化流程收斂為只提交 `.gitea/ai-review/findings.json` 與 `.gitea/ai-review/exclusions.json`，不再同步 action 內建資產或入口檔到目標專案。 - 移除各平台的 triage skill 資產、入口文件與 Docker image 打包設定。 - 更新 README/TODO 與單元測試，使文件與測試都對齊新的問題檔提交行為。 - 清理 `.gitea/ai-review/exclusions.json` 中已過時的 skill 同步相關排除紀錄。 ## 影響範圍 - `app/git.js`：移除 skill tree、入口檔同步與 instruction merge 邏輯，只複製並提交 review 問題檔。 - `app/gitea.js`：移除已不存在的 skill/入口路徑排除清單。 - `Dockerfile`：不再 COPY skill 與平台入口目錄。 - `app/git.test.js`、`app/gitea.test.js`：改驗證只提交問題檔、不覆寫 action source 檔，以及新的 diff filter 行為。 - 專案文件與 ai-review 狀態檔：移除過時的 skill 同步說明與排除紀錄。 ## 驗證 - `npm test`：142 tests passed。 - 已驗證 `.gitea/ai-review/findings.json` 與 `.gitea/ai-review/exclusions.json` 為合法 JSON。 - 已用 hidden 檔掃描確認沒有 skill/triage-findings 相關文字殘留。 ## Commits - `fix(ai-review 同步): 限制自動提交只包含問題檔` - `test(ai-review 同步): 更新只提交問題檔的測試` - `docs(專案說明): 移除 skill 同步相關文件` - `chore(ai-review 狀態): 移除過時 skill 排除紀錄`

jiantw83 added 4 commits 2026-06-18 08:08:08 +00:00

fix(ai-review 同步): 限制自動提交只包含問題檔 6f2bbab041

test(ai-review 同步): 更新只提交問題檔的測試 0e08f5c4d8

docs(專案說明): 移除 skill 同步相關文件 7138250b22

chore(ai-review 狀態): 移除過時 skill 排除紀錄

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

Details

AI / Code Review (pull_request) Successful in 1m0s

Details

198bfce64e

admin commented

2026-06-18 08:08:17 +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 merged commit c7e63c9468 into develop

2026-06-18 08:10:06 +00:00

jiantw83 deleted branch ai-review-resolve/20260618080715

2026-06-18 08:10:06 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: actions/code-review#18