From d8b80d0bfe5ff7b6635d2e070e86949b01882e88 Mon Sep 17 00:00:00 2001 From: gsxdsm Date: Wed, 17 Jun 2026 13:08:05 -0700 Subject: [PATCH] FN-6587: quarantine compound-engineering timeout flakes Quarantine compound-engineering timeout flakes from the broad test workflow. - Add the compound-engineering orchestrator and skill-wiring tests to the quarantine ledger. - Exclude the quarantined tests from the compound-engineering node Vitest project without changing timeouts or retries. - Document the quarantine requirement next to the Vitest excludes. Files changed: .../fusion-plugin-compound-engineering/vitest.config.ts | 15 ++++++++++++++- scripts/lib/test-quarantine.json | 10 ++++++++++ 2 files changed, 24 insertions(+), 1 deletion(-) Fusion-Task-Id: FN-6587 Fusion-Task-Lineage: 5aeb0202-8330-4a93-bdab-675c81b9cb54 --- .../vitest.config.ts | 15 ++++++++++++++- scripts/lib/test-quarantine.json | 10 ++++++++++ 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/plugins/fusion-plugin-compound-engineering/vitest.config.ts b/plugins/fusion-plugin-compound-engineering/vitest.config.ts index 7e7866d0d0..b065a4a722 100644 --- a/plugins/fusion-plugin-compound-engineering/vitest.config.ts +++ b/plugins/fusion-plugin-compound-engineering/vitest.config.ts @@ -9,6 +9,15 @@ const coreSetup = fileURLToPath( ); const dashboardSetup = fileURLToPath(new URL("./src/dashboard/test-setup.ts", import.meta.url)); +/* +FNXC:CompoundEngineeringTests 2026-06-17-12:35: +FN-6587 quarantines the CE broad-pnpm-test timeout flakes without timeout appeasement. Keep these excludes mirrored in scripts/lib/test-quarantine.json and remove or delete the files when the 14-day ratchet resolves. +*/ +const quarantinedCompoundEngineeringTests = [ + "src/__tests__/orchestrator-flow.test.ts", + "src/__tests__/skill-wiring.test.ts", +]; + export default defineConfig({ resolve: { alias: [ @@ -56,7 +65,11 @@ export default defineConfig({ name: "compound-engineering-node", environment: "node", include: ["src/**/__tests__/**/*.test.{ts,tsx}", "src/**/*.test.{ts,tsx}"], - exclude: ["src/dashboard/**/__tests__/**/*.test.{ts,tsx}", "src/dashboard/**/*.test.{ts,tsx}"], + exclude: [ + "src/dashboard/**/__tests__/**/*.test.{ts,tsx}", + "src/dashboard/**/*.test.{ts,tsx}", + ...quarantinedCompoundEngineeringTests, + ], }, }, ], diff --git a/scripts/lib/test-quarantine.json b/scripts/lib/test-quarantine.json index 0d4c12f453..ae9ff0d560 100644 --- a/scripts/lib/test-quarantine.json +++ b/scripts/lib/test-quarantine.json @@ -10,6 +10,16 @@ "file": "packages/engine/src/__tests__/cli-agent-executor.test.ts", "reason": "FN-6492 verification observed the hard-cancel CLI session test fail only in the full @fusion/engine package lane (activeCliTaskSessions false plus ENOTEMPTY temp cleanup), while an immediate file-specific rerun passed; quarantined as a concurrency/temp-cleanup flake per the deletion ratchet.", "quarantinedAt": "2026-06-16" + }, + { + "file": "plugins/fusion-plugin-compound-engineering/src/__tests__/orchestrator-flow.test.ts", + "reason": "FN-6587 broad pnpm test investigation: this CE orchestrator file was reported as timing out at Vitest's 5000ms test limit only in the broad pnpm test workflow; isolated two-file repro and loaded compound-engineering-node runs passed, and the broad command hit its external 900s workflow timeout before reproducing the named CE timeout. Quarantined per deletion-ratchet policy without timeout bumps, retries, or assertion loosening.", + "quarantinedAt": "2026-06-17" + }, + { + "file": "plugins/fusion-plugin-compound-engineering/src/__tests__/skill-wiring.test.ts", + "reason": "FN-6587 broad pnpm test investigation: this CE skill-wiring file was reported as timing out at Vitest's 5000ms test limit only in the broad pnpm test workflow; isolated two-file repro and loaded compound-engineering-node runs passed, and the broad command hit its external 900s workflow timeout before reproducing the named CE timeout. Quarantined per deletion-ratchet policy without timeout bumps, retries, or assertion loosening.", + "quarantinedAt": "2026-06-17" } ] }