fix(ports): preserve unknown port-probe status as fail-closed error

lzyyzznl · lzyyzznl · commit 8705320a6e7f · 2026-06-22T04:58:35.000+08:00
- When checkPortInUse returns "unknown" (all probes failed for non-EADDRINUSE
  reasons), isPortBusy now throws instead of silently returning false
- This prevents forceFreePortAndWait from treating an indeterminate probe
  as "port is free" and skipping cleanup
diff --git a/src/cli/ports.ts b/src/cli/ports.ts
@@ -132,6 +132,9 @@ function killPortWithFuser(port: number, signal: "SIGTERM" | "SIGKILL"): PortPro
 
 async function isPortBusy(port: number): Promise<boolean> {
   const status = await checkPortInUse(port);
+  if (status === "unknown") {
+    throw new Error(`Cannot determine port ${port} status: all probe attempts failed`);
+  }
   return status === "busy";
 }
 

Original file line number	Diff line number	Diff line change
`@@ -132,6 +132,9 @@ function killPortWithFuser(port: number, signal: "SIGTERM" \| "SIGKILL"): PortPro`
`132`	`132`
`133`	`133`	`async function isPortBusy(port: number): Promise<boolean> {`
`134`	`134`	`const status = await checkPortInUse(port);`
	`135`	`+ if (status === "unknown") {`
	`136`	+ throw new Error(`Cannot determine port ${port} status: all probe attempts failed`);
	`137`	`+ }`
`135`	`138`	`return status === "busy";`
`136`	`139`	`}`
`137`	`140`