Skip to content

Commit 41e6534

Browse files
committed
fix(run): remove spinner dependency from runtime output
- decouple spinner from output rendering - fix captureOnly logic blocking stdout when spinner is disabled - ensure live output is always displayed for vix run - remove 'Running script' label for cleaner CLI UX behavior now matches modern runtimes (go, deno, node)
1 parent b473a6c commit 41e6534

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/commands/run/RunProcess.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -768,7 +768,7 @@ namespace vix::commands::RunCommand::detail
768768
::setpgid(pid, pid);
769769

770770
const bool useSpinner = !spinnerLabel.empty();
771-
const bool captureOnly = (!passthroughRuntime && spinnerLabel.empty());
771+
const bool captureOnly = false;
772772
bool spinnerActive = useSpinner;
773773
std::size_t frameIndex = 0;
774774

src/commands/run/RunScript.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -643,7 +643,7 @@ namespace vix::commands::RunCommand::detail
643643

644644
auto rr = run_cmd_live_filtered_capture(
645645
cmdRun,
646-
runLabel,
646+
"",
647647
/*passthroughRuntime=*/isPlainScript,
648648
/*timeoutSec=*/effective_timeout_sec(opt));
649649

0 commit comments

Comments
 (0)