Compare commits

...
Author SHA1 Message Date
chenyunda218 1d9092cdde fix:stalled 分類同步 agents#47——assignee 交付報告待確認改判 waiting-outside
與 agents 86f570a(PR alterminal/agents#48)同語義:最後一則 assignee
留言符合交付報告特徵(結構訊號:逐項查核/驗收結果、彙總 verdict、內文
含「完成定義」段落時簡式交付語也算;∧ 交棒訊號:回報/關閉他件 #N、
或請/可由/待他人確認關閉)時,assignee-idle 改判 waiting-outside。

兩訊號同時命中才翻轉:認領/進度/預告式留言(無交棒對象)保持
assignee-idle,不製造「認領後棄置永不被催」的新死角。

驗證:TestIsDeliveryReport(8 案例,與 gitea.py 離線驗證同集)+
TestScanStalledDeliveryReport(stalled 整合:交付報告翻轉、認領不翻轉)
+全套 go test 綠;live A/B 對安裝版二進位:bear#52 assignee-idle →
waiting-outside,bear#39 parent-tracking 不變。
2026-09-15 04:28:08 +08:00
ydc3148 540718dda7 feat: teai role 2026-09-14 16:13:02 +08:00
alex 8735e35db6 Merge pull request 'fix:stalled 分類同步 agents#44——子項全關閉的父追蹤項一律標 parent-tracking(#47)' (#48) from fix/stalled-parent-wrapup-parity into main 2026-09-14 08:07:55 +08:00
ceo d89caf4af2 fix:stalled 分類同步 agents#44——子項全關閉的父追蹤項一律標 parent-tracking(#47)
considerIssue 原先僅在 anyOpen(children) 時標 parent-tracking,子項全關閉時
退回通用原因(如 assignee-idle),與 gitea.py(agents#44 修正)不一致,
導致巡邏「父項收尾」規則永不觸發。改為 children 非空一律 parent-tracking,
並移除已無引用的 anyOpen。

回歸測試 TestScanStalledParentWrapup 補子項全關閉 fixture;
fakeSource.OpenIssues 過濾 closed 以對齊真實 API 語義。
2026-09-14 08:04:37 +08:00
16 changed files with 615 additions and 132 deletions
+27 -16
View File
@@ -6,7 +6,7 @@ teai 是 Gitea 的命令列(CLI)工具,以更完整、更可靠的方式
- **語言:** Go(與 Gitea 本體一致),僅用標準庫,離線可建置 - **語言:** Go(與 Gitea 本體一致),僅用標準庫,離線可建置
- **目標站點:** `https://gitea.alterminal.com` - **目標站點:** `https://gitea.alterminal.com`
- **狀態:** 基礎建設(#5:internal/gitea)、工作流命令(#6:whoami/orgs/members/mine/pulls/next)、日常操作命令(#8:issues/pulls/labels/milestones/releases/repos/api)與管理者巡邏命令(#7:stalled/xrefs)已實作 - **狀態:** 基礎建設(#5:internal/gitea)、工作流命令(#6:whoami/role/orgs/members/mine/pulls/next)、日常操作命令(#8:issues/pulls/labels/milestones/releases/repos/api)與管理者巡邏命令(#7:stalled/xrefs)已實作
- **範圍(#3):** 不避開 tea 已有的功能;單一工具涵蓋完整操作 - **範圍(#3):** 不避開 tea 已有的功能;單一工具涵蓋完整操作
## 安裝與建置 ## 安裝與建置
@@ -59,8 +59,6 @@ clone 建置時 Go 會把 VCS 狀態寫進 build info:位於 tag 上顯示該
### 登入設定(首次使用) ### 登入設定(首次使用)
teai 與 `tea` 共用同一份組態檔;若 `tea whoami` 已可正常輸出,可跳過本節。
```sh ```sh
teai login add --url https://gitea.alterminal.com teai login add --url https://gitea.alterminal.com
# 未給 --token 時自 stdin 讀一行:貼上 token 後按 Enter(不會進 shell 歷史) # 未給 --token 時自 stdin 讀一行:貼上 token 後按 Enter(不會進 shell 歷史)
@@ -68,8 +66,7 @@ teai whoami # 驗證:應輸出 {"username":"<你的帳號>"}
``` ```
- token 驗證失敗(HTTP 401)不會寫入組態,結束碼 3。 - token 驗證失敗(HTTP 401)不會寫入組態,結束碼 3。
- 組態檔:`--config`/`TEA_CONFIG` 指定,預設 `~/.config/tea/config.yml`;teai 採行級編輯, - 組態檔:`--config`/`TEAI_CONFIG` 指定,預設 `~/.config/teai/config.yml`;檔案權限 `0600`。
不會改壞 tea 也在使用的欄位,檔案權限 `0600`。
- 多站台以 `--name` 區分、`teai login default <名稱>` 切換預設;詳見下方 [`teai login`](#teai-login--管理登入) 一節。 - 多站台以 `--name` 區分、`teai login default <名稱>` 切換預設;詳見下方 [`teai login`](#teai-login--管理登入) 一節。
## 指令設計 ## 指令設計
@@ -83,7 +80,7 @@ teai whoami # 驗證:應輸出 {"username":"<你的帳號>"}
2. **規則單一來源**:「最後一則留言不是自己」等工作流判定規則以 `AGENTS.md` 為準,teai 只實作、不改義。規則要變更,先改 AGENTS.md,teai 跟著改。 2. **規則單一來源**:「最後一則留言不是自己」等工作流判定規則以 `AGENTS.md` 為準,teai 只實作、不改義。規則要變更,先改 AGENTS.md,teai 跟著改。
3. **機器可讀優先**:預設輸出 JSON;`--output table` 僅供人工抽查。 3. **機器可讀優先**:預設輸出 JSON;`--output table` 僅供人工抽查。
4. **可直接替換原型**:`gitea.py` 的每個子命令在 teai 都有語義相同的對應命令,可交叉驗證、平滑遷移。 4. **可直接替換原型**:`gitea.py` 的每個子命令在 teai 都有語義相同的對應命令,可交叉驗證、平滑遷移。
5. **認證沿用 tea**:讀 `tea` 的登入組態(`TEA_CONFIG` 或 `~/.config/tea/config.yml`),不另建認證體系。 5. **自主認證組態**:登入資料寫在 teai 自己的組態(`TEAI_CONFIG` 或 `~/.config/teai/config.yml`),不依賴 tea。
6. **安全預設**:查詢類唯讀;寫入類命令(#8 起)一律要求明確 `--yes`,未確認時不發出任何請求、不產生任何遠端變更。 6. **安全預設**:查詢類唯讀;寫入類命令(#8 起)一律要求明確 `--yes`,未確認時不發出任何請求、不產生任何遠端變更。
### 日常操作(tea 對等指令;#3 不避開 tea 已有功能;#8 已實作) ### 日常操作(tea 對等指令;#3 不避開 tea 已有功能;#8 已實作)
@@ -138,8 +135,8 @@ teai [全域選項] <命令> [參數]
| 選項 | 說明 | | 選項 | 說明 |
| --- | --- | | --- | --- |
| `--url <URL>` | Gitea 站點,預設 `https://gitea.alterminal.com` | | `--url <URL>` | Gitea 站點,預設 `https://gitea.alterminal.com` |
| `--token <TOKEN>` | API token;未給則依序嘗試 `TEAI_TOKEN` 環境變數、tea 登入組態 | | `--token <TOKEN>` | API token;未給則依序嘗試 `TEAI_TOKEN` 環境變數、teai 登入組態 |
| `--config <path>` | tea 組態檔路徑,等同 `TEA_CONFIG` | | `--config <path>` | teai 組態檔路徑,等同 `TEAI_CONFIG` |
| `--output json\|table` | 輸出格式,預設 `json` | | `--output json\|table` | 輸出格式,預設 `json` |
| `--timeout <dur>` | HTTP 請求逾時,預設 `30s` | | `--timeout <dur>` | HTTP 請求逾時,預設 `30s` |
@@ -147,8 +144,9 @@ teai [全域選項] <命令> [參數]
| 命令 | 對應 gitea.py | 說明 | | 命令 | 對應 gitea.py | 說明 |
| --- | --- | --- | | --- | --- | --- |
| `teai login list\|add\|default\|remove` | — | 管理 tea 相容登入組態(見下) | | `teai login list\|add\|default\|remove` | — | 管理登入組態(見下) |
| `teai whoami` | `get_current_username` | 目前帳號 | | `teai whoami` | `get_current_username` | 目前帳號 |
| `teai role` | — | 目前角色(Supervisor 團隊→supervisor,Agents 團隊→worker) |
| `teai orgs` | `for_all_organizations` | 我所屬的組織 | | `teai orgs` | `for_all_organizations` | 我所屬的組織 |
| `teai organizations list\|create\|delete` | — | 組織管理(tea 對等日常操作;`orgs` 是工作流精簡版) | | `teai organizations list\|create\|delete` | — | 組織管理(tea 對等日常操作;`orgs` 是工作流精簡版) |
| `teai members [--has-work]` | `members [--has-work]` | Agents 團隊成員;`--has-work` 只列有未完成工作者 | | `teai members [--has-work]` | `members [--has-work]` | Agents 團隊成員;`--has-work` 只列有未完成工作者 |
@@ -189,9 +187,8 @@ teai next --output table # 人類可讀
### `teai login` — 管理登入 ### `teai login` — 管理登入
teai 與 tea 共用同一份組態檔(`--config`/`TEA_CONFIG`,預設 `~/.config/tea/config.yml`)。 登入組態由 teai 自行管理(`--config`/`TEAI_CONFIG`,預設 `~/.config/teai/config.yml`)。
寫入採行級編輯:只動目標項目的行,未知欄位(`ssh_*` 等)與 `preferences` 區段逐字保留, 寫入採行級編輯:只動目標項目的行,未知欄位與註解逐字保留;檔案權限 `0600`、暫存檔原子替換。
不改壞 tea 也在使用的檔案;檔案權限 `0600`、暫存檔原子替換。
```sh ```sh
teai login list # 列出登入(一律不含 token) teai login list # 列出登入(一律不含 token)
@@ -240,6 +237,11 @@ PR 改帶 `author` 與 `reviewers` 欄位。`reason` 取值:
- issue:`no-assignee`、`assignee-idle`、`no-commenter`、`waiting-outside`、`parent-tracking` - issue:`no-assignee`、`assignee-idle`、`no-commenter`、`waiting-outside`、`parent-tracking`
- PR:`no-reviewer`、`author-idle`、`reviewer-idle`、`no-commenter` - PR:`no-reviewer`、`author-idle`、`reviewer-idle`、`no-commenter`
`assignee-idle` 判定前會先檢查最後一則 assignee 留言是否為**交付報告**(agents #47):
逐項查核/驗收結果(結構訊號)∧ 回報他件/請他人確認關閉(交棒訊號)同時命中 →
改判 `waiting-outside`——已交付待發起人確認不是 assignee 停滯。認領/進度/預告式
留言(無交棒對象)維持 `assignee-idle`,不製造「認領後棄置永不被催」的新死角。
### `teai milestones` — 跨倉庫里程碑總覽 ### `teai milestones` — 跨倉庫里程碑總覽
```sh ```sh
@@ -305,13 +307,21 @@ teai pulls --repo alterminal/teai # 限定單一倉庫
teai mine # 分派給我、最後一則留言不是自己的 open issues(跨所屬組織) teai mine # 分派給我、最後一則留言不是自己的 open issues(跨所屬組織)
``` ```
### `teai orgs` / `teai whoami` ### `teai orgs` / `teai whoami` / `teai role`
```sh ```sh
teai orgs # 我所屬的組織 teai orgs # 我所屬的組織
teai whoami # 目前帳號(讀 tea 組態) teai whoami # 目前帳號
teai role # 目前角色
``` ```
角色由所屬組織的團隊決定:
- 在 **Supervisor** 團隊 → `{"role":"supervisor"}`
- 在 **Agents** 團隊 → `{"role":"worker"}`
- 同時屬於兩者時以 supervisor 為準
- 兩者都不屬於 → 輸出 `null`,exit 0
### `teai xrefs <owner>/<repo> <number>` — 引用解析 ### `teai xrefs <owner>/<repo> <number>` — 引用解析
解析該 issue/PR 內文與全部留言中的 `#N`、`repo#N`、`owner/repo#N` 引用,輸出解析後的目標清單(去重): 解析該 issue/PR 內文與全部留言中的 `#N`、`repo#N`、`owner/repo#N` 引用,輸出解析後的目標清單(去重):
@@ -330,7 +340,7 @@ teai whoami # 目前帳號(讀 tea 組態)
| 碼 | 意義 | | 碼 | 意義 |
| --- | --- | | --- | --- |
| 0 | 成功(含「沒有結果」→ 清單類命令輸出 `[]`、`next` 輸出 `null`;#37 判決) | | 0 | 成功(含「沒有結果」→ 清單類命令輸出 `[]`、`next`/`role` 輸出 `null`;#37 判決) |
| 2 | 用法錯誤(未知命令/參數) | | 2 | 用法錯誤(未知命令/參數) |
| 3 | API 錯誤(連線失敗、401、403、5xx) | | 3 | API 錯誤(連線失敗、401、403、5xx) |
@@ -355,6 +365,7 @@ teai whoami # 目前帳號(讀 tea 組態)
「沒有未完成的工作」。**判決 #37:維持 teai 現行 JSON 一致性(方向 2)**——消費端 「沒有未完成的工作」。**判決 #37:維持 teai 現行 JSON 一致性(方向 2)**——消費端
(agents 倉庫巡邏提示與觸發腳本)應以「輸出 `[]`/`null` 且 exit 0」為「沒有」判據, (agents 倉庫巡邏提示與觸發腳本)應以「輸出 `[]`/`null` 且 exit 0」為「沒有」判據,
不得以「無輸出」判斷;`members [--has-work]` 維持逐行帳號、空時無輸出(兩者一致,不受影響)。 不得以「無輸出」判斷;`members [--has-work]` 維持逐行帳號、空時無輸出(兩者一致,不受影響)。
`teai role` 不在 Supervisor/Agents 時同樣輸出 `null`(exit 0)。
- **清單欄位集(`mine`/`pulls`)**:`gitea.py` 直接傾倒完整 Gitea API 物件 - **清單欄位集(`mine`/`pulls`)**:`gitea.py` 直接傾倒完整 Gitea API 物件
(`_dump` 的副作用,欄位集隨 Gitea 版本浮動);teai 輸出策展摘要欄位 (`_dump` 的副作用,欄位集隨 Gitea 版本浮動);teai 輸出策展摘要欄位
(`mine`:repo/number/title/url;`pulls`:number/title/state/author/ (`mine`:repo/number/title/url;`pulls`:number/title/state/author/
@@ -374,7 +385,7 @@ teai whoami # 目前帳號(讀 tea 組態)
### 安全設計 ### 安全設計
- 本期命令全部唯讀;寫入類命令未來加入時須 `--yes` 明確確認。 - 本期命令全部唯讀;寫入類命令未來加入時須 `--yes` 明確確認。
- token 不入輸出、不入日誌;建議用 `TEAI_TOKEN` 或 tea 組態,避免出現在命令列。 - token 不入輸出、不入日誌;建議用 `TEAI_TOKEN` 或 teai 組態,避免出現在命令列。
- 清單類 API 一律帶 `limit`(Gitea 預設上限 50),避免漏抓。 - 清單類 API 一律帶 `limit`(Gitea 預設上限 50),避免漏抓。
- 「有無實際產出」的判斷一律對照遠端實況(API 回傳),不信任回報文字。 - 「有無實際產出」的判斷一律對照遠端實況(API 回傳),不信任回報文字。
+4 -4
View File
@@ -75,9 +75,9 @@ type command struct {
type Globals struct { type Globals struct {
// URL 是 Gitea 站點網址。 // URL 是 Gitea 站點網址。
URL string URL string
// Token 來自 --token 旗標;空字串表示未提供(依序退回環境變數與 tea 組態)。 // Token 來自 --token 旗標;空字串表示未提供(依序退回環境變數與 teai 組態)。
Token string Token string
// ConfigPath 是 tea 組態路徑(--config,等同 TEA_CONFIG)。 // ConfigPath 是 teai 組態路徑(--config,等同 TEAI_CONFIG)。
ConfigPath string ConfigPath string
// Output 是輸出格式。 // Output 是輸出格式。
Output gitea.Format Output gitea.Format
@@ -464,8 +464,8 @@ func printUsage(w io.Writer) {
fmt.Fprintf(w, "用法:\n teai [全域選項] <命令> [參數]\n\n") fmt.Fprintf(w, "用法:\n teai [全域選項] <命令> [參數]\n\n")
fmt.Fprintf(w, "全域選項:\n") fmt.Fprintf(w, "全域選項:\n")
fmt.Fprintf(w, " --url <URL> Gitea 站點(預設 https://gitea.alterminal.com)\n") fmt.Fprintf(w, " --url <URL> Gitea 站點(預設 https://gitea.alterminal.com)\n")
fmt.Fprintf(w, " --token <TOKEN> API token;未給則依序嘗試 TEAI_TOKEN、tea 登入組態\n") fmt.Fprintf(w, " --token <TOKEN> API token;未給則依序嘗試 TEAI_TOKEN、teai 登入組態\n")
fmt.Fprintf(w, " --config <path> tea 組態檔路徑,等同 TEA_CONFIG\n") fmt.Fprintf(w, " --config <path> teai 組態檔路徑,等同 TEAI_CONFIG\n")
fmt.Fprintf(w, " --output <format> 輸出格式 json|table(預設 json)\n") fmt.Fprintf(w, " --output <format> 輸出格式 json|table(預設 json)\n")
fmt.Fprintf(w, " --timeout <dur> HTTP 逾時(預設 30s)\n\n") fmt.Fprintf(w, " --timeout <dur> HTTP 逾時(預設 30s)\n\n")
fmt.Fprintf(w, "命令:\n") fmt.Fprintf(w, "命令:\n")
+9 -9
View File
@@ -1,6 +1,6 @@
// login.go — teai login 指令:管理 tea 相容的登入組態。 // login.go — teai login 指令:管理 teai 自己的登入組態。
// //
// teai 與 tea 共用同一份組態檔(README 設計原則 5),login 的職責是: // 組態由 teai 自行讀寫(README 設計原則 5),不依賴 tea。login 的職責是:
// 以 token 向站點驗證(GET /user),成功後才寫入組態並記下帳號。 // 以 token 向站點驗證(GET /user),成功後才寫入組態並記下帳號。
// token 只走記憶體與組態檔,不進輸出、日誌或錯誤訊息。 // token 只走記憶體與組態檔,不進輸出、日誌或錯誤訊息。
// //
@@ -25,20 +25,20 @@ import (
// loginStdin 是讀 token 與確認的輸入(測試可替換)。 // loginStdin 是讀 token 與確認的輸入(測試可替換)。
var loginStdin io.Reader = os.Stdin var loginStdin io.Reader = os.Stdin
// loginConfigPath 回傳組態檔路徑:--config/TEA_CONFIG 優先,其次 // loginConfigPath 回傳組態檔路徑:--config/TEAI_CONFIG 優先,其次
// ~/.config/tea/config.yml。找不到可用路徑回空字串。 // ~/.config/teai/config.yml。找不到可用路徑回空字串。
func loginConfigPath(g Globals) string { func loginConfigPath(g Globals) string {
if p := strings.TrimSpace(g.ConfigPath); p != "" { if p := strings.TrimSpace(g.ConfigPath); p != "" {
return expandHome(p) return expandHome(p)
} }
if p := os.Getenv("TEA_CONFIG"); strings.TrimSpace(p) != "" { if p := os.Getenv("TEAI_CONFIG"); strings.TrimSpace(p) != "" {
return expandHome(strings.TrimSpace(p)) return expandHome(strings.TrimSpace(p))
} }
home, err := os.UserHomeDir() home, err := os.UserHomeDir()
if err != nil || home == "" { if err != nil || home == "" {
return "" return ""
} }
return filepath.Join(home, ".config", "tea", "config.yml") return filepath.Join(home, ".config", "teai", "config.yml")
} }
// expandHome 展開路徑開頭的 ~。 // expandHome 展開路徑開頭的 ~。
@@ -178,7 +178,7 @@ func runLoginAdd(env *Env, args []string) error {
if err != nil { if err != nil {
return err return err
} }
if err := gitea.SaveTeaConfigFile(path, out); err != nil { if err := gitea.SaveConfigFile(path, out); err != nil {
return err return err
} }
p := gitea.Printer{Format: env.Globals.Output, Out: env.Out} p := gitea.Printer{Format: env.Globals.Output, Out: env.Out}
@@ -329,7 +329,7 @@ func saveLoginConfig(env *Env, data []byte) error {
if path == "" { if path == "" {
return fmt.Errorf("cannot determine config path") return fmt.Errorf("cannot determine config path")
} }
return gitea.SaveTeaConfigFile(path, data) return gitea.SaveConfigFile(path, data)
} }
// findLoginByHost 在組態內容中找 URL 主機與 siteURL 相同的登入。 // findLoginByHost 在組態內容中找 URL 主機與 siteURL 相同的登入。
@@ -352,7 +352,7 @@ func findLogin(data []byte, name string) (gitea.LoginEntry, bool) {
return gitea.LoginEntry{}, false return gitea.LoginEntry{}, false
} }
// defaultLoginName 以 URL 主機作為登入名稱(同 tea 的習慣)。 // defaultLoginName 以 URL 主機作為登入名稱。
func defaultLoginName(rawURL string) string { func defaultLoginName(rawURL string) string {
s := rawURL s := rawURL
if !strings.Contains(s, "://") { if !strings.Contains(s, "://") {
+32 -1
View File
@@ -25,7 +25,7 @@ func loginTestEnv(t *testing.T, configContent string) (args []string, configPath
t.Fatal(err) t.Fatal(err)
} }
} }
t.Setenv("TEA_CONFIG", configPath) t.Setenv("TEAI_CONFIG", configPath)
return []string{"--config", configPath}, configPath return []string{"--config", configPath}, configPath
} }
@@ -117,6 +117,13 @@ func TestLoginAddVerifiesThenWrites(t *testing.T) {
if info.Mode().Perm() != 0o600 { if info.Mode().Perm() != 0o600 {
t.Fatalf("perm = %v", info.Mode().Perm()) t.Fatalf("perm = %v", info.Mode().Perm())
} }
// 新項目只寫 teai 欄位,不含 tea 的 ssh_*/preferences。
s := string(data)
for _, ban := range []string{"ssh_", "insecure:", "version_check:", "preferences:"} {
if strings.Contains(s, ban) {
t.Fatalf("tea field leaked %q:\n%s", ban, s)
}
}
} }
func TestLoginAddTokenFromStdin(t *testing.T) { func TestLoginAddTokenFromStdin(t *testing.T) {
@@ -187,6 +194,7 @@ preferences:
if strings.Contains(s, "old-tok") || !strings.Contains(s, "token: new-tok") { if strings.Contains(s, "old-tok") || !strings.Contains(s, "token: new-tok") {
t.Fatalf("token not replaced:\n%s", s) t.Fatalf("token not replaced:\n%s", s)
} }
// 未知欄位與其他區段逐字保留(行級編輯,不整檔覆寫)。
if !strings.Contains(s, `ssh_key: "keep-me"`) || !strings.Contains(s, "editor: false") { if !strings.Contains(s, `ssh_key: "keep-me"`) || !strings.Contains(s, "editor: false") {
t.Fatalf("unrelated content lost:\n%s", s) t.Fatalf("unrelated content lost:\n%s", s)
} }
@@ -290,3 +298,26 @@ func TestLoginRegisteredInUsage(t *testing.T) {
t.Fatalf("usage should list login: %s", stdout) t.Fatalf("usage should list login: %s", stdout)
} }
} }
func TestLoginConfigPathIndependentOfTea(t *testing.T) {
t.Setenv("TEA_CONFIG", "/from-tea.yml")
t.Setenv("TEAI_CONFIG", "")
g := defaultGlobals()
got := loginConfigPath(g)
if got == "/from-tea.yml" {
t.Fatal("TEA_CONFIG must not be used")
}
if !strings.HasSuffix(filepath.ToSlash(got), ".config/teai/config.yml") {
t.Fatalf("default path want ~/.config/teai/config.yml, got %q", got)
}
t.Setenv("TEAI_CONFIG", "/from-teai.yml")
if got := loginConfigPath(g); got != "/from-teai.yml" {
t.Fatalf("TEAI_CONFIG: got %q", got)
}
g.ConfigPath = "/explicit.yml"
if got := loginConfigPath(g); got != "/explicit.yml" {
t.Fatalf("--config: got %q", got)
}
}
+25 -1
View File
@@ -1,5 +1,5 @@
// workflow_commands.go — gitea.py 工作流命令的 CLI 接線(#6): // workflow_commands.go — gitea.py 工作流命令的 CLI 接線(#6):
// whoami、orgs、members [--has-work]、mine、pulls [--mine|--reviewer] [--repo]、next。 // whoami、role、orgs、members [--has-work]、mine、pulls [--mine|--reviewer] [--repo]、next。
// //
// 輸出走 internal/gitea 的 Printer(--output json|table);錯誤經 cli.exitCodeFor // 輸出走 internal/gitea 的 Printer(--output json|table);錯誤經 cli.exitCodeFor
// 映射結束碼(用法 2/API 3)。token 解析選序見 internal/gitea/auth.go。 // 映射結束碼(用法 2/API 3)。token 解析選序見 internal/gitea/auth.go。
@@ -20,6 +20,7 @@ func init() {
commands[c.name] = c commands[c.name] = c
} }
register(&command{name: "whoami", usage: "目前帳號", run: runWhoami}) register(&command{name: "whoami", usage: "目前帳號", run: runWhoami})
register(&command{name: "role", usage: "目前角色(Supervisor→supervisor,Agents→worker)", run: runRole})
register(&command{name: "orgs", usage: "我所屬的組織", run: runOrgs}) register(&command{name: "orgs", usage: "我所屬的組織", run: runOrgs})
register(&command{name: "members", usage: "Agents 團隊成員;--has-work 只列有未完成工作者", run: runMembers}) register(&command{name: "members", usage: "Agents 團隊成員;--has-work 只列有未完成工作者", run: runMembers})
register(&command{name: "mine", usage: "分派給我、待跟進的 issues", run: runMine}) register(&command{name: "mine", usage: "分派給我、待跟進的 issues", run: runMine})
@@ -77,6 +78,29 @@ func runWhoami(env *Env, args []string) error {
) )
} }
// runRole:teai role。不在 Supervisor/Agents 團隊時輸出 null(exit 0)。
func runRole(env *Env, args []string) error {
if len(args) != 0 {
return &ErrUsage{Msg: "role 不接受參數"}
}
s, err := newScannerFunc(env)
if err != nil {
return err
}
role, err := s.Role()
if err != nil {
return err
}
p := printer(env)
if role == "" {
return p.PrintNull()
}
return p.PrintOne(
[]gitea.Field{{Name: "role"}},
gitea.Values{role},
)
}
// runOrgs:teai orgs。 // runOrgs:teai orgs。
func runOrgs(env *Env, args []string) error { func runOrgs(env *Env, args []string) error {
if len(args) != 0 { if len(args) != 0 {
+65
View File
@@ -224,6 +224,71 @@ func TestWiringWhoamiOrgs(t *testing.T) {
} }
} }
// TestWiringRoleWorker 驗證 Agents 成員的 role 輸出 worker。
func TestWiringRoleWorker(t *testing.T) {
newStubEnv(t, newStubData())
stdout, _, code := run("role")
if code != 0 {
t.Fatalf("role 應回 0,得到 %d(stdout=%q)", code, stdout)
}
if strings.TrimSpace(stdout) != `{"role":"worker"}` {
t.Fatalf("Agents 成員 role 應輸出 worker,得到 %q", stdout)
}
}
// TestWiringRoleSupervisor 驗證 Supervisor 成員的 role 輸出 supervisor。
func TestWiringRoleSupervisor(t *testing.T) {
c := newStubData()
c.teams["alterminal"] = []workflow.Team{
{ID: 1, Name: "Agents"},
{ID: 2, Name: "Supervisor"},
}
c.members[2] = []string{"alex"}
newStubEnv(t, c)
stdout, _, code := run("role")
if code != 0 {
t.Fatalf("role 應回 0,得到 %d(stdout=%q)", code, stdout)
}
if strings.TrimSpace(stdout) != `{"role":"supervisor"}` {
t.Fatalf("Supervisor 成員 role 應輸出 supervisor,得到 %q", stdout)
}
}
// TestWiringRoleNone 驗證不在任一角色團隊時輸出 null、exit 0。
func TestWiringRoleNone(t *testing.T) {
c := newStubData()
c.currentUser = "outsider"
newStubEnv(t, c)
stdout, _, code := run("role")
if code != 0 {
t.Fatalf("role 無角色應回 0,得到 %d", code)
}
if strings.TrimSpace(stdout) != "null" {
t.Fatalf("role 無角色應輸出 null,得到 %q", stdout)
}
}
// TestWiringRoleRejectsArgs 驗證 role 不接受參數。
func TestWiringRoleRejectsArgs(t *testing.T) {
newStubEnv(t, newStubData())
_, _, code := run("role", "extra")
if code != int(ExitUsage) {
t.Fatalf("role extra 應回 ExitUsage(%d),得到 %d", int(ExitUsage), code)
}
}
// TestWiringRoleTable 驗證 teai role --output table。
func TestWiringRoleTable(t *testing.T) {
newStubEnv(t, newStubData())
stdout, _, code := run("role", "--output", "table")
if code != 0 {
t.Fatalf("role --output table 應回 0,得到 %d", code)
}
if !strings.Contains(stdout, "worker") {
t.Fatalf("role --output table 應含 worker,得到 %q", stdout)
}
}
// TestWiringVersionRejectsArgs 驗證 version 不接受參數(新剖析語義下仍 exit 2)。 // TestWiringVersionRejectsArgs 驗證 version 不接受參數(新剖析語義下仍 exit 2)。
func TestWiringVersionRejectsArgs(t *testing.T) { func TestWiringVersionRejectsArgs(t *testing.T) {
_, _, code := run("version", "--wat") _, _, code := run("version", "--wat")
+28 -29
View File
@@ -1,7 +1,7 @@
// auth.go — 認證來源的解析與選序。 // auth.go — 認證來源的解析與選序。
// //
// 選序(README「全域介面」):--token 旗標 → TEAI_TOKEN 環境變數 → // 選序(README「全域介面」):--token 旗標 → TEAI_TOKEN 環境變數 →
// tea 登入組態(TEA_CONFIG 指定的路徑,或 ~/.config/tea/config.yml)。 // teai 登入組態(TEAI_CONFIG 指定的路徑,或 ~/.config/teai/config.yml)。
// token 一律只在記憶體傳遞,不寫入輸出、日誌或錯誤訊息。 // token 一律只在記憶體傳遞,不寫入輸出、日誌或錯誤訊息。
package gitea package gitea
@@ -23,8 +23,8 @@ const (
SourceFlag SourceFlag
// SourceEnv 是 TEAI_TOKEN 環境變數。 // SourceEnv 是 TEAI_TOKEN 環境變數。
SourceEnv SourceEnv
// SourceTeaConfig 是 tea 登入組態。 // SourceConfig 是 teai 登入組態。
SourceTeaConfig SourceConfig
) )
// String 回傳來源名稱(診斷用)。 // String 回傳來源名稱(診斷用)。
@@ -34,8 +34,8 @@ func (s TokenSource) String() string {
return "flag" return "flag"
case SourceEnv: case SourceEnv:
return "env" return "env"
case SourceTeaConfig: case SourceConfig:
return "tea-config" return "config"
default: default:
return "none" return "none"
} }
@@ -62,12 +62,12 @@ func osReadFile(path string) ([]byte, error) {
type AuthOptions struct { type AuthOptions struct {
// FlagToken 是 --token 旗標值;空字串表示未提供。 // FlagToken 是 --token 旗標值;空字串表示未提供。
FlagToken string FlagToken string
// SiteURL 是目標站點(用來比對 tea 組態中的 login url)。 // SiteURL 是目標站點(用來比對組態中的 login url)。
SiteURL string SiteURL string
// Env 是環境變數查詢;nil 時用 os.LookupEnv。 // Env 是環境變數查詢;nil 時用 os.LookupEnv。
Env EnvLookup Env EnvLookup
// ConfigPath 是 tea 組態的明確路徑(TEA_CONFIG 或 --config)。 // ConfigPath 是 teai 組態的明確路徑(TEAI_CONFIG 或 --config)。
// 空字串表示用預設 ~/.config/tea/config.yml。 // 空字串表示用預設 ~/.config/teai/config.yml。
ConfigPath string ConfigPath string
// Home 是使用者家目錄(展開 ~ 用);空字串表示不展開。 // Home 是使用者家目錄(展開 ~ 用);空字串表示不展開。
Home string Home string
@@ -75,7 +75,7 @@ type AuthOptions struct {
ReadFile ReadFileFunc ReadFile ReadFileFunc
} }
// ResolveToken 依選序解析 token:flag → env(TEAI_TOKEN) → tea 組態。 // ResolveToken 依選序解析 token:flag → env(TEAI_TOKEN) → teai 組態。
// //
// 找不到任何 token 時回傳 ("", SourceNone, nil)——查詢公開端點不需要 // 找不到任何 token 時回傳 ("", SourceNone, nil)——查詢公開端點不需要
// token,是否為錯由呼叫端依情境決定。組態檔存在但解析失敗(格式錯誤) // token,是否為錯由呼叫端依情境決定。組態檔存在但解析失敗(格式錯誤)
@@ -93,8 +93,8 @@ func ResolveToken(opts AuthOptions) (string, TokenSource, error) {
} }
path := opts.ConfigPath path := opts.ConfigPath
if path == "" { if path == "" {
// --config 未給時,先看 TEA_CONFIG 環境變數。 // --config 未給時,先看 TEAI_CONFIG 環境變數。
if p, ok := env("TEA_CONFIG"); ok && strings.TrimSpace(p) != "" { if p, ok := env("TEAI_CONFIG"); ok && strings.TrimSpace(p) != "" {
path = strings.TrimSpace(p) path = strings.TrimSpace(p)
} }
} }
@@ -110,7 +110,7 @@ func ResolveToken(opts AuthOptions) (string, TokenSource, error) {
if home == "" { if home == "" {
return "", SourceNone, nil return "", SourceNone, nil
} }
path = filepath.Join(home, ".config", "tea", "config.yml") path = filepath.Join(home, ".config", "teai", "config.yml")
} }
if path == "" { if path == "" {
return "", SourceNone, nil return "", SourceNone, nil
@@ -124,33 +124,32 @@ func ResolveToken(opts AuthOptions) (string, TokenSource, error) {
// 組態不存在或讀不到:視為沒有這個來源,不是錯誤。 // 組態不存在或讀不到:視為沒有這個來源,不是錯誤。
return "", SourceNone, nil return "", SourceNone, nil
} }
login, err := parseTeaConfig(data, opts.SiteURL) login, err := parseConfig(data, opts.SiteURL)
if err != nil { if err != nil {
return "", SourceNone, fmt.Errorf("gitea: parse tea config %s: %w", path, err) return "", SourceNone, fmt.Errorf("gitea: parse config %s: %w", path, err)
} }
if login.Token == "" { if login.Token == "" {
return "", SourceNone, nil return "", SourceNone, nil
} }
return login.Token, SourceTeaConfig, nil return login.Token, SourceConfig, nil
} }
// teaLogin 是 tea 組態中一筆登入資料(只取 teai 需要的欄位)。 // configLogin 是組態中一筆登入資料(只取解析 token 需要的欄位)。
type teaLogin struct { type configLogin struct {
Name string Name string
URL string URL string
Token string Token string
Default bool Default bool
} }
// parseTeaConfig 從 tea 的 config.yml 內容挑出最匹配 siteURL 的登入: // parseConfig 從 teai 的 config.yml 內容挑出最匹配 siteURL 的登入:
// url 主機相同者優先,其中 default: true 最優,否則取第一筆; // url 主機相同者優先,其中 default: true 最優,否則取第一筆;
// 都不相同時退而取 default: true,再退第一筆。 // 都不相同時退而取 default: true,再退第一筆。
// //
// 僅解析 tea 實際寫出的 YAML 子集:頂層 `logins:` 清單,項目為 // 僅解析 teai 寫出的 YAML 子集:頂層 `logins:` 清單,項目為
// `key: value` 的平面映射。不處理錨點、多文件、巢狀清單——tea 不會寫出 // `key: value` 的平面映射。不處理錨點、多文件、巢狀清單。
// 這些,遇到也不該默默誤讀。 func parseConfig(data []byte, siteURL string) (configLogin, error) {
func parseTeaConfig(data []byte, siteURL string) (teaLogin, error) { var logins []configLogin
var logins []teaLogin
inLogins := false inLogins := false
cur := -1 // 目前項目在 logins 的索引;-1 表示尚未開始任何項目 cur := -1 // 目前項目在 logins 的索引;-1 表示尚未開始任何項目
for lineNo, raw := range strings.Split(string(data), "\n") { for lineNo, raw := range strings.Split(string(data), "\n") {
@@ -171,26 +170,26 @@ func parseTeaConfig(data []byte, siteURL string) (teaLogin, error) {
} }
if strings.HasPrefix(trimmed, "- ") || trimmed == "-" { if strings.HasPrefix(trimmed, "- ") || trimmed == "-" {
// 新的清單項目;`- name: x` 形式同行帶鍵值。 // 新的清單項目;`- name: x` 形式同行帶鍵值。
logins = append(logins, teaLogin{}) logins = append(logins, configLogin{})
cur = len(logins) - 1 cur = len(logins) - 1
rest := strings.TrimSpace(strings.TrimPrefix(trimmed, "-")) rest := strings.TrimSpace(strings.TrimPrefix(trimmed, "-"))
if rest != "" { if rest != "" {
if err := applyLoginField(&logins[cur], rest, lineNo+1); err != nil { if err := applyLoginField(&logins[cur], rest, lineNo+1); err != nil {
return teaLogin{}, err return configLogin{}, err
} }
} }
continue continue
} }
if cur < 0 { if cur < 0 {
// logins: 底下卻不是清單項目——不是預期的結構。 // logins: 底下卻不是清單項目——不是預期的結構。
return teaLogin{}, fmt.Errorf("line %d: expected list item under logins:", lineNo+1) return configLogin{}, fmt.Errorf("line %d: expected list item under logins:", lineNo+1)
} }
if err := applyLoginField(&logins[cur], trimmed, lineNo+1); err != nil { if err := applyLoginField(&logins[cur], trimmed, lineNo+1); err != nil {
return teaLogin{}, err return configLogin{}, err
} }
} }
if len(logins) == 0 { if len(logins) == 0 {
return teaLogin{}, nil return configLogin{}, nil
} }
wantHost := hostOf(siteURL) wantHost := hostOf(siteURL)
// 第一輪:主機相符者,default 優先,否則第一筆。 // 第一輪:主機相符者,default 優先,否則第一筆。
@@ -219,7 +218,7 @@ func parseTeaConfig(data []byte, siteURL string) (teaLogin, error) {
} }
// applyLoginField 把 `key: value` 套用到 login;value 的引號會剝除。 // applyLoginField 把 `key: value` 套用到 login;value 的引號會剝除。
func applyLoginField(l *teaLogin, kv string, lineNo int) error { func applyLoginField(l *configLogin, kv string, lineNo int) error {
key, value, ok := strings.Cut(kv, ":") key, value, ok := strings.Cut(kv, ":")
if !ok { if !ok {
return fmt.Errorf("line %d: expected key: value, got %q", lineNo, kv) return fmt.Errorf("line %d: expected key: value, got %q", lineNo, kv)
+41 -23
View File
@@ -1,5 +1,5 @@
// auth_test.go 驗證認證來源選序(--token → TEAI_TOKEN → tea 組態) // auth_test.go 驗證認證來源選序(--token → TEAI_TOKEN → teai 組態)
// 與 tea 組態解析。 // 與組態解析。
package gitea package gitea
import ( import (
@@ -25,7 +25,7 @@ func fakeReadFile(m map[string]string) ReadFileFunc {
} }
} }
const sampleTeaConfig = `logins: const sampleConfig = `logins:
- name: other.example.com - name: other.example.com
url: https://other.example.com url: https://other.example.com
token: other-token token: other-token
@@ -43,7 +43,7 @@ func TestResolveTokenOrdering(t *testing.T) {
SiteURL: "https://gitea.alterminal.com", SiteURL: "https://gitea.alterminal.com",
Env: fakeEnv(nil), Env: fakeEnv(nil),
ReadFile: fakeReadFile(map[string]string{ ReadFile: fakeReadFile(map[string]string{
"/cfg/config.yml": sampleTeaConfig, "/cfg/config.yml": sampleConfig,
}), }),
ConfigPath: "/cfg/config.yml", ConfigPath: "/cfg/config.yml",
} }
@@ -67,11 +67,11 @@ func TestResolveTokenOrdering(t *testing.T) {
} }
}) })
t.Run("tea config used when flag and env absent", func(t *testing.T) { t.Run("config used when flag and env absent", func(t *testing.T) {
opts := base opts := base
tok, src, err := ResolveToken(opts) tok, src, err := ResolveToken(opts)
if err != nil || tok != "site-token" || src != SourceTeaConfig { if err != nil || tok != "site-token" || src != SourceConfig {
t.Fatalf("want site-token/tea-config, got %q/%v (err %v)", tok, src, err) t.Fatalf("want site-token/config, got %q/%v (err %v)", tok, src, err)
} }
}) })
@@ -88,8 +88,8 @@ func TestResolveTokenOrdering(t *testing.T) {
opts := base opts := base
opts.Env = fakeEnv(map[string]string{"TEAI_TOKEN": " "}) opts.Env = fakeEnv(map[string]string{"TEAI_TOKEN": " "})
tok, src, err := ResolveToken(opts) tok, src, err := ResolveToken(opts)
if err != nil || tok != "site-token" || src != SourceTeaConfig { if err != nil || tok != "site-token" || src != SourceConfig {
t.Fatalf("want site-token/tea-config, got %q/%v (err %v)", tok, src, err) t.Fatalf("want site-token/config, got %q/%v (err %v)", tok, src, err)
} }
}) })
@@ -103,36 +103,54 @@ func TestResolveTokenOrdering(t *testing.T) {
} }
}) })
t.Run("TEA_CONFIG points config path", func(t *testing.T) { t.Run("TEAI_CONFIG points config path", func(t *testing.T) {
opts := base opts := base
opts.ConfigPath = "" // 不用 --config,改用 TEA_CONFIG opts.ConfigPath = "" // 不用 --config,改用 TEAI_CONFIG
opts.Env = fakeEnv(map[string]string{ opts.Env = fakeEnv(map[string]string{
"TEA_CONFIG": "/cfg/config.yml", "TEAI_CONFIG": "/cfg/config.yml",
}) })
tok, src, err := ResolveToken(opts) tok, src, err := ResolveToken(opts)
if err != nil || tok != "site-token" || src != SourceTeaConfig { if err != nil || tok != "site-token" || src != SourceConfig {
t.Fatalf("want site-token/tea-config via TEA_CONFIG, got %q/%v (err %v)", tok, src, err) t.Fatalf("want site-token/config via TEAI_CONFIG, got %q/%v (err %v)", tok, src, err)
} }
}) })
t.Run("explicit --config beats TEA_CONFIG", func(t *testing.T) { t.Run("explicit --config beats TEAI_CONFIG", func(t *testing.T) {
opts := base // ConfigPath = /cfg/config.yml(有 site-token) opts := base // ConfigPath = /cfg/config.yml(有 site-token)
opts.Env = fakeEnv(map[string]string{ opts.Env = fakeEnv(map[string]string{
"TEA_CONFIG": "/cfg/other.yml", "TEAI_CONFIG": "/cfg/other.yml",
}) })
read := fakeReadFile(map[string]string{ read := fakeReadFile(map[string]string{
"/cfg/config.yml": sampleTeaConfig, "/cfg/config.yml": sampleConfig,
"/cfg/other.yml": "logins:\n - name: x\n url: https://gitea.alterminal.com\n token: other\n default: true\n", "/cfg/other.yml": "logins:\n - name: x\n url: https://gitea.alterminal.com\n token: other\n default: true\n",
}) })
opts.ReadFile = read opts.ReadFile = read
tok, src, err := ResolveToken(opts) tok, src, err := ResolveToken(opts)
if err != nil || tok != "site-token" || src != SourceTeaConfig { if err != nil || tok != "site-token" || src != SourceConfig {
t.Fatalf("want site-token from explicit config, got %q/%v (err %v)", tok, src, err) t.Fatalf("want site-token from explicit config, got %q/%v (err %v)", tok, src, err)
} }
}) })
t.Run("TEA_CONFIG is ignored", func(t *testing.T) {
opts := AuthOptions{
SiteURL: "https://gitea.alterminal.com",
Env: fakeEnv(map[string]string{
"TEA_CONFIG": "/cfg/config.yml",
}),
ReadFile: fakeReadFile(map[string]string{
"/cfg/config.yml": sampleConfig,
"/home/.config/teai/config.yml": "logins:\n - name: x\n url: https://gitea.alterminal.com\n token: teai-token\n default: true\n",
}),
Home: "/home",
}
tok, src, err := ResolveToken(opts)
if err != nil || tok != "teai-token" || src != SourceConfig {
t.Fatalf("want teai-token from teai config, got %q/%v (err %v)", tok, src, err)
}
})
} }
func TestParseTeaConfigPicksMatchingLogin(t *testing.T) { func TestParseConfigPicksMatchingLogin(t *testing.T) {
t.Run("host match preferred over default", func(t *testing.T) { t.Run("host match preferred over default", func(t *testing.T) {
cfg := `logins: cfg := `logins:
- name: other - name: other
@@ -144,7 +162,7 @@ func TestParseTeaConfigPicksMatchingLogin(t *testing.T) {
token: right-token token: right-token
default: false default: false
` `
login, err := parseTeaConfig([]byte(cfg), "https://gitea.alterminal.com") login, err := parseConfig([]byte(cfg), "https://gitea.alterminal.com")
if err != nil { if err != nil {
t.Fatalf("parse error: %v", err) t.Fatalf("parse error: %v", err)
} }
@@ -155,7 +173,7 @@ func TestParseTeaConfigPicksMatchingLogin(t *testing.T) {
t.Run("quoted token unquoted", func(t *testing.T) { t.Run("quoted token unquoted", func(t *testing.T) {
cfg := "logins:\n - name: t\n url: https://gitea.alterminal.com\n token: \"quoted-token\"\n default: true\n" cfg := "logins:\n - name: t\n url: https://gitea.alterminal.com\n token: \"quoted-token\"\n default: true\n"
login, err := parseTeaConfig([]byte(cfg), "https://gitea.alterminal.com") login, err := parseConfig([]byte(cfg), "https://gitea.alterminal.com")
if err != nil { if err != nil {
t.Fatalf("parse error: %v", err) t.Fatalf("parse error: %v", err)
} }
@@ -166,14 +184,14 @@ func TestParseTeaConfigPicksMatchingLogin(t *testing.T) {
t.Run("malformed login entry errors", func(t *testing.T) { t.Run("malformed login entry errors", func(t *testing.T) {
cfg := "logins:\n - name: t\n url: https://gitea.alterminal.com\n oops\n" cfg := "logins:\n - name: t\n url: https://gitea.alterminal.com\n oops\n"
if _, err := parseTeaConfig([]byte(cfg), ""); err == nil { if _, err := parseConfig([]byte(cfg), ""); err == nil {
t.Fatal("malformed line should error") t.Fatal("malformed line should error")
} }
}) })
t.Run("no logins yields empty", func(t *testing.T) { t.Run("no logins yields empty", func(t *testing.T) {
cfg := "preferences:\n editor: false\n" cfg := "preferences:\n editor: false\n"
login, err := parseTeaConfig([]byte(cfg), "") login, err := parseConfig([]byte(cfg), "")
if err != nil || login.Token != "" { if err != nil || login.Token != "" {
t.Fatalf("want empty login, got %+v (err %v)", login, err) t.Fatalf("want empty login, got %+v (err %v)", login, err)
} }
+11 -20
View File
@@ -1,9 +1,8 @@
// config.go — tea 相容登入組態的讀取與行級編輯。 // config.go — teai 登入組態的讀取與行級編輯。
// //
// teai 與 tea 共用同一份組態檔(README 設計原則 5),因此寫入採「行級編輯」: // 組態由 teai 自行管理(README 設計原則 5),不依賴 tea。寫入採「行級編輯」:
// 只修改目標登入項目的所屬行,其餘內容(未知欄位、註解、preferences 區段、 // 只修改目標登入項目的所屬行,其餘內容(未知欄位、註解、空行與縮排)逐字
// 空行與縮排)逐字保留,避免改壞 tea 也在使用的檔案。新增項目依 tea 實際 // 保留。新增項目只寫 teai 使用的欄位:name/url/token/default/user。
// 寫出的欄位順序輸出,讓檔案外觀與 tea 一致。
// //
// token 值只會進出這個套件的資料結構與組態檔本身,永遠不出現在錯誤訊息。 // token 值只會進出這個套件的資料結構與組態檔本身,永遠不出現在錯誤訊息。
package gitea package gitea
@@ -18,7 +17,7 @@ import (
"strings" "strings"
) )
// LoginEntry 是組態檔中的一筆登入資料(編輯用,含 tea 的 user 欄位)。 // LoginEntry 是組態檔中的一筆登入資料(編輯用,含 user 欄位)。
type LoginEntry struct { type LoginEntry struct {
Name string Name string
URL string URL string
@@ -43,7 +42,7 @@ type configDoc struct {
} }
// parseConfigDoc 掃描組態內容,定位 logins 清單與各項目的行範圍。 // parseConfigDoc 掃描組態內容,定位 logins 清單與各項目的行範圍。
// 只認識 tea 實際寫出的格式:頂層 `logins:` 加縮排的 `- key: value` 項目; // 只認識 teai 寫出的格式:頂層 `logins:` 加縮排的 `- key: value` 項目;
// 其餘行原樣保留在 lines 中。 // 其餘行原樣保留在 lines 中。
func parseConfigDoc(data []byte) configDoc { func parseConfigDoc(data []byte) configDoc {
doc := configDoc{lines: splitLines(data), loginsHeader: -1, itemIndent: " "} doc := configDoc{lines: splitLines(data), loginsHeader: -1, itemIndent: " "}
@@ -103,7 +102,7 @@ func (d configDoc) findItem(name string) int {
return -1 return -1
} }
// fieldIndent 回傳項目內屬性行的縮排(項目縮排 + 兩格,tea 的格式)。 // fieldIndent 回傳項目內屬性行的縮排(項目縮排 + 兩格)。
func (d configDoc) fieldIndent() string { func (d configDoc) fieldIndent() string {
return d.itemIndent + " " return d.itemIndent + " "
} }
@@ -168,7 +167,7 @@ func ListLoginsFile(path string) ([]LoginEntry, error) {
// UpsertLogin 新增或更新名為 e.Name 的登入。 // UpsertLogin 新增或更新名為 e.Name 的登入。
// //
// 已存在則就地更新 url/token/user/default(保留其他欄位與行);不存在則 // 已存在則就地更新 url/token/user/default(保留其他欄位與行);不存在則
// 依 tea 的欄位順序插入新項目。makeDefault 為 true 時將其設為預設並取消 // 依 teai 的欄位順序插入新項目。makeDefault 為 true 時將其設為預設並取消
// 其他登入的預設旗標;新增後若成為唯一登入也自動成為預設。 // 其他登入的預設旗標;新增後若成為唯一登入也自動成為預設。
// 回傳新內容與最終生效的項目值。 // 回傳新內容與最終生效的項目值。
func UpsertLogin(data []byte, e LoginEntry, makeDefault bool) ([]byte, LoginEntry, error) { func UpsertLogin(data []byte, e LoginEntry, makeDefault bool) ([]byte, LoginEntry, error) {
@@ -326,7 +325,7 @@ func setFieldInItem(lines []string, it loginItemSpan, key, value, fieldIndent st
return append(out, lines[it.end:]...) return append(out, lines[it.end:]...)
} }
// newLoginBlock 產生 tea 格式的新登入項目(欄位順序與 tea 實際寫出的一致)。 // newLoginBlock 產生 teai 格式的新登入項目(name/url/token/default/user)。
func newLoginBlock(itemIndent, name, url, token string, isDefault bool, user string) []string { func newLoginBlock(itemIndent, name, url, token string, isDefault bool, user string) []string {
f := itemIndent + " " f := itemIndent + " "
return []string{ return []string{
@@ -334,15 +333,7 @@ func newLoginBlock(itemIndent, name, url, token string, isDefault bool, user str
f + "url: " + yamlScalar(url), f + "url: " + yamlScalar(url),
f + "token: " + yamlScalar(token), f + "token: " + yamlScalar(token),
f + "default: " + strconv.FormatBool(isDefault), f + "default: " + strconv.FormatBool(isDefault),
f + `ssh_host: ""`,
f + `ssh_key: ""`,
f + "insecure: false",
f + `ssh_certificate_principal: ""`,
f + "ssh_agent: false",
f + `ssh_key_agent_pub: ""`,
f + "version_check: false",
f + "user: " + yamlScalar(user), f + "user: " + yamlScalar(user),
f + "created: 0",
} }
} }
@@ -385,9 +376,9 @@ func joinLines(lines []string) []byte {
return []byte(s) return []byte(s)
} }
// SaveTeaConfigFile 把組態內容原子寫入 path:先寫同目錄暫存檔(0600), // SaveConfigFile 把組態內容原子寫入 path:先寫同目錄暫存檔(0600),
// 再 rename 取代,避免半寫入狀態。必要時建立上層目錄(0700)。 // 再 rename 取代,避免半寫入狀態。必要時建立上層目錄(0700)。
func SaveTeaConfigFile(path string, data []byte) error { func SaveConfigFile(path string, data []byte) error {
dir := filepath.Dir(path) dir := filepath.Dir(path)
if err := os.MkdirAll(dir, 0o700); err != nil { if err := os.MkdirAll(dir, 0o700); err != nil {
return fmt.Errorf("gitea: create config dir: %w", err) return fmt.Errorf("gitea: create config dir: %w", err)
+25 -14
View File
@@ -1,4 +1,4 @@
// config_test.go 驗證 tea 相容組態的行級編輯:新增、更新、設預設、 // config_test.go 驗證登入組態的行級編輯:新增、更新、設預設、
// 移除,以及「未知欄位與其他區段逐字保留」的關鍵性質。 // 移除,以及「未知欄位與其他區段逐字保留」的關鍵性質。
package gitea package gitea
@@ -10,8 +10,8 @@ import (
"testing" "testing"
) )
// realWorldTeaConfig 是 tea 實際寫出的組態(含 ssh_* 等未知欄位與 preferences)。 // extraFieldsConfig 含未知欄位與額外區段,用來驗證行級編輯不會改壞不相干內容。
const realWorldTeaConfig = `logins: const extraFieldsConfig = `logins:
- name: alterminal - name: alterminal
url: https://gitea.alterminal.com url: https://gitea.alterminal.com
token: tok-1 token: tok-1
@@ -32,7 +32,7 @@ preferences:
` `
func TestListLoginsParsesAllEntries(t *testing.T) { func TestListLoginsParsesAllEntries(t *testing.T) {
logins := ListLogins([]byte(realWorldTeaConfig)) logins := ListLogins([]byte(extraFieldsConfig))
if len(logins) != 1 { if len(logins) != 1 {
t.Fatalf("want 1 login, got %d", len(logins)) t.Fatalf("want 1 login, got %d", len(logins))
} }
@@ -43,7 +43,7 @@ func TestListLoginsParsesAllEntries(t *testing.T) {
} }
func TestUpsertLoginUpdatesExistingPreservesRest(t *testing.T) { func TestUpsertLoginUpdatesExistingPreservesRest(t *testing.T) {
out, final, err := UpsertLogin([]byte(realWorldTeaConfig), LoginEntry{ out, final, err := UpsertLogin([]byte(extraFieldsConfig), LoginEntry{
Name: "alterminal", URL: "https://gitea.alterminal.com", Token: "tok-2", User: "ceo", Name: "alterminal", URL: "https://gitea.alterminal.com", Token: "tok-2", User: "ceo",
}, false) }, false)
if err != nil { if err != nil {
@@ -74,7 +74,7 @@ func TestUpsertLoginUpdatesExistingPreservesRest(t *testing.T) {
} }
func TestUpsertLoginAppendsNewAndMakesDefault(t *testing.T) { func TestUpsertLoginAppendsNewAndMakesDefault(t *testing.T) {
out, final, err := UpsertLogin([]byte(realWorldTeaConfig), LoginEntry{ out, final, err := UpsertLogin([]byte(extraFieldsConfig), LoginEntry{
Name: "second", URL: "https://git.example.com", Token: "tok-b", User: "bob", Name: "second", URL: "https://git.example.com", Token: "tok-b", User: "bob",
}, true) }, true)
if err != nil { if err != nil {
@@ -93,10 +93,15 @@ func TestUpsertLoginAppendsNewAndMakesDefault(t *testing.T) {
if !final.Default { if !final.Default {
t.Fatalf("final should be default: %+v", final) t.Fatalf("final should be default: %+v", final)
} }
// tea 的欄位順序仍在。 // 新項目依 teai 欄位順序附加。
s := string(out) s := string(out)
if !strings.Contains(s, " - name: second\n") { if !strings.Contains(s, " - name: second\n") {
t.Fatalf("new item not appended in tea style:\n%s", s) t.Fatalf("new item not appended:\n%s", s)
}
for _, ban := range []string{"ssh_", "insecure:", "version_check:", "created:"} {
if strings.Contains(s[strings.Index(s, "- name: second"):], ban) {
t.Fatalf("new item leaked tea field %q:\n%s", ban, s)
}
} }
} }
@@ -117,6 +122,12 @@ func TestUpsertLoginFirstEverLoginCreatesSection(t *testing.T) {
if !strings.HasPrefix(string(out), "logins:\n") { if !strings.HasPrefix(string(out), "logins:\n") {
t.Fatalf("section not created:\n%s", out) t.Fatalf("section not created:\n%s", out)
} }
s := string(out)
for _, ban := range []string{"ssh_", "insecure:", "version_check:", "created:", "preferences:"} {
if strings.Contains(s, ban) {
t.Fatalf("new config leaked tea field %q:\n%s", ban, s)
}
}
} }
func TestUpsertLoginRejectsBadInput(t *testing.T) { func TestUpsertLoginRejectsBadInput(t *testing.T) {
@@ -158,7 +169,7 @@ func TestSetDefaultLoginSwitchesFlag(t *testing.T) {
} }
func TestSetDefaultLoginUnknownNameErrors(t *testing.T) { func TestSetDefaultLoginUnknownNameErrors(t *testing.T) {
if _, err := SetDefaultLogin([]byte(realWorldTeaConfig), "nope"); err == nil { if _, err := SetDefaultLogin([]byte(extraFieldsConfig), "nope"); err == nil {
t.Fatal("unknown name should error") t.Fatal("unknown name should error")
} }
} }
@@ -195,7 +206,7 @@ preferences:
} }
func TestRemoveLoginLastOne(t *testing.T) { func TestRemoveLoginLastOne(t *testing.T) {
out, promoted, err := RemoveLogin([]byte(realWorldTeaConfig), "alterminal") out, promoted, err := RemoveLogin([]byte(extraFieldsConfig), "alterminal")
if err != nil || promoted != "" { if err != nil || promoted != "" {
t.Fatalf("want clean removal, promoted=%q err=%v", promoted, err) t.Fatalf("want clean removal, promoted=%q err=%v", promoted, err)
} }
@@ -207,10 +218,10 @@ func TestRemoveLoginLastOne(t *testing.T) {
} }
} }
func TestSaveTeaConfigFileAtomicAndMode(t *testing.T) { func TestSaveConfigFileAtomicAndMode(t *testing.T) {
dir := t.TempDir() dir := t.TempDir()
path := filepath.Join(dir, "tea", "config.yml") path := filepath.Join(dir, "teai", "config.yml")
if err := SaveTeaConfigFile(path, []byte("logins: []\n")); err != nil { if err := SaveConfigFile(path, []byte("logins: []\n")); err != nil {
t.Fatal(err) t.Fatal(err)
} }
info, err := os.Stat(path) info, err := os.Stat(path)
@@ -221,7 +232,7 @@ func TestSaveTeaConfigFileAtomicAndMode(t *testing.T) {
t.Fatalf("perm = %v, want 0600", info.Mode().Perm()) t.Fatalf("perm = %v, want 0600", info.Mode().Perm())
} }
// 再寫一次(覆蓋既有檔),不留暫存殘檔。 // 再寫一次(覆蓋既有檔),不留暫存殘檔。
if err := SaveTeaConfigFile(path, []byte("logins: []\n# v2\n")); err != nil { if err := SaveConfigFile(path, []byte("logins: []\n# v2\n")); err != nil {
t.Fatal(err) t.Fatal(err)
} }
entries, _ := os.ReadDir(filepath.Dir(path)) entries, _ := os.ReadDir(filepath.Dir(path))
+67 -1
View File
@@ -1,6 +1,10 @@
package workflow package workflow
import "sort" import (
"regexp"
"sort"
"strings"
)
// sortComments 依 created_at 字典序穩定排序(ISO8601 字串字典序與時間序一致), // sortComments 依 created_at 字典序穩定排序(ISO8601 字串字典序與時間序一致),
// 對應 gitea.py 各處的 comments.sort(key=created_at)。 // 對應 gitea.py 各處的 comments.sort(key=created_at)。
@@ -135,3 +139,65 @@ func LastCommentNotByAssignee(issue *Issue, comments []Comment) bool {
func IssueNeedsWork(issue *Issue, comments []Comment, members []string) bool { func IssueNeedsWork(issue *Issue, comments []Comment, members []string) bool {
return AssigneeOverlaps(issue, members) && LastCommentNotByAssignee(issue, comments) return AssigneeOverlaps(issue, members) && LastCommentNotByAssignee(issue, comments)
} }
// ── 交付報告判定(agents #47,同步 gitea.py _is_delivery_report)──────────
//
// 最後一則 assignee 留言已把工作交棒給發起人/他人確認(逐項查核/驗收
// 結果+回報他件或請確認關閉)時,停滯點不在 assignee:stalled 分類由
// assignee-idle 改判 waiting-outside,管理者巡邏不再對已交付、待確認的
// 處理者觸發自動催促。
var (
// deliveryVerdictRe 對應 gitea.py _DELIVERY_VERDICT_RE:彙總 verdict 語。
deliveryVerdictRe = regexp.MustCompile(`(全數|全部|\d+\s*項)通過|項全過|不通過|未通過`)
// deliveryHandoffRe 對應 gitea.py _DELIVERY_HANDOFF_RE:交棒訊號。
deliveryHandoffRe = regexp.MustCompile(`回報\s*#\d+|關閉\s*#\d+|請[^\n。]{0,40}(確認|關閉)|可由[^\n。]{0,40}關閉|待[^\n。]{0,20}確認`)
// completionDefinitionRe 對應 _has_completion_definition:內文「完成定義」
// 段落(ATX 標題或行首字樣)。
completionDefinitionRe = regexp.MustCompile(`(?m)^#{1,6}\s*完成定義|^完成定義\s*[::]?`)
// simpleDeliveryRe 內文有「完成定義」時的簡式交付語。
simpleDeliveryRe = regexp.MustCompile(`已完成|已交付|已通過|已全數`)
)
// hasCompletionDefinition 內文是否含「完成定義」段落。
func hasCompletionDefinition(body string) bool {
return completionDefinitionRe.MatchString(body)
}
// IsDeliveryReport 判定最後一則 assignee 留言是否符合交付報告特徵
// (agents #47;gitea.py _is_delivery_report 的同語義實作)。
//
// 交付報告=「結構訊號」∧「交棒訊號」同時命中:
// - 結構訊號:逐項查核/驗收/測試結果、結論、或彙總 verdict 語;內文有
// 「完成定義」段落時,簡式交付語(已完成/已交付)也算。
// - 交棒訊號:回報/關閉他件(#N)、或請/可由/待他人確認關閉。
//
// 只命中結構訊號不算:認領/進度/預告式留言(「完成後逐項回報結果至此」)
// 沒有交棒對象,仍是 assignee 自己該做——保持 assignee-idle,修正不製造
// 「認領後棄置永不被催促」的新死角。
func IsDeliveryReport(issueBody string, last *Comment) bool {
if last == nil {
return false
}
body := last.Body
if body == "" {
return false
}
structural := false
for _, marker := range []string{"查核結果", "驗收結果", "測試結果", "逐項", "結論", "完成定義"} {
if strings.Contains(body, marker) {
structural = true
break
}
}
if !structural && deliveryVerdictRe.MatchString(body) {
structural = true
}
if !structural && hasCompletionDefinition(issueBody) && simpleDeliveryRe.MatchString(body) {
structural = true
}
if !structural {
return false
}
return deliveryHandoffRe.MatchString(body)
}
+36
View File
@@ -143,3 +143,39 @@ func TestFullNameFallbacks(t *testing.T) {
t.Errorf("全缺 → 空,得到 %q", got) t.Errorf("全缺 → 空,得到 %q", got)
} }
} }
// TestIsDeliveryReport(agents #47):交付報告=結構訊號 ∧ 交棒訊號。
// 案例與 gitea.py _is_delivery_report 的離線驗證同集(bear#52 實例)。
func TestIsDeliveryReport(t *testing.T) {
body52 := "## 完成定義\n- 逐項回報查核結果至此 issue;\n- 全數通過 → 回報 #39 並由 #39 assignee 關閉"
claim := "認領。我來執行四項驗收查核,完成後逐項回報結果至此。"
report := "## 全階段驗收查核結果(#52)\n**結論:四項全數通過。** 逐項如下…\n四項全過,符合「完成定義」。回報 #39:全數驗收通過,可由 @ceo 關閉 #39 與里程碑。"
reportFail := "## 查核結果\n結論:第 2 項不通過。已開立修復 issue #60,請 @ceo 確認分配。"
cases := []struct {
name string
issueBody string
comment string
want bool
}{
{"交付報告+回報#39+@ceo 關閉", body52, report, true},
{"失敗型交付(開修復 issue+請確認)", body52, reportFail, true},
{"認領留言(逐項+回報 但無交棒)", body52, claim, false},
{"純進度留言", body52, "進行中,已看完 P1,預計今晚交付。", false},
{"已完成但無交棒", body52, "已完成,程式碼都在分支上。", false},
{"無完成定義內文+逐項交棒", "一般 issue 內文", "逐項測試結果全數通過,請 @ceo 確認後關閉。", true},
{"無完成定義+僅交棒(無結構)", "一般 issue 內文", "請 @ceo 確認後關閉。", false},
{"空留言", body52, "", false},
}
for _, tc := range cases {
var last *Comment
if tc.comment != "" {
last = &Comment{Body: tc.comment}
}
if got := IsDeliveryReport(tc.issueBody, last); got != tc.want {
t.Errorf("%s:IsDeliveryReport = %v,want %v", tc.name, got, tc.want)
}
}
if IsDeliveryReport(body52, nil) {
t.Error("nil 最後留言應為 false")
}
}
+69 -3
View File
@@ -9,8 +9,17 @@ import (
"sort" "sort"
) )
// AgentsTeamName 是工作流團隊的名稱(AGENTS.md:調度成員由 Agents 團隊決定)。 const (
const AgentsTeamName = "Agents" // AgentsTeamName 是工作流團隊的名稱(AGENTS.md:調度成員由 Agents 團隊決定)。
AgentsTeamName = "Agents"
// SupervisorTeamName 是管理者團隊的名稱;成員角色為 supervisor。
SupervisorTeamName = "Supervisor"
// RoleSupervisor 是 Supervisor 團隊成員的角色值。
RoleSupervisor = "supervisor"
// RoleWorker 是 Agents 團隊成員的角色值。
RoleWorker = "worker"
)
// Client 抽象 teai 需要的 Gitea API 讀取操作(僅唯讀)。 // Client 抽象 teai 需要的 Gitea API 讀取操作(僅唯讀)。
// #5 的 internal/gitea HTTP 用戶端實作此介面;workflow 不依賴 HTTP 細節。 // #5 的 internal/gitea HTTP 用戶端實作此介面;workflow 不依賴 HTTP 細節。
@@ -49,7 +58,7 @@ type PullItem struct {
Pull PullRequest Pull PullRequest
} }
// Scanner 在 Client 之上組合出各命令的資料(whoami/orgs/members/mine/pulls/next)。 // Scanner 在 Client 之上組合出各命令的資料(whoami/role/orgs/members/mine/pulls/next)。
type Scanner struct { type Scanner struct {
Client Client Client Client
} }
@@ -59,6 +68,63 @@ func (s *Scanner) Whoami() (string, error) {
return s.Client.CurrentUser() return s.Client.CurrentUser()
} }
// Role 依目前帳號所屬團隊回傳角色:Supervisor 團隊為 supervisor,
// Agents 團隊為 worker。同時屬於兩者時以 supervisor 為準。
// 兩者都不屬於時回空字串(CLI 輸出 null)。
func (s *Scanner) Role() (string, error) {
username, err := s.Whoami()
if err != nil {
return "", err
}
orgs, err := s.Orgs()
if err != nil {
return "", err
}
inAgents := false
for _, org := range orgs {
teams, err := s.Client.ListTeams(org)
if err != nil {
return "", err
}
for _, team := range teams {
switch team.Name {
case SupervisorTeamName, AgentsTeamName:
default:
continue
}
ok, err := s.usernameInTeam(team.ID, username)
if err != nil {
return "", err
}
if !ok {
continue
}
if team.Name == SupervisorTeamName {
return RoleSupervisor, nil
}
inAgents = true
}
}
if inAgents {
return RoleWorker, nil
}
return "", nil
}
// usernameInTeam 檢查帳號是否為該團隊成員。
func (s *Scanner) usernameInTeam(teamID int64, username string) (bool, error) {
members, err := s.Client.ListTeamMembers(teamID)
if err != nil {
return false, err
}
for _, m := range members {
if m == username {
return true, nil
}
}
return false, nil
}
// Orgs 回傳目前帳號所屬的組織(去重、排序)。 // Orgs 回傳目前帳號所屬的組織(去重、排序)。
func (s *Scanner) Orgs() ([]string, error) { func (s *Scanner) Orgs() ([]string, error) {
names, err := s.Client.ListOrgs() names, err := s.Client.ListOrgs()
+85
View File
@@ -288,6 +288,91 @@ func TestErrorPropagation(t *testing.T) {
} }
} }
func TestRoleWorker(t *testing.T) {
s, _ := newTestScanner()
got, err := s.Role()
if err != nil {
t.Fatal(err)
}
if got != RoleWorker {
t.Errorf("Agents 團隊成員應為 worker,得到 %q", got)
}
}
func TestRoleSupervisor(t *testing.T) {
s, fc := newTestScanner()
fc.teams["alterminal"] = []Team{
{ID: 1, Name: "Agents"},
{ID: 2, Name: "Supervisor"},
}
fc.teamMembers[2] = []string{"alex"}
got, err := s.Role()
if err != nil {
t.Fatal(err)
}
if got != RoleSupervisor {
t.Errorf("Supervisor 團隊成員應為 supervisor,得到 %q", got)
}
}
func TestRoleSupervisorTakesPrecedence(t *testing.T) {
s, fc := newTestScanner()
// alex 同時在 Agents(newTestScanner 預設)與 Supervisor。
fc.teams["alterminal"] = []Team{
{ID: 1, Name: "Agents"},
{ID: 2, Name: "Supervisor"},
}
fc.teamMembers[2] = []string{"alex", "ceo"}
got, err := s.Role()
if err != nil {
t.Fatal(err)
}
if got != RoleSupervisor {
t.Errorf("同時屬於兩者時應為 supervisor,得到 %q", got)
}
}
func TestRoleNone(t *testing.T) {
s, fc := newTestScanner()
fc.currentUser = "outsider"
got, err := s.Role()
if err != nil {
t.Fatal(err)
}
if got != "" {
t.Errorf("不在 Supervisor/Agents 應回空字串,得到 %q", got)
}
}
func TestRoleWorkerWhenSupervisorExists(t *testing.T) {
s, fc := newTestScanner()
fc.teams["alterminal"] = []Team{
{ID: 1, Name: "Agents"},
{ID: 2, Name: "Supervisor"},
}
fc.teamMembers[2] = []string{"ceo"}
got, err := s.Role()
if err != nil {
t.Fatal(err)
}
if got != RoleWorker {
t.Errorf("只在 Agents、不在 Supervisor 應為 worker,得到 %q", got)
}
}
func TestRoleErrorPropagation(t *testing.T) {
s, fc := newTestScanner()
fc.failOn["teams"] = errors.New("boom")
if _, err := s.Role(); err == nil {
t.Errorf("ListTeams 錯誤應向上傳播")
}
delete(fc.failOn, "teams")
fc.failOn["members"] = errors.New("boom")
if _, err := s.Role(); err == nil {
t.Errorf("ListTeamMembers 錯誤應向上傳播")
}
}
func TestPullsDefault(t *testing.T) { func TestPullsDefault(t *testing.T) {
s, fc := newTestScanner() s, fc := newTestScanner()
fc.pulls["alterminal/teai"] = []PullRequest{ fc.pulls["alterminal/teai"] = []PullRequest{
+12 -11
View File
@@ -526,11 +526,21 @@ func considerIssue(src Source, org, repo string, issue Issue, comments []Comment
case len(assignees) == 0: case len(assignees) == 0:
reason = "no-assignee" reason = "no-assignee"
case lastCommentBy != "" && containsString(assignees, lastCommentBy): case lastCommentBy != "" && containsString(assignees, lastCommentBy):
reason = "assignee-idle" if IsDeliveryReport(issue.Body, last) {
// 交付報告待確認(agents #47):assignee 已交付並交棒(回報他件/
// 請發起人確認關閉),停滯點在等待外部確認,不是 assignee 停滯。
reason = "waiting-outside"
} else {
reason = "assignee-idle"
}
case lastCommentBy == "": case lastCommentBy == "":
reason = "no-commenter" reason = "no-commenter"
} }
if len(children) > 0 && anyOpen(children) { if len(children) > 0 {
// 父追蹤項一律標 parent-tracking(agents #44,與 gitea.py 同步):
// 有 open 子項在推進時,父項留言閒置屬等待子項;子項全關閉時只剩
// 父項收尾(回報驗收並關閉),兩者都不是 assignee 停滯。統一標記
// 讓「父項收尾」規則能正確匹配,不退回通用原因(如 assignee-idle)。
reason = "parent-tracking" reason = "parent-tracking"
} }
item := Item{ item := Item{
@@ -606,15 +616,6 @@ func considerPull(src Source, org, repo string, pr PullRequest, comments []Comme
return item, true return item, true
} }
func anyOpen(children []Child) bool {
for _, c := range children {
if c.State == "open" {
return true
}
}
return false
}
// round1 舍入到小數一位(對應 Python round(x, 1))。 // round1 舍入到小數一位(對應 Python round(x, 1))。
// 經 strconv 的十進位正確舍入(半偶數),與 Python 的浮點 round 行為一致, // 經 strconv 的十進位正確舍入(半偶數),與 Python 的浮點 round 行為一致,
// 避免 (x*10+0.5) 截斷在 .x5 邊界產生偏差。 // 避免 (x*10+0.5) 截斷在 .x5 邊界產生偏差。
+79
View File
@@ -42,6 +42,9 @@ func (f *fakeSource) OrganizationRepositories(org string) ([]Repo, error) {
func (f *fakeSource) OpenIssues(owner, repo string) ([]Issue, error) { func (f *fakeSource) OpenIssues(owner, repo string) ([]Issue, error) {
var out []Issue var out []Issue
for _, i := range f.issues[owner+"/"+repo] { for _, i := range f.issues[owner+"/"+repo] {
if i.State == "closed" {
continue // 對齊真實 API:OpenIssues 只回 open;closed 條目僅供 TryGetIssue 解引用。
}
out = append(out, i) out = append(out, i)
} }
// 對齊真實 API:issues 依 id(=number)遞增回傳,消除 map 迭代隨機性。 // 對齊真實 API:issues 依 id(=number)遞增回傳,消除 map 迭代隨機性。
@@ -245,6 +248,46 @@ func TestScanStalledNudgeCooldown(t *testing.T) {
} }
} }
// TestScanStalledParentWrapup(teai #47,同步 agents #44):子項全關閉的父追蹤項
// 不退回通用原因(assignee-idle),一律標 parent-tracking——「父項收尾」情境。
func TestScanStalledParentWrapup(t *testing.T) {
now := at(0)
// 父項 3 最後留言來自 assignee ceo(舊行為會誤標 assignee-idle);
// 唯一子項 7 已關閉,父子最後活動皆 5 小時前。
src := &fakeSource{
orgs: []string{"alterminal"},
repos: map[string][]Repo{"alterminal": repoWith("teai", true)},
issues: map[string]map[int64]Issue{
"alterminal/teai": {
3: {Number: 3, Title: "父", State: "open", Body: "子項:#7", CreatedAt: iso(-700), UpdatedAt: iso(-300), Assignees: []User{{UserName: "ceo"}}, Repository: &Repository{FullName: "alterminal/teai"}},
7: {Number: 7, Title: "子", State: "closed", CreatedAt: iso(-650), UpdatedAt: iso(-300), Repository: &Repository{FullName: "alterminal/teai"}},
},
},
comments: map[string][]Comment{
"alterminal/teai/3": {{User: &User{UserName: "ceo"}, CreatedAt: iso(-300)}},
},
}
got, err := ScanStalled(src, "admin", 4.0, now)
if err != nil {
t.Fatalf("ScanStalled: %v", err)
}
var parent *Item
for i := range got {
if got[i].Number == 3 {
parent = &got[i]
}
}
if parent == nil {
t.Fatalf("父項 3 應列入(5h 無活動),got %v", got)
}
if parent.Reason != "parent-tracking" {
t.Errorf("子項全關閉的父項 reason = %q, want parent-tracking", parent.Reason)
}
if len(parent.ChildActivity) == 0 {
t.Errorf("父項 3 應附 child_activity,got %v", parent.ChildActivity)
}
}
// TestScanStalledParentTracking 父追蹤項:子項活動計入 last_activity、reason 標記。 // TestScanStalledParentTracking 父追蹤項:子項活動計入 last_activity、reason 標記。
func TestScanStalledParentTracking(t *testing.T) { func TestScanStalledParentTracking(t *testing.T) {
now := at(0) now := at(0)
@@ -552,3 +595,39 @@ func TestRound1(t *testing.T) {
} }
} }
} }
// TestScanStalledDeliveryReport(agents #47):assignee 最後留言為交付報告
// (結構訊號+交棒訊號)→ waiting-outside,不是 assignee-idle;
// 同型認領留言(無交棒)維持 assignee-idle——修正不製造新死角。
func TestScanStalledDeliveryReport(t *testing.T) {
now := at(0)
defBody := "## 完成定義\n- 逐項回報查核結果至此 issue;\n- 全數通過 → 回報 #39 並由 #39 assignee 關閉"
src := &fakeSource{
orgs: []string{"alterminal"},
repos: map[string][]Repo{"alterminal": repoWith("bear", true)},
issues: map[string]map[int64]Issue{
"alterminal/bear": {
5: {Number: 5, Title: "驗收", State: "open", Body: defBody, CreatedAt: iso(-600), UpdatedAt: iso(-300), Assignees: []User{{UserName: "queena"}}, Repository: &Repository{FullName: "alterminal/bear"}},
6: {Number: 6, Title: "認領中", State: "open", Body: defBody, CreatedAt: iso(-600), UpdatedAt: iso(-300), Assignees: []User{{UserName: "dan"}}, Repository: &Repository{FullName: "alterminal/bear"}},
},
},
comments: map[string][]Comment{
"alterminal/bear/5": {{User: &User{UserName: "queena"}, CreatedAt: iso(-300), Body: "## 查核結果\n結論:四項全數通過。回報 #39:可由 @ceo 關閉。"}},
"alterminal/bear/6": {{User: &User{UserName: "dan"}, CreatedAt: iso(-300), Body: "認領。完成後逐項回報結果至此。"}},
},
}
got, err := ScanStalled(src, "admin", 4.0, now)
if err != nil {
t.Fatalf("ScanStalled: %v", err)
}
byNumber := map[int64]Item{}
for _, it := range got {
byNumber[it.Number] = it
}
if it, ok := byNumber[5]; !ok || it.Reason != "waiting-outside" {
t.Errorf("issue 5(交付報告)reason 應為 waiting-outside,got %+v", it)
}
if it, ok := byNumber[6]; !ok || it.Reason != "assignee-idle" {
t.Errorf("issue 6(認領留言)reason 應為 assignee-idle,got %+v", it)
}
}