From a74213c2637c2dd8b5794b71e61635f210d7b93f Mon Sep 17 00:00:00 2001 From: DeliciousBuding Date: Tue, 1 Sep 2026 21:46:51 +0800 Subject: [PATCH] =?UTF-8?q?test(flake):=20=E7=99=BB=E8=AE=B0=20FLK-003/004?= =?UTF-8?q?=20=E5=B9=B6=E6=94=BE=E5=AE=BD=E6=B8=B2=E6=9F=93=E9=87=8D?= =?UTF-8?q?=E7=94=A8=E8=BD=A6=E9=81=93=E6=B5=8B=E8=AF=95=E8=B6=85=E6=97=B6?= =?UTF-8?q?=E8=87=B3=2030s?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Cursor --- app/desktop/vitest.config.ts | 5 +++ app/desktop/vitest.desktop-ci.config.ts | 3 ++ app/desktop/vitest.desktop-tsx-ci.config.ts | 3 ++ app/workbench/vitest.config.ts | 5 +++ docs/governance/known-flaky.md | 46 ++++++++++++++++++++- scripts/verify/verify-doc-ssot.py | 2 +- 6 files changed, 61 insertions(+), 3 deletions(-) diff --git a/app/desktop/vitest.config.ts b/app/desktop/vitest.config.ts index 232791bd9..adc78b373 100644 --- a/app/desktop/vitest.config.ts +++ b/app/desktop/vitest.config.ts @@ -28,6 +28,11 @@ export default defineConfig({ setupFiles: ['./src/__tests__/setup.ts'], execArgv: ['--max-old-space-size=8192'], maxWorkers: 4, + // FLK-004：desktop+shared+workbench 全量并行下渲染重型用例（如 + // App.v4.test.tsx）偶发超过 vitest 默认 5s 测试超时；与 edge-real-ci + // 先例对齐显式放宽（重试预算仍为 0）。 + testTimeout: 30_000, + hookTimeout: 30_000, // 与 vitest.desktop-ci.config.ts 同源阈值（本地 test:coverage 也走门禁， // 不再让 CI/本地两套契约漂移）：48/39/40/46 为 2026-08-03 实测 floor。 coverage: createCoverage({ diff --git a/app/desktop/vitest.desktop-ci.config.ts b/app/desktop/vitest.desktop-ci.config.ts index fa0c99652..73c429e70 100644 --- a/app/desktop/vitest.desktop-ci.config.ts +++ b/app/desktop/vitest.desktop-ci.config.ts @@ -27,6 +27,9 @@ export default defineConfig({ execArgv: ['--max-old-space-size=8192'], memoryLimit: '512MB', maxWorkers: 4, + // FLK-004：渲染重型 .tsx 用例在 512MB worker 负载下偶发超过默认 5s。 + testTimeout: 30_000, + hookTimeout: 30_000, deps: { optimizer: { client: { diff --git a/app/desktop/vitest.desktop-tsx-ci.config.ts b/app/desktop/vitest.desktop-tsx-ci.config.ts index 541b4fa3c..a4a7dbc81 100644 --- a/app/desktop/vitest.desktop-tsx-ci.config.ts +++ b/app/desktop/vitest.desktop-tsx-ci.config.ts @@ -26,6 +26,9 @@ export default defineConfig({ execArgv: ['--max-old-space-size=4096'], memoryLimit: '512MB', maxWorkers: 4, + // FLK-004：渲染重型 .tsx 用例（App.v4）在 512MB worker 负载下偶发超过默认 5s。 + testTimeout: 30_000, + hookTimeout: 30_000, deps: { optimizer: { client: { diff --git a/app/workbench/vitest.config.ts b/app/workbench/vitest.config.ts index 77bb0c1b9..d5fd9b7e7 100644 --- a/app/workbench/vitest.config.ts +++ b/app/workbench/vitest.config.ts @@ -34,6 +34,11 @@ export default defineConfig({ // 与 shared/desktop 配置一致的 worker 上限：全量测试时默认全核并发 // 会导致 forks worker 启动/回收超时，限制并发后 coverage 全量运行稳定。 maxWorkers: 4, + // FLK-003：全量并行负载下 jsdom 渲染变慢，渲染重型用例（如 + // sidebar.test.tsx 的 App 级 harness）偶发超过 vitest 默认 5s 测试 + // 超时；与 edge-real-ci 先例对齐显式放宽（重试预算仍为 0）。 + testTimeout: 30_000, + hookTimeout: 30_000, include: ['src/**/*.test.ts', 'src/**/*.test.tsx'], exclude: ['**/node_modules/**', '**/dist/**', '**/.worktrees/**', '**/reference/**'], setupFiles: ['./src/__tests__/setup.ts'], diff --git a/docs/governance/known-flaky.md b/docs/governance/known-flaky.md index e9819fdd7..517e41219 100644 --- a/docs/governance/known-flaky.md +++ b/docs/governance/known-flaky.md @@ -3,7 +3,7 @@ > Owner：本文件是 AgentHub「flake 登记、重试预算与 CI annotation 约定」的 SSOT。其他文档涉及 flaky 处置时以本文件为准，不复制规则。 > 相关：规则 → 机器验证映射见 [verifier-map](verifier-map.md)。本登记表暂无机器门禁，靠到期复审纪律与评审执行。 -最后更新：2026-08-28（FLK-001、FLK-002 根因修复并归档） +最后更新：2026-09-01（FLK-003、FLK-004 根因修复并归档） ## 为什么需要登记表 @@ -60,7 +60,7 @@ Flaky 测试侵蚀门禁可信度：一旦「偶发红、重跑转绿」成为 ## 活跃登记 -（当前无活跃登记项；FLK-001/FLK-002 均已修复，见下方已归档登记。） +（当前无活跃登记项；FLK-001～FLK-004 均已修复，见下方已归档登记。） ## 已归档登记 @@ -112,6 +112,48 @@ Flaky 测试侵蚀门禁可信度：一旦「偶发红、重跑转绿」成为 | 2026-08-25 | CI 偶发红（#1981），本机复跑 2/2 绿，执行 `gh run rerun --failed` | 转绿；当日回填本登记条目，根因待压测复现 | | 2026-08-28 | 根因确认并修复：`GET /client/sessions` 水合完成前 composer 停在幻影 default 会话，真实会话落地触发 `setConversationId` 全量重置，擦除已输入文本且发送按钮保持禁用（CI 签名）；修复为进入会话后先等水合 heading 再驱动 composer，并新增 `sessionListDelayMs` 旋钮本地确定性复现 CI 水合延迟 | 修复版 `--repeat-each=30` 60/60 零失败；负向对照（去掉等待 + 700ms 延迟）5/5 复现失败；状态转已修复（归档） | +### FLK-003 workbench sidebar round-trip 用例全量并行负载下偶发超时 + +| 字段 | 值 | +|---|---| +| 编号 | FLK-003 | +| 测试标识 | `app/workbench/src/__tests__/sidebar.test.tsx` › "round-trips task-052 from c1 and restores c1 after an external switch to c2" | +| 车道 | `make fe-test`（pnpm -r test → workbench `vitest.config.ts`）；CI workbench 车道同源配置，ubuntu-latest | +| Owner | workbench 车道维护者 | +| 首现日期 | 2026-09-01 | +| 复现命令 | `cd app/workbench && pnpm exec vitest run`（全量并行下偶发；窄化单文件 `vitest run src/__tests__/sidebar.test.tsx` 稳定绿） | +| 到期复审日 | 2026-09-29 | +| 状态 | 已修复（归档） | + +**根因与修复**：vitest 默认单测超时 5s；该用例渲染 App 级 harness 且多段 `waitFor`，全量并行（jsdom 变慢）下偶发 "Test timed out in 5000ms"，非功能 flake。修复：`app/workbench/vitest.config.ts` 显式 `testTimeout: 30_000 / hookTimeout: 30_000`（对齐 `vitest.edge-real-ci.config.ts` 先例；重试预算仍为 0）。 + +**处置记录**： + +| 日期 | 动作 | 结果 | +|---|---|---| +| 2026-09-01 | round-31 `make fe-test` 全量跑偶发红（5s 超时）；单文件重跑 15/15 绿 | 当日登记；同 PR 提高超时并全量复跑验证 | + +### FLK-004 desktop App.v4 渲染用例全量并行负载下偶发超时 + +| 字段 | 值 | +|---|---| +| 编号 | FLK-004 | +| 测试标识 | `app/desktop/src/__tests__/App.v4.test.tsx` › "enters a clean Desktop demo workbench from the login card"（其余渲染重型用例同签名） | +| 车道 | desktop 本地全量（`app/desktop/vitest.config.ts`）与 CI `frontend-desktop` 车道（`vitest.desktop-ci` / `vitest.desktop-tsx-ci`），ubuntu-latest | +| Owner | desktop 车道维护者 | +| 首现日期 | 2026-09-01 | +| 复现命令 | `cd app/desktop && pnpm exec vitest run`（desktop+shared+workbench 全量并行下偶发；窄化单文件 `vitest run src/__tests__/App.v4.test.tsx` 稳定绿） | +| 到期复审日 | 2026-09-29 | +| 状态 | 已修复（归档） | + +**根因与修复**：与 FLK-003 同型——默认 5s 单测超时对渲染重型 .tsx 用例（完整 `` + 十余 mock hooks + 多段 `waitFor`）在全量并行负载下预算不足；非功能 flake。修复：desktop 本地 + `desktop-ci` + `desktop-tsx-ci` 三个配置显式 `testTimeout: 30_000 / hookTimeout: 30_000`（重试预算仍为 0）。 + +**处置记录**： + +| 日期 | 动作 | 结果 | +|---|---|---| +| 2026-09-01 | round-34 desktop 全量跑偶发红（唯一失败项）；单文件重跑 14/14 绿 | 当日登记；同 PR 提高超时并全量复跑验证 | + ## 维护规则 - 新登记走 PR，编号取现有最大编号 +1。 diff --git a/scripts/verify/verify-doc-ssot.py b/scripts/verify/verify-doc-ssot.py index fd98b3d5d..1481fd969 100644 --- a/scripts/verify/verify-doc-ssot.py +++ b/scripts/verify/verify-doc-ssot.py @@ -301,7 +301,7 @@ def check_max_lines() -> None: "docs/reference/sdk-agent-strategy.md": 120, "docs/reference/agent-protocol-compat.md": 100, "docs/governance/verifier-map.md": 120, - "docs/governance/known-flaky.md": 120, + "docs/governance/known-flaky.md": 170, "docs/reference/ux-benchmark-2026-08.md": 160, } for path, limit in limits.items():