test: raise AVA concurrency now that the global-composer pin is obsolete#216
Merged
Conversation
This was referenced Jul 18, 2026
Merged
Collaborator
Author
|
This pull request is part of a Mergify stack:
|
This was referenced Jul 18, 2026
Merged
Run four isolated AVA workers: test writers use distinct temporary directories, watcher servers bind ephemeral ports, and no suite changes the process cwd. This retains headroom for filesystem watchers while cutting the measured suite wall time from 61.12s serial to 32.60s, 32.79s, 32.02s, 31.74s, and 32.97s. Parallel validation exposed a startup race in the live-bridge journal poll: the test read its transaction directory before the writer created it. Treat ENOENT as an empty poll iteration so the existing bounded wait covers normal async startup. Verification: five consecutive pnpm test:ava runs, each 293 passed; pnpm verify (lint, typecheck, build, 293 AVA, 45 codegen Vitest, 7 utils Vitest). Co-authored-by: Cursor <[email protected]> Change-Id: Ib58dca643335b671687601e01bda25e0b4013c32
tonyketcham
force-pushed
the
stack/tonyketcham/feat/unified-watch-effort-graph/raise-ava-concurrency-now-global-composer-pin--b58dca64
branch
from
July 18, 2026 07:36
4825b74 to
b599488
Compare
Collaborator
Author
Revision history
|
Collaborator
Author
|
@Mergifyio queue |
Contributor
Merge Queue Status
This pull request spent 52 minutes 7 seconds in the queue, with no time running CI. Waiting for
All conditions
ReasonThe merge conditions cannot be satisfied due to failing checks Failing checks:
HintYou may have to fix your CI before adding the pull request to the queue again. Requeued — the merge queue status continues in this comment ↓. |
Base automatically changed from
stack/tonyketcham/feat/unified-watch-effort-graph/drop-unused-graphql-compose-json-lru-cache-deps--6d9ecf9f
to
main
July 18, 2026 09:24
…graph/raise-ava-concurrency-now-global-composer-pin--b58dca64
Contributor
Merge Queue Status
Waiting for any of
All conditions
|
Collaborator
Author
|
@Mergifyio queue |
Contributor
Merge Queue Status
Required conditions to enter a queue
|
Retry only ENOENT watcher subscription failures with bounded backoff so parallel test workers do not fail when temporary directories are removed. Co-authored-by: Cursor <[email protected]> Change-Id: I7e092e6e30ec70dcc30db39777cba6d06bf5058d
Collaborator
Author
|
@Mergifyio queue |
Contributor
☑️ Command
|
Collaborator
Author
|
@Mergifyio refresh |
Contributor
✅ Pull request refreshed |
Collaborator
Author
|
@Mergifyio queue |
Contributor
☑️ Command
|
Contributor
Merge Queue Status
This pull request spent 16 seconds in the queue, including 2 seconds running CI. Required conditions to merge
|
tonyketcham
deleted the
stack/tonyketcham/feat/unified-watch-effort-graph/raise-ava-concurrency-now-global-composer-pin--b58dca64
branch
July 18, 2026 10:00
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
1 participant
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.
Run four isolated AVA workers: test writers use distinct temporary directories, watcher servers bind ephemeral ports, and no suite changes the process cwd. This retains headroom for filesystem watchers while cutting the measured suite wall time from 61.12s serial to 32.60s, 32.79s, 32.02s, 31.74s, and 32.97s.
Parallel validation exposed a startup race in the live-bridge journal poll: the test read its transaction directory before the writer created it. Treat ENOENT as an empty poll iteration so the existing bounded wait covers normal async startup.
Verification: five consecutive pnpm test:ava runs, each 293 passed; pnpm verify (lint, typecheck, build, 293 AVA, 45 codegen Vitest, 7 utils Vitest).
Co-authored-by: Cursor [email protected]
Depends-On: #215