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

feat: 嚴重問題改用 Gitea 行內 review comment 標註檔案行數 #12

Closed
jiantw83 wants to merge 0 commits from feat/inline-critical-comments into develop
pull from: feat/inline-critical-comments
merge into: :develop
Conversation 3 Commits - Files Changed -
+182 -46
jiantw83 commented 2026-06-16 06:45:04 +00:00
Member
Copy Link
Copy Source

每個新的嚴重問題改以行內 review comment 標註在問題所在的檔案與行數上，
留言內容為等級／審查員／建議；無法解析出行號（未標行號或一次列出多個
檔案），或行內留言失敗（該行不在 diff 範圍）時，降級為原本的一般 PR comment。

  • gitea.js：新增 postPullReviewComment，呼叫 pull reviews API，以 new_position
    對應新版檔案行號、commit_id 帶 PR_HEAD_SHA
  • comments.js：新增 parseLocation（支援 file:19 / file:70-82，取起始行）與
    行內留言內容組裝；postNewCriticalComments 先試行內、失敗降級，deps 可注入
  • 補 11 個測試（API payload、parseLocation 各情境、行內成功與兩種降級路徑）
  • README 更新流程第 7 步說明

app/ 測試 123 pass。

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

每個新的嚴重問題改以行內 review comment 標註在問題所在的檔案與行數上， 留言內容為等級／審查員／建議；無法解析出行號（未標行號或一次列出多個 檔案），或行內留言失敗（該行不在 diff 範圍）時，降級為原本的一般 PR comment。 - gitea.js：新增 postPullReviewComment，呼叫 pull reviews API，以 new_position 對應新版檔案行號、commit_id 帶 PR_HEAD_SHA - comments.js：新增 parseLocation（支援 file:19 / file:70-82，取起始行）與 行內留言內容組裝；postNewCriticalComments 先試行內、失敗降級，deps 可注入 - 補 11 個測試（API payload、parseLocation 各情境、行內成功與兩種降級路徑） - README 更新流程第 7 步說明 app/ 測試 123 pass。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
jiantw83 added 1 commit 2026-06-16 06:45:05 +00:00
feat: 嚴重問題改用 Gitea 行內 review comment 標註檔案行數
AI / 計算版本號 (pull_request) Successful in 4s
Details
AI / Code Review (pull_request) Successful in 1m35s
Details
caebd2b112 
每個新的嚴重問題改以行內 review comment 標註在問題所在的檔案與行數上，
留言內容為等級／審查員／建議；無法解析出行號（未標行號或一次列出多個
檔案），或行內留言失敗（該行不在 diff 範圍）時，降級為原本的一般 PR comment。

- gitea.js：新增 postPullReviewComment，呼叫 pull reviews API，以 new_position
  對應新版檔案行號、commit_id 帶 PR_HEAD_SHA
- comments.js：新增 parseLocation（支援 file:19 / file:70-82，取起始行）與
  行內留言內容組裝；postNewCriticalComments 先試行內、失敗降級，deps 可注入
- 補 11 個測試（API payload、parseLocation 各情境、行內成功與兩種降級路徑）
- README 更新流程第 7 步說明

app/ 測試 123 pass。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
admin commented 2026-06-16 06:45:15 +00:00
Owner
Copy Link
Copy Source

🤖 AI Code Review 團隊

👤 名稱 🎯 職責 🧠 個性
Leo 可維護性審查員 有遠見、重視長期維護成本，常常思考「六個月後的自己能看懂嗎？」
Zara 效能優化專家 追求極致效能，對任何不必要的資源消耗都感到不舒服，喜歡用數據說話
Rex 資安審查員 謹慎、多疑、對任何潛在風險都保持高度警覺，寧可誤報也不放過漏洞
Aria 程式碼風格審查員 嚴謹、注重細節、對程式碼整潔度有高度要求，說話直接但不失禮貌
Maya 測試品質審查員 對測試覆蓋率有執念，相信沒有測試的程式碼等於沒有完成，溫和但堅持

🔍 服務：gemini 模型：gemini-2.5-flash

## 🤖 AI Code Review 團隊 | 👤 名稱 | 🎯 職責 | 🧠 個性 | |--------|--------|--------| | **Leo** | 可維護性審查員 | 有遠見、重視長期維護成本，常常思考「六個月後的自己能看懂嗎？」 | | **Zara** | 效能優化專家 | 追求極致效能，對任何不必要的資源消耗都感到不舒服，喜歡用數據說話 | | **Rex** | 資安審查員 | 謹慎、多疑、對任何潛在風險都保持高度警覺，寧可誤報也不放過漏洞 | | **Aria** | 程式碼風格審查員 | 嚴謹、注重細節、對程式碼整潔度有高度要求，說話直接但不失禮貌 | | **Maya** | 測試品質審查員 | 對測試覆蓋率有執念，相信沒有測試的程式碼等於沒有完成，溫和但堅持 | > 🔍 服務：gemini 模型：gemini-2.5-flash
admin commented 2026-06-16 06:46:36 +00:00
Owner
Copy Link
Copy Source

