Don't Tear Down RPC Connection On Per-Call Timeout

A single slow call would call failConnection(), set closedError, and
reject every subsequent call instantly with "Remote ADE service
connection failed: timed out waiting for method ...". One stuck call
on a heavy mutation (lane delete, sync) made every following folder
open look like an 8s timeout even when the daemon answered in 5 ms.

Per-call timeouts now reject only the one promise. The socket stays
open and onDisconnect only fires on actual transport errors/close.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: arul28

apps/desktop/src/main/services/remoteRuntime/runtimeRpcClient.test.ts

@@ -104,7 +104,7 @@ describe("RuntimeRpcClient", () => {
     await expect(client.call("projects.list", {})).rejects.toThrow("broken pipe");
   });
 
-  it("fails the connection when a call times out", async () => {
+  it("rejects only the timed-out call and keeps the connection open", async () => {
     vi.useFakeTimers();
     try {
       const transport = new MockTransport();
@@ -114,15 +114,17 @@ describe("RuntimeRpcClient", () => {
 
       const pending = client.call("projects.list", {}, { timeoutMs: 25 });
       const assertion = expect(pending).rejects.toThrow(
-        "Remote ADE service connection failed: timed out waiting for method projects.list.",
+        "Remote ADE service timed out waiting for method projects.list (25ms).",
       );
       await vi.advanceTimersByTimeAsync(25);
 
       await assertion;
-      expect(onDisconnect).toHaveBeenCalledTimes(1);
-      await expect(client.call("projects.list", {})).rejects.toThrow(
-        "Remote ADE service connection failed: timed out waiting for method projects.list.",
-      );
+      expect(onDisconnect).not.toHaveBeenCalled();
+
+      // Subsequent calls go out fine — connection is still alive.
+      const followup = client.call("projects.list", {});
+      transport.emitData({ jsonrpc: "2.0", id: 2, result: { ok: true } });
+      await expect(followup).resolves.toEqual({ ok: true });
     } finally {
       vi.useRealTimers();
     }

apps/desktop/src/main/services/remoteRuntime/runtimeRpcClient.ts

@@ -81,10 +81,11 @@ export class RuntimeRpcClient {
     }
     return new Promise((resolve, reject) => {
       const timer = setTimeout(() => {
-        this.failConnection(
-          new Error(
-            `Remote ADE service connection failed: timed out waiting for method ${method}.`,
-          ),
+        const pending = this.pending.get(id);
+        if (!pending) return;
+        this.pending.delete(id);
+        pending.reject(
+          new Error(`Remote ADE service timed out waiting for method ${method} (${timeoutMs}ms).`),
         );
       }, timeoutMs);
       this.pending.set(id, { resolve, reject, timer });