fix(diagnostics): 處理 ai review findings #32

Open

jiantw83 wants to merge 85 commits from develop into master

pull from: develop

merge into: :master

:master

:develop

Conversation 0 Commits 85 Files Changed 30

+4445 -370

jiantw83 commented 2026-07-28 06:03:11 +00:00

Member

Copy Link

Copy Source

Summary

• 修正 debug 診斷輸出：先限長再遮罩，並擴充 Email、JWT、private key 等常見 PII/secret 遮罩。
• 改用 shallow 檔案存在判斷，避免 merge-base 策略額外呼叫 git 查詢 shallow 狀態。
• 收斂 token 權限說明，移除 README 函式清單的易漂移行號連結。
• 清空已處理 ai-review findings，並把非本次安全最小改動項目記錄到 exclusions。

Tests

• npm test

## Summary - 修正 debug 診斷輸出：先限長再遮罩，並擴充 Email、JWT、private key 等常見 PII/secret 遮罩。 - 改用 shallow 檔案存在判斷，避免 merge-base 策略額外呼叫 git 查詢 shallow 狀態。 - 收斂 token 權限說明，移除 README 函式清單的易漂移行號連結。 - 清空已處理 ai-review findings，並把非本次安全最小改動項目記錄到 exclusions。 ## Tests - `npm test`

jiantw83 added 85 commits 2026-07-28 06:03:12 +00:00

docs(doc-funcs): 補齊 action 與 workflow 文件 3fe643c003

fix(gitrepo): 補抓淺層 checkout 的 base 歷史 c4c23d4531

chore: update ai-review findings [ai-review-bot][success] ccbe615c2a

refactor(審查流程): 將舊留言標記移到偵測 AI 工具之前 d751cee69d

docs(審查步驟編號): 依新順序重編步驟編號與說明 8662e8ca80

chore: update ai-review findings [ai-review-bot][success] af6b9a197f

feat(建問題模式): 審查留言改發到 issue、跳過舊留言處理並回貼 PR 連結 91fff79f22

chore(ci): 啟用建問題模式 3631bf3592

feat(建問題模式): 無問題時靜默通過，不在 PR 留任何審查留言 94b32737c5

chore(審查記錄): AI 失敗時輸出 exit code 與 stderr／stdout 供診斷 4a68743b66

fix(ci): 傳入各 AI 工具的 OAuth 憑證修正未登入問題 e37a96433d

chore: update ai-review findings [ai-review-bot][failure] d464998e4f

fix(ai-review 診斷輸出): AI 失敗預設不輸出原始 stdout/stderr、除錯開關下遮罩機密 ... 5c8332928b

解決議題 #7 🔴 嚴重（Assassin/security）：agentFailureDetail 原本無遮罩擷取 stderr/stdout 各 500 字寫入 CI log，可能長期外洩 token、PII 或原始碼祕密。

- 預設只輸出 exit code／signal／killed 等不含機密的分類資訊。
- 僅在明確開啟 ACTIONS_STEP_DEBUG=true 時，才附上經新增 redactSecrets 遮罩（Authorization／token／URL 帳密／長金鑰）並去除控制字元的片段。
- 另新增 postOthersToIssue：建問題模式下警告＋建議改逐條發到 issue，讓每條問題可個別回覆（feat，配合 index.js）。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

refactor(resolveMergeBase): 補抓歷史每步早停並彙整策略診斷 ... 35887f1a68

解決議題 #7 🟠 警告（Leo/maintainability、Rogue/efficiency）：resolveMergeBase 原先無條件連做 --unshallow 與兩次 --deepen，且忽略各次 fetch 成敗、只把首次 merge-base 錯誤設為 cause。

