test(edge-server): 全仓最大测试文件 process_executor_test.go（3283行/67Test）按生产域归位——拆出 10 个镜像生产文件的新测试文件 + 3 个既有域文件各追加 1 个，巨石瘦身到 510 行；四路守恒 + 逐行守恒 2973→2970（少的 3 行是分节注释、0 行新增）、import 0 越界、0 src 改动；3 处 sleep 经 --update-baseline 重新键控（全局 47 不变）；13 个脱敏 Test 因 #2295 刻意不搬

[DeliciousBuding]

一句话

edge-server/internal/lifecycle/process_executor_test.go（3283 行 / 67 Test，全仓最大的测试文件）按生产域归位：拆出 10 个镜像生产文件的新测试文件 + 3 个既有域文件各追加 1 个 Test，共享夹具与 4 个 mock adapter 收进既有 testutil_test.go，巨石瘦身到 510 行（16 文件，+2932 / −2815，净 +117 行＝拆文件的 package/import 头固有成本）。纯搬家，0 个 src 改动。

该包 30 个测试文件里 29 个早已按域命名（env_sanitizer_test.go、evidence_gate_test.go、process_executor_hub_callback_test.go、result_aggregator_test.go…），process_executor_test.go 是最后一个"一个文件装整包"的 leftovers。同批已合的 #2288（edge-api 3792→184）、#2294（hub agentteam 3176→0、repository 2221→0）做的是同一件事。

拆分映射（每个新文件镜像哪个生产文件）

新/改文件	Test 数	镜像的生产文件（判据＝Test 实际调用的符号定义处）
process_executor_start_test.go	9	process_executor_start.go / _run.go / _publish.go（Start→RunStarted→output→terminal 发布链）
process_profile_test.go	8	process_profile.go（validatePlaceholders/expandPlaceholders/runPlaceholderValue/parseEnvTemplate）
process_executor_pure_adapter_test.go	7	process_executor_pure_adapter.go（adapter 契约、fixture 流映射、NeedsStdin 闸门）
process_executor_cancel_test.go	6	process_executor_cancel.go（Cancel/grace/race/级联）
process_executor_finish_test.go	5	process_executor_finish.go:166 sendSubAgentResult
process_executor_build_test.go	2	process_executor_build.go（RunCheckpoint 在 build 阶段发出）
fault_escalation_test.go	2	fault_escalation.go
process_executor_helper_test.go	2	假子进程 helper（TestHelperProcess 模式，5s real_protocol sleep 在此）
process_arg_log_test.go	2	process_arg_log.go:16 summarizeProcessArgsForLog
runtime_evidence_test.go	1	runtime_evidence.go
hub_output_text_test.go（既有，追加 1）	+1	hub_output_text.go splitHubCallbackText
thread_transcript_test.go（既有，追加 1）	+1	thread_transcript.go
testutil_test.go（既有，追加 22 个单元）	0	共享夹具：recordingLifecycleEmitter、recordingContextAdapter、fixtureSDKStreamAdapter、needsStdinTestAdapter、newTestProcessExecutor、collectEventsUntilRunDone、outputChunksText、withoutEnvKey、hasArg/hasEventType/findEventType/eventTypeList、stringSliceContains、两个 Eventually 预算常量
process_executor_test.go（瘦身 3283→510）	8 + 13	process_executor.go：NewProcessExecutor 构造/依赖校验/重复 Start/并发上限 8 个；另 13 个 TestSanitizeSubAgentResult_* 刻意留在此处，见下

为什么 13 个 TestSanitizeSubAgentResult_* 没搬（如实）

它们的夹具是故意的假凭据字面量（sk-proj-abc123…、AIzaSyDabc123…、ghp_abc123…、AKIA1234567890ABCDEF、eyJhbGci…），而 scripts/verify/check-secrets.sh 的字面量规则对 range diff 的新增行无条件生效、且除 *.env.example 外没有 fixture allowlist ⇒ 逐字搬家实测判红 9 行。这些字面量自导入提交 372a2119（2026-08-14）就在仓里，range 门禁从未判过它们。

• 我没有改夹具内容绕过（本仓纪律），也没有自行放宽安全门禁（属策略变更）。
• 处置：13 个 Test 原地保留，另开 check-secrets.sh 字面量规则无 fixture allowlist：脱敏测试的假凭据夹具让这类测试文件「不可搬家」，且良性 ID task-backfill-mismatch-conflict 被判成 API key（round-72 实测两发） #2295 把根因、复现、三个可选处置（精确 (path, literal) allowlist / 只修 sk- 左边界精度 / 改夹具——不建议）交 operator 与安全 owner 裁决。
• 代价如实写在这里：巨石因此停在 510 行而不是 188 行。

守恒证据（全部本机实跑，diff rc=0）

守恒项	master	本分支	diff
整包 func Test* 名集合	283（uniq 283）	283（uniq 283）	rc=0
整包小写顶层 helper 集合	47	47	rc=0
go test -v 的 === RUN 排序集合（含子测试路径）	379	379	rc=0
--- PASS 名集合	282	282	rc=0
--- SKIP	1	1	同名：TestSanitizedEnvIncludesWindowsSpecificVars（Windows-only）
--- FAIL	0	0	—
逐行搬运守恒	2973 非空行（去 package/import 头）	2970	少的 3 行全是巨石内的分节注释（// ── Result aggregation tests ───、// ── SanitizeSubAgentResult tests ───、// --- Process executor error path tests ---）——域已成文件，分节标记随之失去意义；分支侧 0 行新增（B-only = 0）
import 纪律	原文件 import 并集 22 个	11 个新/改文件的 import 全在该并集内	越界 = 0（goimports 只裁剪、没有注入任何新依赖）
src 改动	—	0 个非 _test.go 文件	git diff --name-only origin/master 过滤后为空

