Repository navigation
test(runtime): await validator preflight in controller fixture - #1334
Merged
aviggiano merged 1 commit intoOct 26, 2026
Merged
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
aviggiano
deleted the
codex/fix-validator-preflight-controller-test-1315
branch
October 26, 2026 20:53
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
The controller integration fixture added by #1315 still calls
preflightJsonValidatoras a synchronous three-argument function. #1318 made it asynchronous and requires anAbortSignal. The stale fixture leaves a rejected promise unobserved, so Smithers recordsTASK_ABORTEDand cancels the run before the test can check its intendedTOOL_UNAVAILABLEfailure.Await the current validator preflight, pass a signal, use the real
runProcessGroup, and wrap its failure through the same named preparation step used by the generated workflow. This keeps the existing assertions for one failed attempt, no retry, and the exactTOOL_UNAVAILABLEcode.This fixes the single support-lane failure in #1329's exact-head run 40396550104. The same test failed on pre-#1329
unstableatb33cb2d, both with and without the scratch runner.Validation
node scripts/run-tests.mjs smithers-controller-engine.integration: 3/3 passed, including the formerly failing case.ee990b2, Prettier, andgit diff --checkpassed.

Confidence Score: 5/5
This fixture update appears safe to merge.Summary
Updates the controller fixture to await
preflightJsonValidator, pass anAbortSignal, and userunProcessGroup.Reviews (1) · Last reviewed commit: "test(runtime): await validator preflight..." · Reviewed by Greptile