unsloth/unsloth_cli
Daniel Han 2453134084 cli(windows): pass sys.stdio handles explicitly to powershell.exe
The previous Write-Host capture attempts (47432b0b -Command + *>&1
and f2c2b3f3 [Console]::Out mirror in setup.ps1) still produced an
empty update.log on windows-latest because the powershell.exe child
had no stdio handles at all to write to.

Root cause: subprocess.run on Windows with the default close_fds=True
(Python 3.7+ default) sets bInheritHandles=False on CreateProcess.
Combined with CREATE_NO_WINDOW (added by _windows_hidden_subprocess_
kwargs in non-TTY runs), the child gets:
  - no console (CREATE_NO_WINDOW)
  - no inherited std handles (bInheritHandles=False)
GetStdHandle in the child returns INVALID_HANDLE_VALUE, so even
[Console]::Out.WriteLine and Write-Output -- not just Write-Host --
write into the void.

Fix: pass stdout=sys.stdout, stderr=sys.stderr (and stdin) when
running the setup script on Windows. With explicit handles, Python's
subprocess sets up PROC_THREAD_ATTRIBUTE_HANDLE_LIST containing the
std handles + bInheritHandles=True, so the child inherits exactly
the three std handles regardless of close_fds=True. CREATE_NO_WINDOW
still applies (no transient console window), but the child can now
write to the inherited stdout file handle, which lands on bash's
`tee logs/update.log` in CI.

A small _stream_for_subprocess helper guards against test harnesses
that swap sys.stdout for a stream without a real fileno (pytest
capsys, in-memory IO buffers, etc) -- those fall back to None so
subprocess uses its default.

Verified locally on PowerShell 7.4.6 / Linux that the explicit
stdout handoff doesn't regress the existing direct-inherit path,
and the marker line "prebuilt up to date and validated" reaches
both the child's stdout and a parent `tee` consumer.
2026-05-08 03:24:52 +00:00
..
commands cli(windows): pass sys.stdio handles explicitly to powershell.exe 2026-05-08 03:24:52 +00:00
__init__.py Studio: forward llama-server args from unsloth studio run , activate unsloth run , and allow passing model:quant to load models (#5271) 2026-05-04 17:08:04 +04:00
_tool_policy.py unsloth run: add --enable-tools/--disable-tools server-side tool policy (#5277) 2026-05-05 12:45:15 +04:00
config.py Rename cli/ to unsloth_cli/ to fix namespace collision with stringzilla (#4393) 2026-03-17 20:40:21 -07:00
options.py Rename cli/ to unsloth_cli/ to fix namespace collision with stringzilla (#4393) 2026-03-17 20:40:21 -07:00