Merge pull request 'feat/refactor/kiro/fix-1' (#75 ) from feat/refactor/kiro/fix-1 into feat/refactor/kiro/fix

Reviewed-on: jiantw83/code-review#75
chore: update ai-review findings [skip ci]
2026-05-12 02:27:31 +00:00 · 2026-05-12 02:26:16 +00:00 · 2026-05-12 02:26:16 +00:00 · 2026-05-12 02:26:16 +00:00 · 2026-05-12 02:26:16 +00:00 · 2026-05-12 02:26:16 +00:00
2 changed files with 5 additions and 5 deletions
@@ -6,8 +6,8 @@

 1. 服務名稱、模型名稱、角色資訊(個性、符合個性的英文名稱、工作內容)，Comment 到 Push Request
 2. 每個角色個別分析 Git Diff 的內容產生新問題表格(問題等級、角色名稱、問題位置或行數、修改建議)
-3. 讀取所有未解決的舊問題(問題檔案 `.gitea/ai-review/findings.json` 存在於使用此 Action 的專案固定位置)加上新問題後，去除重複產生本次 Push Request 的問題表格(PR問題表格)覆蓋問題檔案
-4. 讀取排除問題檔案(`.gitea/ai-review/exclusions.json` 存在於使用此 Action 的專案固定位置)，用來過濾PR問題表格中不需要處理的問題
+3. 讀取所有未解決的舊問題(問題檔案存在於使用此 Action 的專案固定位置)加上新問題後，去除重複產生本次 Push Request 的問題表格(PR問題表格)覆蓋問題檔案
+4. 讀取排除問題檔案，用來過濾PR問題表格中不需要處理的問題
 5. 從PR問題表格中取出所有舊問題，依照等級排序後 Comment 到 Push Request
 6. 從PR問題表格中取出所有新問題，排除嚴重等級的問題後 Comment 到 Push Request
 7. 從PR問題表格中取出所有新問題，將每個嚴重等級的問題 Comment 到 Push Request
@@ -16,13 +16,13 @@
 - 完成

 ## 階段四：AI 排除問題過濾
- 目標：讀取排除問題檔案（`.gitea/ai-review/exclusions.json`）進行規則過濾，並呼叫 AI 判斷剩餘問題是否為誤報或不適用，兩層過濾後產生最終問題清單。
+- 目標：讀取排除問題檔案（exclusions.json）進行規則過濾，並呼叫 AI 判斷剩餘問題是否為誤報或不適用，兩層過濾後產生最終問題清單。
 - 驗收：log 中能看到排除問題檔案讀取成功或不存在的訊息、規則過濾數量變化，以及「AI 誤報過濾: N -> M 筆」或降級訊息。
 - 完成

 ## 階段五：findings 寫入與 comment 發布
- 目標：`.gitea/ai-review/findings.json` 正確寫入，comment 發布順序正確（舊問題→非嚴重→嚴重），每步有 log。
- 驗收：log 中能看到 `.gitea/ai-review/findings.json` 寫入、comment sync 的詳細訊息與順序。
+- 目標：findings.jsonl 正確寫入，comment 發布順序正確（舊問題→非嚴重→嚴重），每步有 log。
+- 驗收：log 中能看到 findings.json 寫入、comment sync 的詳細訊息與順序。
 - 完成

 ## 階段六：記憶區 commit/push 與錯誤處理
Author	SHA1	Message	Date
jiantw83	02a6a109da	Merge pull request 'feat/refactor/kiro/fix-1' (#75 ) from feat/refactor/kiro/fix-1 into feat/refactor/kiro/fix Reviewed-on: jiantw83/code-review#75	2026-05-12 02:27:31 +00:00
AI Review Bot	67e1e83210	chore: update ai-review findings [skip ci]	2026-05-12 02:26:16 +00:00
jiantw83	0116bad4e3	fix: add Leo/Zara false positive exclusion; add cloneRepo unit tests	2026-05-12 02:26:16 +00:00
AI Review Bot	60a4854d56	chore: update ai-review findings [skip ci]	2026-05-12 02:26:16 +00:00
AI Review Bot	1e82594db2	chore: update ai-review findings [skip ci]	2026-05-12 02:26:16 +00:00
jiantw83	f0f417bd2e	refactor: rename Step4 to AI 排除問題過濾	2026-05-12 02:26:16 +00:00
AI Review Bot	bf1c081c40	chore: update ai-review findings [skip ci]	2026-05-12 02:26:16 +00:00
jiantw83	4ac614686c	docs: update TODO stage4 description and fix findings filename typo	2026-05-12 02:26:16 +00:00
jiantw83	e07f1f8a03	feat: add AI false positive filtering in Step4	2026-05-12 02:26:16 +00:00
jiantw83	f3f24f0af2	fix: use includes matching for exclusions location and suggestion	2026-05-12 02:26:16 +00:00
AI Review Bot	b020cbe9e1	chore: update ai-review findings [skip ci]	2026-05-12 02:26:16 +00:00
jiantw83	8779df9e8d	fix: clone repo before Step3/4 to read findings and exclusions from head branch	2026-05-12 02:26:16 +00:00
AI Review Bot	7333f0a98a	chore: update ai-review findings [skip ci]	2026-05-12 02:26:16 +00:00
jiantw83	ab688b4764	chore: add exclusions for Rex false positive on git.js token handling	2026-05-12 02:26:16 +00:00
jiantw83	1633a9ef7b	docs: mark all TODO stages complete	2026-05-12 02:26:16 +00:00
jiantw83	0bf90d44df	fix: align flow with README, add Step4 exclusions filter, fix step numbers	2026-05-12 02:26:16 +00:00
jiantw83	799d398b95	refactor: reorganize TODO stages for clarity and accuracy in workflow steps Co-authored-by: Copilot <copilot@github.com>	2026-05-12 02:26:16 +00:00
jiantw83	1c2fc679b4	refactor: update processing steps in README for clarity and accuracy	2026-05-12 02:26:16 +00:00
jiantw83	75d4a44fa6	refactor: remove outdated AI Code configurations for Kilo, Roo, Cline, Continue, and Kade	2026-05-12 02:26:16 +00:00
AI Review Bot	ca6edea43d	chore: update ai-review findings [skip ci]	2026-05-12 02:26:16 +00:00
jiantw83	e22223e501	fix: update askpass script to securely read token from env var	2026-05-12 02:26:08 +00:00
AI Review Bot	3fef7df7a5	chore: update ai-review findings [skip ci]	2026-05-12 01:09:42 +00:00
jiantw83	0aefa66224	feat: refactor commitAndPush to use a runner function and improve token security; add tests for git operations	2026-05-12 01:08:39 +00:00
jiantw83	66d93abe24	Merge pull request 'feat: master 不會觸發 review.yaml' (#66 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#66	2026-05-11 14:12:38 +00:00
jiantw83	0063f3282f	Merge pull request 'refactor: 改用 execSync' (#65 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#65	2026-05-11 13:56:13 +00:00
jiantw83	8c3d0d9a6d	Merge pull request 'fix: add bash to Dockerfile dependencies' (#64 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#64	2026-05-11 10:54:08 +00:00
jiantw83	3849bb2168	Merge pull request 'fix: switch Dockerfile base image to alpine and install dependencies using apk' (#63 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#63	2026-05-11 10:46:43 +00:00
jiantw83	379938d6dc	Merge pull request 'fix: Dockerfile 加入 git 安裝驗證，git.js 改回 git binary' (#62 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#62	2026-05-11 10:38:59 +00:00
jiantw83	5bf39966d0	Merge pull request 'fix: 換用 node:20 完整版（內建 git，不需要 apt-get）' (#61 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#61	2026-05-11 10:34:11 +00:00
jiantw83	3509a882e1	Merge pull request 'chore: add newline at end of TODO.md for better formatting' (#60 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#60	2026-05-11 10:19:46 +00:00
jiantw83	1d2e8236de	Merge pull request 'chore: remove duplicate log assistance note in TODO.md' (#59 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#59	2026-05-11 10:18:13 +00:00
jiantw83	d8423c74b1	Merge pull request 'chore: remove test findings' (#58 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#58	2026-05-11 10:12:32 +00:00
jiantw83	94e974b5dc	Merge pull request 'feat/refactor/kiro/1' (#57 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#57	2026-05-11 10:10:23 +00:00
jiantw83	a9a0b43ea5	Merge pull request 'debug: commit/push 失敗時顯示詳細錯誤' (#56 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#56	2026-05-11 10:04:49 +00:00
jiantw83	aa8234b5c7	Merge pull request 'fix: commitAndPush 加上 await' (#55 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#55	2026-05-11 10:01:12 +00:00
jiantw83	b0f2d45c11	Merge pull request 'fix: add newline at end of TODO.md for proper formatting' (#54 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#54	2026-05-11 09:59:14 +00:00
jiantw83	3fd9a7e13d	Merge pull request 'feat: 階段五六 - findings commit/push 到來源分支，critical 問題 exit 1' (#53 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#53	2026-05-11 09:54:39 +00:00
jiantw83	39cc5c932c	Merge pull request 'feat: 階段四 - findings 寫入與 comment 依序發布（舊問題→非嚴重→嚴重）' (#52 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#52	2026-05-11 09:48:41 +00:00
jiantw83	255adbabe4	Merge pull request 'feat: 階段三 - AI 語意去重，失敗時降級保留所有問題' (#51 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#51	2026-05-11 09:44:10 +00:00
jiantw83	a10fc8f176	Merge pull request 'feat/refactor/kiro/1' (#50 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#50	2026-05-11 09:40:12 +00:00
jiantw83	9b39908394	Merge pull request 'feat: 階段一 - 基本流程串接骨架' (#47 ) from feat/refactor/kiro/1 into feat/refactor/main Reviewed-on: jiantw83/code-review#47	2026-05-11 07:53:38 +00:00