From f122634b5678566ea9203444eb977c0ff8a2a618 Mon Sep 17 00:00:00 2001
From: Kris Wong <kris.wong@closedloop.ai>
Date: Thu, 26 Mar 2026 15:07:38 -0500
Subject: [PATCH 1/2] fix unit tests

---
 .../src/server/operations/symphony-loop.ts       | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/apps/desktop/src/server/operations/symphony-loop.ts b/apps/desktop/src/server/operations/symphony-loop.ts
index 88a7ae51..0bd94b8f 100644
--- a/apps/desktop/src/server/operations/symphony-loop.ts
+++ b/apps/desktop/src/server/operations/symphony-loop.ts
@@ -305,22 +305,30 @@ function buildClaudePipeline(
   if (formatter) {
     // Full pipeline matching run-loop.sh:
     // claude ... 2>stderr | grep JSON | tee jsonl | formatter
+    //
+    // Without pipefail, bash returns the exit code of the LAST pipeline
+    // command (formatter/tee), masking claude's non-zero exit.  We use
+    // pipefail so claude's failure propagates.  However, grep returns 1
+    // when there are no matches (e.g. claude exits 0 but emits no JSON),
+    // which would false-positive as a failure.  Wrapping grep in a
+    // subshell with `|| true` neutralises the no-match exit code while
+    // still letting claude's real failures through.
     const pipeline = [
       `${claudeCmd} 2>${shellEscape(stderrFile)}`,
-      "grep --line-buffered '^{'",
+      `(grep --line-buffered '^{' || true)`,
       `tee -a ${shellEscape(jsonlFile)}`,
       `python3 ${shellEscape(formatter)}`,
     ].join(" | ");
-    return { cmd: "bash", args: ["-c", pipeline] };
+    return { cmd: "bash", args: ["-c", `set -o pipefail; ${pipeline}`] };
   }
 
   // No formatter — wrap in bash pipeline so grep|tee still writes claude-output.jsonl
   const pipeline = [
     `${claudeCmd} 2>${shellEscape(stderrFile)}`,
-    "grep --line-buffered '^{'",
+    `(grep --line-buffered '^{' || true)`,
     `tee -a ${shellEscape(jsonlFile)}`,
   ].join(" | ");
-  return { cmd: "bash", args: ["-c", pipeline] };
+  return { cmd: "bash", args: ["-c", `set -o pipefail; ${pipeline}`] };
 }
 
 /** Find the local repo path for a given fullName (e.g. "org/repo"). */

From 56ba45b5d36838a4bc73d77d7fcee0d181edf465 Mon Sep 17 00:00:00 2001
From: Kris Wong <kris.wong@closedloop.ai>
Date: Thu, 26 Mar 2026 15:17:45 -0500
Subject: [PATCH 2/2] here's your version bump

---
 apps/desktop/package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/desktop/package.json b/apps/desktop/package.json
index 1871789c..f45df792 100644
--- a/apps/desktop/package.json
+++ b/apps/desktop/package.json
@@ -1,6 +1,6 @@
 {
   "name": "desktop",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "description": "ClosedLoop Desktop",
   "author": "ClosedLoop AI <support@closedloop.ai>",
   "private": true,