@m64/nats-agent-dashboard 0.1.2 → 0.1.3

package/bin/dashboard.js

@@ -43,9 +43,25 @@ let natsArg = null
 let ownerArg = null
 let openBrowser = true
 
+/** Parse a port value; returns null if missing or invalid. */
+function parsePort(raw) {
+  if (raw == null || raw === '') return null
+  const n = Number(raw)
+  if (!Number.isFinite(n) || n < 0 || n > 65535) return null
+  return n
+}
+
 for (let i = 0; i < args.length; i++) {
   const a = args[i]
-  if (a === '--port' || a === '-p') portArg = Number(args[++i])
+  if (a === '--port' || a === '-p') {
+    const raw = args[++i]
+    const parsed = parsePort(raw)
+    if (parsed == null) {
+      console.error(`invalid port: "${raw}" (must be 0-65535)`)
+      process.exit(1)
+    }
+    portArg = parsed
+  }
   else if (a === '--nats-url' || a === '-u') natsArg = args[++i]
   else if (a === '--owner') ownerArg = args[++i]
   else if (a === '--no-open') openBrowser = false
@@ -85,7 +101,10 @@ const server = createServer((req, res) => {
   res.end(html)
 })
 
-const startPort = portArg ?? Number(process.env.PORT) ?? 5173
+// `??` only fires on null/undefined, NOT on NaN. Without parsePort(), a
+// missing PORT env var would give `Number(undefined) === NaN`, which then
+// skips the `?? 5173` fallback and lands `NaN` in server.listen().
+const startPort = portArg ?? parsePort(process.env.PORT) ?? 5173
 
 tryPort(startPort)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@m64/nats-agent-dashboard",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "type": "module",
   "description": "Static Vue 3 dashboard for the NATS AI agent network — discover, prompt and manage agents across pi-channel, pi-exec, claude-channel and OpenClaw runtimes from a single screen, with no backend. Ships as a single HTML file or via npx.",
   "license": "Apache-2.0",