Skip to content

Conversation

@ammar-agent
Copy link
Collaborator

@ammar-agent ammar-agent commented Jan 20, 2026

Summary

  • avoid backgrounding already-exited bash runs by draining foreground output
  • return an immediate error for mux.md URLs missing a key to avoid network fetches
  • retry the executeBash env check in Windows integration tests to reduce flake
  • capture the deterministic coordination guidance in AGENTS.md

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

  • make static-check
  • bun test src/node/services/tools/web_fetch.test.ts -t "should return error for mux.md URLs without valid key format"

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

@ammario ammario merged commit 472d890 into main Jan 21, 2026
22 checks passed
@ammario ammario deleted the mux-26fp branch January 21, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants