From 5b4af29420d21332bbea273a1309b046e02e4527 Mon Sep 17 00:00:00 2001
From: Mika Kuns <mika.kuns@fb-tuning.de>
Date: Wed, 15 Apr 2026 09:32:26 +0200
Subject: [PATCH] fix(installer): check exit code (not stdout) for
 ERROR_SERVICE_ALREADY_RUNNING

---
 src/ClaudeDo.Installer/Steps/StartServiceStep.cs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/ClaudeDo.Installer/Steps/StartServiceStep.cs b/src/ClaudeDo.Installer/Steps/StartServiceStep.cs
index 270c179..0b97ce7 100644
--- a/src/ClaudeDo.Installer/Steps/StartServiceStep.cs
+++ b/src/ClaudeDo.Installer/Steps/StartServiceStep.cs
@@ -12,11 +12,11 @@ public sealed class StartServiceStep : IInstallStep
     {
         progress.Report($"Starting {ServiceName}...");
 
-        var (exit, output) = await ProcessRunner.RunAsync("sc.exe", $"start {ServiceName}", null, progress, ct);
+        var (exit, _) = await ProcessRunner.RunAsync("sc.exe", $"start {ServiceName}", null, progress, ct);
         if (exit == 0) return StepResult.Ok();
 
-        // Exit 1056 = already running — that's fine too.
-        if (output.Contains("1056", StringComparison.OrdinalIgnoreCase))
+        // Exit 1056 = ERROR_SERVICE_ALREADY_RUNNING — that's fine too.
+        if (exit == 1056)
         {
             progress.Report("Service was already running.");
             return StepResult.Ok();