Repository navigation
test(spec): the connector-author-shape compile test carries its own 180 s timeout - #22969
Conversation
…80 s timeout instead of riding the 60 s default The one `it` that compiles the document's connector examples through a ts.createProgram over the spec's types read 55,680 ms on main and 61,767 ms on a cold CI re-run against the package default of 60,000 ms. It now has its own timeout with the measured figures in a comment; the global testTimeout is untouched so the next slow test stays visible. Claude-Session: https://claude.ai/code/session_019G5ZnRrTMiwcHG6iZM19zf Co-authored-by: Claude <noreply@anthropic.com>
📓 Docs Drift CheckNothing in this diff resolved to a documentable surface (no symbol, route or SDK anchor derived from 0 changed package(s)), so this run has no opinion about the docs. What this run could not see
Coarse fallback — 0 page(s) merely mention a changed package (the pre-#9192 predicate, kept for the deliberately-wide backstop): |
|
The basis is this seat's own review, as the review checklist prescribes when a dev's label write is refused by its session's permission check. The dev's report records the refused command verbatim.
The maintainer confirmed in this seat's chat that the call is the seat's. |
Fixes #22929
Clause-②: no
What changed
One file,
packages/spec/src/integration/connector-author-shape.test.ts, +14 / -1: theitnamedcompiles every full connector example verbatim, import line includednow passes180_000as its own timeout, with a comment carrying the measured figures and the reasoning for the bound. Test-only. The package-widetestTimeout: 60_000inpackages/spec/vitest.config.tsis untouched, and the test is neither skipped nor split.Why 180 s
Test Core (1/6), as filed on the card: 55,680 ms onmain(one cached replay, not three readings) and 61,767 ms on a cold spec re-run on PR feat(spec): ServiceSlotContracts types the webhooks and trigger-api slots #22922, which timed out at 60,000 ms. I could not re-read the job log from this container (the log blob host is not reachable through the proxy); the check run itself confirms that job failed in thepackages/specpnpm run teststep, and the figures here are the card's.--maxWorkers=2on a shared box: 13,796 ms before the change and 10,969 ms after.shared/alias-integrity.test.tsalready gives its own heavybeforeAll.Evidence the timeout is the one that governs (ablation, one-off, nothing kept)
Direction stated before running: with the own timeout removed and the global forced down to 2,000 ms on the command line, the
itgoes red; restored, it goes green under the same forced global.}, 180_000);replaced by});, mutation proven on disk by anchor count 1 to 0 and blob change)vitest run --maxWorkers=2 --testTimeout=2000 ...connector-author-shape.test.tsTest timed out in 2000mson the compileit, 10 other tests pass88e0e6551c93,git diff HEADempty)it12,452 ms under the 2,000 ms globalThe observed direction is the predicted one (red, then green). The test subject here is the test file itself, resolved from
src, so there is nodistleg to prove.Verification
pnpm --filter @objectstack/spec exec vitest run --maxWorkers=2 src/integration/connector-author-shape.test.tsat57d5074ff: 11 passed, compileit10,969 ms.node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstackderived 77 commands for this change set; all 77 were run at57d5074ffwith exit codes recorded before any pipe and reconciled with--ran: 76 exit 0, 1 NOT MEASURED, 0 unrun.pnpm check:dual-build-cjs-loadsexited 3 (prerequisite: it reads the builtdistof 88 other packages, which needs a whole-workspacepnpm build). A test file enters no package's emitted output, andBuild Corein CI covers it. Two other gates that first exited 3 for an unbuilt prerequisite (check:doc-formula-expressions,check:lean-entry-closure) were rerun after a targeted build and exited 0.Test Coreshard running it green, which is CI's reading of this PR.mainmoved four commits after the branch point (da9c2bf2f); none touches the test,docs/SYNC_ARCHITECTURE.md,packages/spec/src/integration/or the vitest config.mainwas not merged in.Publishes nothing
npm pack --dry-runon@objectstack/speclists 2,079 files: 0 are test files or this test, withsrc/integration/connector.zod.tspresent as the positive control; a search ofdistfor the test's title finds 0 files, and forConnectorSchemafinds files. No changeset is owed, andskip-changesetapplies.Acceptance notes
Nothing out of scope was found.
Generated by Claude Code