📋 舊有未解決問題（3 筆）

等級 審查員 位置 建議
🟡 警告 Aria app/preflight.test.js:25 測試描述使用英文。請確保專案在測試描述的語言上保持一致性。如果專案主要使用繁體中文（如 app/preflight.js 中的 JSDoc 和日誌），則應將此測試描述翻譯為繁體中文。
🔵 建議 Aria app/preflight.test.js:1-4 匯入語句的排序不一致。建議遵循一致的排序規則，例如：內建模組、第三方模組、本地模組，並在各組內按字母順序排序。
🔵 建議 Aria app/preflight.test.js:14 函數名稱 clearLLMEnv 雖然可理解，但可以更具描述性，例如 clearLlmEnvironmentVariables 或 resetLlmEnv。
## 📋 舊有未解決問題（3 筆） | 等級 | 審查員 | 位置 | 建議 | |------|--------|------|------| | 🟡 警告 | Aria | app/preflight.test.js:25 | 測試描述使用英文。請確保專案在測試描述的語言上保持一致性。如果專案主要使用繁體中文（如 app/preflight.js 中的 JSDoc 和日誌），則應將此測試描述翻譯為繁體中文。 | | 🔵 建議 | Aria | app/preflight.test.js:1-4 | 匯入語句的排序不一致。建議遵循一致的排序規則，例如：內建模組、第三方模組、本地模組，並在各組內按字母順序排序。 | | 🔵 建議 | Aria | app/preflight.test.js:14 | 函數名稱 clearLLMEnv 雖然可理解，但可以更具描述性，例如 clearLlmEnvironmentVariables 或 resetLlmEnv。 |
admin added 1 commit 2026-06-16 06:46:47 +00:00
chore: update ai-review findings [ai-review-bot][success]
AI / 計算版本號 (pull_request) Successful in 2s
Details
AI / Code Review (pull_request) Successful in 2s
Details
8419e60848
jiantw83 added 1 commit 2026-06-16 08:00:35 +00:00
chore(ai-review): 三條 preflight 測試誤判寫入 exclusions 並清空 findings
AI / 計算版本號 (pull_request) Successful in 4s
Details
AI / Code Review (pull_request) Successful in 1m17s
Details
f047b4473e
admin commented 2026-06-16 08:00:54 +00:00
Owner
Copy Link
Copy Source

🤖 AI Code Review 團隊

👤 名稱 🎯 職責 🧠 個性
Leo 可維護性審查員 有遠見、重視長期維護成本，常常思考「六個月後的自己能看懂嗎？」
Zara 效能優化專家 追求極致效能，對任何不必要的資源消耗都感到不舒服，喜歡用數據說話
Rex 資安審查員 謹慎、多疑、對任何潛在風險都保持高度警覺，寧可誤報也不放過漏洞
Aria 程式碼風格審查員 嚴謹、注重細節、對程式碼整潔度有高度要求，說話直接但不失禮貌
Maya 測試品質審查員 對測試覆蓋率有執念，相信沒有測試的程式碼等於沒有完成，溫和但堅持

🔍 服務：gemini 模型：gemini-2.5-flash

## 🤖 AI Code Review 團隊 | 👤 名稱 | 🎯 職責 | 🧠 個性 | |--------|--------|--------| | **Leo** | 可維護性審查員 | 有遠見、重視長期維護成本，常常思考「六個月後的自己能看懂嗎？」 | | **Zara** | 效能優化專家 | 追求極致效能，對任何不必要的資源消耗都感到不舒服，喜歡用數據說話 | | **Rex** | 資安審查員 | 謹慎、多疑、對任何潛在風險都保持高度警覺，寧可誤報也不放過漏洞 | | **Aria** | 程式碼風格審查員 | 嚴謹、注重細節、對程式碼整潔度有高度要求，說話直接但不失禮貌 | | **Maya** | 測試品質審查員 | 對測試覆蓋率有執念，相信沒有測試的程式碼等於沒有完成，溫和但堅持 | > 🔍 服務：gemini 模型：gemini-2.5-flash
jiantw83 closed this pull request 2026-06-16 08:17:22 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
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
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: old-actions/code-review#12