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

整理 AI review findings 與 Docker action 設定 #14

Open
jiantw83 wants to merge 8 commits from ai-review-resolve/develop-20260629-104700 into develop
pull from: ai-review-resolve/develop-20260629-104700
merge into: :develop
Conversation 3 Commits 8 Files Changed 13
+51 -174
jiantw83 commented 2026-06-29 10:49:56 +00:00
Member
Copy Link
Copy Source

變更摘要

  • 移除 git 網路操作強制關閉 SSL 驗證，避免遠端存取被中間人污染或外洩 token。
  • OpenRouter 額度查詢改回預設 HTTPS 憑證驗證，避免外部 API key 在不安全連線中傳送。
  • 明確化 LLM CLI 參數組裝與 preflight 成功訊息，補齊 verifyLLM 回傳欄位文件。
  • 整理 Docker action 執行設定，並將已處理的 AI review findings 清空，新增一筆相容性排除。

影響範圍

  • app/git.js、app/usage.js 與對應測試更新 TLS 驗證行為。
  • app/config.js、app/llm.js、app/preflight.js 更新 CLI 與內部憑證相關說明。
  • action.yaml、Dockerfile、.gitea/workflows/ci.yaml 與 package.json 整理 Docker action 化設定。
  • .gitea/ai-review/findings.json、exclusions.json 更新 review 狀態。

測試

  • node -e JSON 驗證 .gitea/ai-review/findings.json 與 exclusions.json
  • npm --prefix app test

Commits

  • fix(TLS 驗證): 移除 git 與 OpenRouter 的不安全憑證略過設定
  • refactor(LLM CLI): 明確化 CLI 參數與驗證輸出
  • chore(Docker action): 整理 Docker action 執行設定
  • chore(ai-review 狀態): 清空已處理 findings 並登記排除項目
## 變更摘要 - 移除 git 網路操作強制關閉 SSL 驗證，避免遠端存取被中間人污染或外洩 token。 - OpenRouter 額度查詢改回預設 HTTPS 憑證驗證，避免外部 API key 在不安全連線中傳送。 - 明確化 LLM CLI 參數組裝與 preflight 成功訊息，補齊 verifyLLM 回傳欄位文件。 - 整理 Docker action 執行設定，並將已處理的 AI review findings 清空，新增一筆相容性排除。 ## 影響範圍 - app/git.js、app/usage.js 與對應測試更新 TLS 驗證行為。 - app/config.js、app/llm.js、app/preflight.js 更新 CLI 與內部憑證相關說明。 - action.yaml、Dockerfile、.gitea/workflows/ci.yaml 與 package.json 整理 Docker action 化設定。 - .gitea/ai-review/findings.json、exclusions.json 更新 review 狀態。 ## 測試 - node -e JSON 驗證 .gitea/ai-review/findings.json 與 exclusions.json - npm --prefix app test ## Commits - fix(TLS 驗證): 移除 git 與 OpenRouter 的不安全憑證略過設定 - refactor(LLM CLI): 明確化 CLI 參數與驗證輸出 - chore(Docker action): 整理 Docker action 執行設定 - chore(ai-review 狀態): 清空已處理 findings 並登記排除項目
jiantw83 added 4 commits 2026-06-29 10:49:57 +00:00
fix(TLS 驗證): 移除 git 與 OpenRouter 的不安全憑證略過設定 68ce92b0e5
refactor(LLM CLI): 明確化 CLI 參數與驗證輸出 0b59ca6023
chore(Docker action): 整理 Docker action 執行設定 45b6abacc8
chore(ai-review 狀態): 清空已處理 findings 並登記排除項目
CI / Release Tag Version (pull_request) Successful in 6s
Details
CI / AI Code Review (pull_request) Failing after 12s
Details
2703840018
jiantw83 added 1 commit 2026-06-30 05:51:03 +00:00
feat(Dockerfile): 修改映像檔來源
CI / Release Tag Version (pull_request) Successful in 2m23s
Details
CI / AI Code Review (pull_request) Failing after 34s
Details
5a07f03abb
jiantw83 added 1 commit 2026-07-01 01:44:58 +00:00
feat(Dockerfile): 固定版本號
CI / Release Tag Version (pull_request) Successful in 7s
Details
CI / AI Code Review (pull_request) Failing after 6s
Details
b3878b9860
jiantw83 added 1 commit 2026-07-01 01:51:28 +00:00
fix(Dockerfile): 修正 RUN 指令換行與清理路徑導致 build 失敗
CI / Release Tag Version (pull_request) Successful in 6s
Details
CI / AI Code Review (pull_request) Failing after 54s
Details
2f54271c38
admin commented 2026-07-01 01:52:28 +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
jiantw83 added 1 commit 2026-07-01 02:09:32 +00:00
feat(Dockerfile): 更新映像檔版本號
CI / Release Tag Version (pull_request) Successful in 6s
Details
CI / AI Code Review (pull_request) Failing after 2m1s
Details
3a904f9ede
admin commented 2026-07-01 02:11:04 +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 commented 2026-07-01 02:15:11 +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
Some required checks failed
CI / Release Tag Version (pull_request) Successful in 6s
Details
CI / AI Code Review (pull_request) Failing after 2m1s
Required
Details
Some required checks were not successful.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin ai-review-resolve/develop-20260629-104700:ai-review-resolve/develop-20260629-104700
git checkout ai-review-resolve/develop-20260629-104700
Sign in to join this conversation.
Reviewers
No Reviewers
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-docker-actions/ai-code-review#14