- 改為資料驅動策略清單，每個補抓策略成功後立即重試 merge-base，一成功即回傳，避免多餘遠端往返（--unshallow 成功就不再 deepen）。
- 全部用盡才拋錯，訊息彙整各策略成功／失敗診斷（僅策略名與成敗，不含 git 原始輸出以免洩漏遠端資訊），並以 error.cause 保留首次錯誤。
- 本檔另含既有的 push-token（PAT）推送參數 pushToken（feat），維持原推送降級行為。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

feat(建問題模式): PR 相依追蹤 issue、標籤建立時帶入、議題問題逐條可回覆 ... e44a8ed0cb

整併議題 #7 的建問題模式流程調整與多項 findings 修復（index.js、gitea.js 交錯同檔，整檔歸此）。

- addIssueDependency：建立追蹤 issue 後讓 PR 相依於該 issue，issue 關閉前無法合併（需 repo 啟用問題相依）。
- 🔵 #10（Rogue/efficiency）：標籤改於建立 issue 前先 selectLabels、createIssue 時一次帶入，移除事後 addLabelsToIssue 補掛的多餘 API 往返（addLabelsToIssue 保留為通用工具）。
- 使用者補充：建問題模式警告＋建議改逐條發到 issue（postOthersToIssue），每條問題可個別回覆。
- 🔴 #1（Mage/logic）＋使用者補充：resolveOldComments 延後到審查成功產生結果、發布問題留言前才執行，並以 currentRunCommentIds 排除本回合新留言，避免前置步驟失敗時舊結果被清卻無新結果。
- 🔵 #9（Bard/style）：修正「無保留問題」段落註解與實作（靜默通過、不留言）一致。
- 同步更新相關 JSDoc（含 gitea.js listLabels／createIssue／addLabelsToIssue 使用情境）。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

feat(push-token): 新增 push-token 輸入以 PAT 推送結果 commit 觸發 CI ... d3a09d892e

以自動 token（gitea.token／GITHUB_TOKEN）推送的 commit 不會再觸發 CI，導致新 head 缺檢查而卡合併；新增 push-token（PAT）由呼叫端以 secrets 傳入，改以 PAT 身分推送使 PR synchronize 事件再觸發 CI，由步驟 1 快速回報把結果蓋到新 head。留空＝退回 token 走 origin 推送。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

docs(templates): 角色登場步驟編號統一為 5／7 ... dbeb56589c

解決議題 #7 🔵 建議（Bard/style）：focusLabel 與 rolesComment 文件仍標「步驟 5／6」，但角色登場已調整為攻擊方步驟 5、防守方步驟 7；統一改為 5／7 與鄰近模板文件一致。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore(ci): 改以 PAT（push-token）推送 findings commit 觸發 CI ... 10d826d1b1

各 job 的 token 改用 gitea.token、並新增 push-token: secrets.TOKEN，讓結果 commit 以 PAT 推送再觸發 CI，避免自動 token 推送不觸發而卡合併。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] c980add807

chore(ai-review 狀態): 寫回議題 #7 待人工處理問題至 findings ... b2129e0bc6

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] dae379273a

docs(review): 修正 postSevereToIssue 註解為建問題模式警告／建議逐條留言 ... cfbbbe5f6c

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] a6534ee076

fix(推送結果 commit): 帶認證推送改經 env 傳入並遮蔽 push 失敗錯誤 ... 8349fe8b71

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

perf(AI 失敗診斷): 除錯輸出先截長度上限再遮罩降低掃描成本 ... f341dc6e07

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

refactor(gitea client): 移除無呼叫端的 addLabelsToIssue ... 0a51affa72

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

docs(readme): 流程圖步驟順序反映舊留言標記延後執行 ... dcd3e931f4

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore(ai-review 狀態): 記入議題 #8 不採納排除並寫回待人工處理 findings ... 4c2b6d4265

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] 7823718ff4

chore(ci): 審查 job 的 token 改用 secrets.TOKEN ... 7b5c7c3e4f

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

fix(AI 失敗診斷): CLI 失敗預設輸出經遮罩的 stderr 片段以利除錯 ... 71b93a885a

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] afed9a3cce

