在 #5544 的实施里量到(该单只让 tsc 读集成套件,不改运行通道,故不在其范围内修)。基线:origin/main @ 229d29ea4。
事实
packages/client/vitest.integration.config.ts 写着:
// Run integration tests sequentially to avoid race conditions
pool: 'forks',
poolOptions: {
forks: {
singleFork: true
}
}而本仓 @objectstack/client 的 devDependency 是 vitest ^4.1.10。在该套件上跑一次收集就能看到 vitest 自己报:
$ npx vitest list --config vitest.integration.config.ts
DEPRECATED `test.poolOptions` was removed in Vitest 4. All previous `poolOptions`
are now top-level options. Please, refer to the migration guide:
https://vitest.dev/guide/migration#pool-rework
即:singleFork: true 这条没有被读取。注释声明的「串行执行以避免竞态」是 declared,不是 enforced —— Prime Directive #10 的形状,发生在测试配置层。
poolOptions 在全仓只此一处(grep -rn poolOptions --include=*.ts --include=*.mts --include=*.js,排除 node_modules),所以这是孤例,不是一次批量迁移的遗留。
为什么现在没人踩到
该套件由 pnpm --filter @objectstack/client test:integration 单独跑,且需要一台外部仓库提供的活服务器(tests/integration/README.md 写明),CI 的常规 test 任务用 vitest.config.ts 把 tests/integration/** 排除掉。于是没有任何自动通道跑到它,这条失效的串行声明今天不会让任何人变红——属于观察类(finding),不是当下缺陷。但它是将来有人接上服务器时第一批会咬人的东西:并行跑一组共享同一台服务器、同一份测试数据的用例,正是注释想避免的竞态。
处置
按 Vitest 4 迁移指南把 poolOptions 里的选项提到顶层(pool: 'forks' 保留,singleFork 的等价物按迁移指南定 —— 目测是顶层 fileParallelism: false 或 maxWorkers: 1,需照文档核实而不是照猜)。落点只有这一个文件。
顺带:同目录 README.md 列了 17 个集成测试文件的规划表,而目录里实际只有 01-discovery.test.ts 一个 —— 那是另一件事,不在本单。
关联:#5544(纳入 tsconfig + 修 client.discovery 漂移)、#5449。
在 #5544 的实施里量到(该单只让 tsc 读集成套件,不改运行通道,故不在其范围内修)。基线:
origin/main@229d29ea4。事实
packages/client/vitest.integration.config.ts写着:而本仓
@objectstack/client的 devDependency 是vitest ^4.1.10。在该套件上跑一次收集就能看到 vitest 自己报:即:
singleFork: true这条没有被读取。注释声明的「串行执行以避免竞态」是 declared,不是 enforced —— Prime Directive #10 的形状,发生在测试配置层。poolOptions在全仓只此一处(grep -rn poolOptions --include=*.ts --include=*.mts --include=*.js,排除 node_modules),所以这是孤例,不是一次批量迁移的遗留。为什么现在没人踩到
该套件由
pnpm --filter @objectstack/client test:integration单独跑,且需要一台外部仓库提供的活服务器(tests/integration/README.md写明),CI 的常规test任务用vitest.config.ts把tests/integration/**排除掉。于是没有任何自动通道跑到它,这条失效的串行声明今天不会让任何人变红——属于观察类(finding),不是当下缺陷。但它是将来有人接上服务器时第一批会咬人的东西:并行跑一组共享同一台服务器、同一份测试数据的用例,正是注释想避免的竞态。处置
按 Vitest 4 迁移指南把
poolOptions里的选项提到顶层(pool: 'forks'保留,singleFork的等价物按迁移指南定 —— 目测是顶层fileParallelism: false或maxWorkers: 1,需照文档核实而不是照猜)。落点只有这一个文件。顺带:同目录
README.md列了 17 个集成测试文件的规划表,而目录里实际只有01-discovery.test.ts一个 —— 那是另一件事,不在本单。关联:#5544(纳入 tsconfig + 修
client.discovery漂移)、#5449。