Commit dce48b2
fix: pipe stdio in engine.ts to suppress subprocess noise (#54)
* test: add E2E tests for engine.ts stdio piping
Verify that all execFileSync calls in engine.ts use { stdio: "pipe" }
to prevent subprocess output (uv, pip install, python --version, etc.)
from leaking into the TUI's text input area.
Tests use real subprocess calls via spawnSync to confirm the piping
behavior end-to-end, plus a source-level check that every call site
in engine.ts includes the stdio option.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: add stdio: "pipe" to all execFileSync calls in engine.ts
Without this option, subprocess output from uv, pip install, tar, and
python/uv --version leaks into the TUI's text input area. Piping stdio
suppresses this noise while still capturing the return value.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: update docs URL to GitHub Pages
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: include stderr in telemetry error messages for piped execFileSync
With stdio: "pipe", e.message is a generic "Command failed" string.
The actual diagnostics are in e.stderr, so prefer that for telemetry.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>1 parent d3301b1 commit dce48b2
2 files changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
179 | 179 | | |
180 | 180 | | |
181 | 181 | | |
182 | | - | |
| 182 | + | |
183 | 183 | | |
184 | 184 | | |
185 | 185 | | |
| |||
196 | 196 | | |
197 | 197 | | |
198 | 198 | | |
199 | | - | |
| 199 | + | |
200 | 200 | | |
201 | 201 | | |
202 | 202 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
555 | 555 | | |
556 | 556 | | |
557 | 557 | | |
558 | | - | |
| 558 | + | |
559 | 559 | | |
560 | 560 | | |
561 | 561 | | |
| |||
0 commit comments