Summary

Fix exec tool returning (no output) on Windows, a regression from the supervisor refactor in cd44a0d.

Fixes #18035
Fixes #17806

Root Cause

The new createChildAdapter always spawns with detached: true. On Windows Scheduled Tasks (headless, no console), detached: true creates a new process group that can prevent stdout/stderr pipes from connecting. The old runCommandWithTimeout in exec.ts never used detached: true.

The no-detach fallback only triggers on EBADF spawn errors. The pipe disconnection issue does not throw — it silently produces empty output.

Fix

On Windows (process.platform === "win32"), default to detached: false. On POSIX, keep detached: true with the no-detach fallback.

Tests

• Updated child.test.ts with platform-aware assertions.

Sign-Off

• Models: Claude (Anthropic)
• Keyword: lobster-biscuit

Greptile Summary

Fixed Windows exec output regression by disabling detached: true spawn mode. On Windows Scheduled Tasks (headless environments), detached mode creates a new process group that silently prevents stdout/stderr pipes from connecting. The fix conditionally uses detached: false on Windows while maintaining detached: true on POSIX systems with a fallback mechanism.

Confidence Score: 5/5

• Safe to merge - targeted Windows-specific fix with proper platform checks and test coverage
• The fix is minimal, well-documented, and directly addresses the root cause. Tests are updated with platform-aware assertions. No breaking changes or security concerns.
• No files require special attention

_{Last reviewed commit: 1163407}