fix(AI 失敗診斷): 失敗時預設併印遮罩後 stdout（claude-code 錯誤寫在 stdout） ... 491c170d36

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][success] 85235e43a4

chore(ci): 三個測試 job 指定各工具最省 model 以降低消耗 ... 36bf2d8644

Antigravity→gemini-3.5-flash、Codex→gpt-5.5、Claude→claude-haiku-4-5

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore(ci): Claude 測試 job 改用 claude-opus-4-8 ... 321a717f91

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][success] 9e6e8bb793

fix(推送觸發 CI): 合併 push-token 進 token，findings 一律以 token 認證推送觸發 CI ... 05178be520

- 移除 push-token input，token 兼作 Gitea API 認證與 findings 推送
- commitAndPushFindings 一律以 token 明確認證推送（不走 origin 自動 token）；
  只要 token 為能觸發 CI 的 PAT，結果 commit 即再觸發 PR 的 CI，避免新 head 缺檢查卡合併
- ci.yaml 三個 job 移除 push-token，保留 token: ${{ secrets.TOKEN }}

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][success] 20a2b57998

fix(建問題模式): 僅在有嚴重問題時讓 PR 相依於 issue，警告／建議不阻擋合併 ... b277ff2f7c

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] be72aa0944

fix(結果回報): 審查本輪不直接 exit 1，失敗只由步驟 1 讀結果 commit 訊息回報 ... 3ef8a30291

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] ed883f6f20

perf(resolveMergeBase): 先 deepen base/HEAD，--unshallow 降為最後手段 ... c63b1a6da2

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore(ai-review 狀態): 寫回議題 #9 待人工處理問題至 findings ... deb20bb078

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] 4b779ee00b

fix(推送觸發 CI): push 前重置 checkout 的自動 token extraheader，改以 PAT 身分推送 ... b5264141c4

checkout 於 http.<serverUrl>/.extraheader 持久化自動 Actions token；沿用它推送會被
Gitea 視為自動 token 觸發而不再觸發 workflow。改為推送前於同 scope 先空值重置、再注入
PAT 的 Authorization，使 findings 結果 commit 以 PAT 身分推送、觸發 PR synchronize。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] d354f30c27

fix(resolveMergeBase): deepen HEAD 改以目前 HEAD 的 SHA 補抓 PR head 歷史 ... 83dd56f37f

原策略 fetch 遠端符號 HEAD，會被伺服器解析為遠端預設分支，
只加深預設分支歷史、補不到目前 checkout 的 PR head，淺層時仍可能算不出 merge-base。
改先 rev-parse 取得 HEAD 的 commit SHA，以該 SHA 加深 PR head 側歷史。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

refactor(ai-review): 重新命名留言閉包並抽出診斷輸出長度常數 ... 0c108ab7f9

- postComment → queueOrPostComment：涵蓋建問題模式下可能只暫存不立即發布的語義。
- ensureIssueCreated → createIssueAndFlushBufferedComments：明示建立 issue 並清空暫存留言的完整行為。
- review.js agentFailureDetail 的 500 字上限抽為具名常數 AGENT_DIAGNOSTIC_OUTPUT_LIMIT。
純內部命名與可讀性調整，無外部行為變更。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

docs(ai-review): 同步 JSDoc 對 main() 閉包改名的引用 ... cc9c8afed7

配合 postComment／ensureIssueCreated 改名，更新 gitea.js 與 templates.js
JSDoc 內對兩個閉包的引用名稱。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore(ai-review 狀態): 記錄 issue #10 誤報排除並回寫待人工 findings ... d8dd5ce92e

- exclusions.json 新增一條誤報（gitrepo.js:276 使用者名稱固定 ai-review-bot，依管理員指示）。
- 新增 findings 檔追蹤 4 條待人工處理問題（步驟編號硬編碼、兩處缺測試、debug 輸出取捨）。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] 0c29daa014

