From 8db67c10a2d680dbd6b937683eabdaa8e581e151 Mon Sep 17 00:00:00 2001
From: Michael Neale <michael.neale@gmail.com>
Date: Tue, 17 Mar 2026 21:05:35 +1100
Subject: [PATCH] fix: unregister goosed child process's  listener (#7956)

---
 ui/desktop/src/goosed.ts | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/ui/desktop/src/goosed.ts b/ui/desktop/src/goosed.ts
index a5265268fb..9f211e81f6 100644
--- a/ui/desktop/src/goosed.ts
+++ b/ui/desktop/src/goosed.ts
@@ -303,6 +303,19 @@ export const startGoosed = async (options: StartGoosedOptions): Promise<GoosedRe
     });
   });
 
+  // Once we have the fingerprint (or the process exits before emitting one),
+  // remove the stdout listener. Leaving it attached for the lifetime of the
+  // long-running goosed process means every chunk of stdout data triggers
+  // Node's internal EmitToJSStreamListener::OnStreamRead which converts raw
+  // bytes into a JS string via v8::String::NewFromTwoByte. Over multi-hour
+  // sessions this has been observed to hit a V8 assertion and crash the
+  // Electron main process. Removing the listener and calling resume()
+  // lets the pipe drain harmlessly without buffering into Node/V8.
+  void fingerprintReady.then(() => {
+    goosedProcess.stdout?.removeAllListeners('data');
+    goosedProcess.stdout?.resume();
+  });
+
   const onStderrData = (data: Buffer) => {
     const lines = data.toString().split('\n');
     for (const line of lines) {