fix: clear Tailscale probe timeout

zhangLei99586 · web-flow · commit 324193652651 · 2026-07-01T04:08:39.000-07:00
The checkBinary helper used Promise.race with a setTimeout-based timeout
but never cleared the timer when runExec completed first. Wrap the race
in a try/finally that clears the timer handle to avoid a dangling timer.
diff --git a/src/infra/tailscale.ts b/src/infra/tailscale.ts
@@ -40,12 +40,17 @@ export async function findTailscaleBinary(): Promise<string | null> {
     }
     try {
       // Use Promise.race with runExec to implement timeout
-      await Promise.race([
-        runExec(path, ["--version"], { timeoutMs: 3000 }),
-        new Promise<never>((_, reject) => {
-          setTimeout(() => reject(new Error("timeout")), 3000);
-        }),
-      ]);
+      let timer: ReturnType<typeof setTimeout> | undefined;
+      try {
+        await Promise.race([
+          runExec(path, ["--version"], { timeoutMs: 3000 }),
+          new Promise<never>((_, reject) => {
+            timer = setTimeout(() => reject(new Error("timeout")), 3000);
+          }),
+        ]);
+      } finally {
+        if (timer) clearTimeout(timer);
+      }
       return true;
     } catch {
       return false;
