Merge branch 'main' into sb/zen-helpsb/zen-help

- Combine HelpCommand (this branch) with HistoryCommand (main) in zen CLI dispatcher
- Keep filter-aware TuiPickOne rewrite; adopt main's ASCII arrow glyphs in doc comment

1 files changed, 9 insertions, 7 deletions

diff --git a/src/zencompute/runners/macrunner.cpp b/src/zencompute/runners/macrunner.cpp
index c2ccca9a6..ab24d4672 100644
--- a/src/zencompute/runners/macrunner.cpp
+++ b/src/zencompute/runners/macrunner.cpp
@@ -211,18 +211,19 @@ MacProcessRunner::SubmitAction(Ref<RunnerAction> Action)
 
 	if (ChildPid == 0)
 	{
-		// Child process
+		// Child process - lower priority so workers don't starve the main server
+		nice(5);
 
 		if (m_Sandboxed)
 		{
-			// Close read end of error pipe — child only writes
+			// Close read end of error pipe - child only writes
 			close(ErrorPipe[0]);
 
 			// Apply Seatbelt sandbox profile
 			char* ErrorBuf = nullptr;
 			if (sandbox_init(SandboxProfile.c_str(), 0, &ErrorBuf) != 0)
 			{
-				// sandbox_init failed — write error to pipe and exit
+				// sandbox_init failed - write error to pipe and exit
 				if (ErrorBuf)
 				{
 					WriteErrorAndExit(ErrorPipe[1], ErrorBuf, 0);
@@ -261,7 +262,7 @@ MacProcessRunner::SubmitAction(Ref<RunnerAction> Action)
 
 	if (m_Sandboxed)
 	{
-		// Close write end of error pipe — parent only reads
+		// Close write end of error pipe - parent only reads
 		close(ErrorPipe[1]);
 
 		// Read from error pipe. If execve succeeded, pipe was closed by O_CLOEXEC
@@ -281,7 +282,8 @@ MacProcessRunner::SubmitAction(Ref<RunnerAction> Action)
 			// Clean up the sandbox in the background
 			m_DeferredDeleter.Enqueue(Action->ActionLsn, std::move(Prepared->SandboxPath));
 
-			ZEN_ERROR("Sandbox setup failed for action {}: {}", Action->ActionLsn, ErrBuf);
+			Action->FailureReason = fmt::format("sandbox setup failed: {}", ErrBuf);
+			ZEN_ERROR("action {} ({}): {}", Action->ActionId, Action->ActionLsn, Action->FailureReason);
 
 			Action->SetActionState(RunnerAction::State::Failed);
 			return SubmitResult{.IsAccepted = false};
@@ -469,7 +471,7 @@ MacProcessRunner::SampleProcessCpu(RunningAction& Running)
 	const uint64_t CurrentOsTicks = Info.pti_total_user + Info.pti_total_system;
 	const uint64_t NowTicks		  = GetHifreqTimerValue();
 
-	// Cumulative CPU seconds (absolute, available from first sample): ns → seconds
+	// Cumulative CPU seconds (absolute, available from first sample): ns -> seconds
 	Running.Action->CpuSeconds.store(static_cast<float>(static_cast<double>(CurrentOsTicks) / 1'000'000'000.0), std::memory_order_relaxed);
 
 	if (Running.LastCpuSampleTicks != 0 && Running.LastCpuOsTicks != 0)
@@ -478,7 +480,7 @@ MacProcessRunner::SampleProcessCpu(RunningAction& Running)
 		if (ElapsedMs > 0)
 		{
 			const uint64_t DeltaOsTicks = CurrentOsTicks - Running.LastCpuOsTicks;
-			// ns → ms: divide by 1,000,000; then as percent of elapsed ms
+			// ns -> ms: divide by 1,000,000; then as percent of elapsed ms
 			const float CpuPct = static_cast<float>(static_cast<double>(DeltaOsTicks) / 1'000'000.0 / ElapsedMs * 100.0);
 			Running.Action->CpuUsagePercent.store(CpuPct, std::memory_order_relaxed);
 		}