@palettelab/cli 0.3.9 → 0.3.10

package/lib/ports.js

@@ -13,13 +13,38 @@ function canBindPort(port, host = "0.0.0.0") {
   })
 }
 
+function canConnectPort(port, host, timeoutMs = 250) {
+  return new Promise((resolve) => {
+    const socket = net.createConnection({ port, host })
+    const done = (connected) => {
+      socket.removeAllListeners()
+      socket.destroy()
+      resolve(connected)
+    }
+    socket.setTimeout(timeoutMs)
+    socket.once("connect", () => done(true))
+    socket.once("timeout", () => done(false))
+    socket.once("error", () => done(false))
+  })
+}
+
+async function isPortAvailable(port, { bindHost = "0.0.0.0" } = {}) {
+  // Docker publishes host ports on localhost-facing addresses. A process bound
+  // only to 127.0.0.1 or ::1 can be missed by a single 0.0.0.0 bind probe on
+  // some host setups, so test the addresses users actually open in browsers.
+  for (const host of ["127.0.0.1", "::1", "localhost"]) {
+    if (await canConnectPort(port, host)) return false
+  }
+  return canBindPort(port, bindHost)
+}
+
 async function findFreePort(preferred, { host = "0.0.0.0", maxAttempts = 100 } = {}) {
   const start = Number(preferred)
   if (!Number.isInteger(start) || start <= 0 || start > 65535) {
     throw new Error(`invalid port: ${preferred}`)
   }
   for (let port = start; port < start + maxAttempts && port <= 65535; port++) {
-    if (await canBindPort(port, host)) return port
+    if (await isPortAvailable(port, { bindHost: host })) return port
   }
   throw new Error(`no free port found from ${start} to ${Math.min(start + maxAttempts - 1, 65535)}`)
 }
@@ -39,4 +64,4 @@ async function resolveDevPorts({
   }
 }
 
-module.exports = { canBindPort, findFreePort, resolveDevPorts }
+module.exports = { canBindPort, canConnectPort, isPortAvailable, findFreePort, resolveDevPorts }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@palettelab/cli",
-  "version": "0.3.9",
+  "version": "0.3.10",
   "description": "Developer CLI for building Palette platform plugins — no platform source access required.",
   "bin": {
     "pltt": "bin/pltt.js"