Surfaced by the dev delivering #18590 (PR #18746) as an out-of-scope finding, then dedupe-searched and re-framed by the domain:cli execution PM seat before filing — ⛔ the instance is the dev's reading; the recurrence below is the seat's. ⛔ Filed unlabelled by domain — routing and grading are triage's.
The instance
packages/cli/src/commands/datasource/envelope-unwrap.test.ts fails to COLLECT under load: its beforeAll dies with Hook timed out in 10000ms while loading oclif's Config.load({ root: CLI_ROOT }). On a quiet box the same file is green — re-measured: Test Files 2 passed (2), Tests 16 passed (16).
⇒ a test that never runs its bodies, reported as a failure of the suite, decided by how busy the runner is.
⭐ Why this is worth a card: it is the FIFTH site of one class, and every previous fix was per-site
The dedupe search found the class already settled three times, all closed/completed:
| card |
site |
budget |
state |
| #5421 |
runtime-lazy-deps.test.ts — "gating a flow in-process" |
default 5s, flakes under load |
closed |
| #9311 |
types/src/node.test.ts and dogfood/test/semantic-roles.dogfood.test.ts |
5s and 10s hookTimeout |
closed |
| #18590 |
packages/cli/test/published-entry-node-env-source-reroute.test.ts |
default 5s against 2.5–4.1s legs |
fixed this hour, per-site |
| this card |
packages/cli/src/commands/datasource/envelope-unwrap.test.ts |
10s hookTimeout on an oclif Config.load |
— |
⭐ #5421's own title records that the remedy already existed one file over: "its cold-load sibling already carries COLD_LOAD_TIMEOUT_MS". ⭐ #9311 already named a 10s hookTimeout — the very budget this instance dies on.
⇒ the class is not "a flaky test". It is: a default budget nobody chose is silently made the judge of a real cold-boot cost, and the correct per-file bound is often already written down somewhere adjacent. Four closures have not stopped a fifth.
What is measurably different about this one
⛔ What this card does NOT decide
Dedupe
Search ran over this repository. The instance query returned 0; the control that can fail is a sibling query in the same session returning #18590 plus 11 more, so the zero is a reading and ⛔ not a dead search. The class query returned exactly the three closed cards tabled above, and ⛔ no open card names this file or this hook budget.
Dedupe words: envelope-unwrap · Hook timed out in 10000ms · Config.load · oclif beforeAll · cli unit tier collection failure · unchosen default budget · COLD_LOAD_TIMEOUT_MS.
Generated by Claude Code
Surfaced by the dev delivering #18590 (PR #18746) as an out-of-scope finding, then dedupe-searched and re-framed by the
domain:cliexecution PM seat before filing — ⛔ the instance is the dev's reading; the recurrence below is the seat's. ⛔ Filed unlabelled by domain — routing and grading are triage's.The instance
packages/cli/src/commands/datasource/envelope-unwrap.test.tsfails to COLLECT under load: itsbeforeAlldies withHook timed out in 10000mswhile loading oclif'sConfig.load({ root: CLI_ROOT }). On a quiet box the same file is green — re-measured:Test Files 2 passed (2), Tests 16 passed (16).⇒ a test that never runs its bodies, reported as a failure of the suite, decided by how busy the runner is.
⭐ Why this is worth a card: it is the FIFTH site of one class, and every previous fix was per-site
The dedupe search found the class already settled three times, all closed/completed:
runtime-lazy-deps.test.ts— "gating a flow in-process"types/src/node.test.tsanddogfood/test/semantic-roles.dogfood.test.tspackages/cli/test/published-entry-node-env-source-reroute.test.tspackages/cli/src/commands/datasource/envelope-unwrap.test.tsConfig.load⭐ #5421's own title records that the remedy already existed one file over: "its cold-load sibling already carries
COLD_LOAD_TIMEOUT_MS". ⭐ #9311 already named a 10s hookTimeout — the very budget this instance dies on.⇒ the class is not "a flaky test". It is: a default budget nobody chose is silently made the judge of a real cold-boot cost, and the correct per-file bound is often already written down somewhere adjacent. Four closures have not stopped a fifth.
What is measurably different about this one
hookTimeout), not the test budget — so the failure is a collection failure: ⛔ zero test bodies run, and the reported shape is not "a test failed" but "the file failed".pass 1— so the rerun property it guards has never been measured #18590's sibling was integration. ⇒ it can redden an ordinary PR, not only a nightly.⛔ What this card does NOT decide
pass 1— so the rerun property it guards has never been measured #18590's own ruling, "raise the timeout until it passes" is the move that hides a real failure later — whatever is chosen here has to survive that objection.pass 1— so the rerun property it guards has never been measured #18590's dev named one adjacent candidate (packages/cli/test/published-entry-stderr-error-listener.test.ts, the only other spawning file in that directory with no explicit per-test timeout — 27 of 48 integration-tier files carry one) and explicitly did not measure it. ⛔ A population claim needs that census; this card makes none.Dedupe
Search ran over this repository. The instance query returned 0; the control that can fail is a sibling query in the same session returning #18590 plus 11 more, so the zero is a reading and ⛔ not a dead search. The class query returned exactly the three closed cards tabled above, and ⛔ no open card names this file or this hook budget.
Dedupe words:
envelope-unwrap·Hook timed out in 10000ms·Config.load·oclif beforeAll·cli unit tier collection failure·unchosen default budget·COLD_LOAD_TIMEOUT_MS.Generated by Claude Code