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

fix(審查流程): 修正 bot 跳過、JSON 驗證與排除比對邊界 #4

Merged
admin merged 1 commits from develop into master 2026-07-11 12:20:29 +00:00
Conversation 0 Commits 1 Files Changed 10
+94 -15
jiantw83 commented 2026-07-11 12:19:38 +00:00
Member
Copy Link
Copy Source

變更摘要

  • 修正 parseLocation() 對 line=0 的容錯，避免無效行號送進 Gitea。
  • 修正 shouldSkipBotCommit() 只在 bot success commit 時跳過，failure commit 仍會讓 workflow 失敗。
  • 修正 validateJSONArrayFile()，要求修復後內容必須是 JSON 陣列。
  • 修正純文字 exclusions 比對，以及同一行多個 bot findings 的保留邏輯。

影響範圍

  • src/comments.js
  • src/gitea.js
  • src/json.js
  • src/findings.js
  • src/resolve.js
  • 對應測試已補齊

驗證

  • npm test
## 變更摘要 - 修正 `parseLocation()` 對 `line=0` 的容錯，避免無效行號送進 Gitea。 - 修正 `shouldSkipBotCommit()` 只在 bot success commit 時跳過，failure commit 仍會讓 workflow 失敗。 - 修正 `validateJSONArrayFile()`，要求修復後內容必須是 JSON 陣列。 - 修正純文字 exclusions 比對，以及同一行多個 bot findings 的保留邏輯。 ## 影響範圍 - `src/comments.js` - `src/gitea.js` - `src/json.js` - `src/findings.js` - `src/resolve.js` - 對應測試已補齊 ## 驗證 - `npm test`
jiantw83 added 1 commit 2026-07-11 12:19:38 +00:00
fix(審查流程): 修正 bot 跳過、JSON 驗證與排除比對邊界
CI / 1. BUILD (pull_request) Successful in 3s
Details
CI / 2. TEST (pull_request) Has been skipped
Details
CI / 3. RESULT (pull_request) Has been skipped
Details
8f909e5397
admin merged commit 9ba12ceca7 into master 2026-07-11 12:20:29 +00:00
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
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: actions/ai-code-review#4