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

feat: 導入 AI 程式碼審查 action 並修正進入點與參數接線 #1

Merged
admin merged 30 commits from ai-review-resolve/develop-20260702-160700 into develop 2026-07-03 10:04:33 +00:00
Conversation 18 Commits 30 Files Changed 456
+1 -1
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 7c35dc0601 - Show all commits
src/config.js
+1 -1
View File
3
 @@ -30,7 +30,7 @@ const PR = EVENT.pull_request || {};
// 無對應輸入的欄位（server url / repository / PR_*）則走專用 env → runner 內建 / 事件 payload。
// 使用端 workflow 只需傳 `with: token`。
admin marked this conversation as resolved
admin commented 2026-07-03 04:08:32 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Maya
問題：這裡是整個 action 讀取 INPUT_*、GITEA_* 與事件 payload 的入口，但測試只覆蓋了 getLLMConfig()，沒有把 GITEA_TOKEN、GITEA_COMMENT_TOKEN、PR_NUMBER、PR_HEAD_SHA 這些環境與 payload 的優先序鎖住。特別是 comment token 退回主 token、以及 event 檔讀不到時回到空值的情境，都是 CI 最容易因環境差異壞掉的地方。
建議：新增 config 相關測試，分別用假 process.env 和暫存 event payload 檔驗證：INPUT_* 會蓋過 ambient env、GITEA_COMMENT_TOKEN 缺值時會 fallback 到主 token、GITEA_EVENT_PATH / GITHUB_EVENT_PATH 讀取失敗時不會拋錯且回傳預設值。

**嚴重等級**：🟡 警告 **審查員**：Maya **問題**：這裡是整個 action 讀取 `INPUT_*`、`GITEA_*` 與事件 payload 的入口，但測試只覆蓋了 `getLLMConfig()`，沒有把 `GITEA_TOKEN`、`GITEA_COMMENT_TOKEN`、`PR_NUMBER`、`PR_HEAD_SHA` 這些環境與 payload 的優先序鎖住。特別是 comment token 退回主 token、以及 event 檔讀不到時回到空值的情境，都是 CI 最容易因環境差異壞掉的地方。 **建議**：新增 config 相關測試，分別用假 `process.env` 和暫存 event payload 檔驗證：`INPUT_*` 會蓋過 ambient env、`GITEA_COMMENT_TOKEN` 缺值時會 fallback 到主 token、`GITEA_EVENT_PATH` / `GITHUB_EVENT_PATH` 讀取失敗時不會拋錯且回傳預設值。
export const GITEA_TOKEN = process.env.INPUT_TOKEN || process.env.GITEA_TOKEN || '';
export const GITEA_COMMENT_TOKEN = process.env.INPUT_COMMENT_TOKEN || process.env.GITEA_COMMENT_TOKEN || '';
export const GITEA_COMMENT_TOKEN = process.env.INPUT_COMMENT_TOKEN || process.env.GITEA_COMMENT_TOKEN || process.env.INPUT_TOKEN || process.env.GITEA_TOKEN || '';
export const GITEA_SERVER_URL = process.env.GITEA_SERVER_URL || process.env.GITHUB_SERVER_URL || 'https://gitea.com';
export const GITEA_REPOSITORY = process.env.GITEA_REPOSITORY || process.env.GITHUB_REPOSITORY || '';
export const PR_NUMBER = process.env.PR_NUMBER || (PR.number != null ? String(PR.number) : '');
1