fix(cli): isPortBusy now probes all 4 address families to catch IPv4-only listeners

lzyyzznl · claude · lzyyzznl · commit da003653cffd · 2026-06-18T14:48:37.000+08:00
Previously, isPortBusy (used by --force port detection) only tried a bare tryListenOnPort which binds IPv6 wildcard (::) by default, missing ports occupied by IPv4-only listeners on 127.0.0.1 or 0.0.0.0. Now probes 127.0.0.1, 0.0.0.0, ::1, :: — same multi-host approach as checkPortInUse in ports-inspect.ts. EADDRNOTAVAIL/EAFNOSUPPORT hosts are silently skipped (e.g. IPv6 not enabled). Fixes #94426 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
diff --git a/src/cli/ports.ts b/src/cli/ports.ts
@@ -130,16 +130,23 @@ function killPortWithFuser(port: number, signal: "SIGTERM" | "SIGKILL"): PortPro
 }
 
 async function isPortBusy(port: number): Promise<boolean> {
-  try {
-    await tryListenOnPort({ port, exclusive: true });
-    return false;
-  } catch (err: unknown) {
-    const code = (err as NodeJS.ErrnoException).code;
-    if (code === "EADDRINUSE") {
-      return true;
+  const hosts = ["127.0.0.1", "0.0.0.0", "::1", "::"];
+  for (const host of hosts) {
+    try {
+      await tryListenOnPort({ port, host, exclusive: true });
+    } catch (err: unknown) {
+      const code = (err as NodeJS.ErrnoException).code;
+      if (code === "EADDRINUSE") {
+        return true;
+      }
+      if (code === "EADDRNOTAVAIL" || code === "EAFNOSUPPORT") {
+        // Host not available on this system (e.g., IPv6 not enabled), skip
+        continue;
+      }
+      throw err instanceof Error ? err : new Error(String(err));
     }
-    throw err instanceof Error ? err : new Error(String(err));
   }
+  return false;
 }
 
 export function parseLsofOutput(output: string): PortProcess[] {
diff --git a/src/cli/program.force.test.ts b/src/cli/program.force.test.ts
@@ -236,7 +236,7 @@ describe("gateway --force helpers", () => {
       .mockRejectedValueOnce(busyErr)
       .mockRejectedValueOnce(busyErr)
       .mockRejectedValueOnce(busyErr)
-      .mockResolvedValueOnce(undefined);
+      .mockResolvedValue(undefined);
 
     const promise = forceFreePortAndWait(18789, {
       timeoutMs: 300,
