優化 Step2：改用 skill RPG 攻防腳色系統（新增 Mage 邏輯角色、Step3/4 套 Paladin 裁決人設） #15

Merged

jiantw83 merged 14 commits from develop into master 2026-06-16 09:05:55 +00:00

Conversation 0 Commits 14 Files Changed 22

+713 -200

jiantw83 commented 2026-06-16 09:05:24 +00:00

Member

Copy Link

Copy Source

No description provided.

jiantw83 added 14 commits 2026-06-16 09:05:25 +00:00

feat: 嚴重問題改用 Gitea 行內 review comment 標註檔案行數 ... 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>

chore: update ai-review findings [ai-review-bot][success] 8419e60848

chore(ai-review): 三條 preflight 測試誤判寫入 exclusions 並清空 findings f047b4473e

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

Merge pull request 'chore(ai-review): 三條 preflight 測試誤判寫入 exclusions 並清空 findings' (#13) from feat/inline-critical-comments into develop ... 5ac73091cd

Reviewed-on: #13

feat(腳色系統): 改用 skill RPG 攻防腳色、新增 Mage 邏輯角色並讓 Step3/4 套上 Paladin 裁決人設 1602853c99

test(roles 與 comments): 新增腳色載入器測試與嚴重問題 comment 的邊界/多筆案例 d8b681e63b

chore(ai-review): 遷移 exclusions 舊腳色名並清空 findings 9e3c7f61bf

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

fix(腳色載入器): 壞角色檔改記錄警告並略過、快取解析結果並補 focus/personality 缺漏防護 60001499da

test(roles): 補 focus/personality 缺漏時的輸出防護測試 fa95a463f8

chore(ai-review): 清空 findings 97888f8b22

chore: update ai-review findings [ai-review-bot][success] 862f4e46ef

Merge pull request '優化 Step2：改用 skill RPG 攻防腳色系統（新增 Mage 邏輯角色、Step3/4 套 Paladin 裁決人設）' (#14) from feat/optimize-step2 into develop ... ffc9038923

Reviewed-on: #14

jiantw83 scheduled this pull request to auto merge when all checks succeed 2026-06-16 09:05:33 +00:00

admin approved these changes 2026-06-16 09:05:52 +00:00

jiantw83 merged commit 92f10c7970 into master 2026-06-16 09:05:55 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

admin

Labels

No items

No Label

Milestone

No items

No Milestone

Assignees

Clear assignees

admin (系統管理員) jiantw83 (Jeffery)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: actions/code-review#15