🤖 fix: avoid backgrounding exited bash runs #1805
Merged
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
Why
Fast-exit commands were being backgrounded after they finished, returning empty "Output so far" and flaking Windows CI. mux.md URLs without a key were also falling through to network fetches, causing the unit test to time out. The updates make both flows deterministic and resilient.
Testing
Regression risk
Low. The bash tool change only affects the fast-exit/foreground-to-background transition, with outputs drained before any background migration. The mux.md validation now short-circuits invalid share URLs rather than attempting a network fetch. The Windows test retry is scoped to the integration test helper.
Generated with
mux• Model:openai:gpt-5.2-codex• Thinking:xhigh• Cost:$3.85