Repository navigation
test(scenarios): give the built-consumer child-process cases a 120s timeout - #30996
Merged
Merged
Conversation
…l's 120s timeout Co-authored-by: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016WV4cwgUKpQgCE3UWsA74d
Contributor
📓 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): |
Contributor
Author
|
domain:devx#1 · os-elon-musk · session_016WV4cymvo9YoyLwKCXXBNw 席位核对 · ACCEPT · #30915 follow-up · 2026-11-06T16:26ZHead-sha:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Elon · project thread
domain:devx#1 · os-elon-musk · session_016WV4cwgUKpQgCE3UWsA74d
Refs #30915
Clause-②: no
Metadata-protocol: none
维护者速读
built-consumer.test.ts里两个起子进程的用例(ESM/CJS)给了显式超时 120 秒,和同文件beforeAll一致。packages/qa/scenarios/test/build/built-consumer.test.ts的it.each结尾。pnpm --filter @objectstack/scenarios build && pnpm --filter @objectstack/scenarios exec vitest run test/build,预期 5/5。What changes
packages/qa/scenarios/test/build/built-consumer.test.ts(added by #30938): theit.eachESM / CJS cases spawn a Node child that loads the scorer graph throughmodule.registerHooksresolve hooks. They ran on vitest's 5s default while onlybeforeAllhad120_000. Under the full-suite merge queue (build 42243079369, job 127979322328, Test Core 5/8) the file took 9415ms and the CJS case failed withTest timed out in 5000ms. The cases now pass120_000, the budgetbeforeAllalready uses. No assertion changed and nothing retries.Verification
At
9ab662dda1(onorigin/main000a4c0d36):pnpm --filter @objectstack/scenarios buildexit 0 (check-dts-emitted10/10),vitest run test/build5 passed (5),typecheckexit 0.dispatch-gates.mjs --commands --repo objectstack-ai/objectstackderived 47 commands for this one file; they were still running when this PR opened. Results go to the seat, and CI runs the full set.Test-only change in a private package:
skip-changeset.🤖 Generated with Claude Code
https://claude.ai/code/session_016WV4cwgUKpQgCE3UWsA74d
Generated by Claude Code