chore(ai-review findings): 寫回議題 #11–#23 只來自議題的待人工 findings ... d7cecc8fa8

以 --issue all 處理 13 個建問題模式追蹤議題，逐條對照目前程式碼與 exclusions.json 後：
✅ 已解決 18 條（現行程式碼已修）、🚫 誤報 28 條（已列入 exclusions，不重複新增）、
⏭️ 待人工 44 條（設計／效能／慣例取捨，依主題去重寫回 42 條追蹤）。各議題已留言處理進度並關閉。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] 05153bba8e

refactor(review): 診斷輸出長度常數提升為模組層級 ... 5ac73f290e

依議題 #24 Bard 建議，將 agentFailureDetail 內的 INPUT_LIMIT／
AGENT_DIAGNOSTIC_OUTPUT_LIMIT 移至模組頂層，與既有 PER_FILE_DIFF_LIMIT／
TOTAL_DIFF_LIMIT 並列集中管理；值與行為不變。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chore: update ai-review findings [ai-review-bot][failure] a8561b4257

fix(ai-review): 強化安全防護與建問題留言處理 6a26984bee

test(ai-review): 補上安全與 Gitea 契約測試 55b349da07

chore(ai-review 狀態): 回寫已處理 findings e0ae6f886f

chore: update ai-review findings [ai-review-bot][failure] 24b248884f

fix(ai-review): 避免建問題模式嚴重結果 fail-open 555611db07

test(ai-review): 補上建問題模式結果檔測試 8b50a0d643

chore(ai-review 狀態): 回寫本輪已處理 findings 9e7f8a2a0a

chore: update ai-review findings [ai-review-bot][failure] 941d763129

fix(ai-review): 對齊建問題模式留言順序與命名 9d05af647b

chore(ai-review 狀態): 回寫本輪已處理 findings d424447d15

chore: update ai-review findings [ai-review-bot][failure] 55da29a86f

fix(ai-review): 避免建問題模式阻擋失效與連結累積 aaf2e484d5

refactor(ai-review 診斷): 收斂失敗輸出處理與推送註解 75ca8cdb83

chore(ai-review 狀態): 回寫本輪已處理 findings 216bc39255

chore: update ai-review findings [ai-review-bot][failure] 76617dd9dc

refactor(gitref): 將分支名稱驗證改為正式模組 1f012c6cfb

style(action manifest): 統一中文斜線標點 8e18bbacc2

chore(ai-review 狀態): 回寫本輪 findings 與排除事項 43ad2b56d2

chore: update ai-review findings [ai-review-bot][failure] b60f50f2b4

fix(ai-review): 結果推送失敗時回報工作流失敗 449b17480b

chore(ai-review 狀態): 回寫本輪已處理 findings 44c33b6c1e

chore: update ai-review findings [ai-review-bot][success] 25f76e6e6e

Merge pull request 'feat(ai-code-review): 建問題模式導向 issue，並整併流程調整、文件與 CI' (#6) from ai-review-resolve/develop-20260717-185330 into develop ... cce35b3bef

Reviewed-on: #6

fix(diagnostics): 限縮 debug 診斷輸出並擴充遮罩 3c1cd55444

perf(gitrepo): 改用 shallow 檔案判斷避免額外 git 子行程 db83b54c2d

test(diagnostics): 補上診斷限長與 PII 遮罩測試 885b79f8b3

docs(token): 收斂 token 權限建議並移除 README 行號連結 2d3128eb45

chore(ai-review): 清空已處理 findings 並更新 exclusions 34aecf6e43

All checks were successful

Hide all checks

node-actions/template: CI / BUILD (pull_request) Successful in 6s

Required

Details

This pull request doesn't have enough required approvals yet. 0 of 1 approvals granted from users or teams on the allowlist.

You are not authorized to merge this pull request.

This branch is out-of-date with the base branch

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin develop:develop

git checkout develop

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

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: node-actions/ai-code-review#32