time.Sleep 双基线重新键控（本 PR 唯一的非测试改动）

3 处 sleep 随域搬家：80ms（CancelGraceNotImmediateKill，negative_window）+ 10ms（StartCancelRace，simulated_slow）→ process_executor_cancel_test.go；5000ms（TestProcessExecutorHelper 的假子进程 sleep 模式，real_protocol）→ process_executor_helper_test.go。verify-test-sleep-ratchet.py 对"有 sleep 但无预算条目"是 fail-closed，所以走它自己规定的被跟踪审批步骤：

$ python3 scripts/verify/verify-test-sleep-ratchet.py --update-baseline
  CHG  edge-server/internal/lifecycle/process_executor_test.go : 3 -> 0
  NEW  edge-server/internal/lifecycle/process_executor_cancel_test.go : 2
  NEW  edge-server/internal/lifecycle/process_executor_helper_test.go : 1
  DEL  edge-server/internal/lifecycle/process_executor_test.go（源码已无 sleep，条目移除）

随后人工补全 --update-baseline 留空的审批字段（owner/review/reason）与 kind（工具对新条目一律写 unclassified）：

• 全局 sleep 总数不变：47 → 47；文件数 21 → 22（一个键变成两个）。
• 逐值逐 kind 保留：80/negative_window、10/simulated_slow、5000/real_protocol；total_ms 5090 → 90 + 5000 = 5090。
• reason 里写明原始审批出处（2026-08-29，test(edge): process_executor 轮询睡眠改 Eventually（sleep 治理第二批） #2038）与本次 re-key 的原因（round-72 按域拆分，纯搬家、零新增 sleep）。
• 复跑 verify-test-sleep-ratchet.py → rc=0 PASS test-sleep ratchet + value budget hold (47 sleeps across 22 files, count baseline 22 files)。

门禁（本机 huawei-dev 实跑）

命令	结果
go build ./...（edge-server）	rc=0
go vet ./internal/lifecycle/	rc=0
gofmt -l internal/lifecycle	空输出
go test ./internal/lifecycle/ -v -count=1	rc=0（379 RUN / 282 PASS / 1 SKIP / 0 FAIL，7.8s）
golangci-lint run ./internal/lifecycle/...（v2.12.2，仓库配置）	1 issue＝process_executor_build.go 的 gocyclo 22，与 edge-lint-baseline.json 里那条逐字相同（既有、生产文件、非本 PR 引入）；新拆的测试文件 0 finding
python3 scripts/verify/verify-edge-lint-ratchet.py	rc=0（9 findings, all baseline-registered）
python3 scripts/verify/verify-test-sleep-ratchet.py	rc=0（47 sleeps / 22 files）
bash /tmp/run-validate-H.sh <worktree>（61 条 CI validate 命令）	PASS=61 FAIL=0 SKIP(merge-ref)=1（1m01s）
bash scripts/verify/verify-commit-messages.sh origin/master HEAD	rc=0
git diff --check origin/master...HEAD	rc=0
bash scripts/verify/check-secrets.sh --range "origin/master...HEAD" 和 --staged	双双 rc=0（Secret guard passed.）——正因 13 个 sanitize Test 没搬

暗卷（变异测试，证明搬家后断言仍在管，且落点在新文件）

1. process_arg_log.go 的 appendConfigKeyName：把 value-like 参数的 redactedCount + 1 改成 redactedCount（脱敏计数不再累加）→ --- FAIL: TestSummarizeProcessArgsForLogRedactsValueLikeArgs，落点 process_arg_log_test.go:105（新文件）；git checkout -- 还原后 ok。
2. process_profile.go 的 validatePlaceholders：函数体改成 return nil（不再拒绝未知占位符）→ --- FAIL: TestProcessExecutorRejectsUnknownPlaceholder、--- FAIL: TestProcessExecutorRejectsInvalidEnvTemplate，落点 process_profile_test.go:194、:217（新文件）；还原后 ok 7.733s，git status --short 无 src 残留。

未验证 / 边界

• 未跑全仓 go test ./...、未跑全量 lint（4C8G、磁盘 93%）；只跑被改包 + 61 条 validate。
• 未跑 CI 的真实 Windows job（process_executor_windows_test.go 未动）、未跑 PG16/Redis 集成（本包不依赖）。
• 净 +117 行是 10 个新文件的 package/import 头 + 空行开销，不是新增断言；逐行守恒证明分支侧 0 行新增内容。
• TestProcessExecutorHelper 是假子进程入口（-test.run=^TestProcessExecutorHelper$ 重入自身），5s sleep 属 real_protocol，搬家不改其语义。
• 关联 check-secrets.sh 字面量规则无 fixture allowlist：脱敏测试的假凭据夹具让这类测试文件「不可搬家」，且良性 ID task-backfill-mismatch-conflict 被判成 API key（round-72 实测两发） #2295（secret 门禁 fixture allowlist 裁决）；同批 round-72 另有 hub service/agent 与 edge store 的同类归位。

[coderabbitai]

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: 08efdd9c-cb1c-4877-9c03-ad80a2640ee8

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

• X
• Mastodon
• Reddit
• LinkedIn

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