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

改用 AI 助理 CLI 並強化 AI Code Review 執行流程 #13

Merged
admin merged 19 commits from ai-review-resolve/develop-20260626-091820 into develop 2026-06-29 08:49:20 +00:00
Conversation 11 Commits 19 Files Changed 17
+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 34a9eb6d4e - Show all commits
app/llm.js
+1 -1
View File
@@ -25,7 +25,7 @@ function buildPrompt(systemPrompt, userContent) {
function cliArgs(provider, model, promptFile = null, prompt = null) { function cliArgs(provider, model, promptFile = null, prompt = null) {
if (provider === 'codex') { if (provider === 'codex') {
admin commented 2026-06-29 08:07:59 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Bard
問題：cliArgs(provider, model, promptFile = null, prompt = null) 這個簽名把不同 provider 的參數意義混在一起，光看位置很難分辨哪個值在某條分支才有用，讀起來像把三種節拍硬塞進同一小節。
建議：改成物件參數或拆成各 provider 專用的組裝函式，讓每個欄位的用途能被名字直接說明，而不是靠呼叫順序猜測。

**嚴重等級**：🟡 警告 **審查員**：Bard **問題**：`cliArgs(provider, model, promptFile = null, prompt = null)` 這個簽名把不同 provider 的參數意義混在一起，光看位置很難分辨哪個值在某條分支才有用，讀起來像把三種節拍硬塞進同一小節。 **建議**：改成物件參數或拆成各 provider 專用的組裝函式，讓每個欄位的用途能被名字直接說明，而不是靠呼叫順序猜測。
return ['exec', '--model', model, '--sandbox', 'read-only', '--ask-for-approval', 'never', '--skip-git-repo-check', '-']; return ['exec', '--model', model, '--sandbox', 'read-only', '--skip-git-repo-check', '-'];
} }
if (provider === 'claude') { if (provider === 'claude') {
return ['--print', '--model', model, '--permission-mode', 'dontAsk', '--no-session-persistence']; return ['--print', '--model', model, '--permission-mode', 'dontAsk', '--no-session-persistence'];