Repository navigation
test: suite no longer races a starved runner (0.0.60) - #65
Merged
Merged
Conversation
This was referenced Sep 20, 2026
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.
summary
test (macos-latest, 22)job (flaky: codex companion process tests fail on the macos-latest node 22 job under load #64). 0.0.59 fixed the codex late close window and the monitors' handler order, and main still went red ontests/job-collect.test.mjsdoes not infer terminal state from prose, which gave a spawned fake companioncapMs: 100pson every write and made that one case take 40 seconds on an idle machine; seeding is a plain file write now, and the codex harness watchdog for a companion run moves from 20 to 60 secondstests/fusion-stats.test.mjs(a helper that runs the monitor until its state file or tagged output appears, replacing a fixed 400msspawnSynctimeout); everything else is a few lines per file, plus the 0.0.60 release (changelog and the seven version fields)test plan
already verified
npm testidle on the merged tree -> 1200 tests, 1199 pass, 0 fail, 1 skipped (the existing skip), 123 secondsnode --test tests/plugin-versions.test.mjs tests/verified-versions.test.mjs tests/codex-companion.test.mjsafter the version bump -> 106/106reviewer should verify
test (macos-latest, 22)on this PR and on main's push run after the merge; both should be green on the first attemptnotes
taskrun shells out tops32 times at startup, and a workspace with 600 job records adds about 5 seconds to a gpt-6-astra write task because the flagship stat scans every record