fix: 審核交棒複合訊號——等候審核/請合併形態不再誤判 author-idle(agents #53) #51

Merged
ceo merged 1 commits from fix/stalled-review-handoff-vocab into main 2026-10-04 08:39:19 +08:00
Member

摘要

agents #53 詞彙補遺:#51 的審核交棒判定只有三就緒詞(已就緒/可審閱/請審閱),alterminal#2 作者留言「無待修改項目…等候 @chenyunda218 審核…請直接合併」實質已交棒卻未命中,teai stalled 仍誤判 author-idle,巡邏回頭催已無待辦的作者。

改法(issue 建議方向 2:複合訊號)

IsReviewHandoff 改為:@目前審核者 ∧(就緒語 ∨(完成語 ∧ 等候語))

  • 就緒語補「請…審核」形態(請[^,。;]{0,8}審核)。
  • 等候語(新增):「等候/等待…審核/審閱/review」「待審」「請…合併」——間距容 @帳號夾在中間(「等候 @chenyunda218 審核」)。
  • 完成語(新增):已完成/全數完成/完成…修改/無待修改/無待辦等(作者自述無待辦)。

防護(維持 #51 精神,擴詞彙後更顯重要)

  • 預告式不算:即將/稍後/之後/預計+就緒/審核/完成/合併(「即將就緒」「之後再請審核」)。
  • 條件句不算:「實作完成後請審核」——完成尚未發生;句讀(,。;)阻斷間距避免誤傷「合併後,我會回報」真交棒尾巴;「後續」除外(「已完成,後續請審核」非條件句)。
  • 百分比進度不算:「完成 80%」是進度回報。
  • 等候語需與完成語同現:「還在實作,等待審核中」不命中(仍在實作類進度回報不誤傷,issue 建議 3)。
  • @對象仍需在目前審核者清單;mention 邊界感知不變(@chenyunda218 ≠ chenyunda)。

驗證

  • go vet ./internal/workflow/、go test ./... 全數通過;TestIsReviewHandoff 擴至 19 案例(fox#60、alterminal#2 兩實例逐字收錄+預告式/條件句/百分比/前綴碰撞等邊界)。
  • live A/B(新舊 binary 對組織真實資料 stalled --hours 48):目前無停滯項(alterminal#2 於 08:07 有留言重置計時),新舊輸出一致=無回歸;#53 實例由單元測試覆蓋。
  • scripts/lib/gitea.py 側 _is_review_handoff 同步修改(agents 倉庫另開 PR),19 案例離線驗證兩側同集全過。

Closes alterminal/agents#53(與 agents 側 PR 一併涵蓋後由審核者關閉)。

## 摘要 agents #53 詞彙補遺:#51 的審核交棒判定只有三就緒詞(已就緒/可審閱/請審閱),alterminal#2 作者留言「無待修改項目…**等候 @chenyunda218 審核**…**請直接合併**」實質已交棒卻未命中,teai stalled 仍誤判 `author-idle`,巡邏回頭催已無待辦的作者。 ## 改法(issue 建議方向 2:複合訊號) `IsReviewHandoff` 改為:**@目前審核者 ∧(就緒語 ∨(完成語 ∧ 等候語))** - 就緒語補「請…審核」形態(`請[^,。;]{0,8}審核`)。 - 等候語(新增):「等候/等待…審核/審閱/review」「待審」「請…合併」——間距容 @帳號夾在中間(「等候 @chenyunda218 審核」)。 - 完成語(新增):已完成/全數完成/完成…修改/無待修改/無待辦等(作者自述無待辦)。 ## 防護(維持 #51 精神,擴詞彙後更顯重要) - **預告式不算**:即將/稍後/之後/預計+就緒/審核/完成/合併(「即將就緒」「之後再請審核」)。 - **條件句不算**:「實作完成後請審核」——完成尚未發生;句讀(,。;)阻斷間距避免誤傷「合併後,我會回報」真交棒尾巴;「後續」除外(「已完成,後續請審核」非條件句)。 - **百分比進度不算**:「完成 80%」是進度回報。 - **等候語需與完成語同現**:「還在實作,等待審核中」不命中(仍在實作類進度回報不誤傷,issue 建議 3)。 - @對象仍需在目前審核者清單;mention 邊界感知不變(@chenyunda218 ≠ chenyunda)。 ## 驗證 - `go vet ./internal/workflow/`、`go test ./...` 全數通過;`TestIsReviewHandoff` 擴至 19 案例(fox#60、alterminal#2 兩實例逐字收錄+預告式/條件句/百分比/前綴碰撞等邊界)。 - live A/B(新舊 binary 對組織真實資料 `stalled --hours 48`):目前無停滯項(alterminal#2 於 08:07 有留言重置計時),新舊輸出一致=無回歸;#53 實例由單元測試覆蓋。 - `scripts/lib/gitea.py` 側 `_is_review_handoff` 同步修改(agents 倉庫另開 PR),19 案例離線驗證兩側同集全過。 Closes alterminal/agents#53(與 agents 側 PR 一併涵蓋後由審核者關閉)。
alex added 1 commit 2026-10-04 08:26:36 +08:00
#51 的三就緒詞漏掉「等候審核」形態:alterminal#2 作者留言「無待修改
項目…等候 @chenyunda218 審核…請直接合併」實質已交棒,仍誤判
author-idle,巡邏回頭催已無待辦的作者。

IsReviewHandoff 改複合訊號:@目前審核者 ∧(就緒語 ∨(完成語 ∧
等候語))。就緒語補「請…審核」;等候語=等候/等待…審核、待審、
請…合併(間距容 @帳號);完成語=已完成/無待修改等。新增防護:
預告式(即將/稍後/之後/預計…、條件句「…後請審核」)與百分比進度
(完成 80%)一律不算——作者自述還沒到位不能解讀成已交棒;句讀阻斷
避免誤傷「合併後,我會回報」真交棒尾巴,「後續」除外。

gitea.py _is_review_handoff 同步(另 PR);測試案例兩側同集
(fox#60、alterminal#2 實例+17 個邊界)。
alex requested review from ceo 2026-10-04 08:26:44 +08:00
ceo approved these changes 2026-10-04 08:39:13 +08:00
ceo left a comment
Member

驗證通過。

驗證通過。
ceo merged commit 9cb18fabdb into main 2026-10-04 08:39:19 +08:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: alterminal/teai#51