Summary

• The test "keeps sandbox as the implicit exec host default without forcing gateway approvals" (added in 45febec) uses yieldMs: 10 which causes intermittent failures on Windows CI because echo done doesn't always complete within 10ms.
• Removed yieldMs: 10 so the exec command runs to completion, matching the pattern used in bash-tools.exec.path.test.ts.
• No behavioral change — test still verifies sandbox default and gateway rejection.

Change Type (select all)

• Bug fix
• Feature
• Refactor
• Docs
• Security hardening
• Chore/infra

Scope (select all touched areas)

• Gateway / orchestration
• Skills / tool execution
• Auth / tokens
• Memory / storage
• Integrations
• API / contracts
• UI / DX
• CI/CD / infra

Linked Issue/PR

• Related Agent runner: align compaction floor guidance #24059

Security Impact (required)

• New permissions/capabilities? No
• Secrets/tokens handling changed? No
• New/changed network calls? No
• Command/tool execution surface changed? No
• Data access scope changed? No

Human Verification (required)

Test-only change removing a timing-sensitive yieldMs: 10 that caused flakiness on Windows.

Compatibility / Migration

• Backward compatible? Yes
• Config/env changes? No
• Migration needed? No

Risks and Mitigations

• Risk: None.
  • Mitigation: N/A.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com

Greptile Summary

Removed timing-sensitive yieldMs: 10 parameter from Windows-flaky test. The test "keeps sandbox as the implicit exec host default without forcing gateway approvals" (added in 45febec) was causing intermittent failures on Windows CI because echo done doesn't always complete within 10ms. Without yieldMs, the exec command runs to completion, matching the pattern in similar tests like bash-tools.exec.path.test.ts (line 87) and other tests in this file (lines 618, 625, 642).

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The change removes only a timing parameter that was causing flakiness without affecting test logic. The test still verifies the sandbox default and gateway rejection behavior. The removal aligns with existing patterns in the codebase where simple commands like echo done run to completion without artificial time limits.
• No files require special attention

_{Last reviewed commit: 143b80d}

_{(5/5) You can turn off certain types of comments like style here!}