Skip to content

test(hub-test): hub-server agent 包最后一个巨型测试文件 agent_test.go 按生产域拆入 7 个新域文件并 git rm——67 Test 逐字搬移(dispatch 19 / trigger-contract 12 / cancel 2 / edge-callback 19 / final-message 10 / service 5),共享夹具与 mock 收进 helpers_test.go,四路守恒 diff rc=0,golangci-lint 0 issues - #2312

Merged
DeliciousBuding merged 1 commit into
masterfrom
test/hub-agent-test-split
Sep 3, 2026

Conversation

@DeliciousBuding

Copy link
Copy Markdown
Collaborator

Co-authored-by: Cursor cursor@vectorcontrol.tech

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: eb40f30c-48dc-40a3-a96f-2acc82406598

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@DeliciousBuding

Copy link
Copy Markdown
Collaborator Author

round-73 主机复核(这两个分支自 round-72 起 committed-but-never-PR'd,落地前重测过)

这两个 PR 是 round-72 已经完成、但一直没开 PR 的纯搬家工作。round-73 的收尾原则是「不许留悬空分支」,所以在落地前把它们重新拉到当前 master 上复核了一遍,而不是直接推:

  • rebase 到当前 master 干净(各自落后 12 / 13 个 commit,0 冲突);
  • 本地 61 项 validate PASS=61 FAIL=0 SKIP(merge-ref)=1(两个分支各跑一遍);
  • 测试守恒对上了 round-71 记录的基线(这是本仓拆/搬测试文件的验收口径):
    • edge-server/internal/storego test -count=1 ok,-v=== RUN = 306(基线 306);
    • hub-server/internal/service/agentgo test -count=1 ok,-v=== RUN = 247(基线 247);
  • 提交身份 DeliciousBuding <DeliciousBuding@users.noreply.github.com>Co-authored-by trailer 与 master 上既有先例一致(master 近 200 个 commit 里已有 Cursor / Claude Code / Codex 三种)。

为什么现在才落地:round-73 的优先级是 Correctness → Contract/Decision → Developer feedback → Measured performance → Cleanup,这两发属最后一档,所以排在 #2298/#2299/#2301/#2302/#2305/#2306/#2307/#2308/#2309/#2310 之后。核心目标收口后才动它们;再拖下去只会让 rebase 成本继续涨、把已经付过的守恒验证成本烂在本地分支里。

0 行 src 改动,两个 PR 的写集不重叠(edge-server/internal/store vs hub-server/internal/service/agent)。

@DeliciousBuding
DeliciousBuding enabled auto-merge (squash) September 3, 2026 23:18
@DeliciousBuding
DeliciousBuding force-pushed the test/hub-agent-test-split branch from c16d576 to a3b912a Compare September 3, 2026 23:24
…新域文件并 git rm——67 Test 逐字搬移(dispatch 19 / trigger-contract 12 / cancel 2 / edge-callback 19 / final-message 10 / service 5),共享夹具与 mock 收进 helpers_test.go,四路守恒 diff rc=0,golangci-lint 0 issues

Co-authored-by: Cursor <cursor@vectorcontrol.tech>
@DeliciousBuding
DeliciousBuding force-pushed the test/hub-agent-test-split branch from a3b912a to fc65300 Compare September 3, 2026 23:33
@DeliciousBuding
DeliciousBuding merged commit a3e57bf into master Sep 3, 2026
41 checks passed
@DeliciousBuding
DeliciousBuding deleted the test/hub-agent-test-split branch September 3, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant