@goldenlustre/agent-studio 1.0.5 → 1.0.6

package/.next/trace-build

@@ -1 +1 @@
-[{"name":"run-webpack","duration":65089888,"timestamp":190223941897,"id":14,"parentId":1,"tags":{},"startTime":1789547702791,"traceId":"32dfe4e4005e962a"},{"name":"run-typescript","duration":30360690,"timestamp":190289039773,"id":8510,"parentId":1,"tags":{},"startTime":1789547767889,"traceId":"32dfe4e4005e962a"},{"name":"static-check","duration":4866452,"timestamp":190319422034,"id":8513,"parentId":1,"tags":{},"startTime":1789547798271,"traceId":"32dfe4e4005e962a"},{"name":"static-generation","duration":11859811,"timestamp":190324850162,"id":8641,"parentId":1,"tags":{},"startTime":1789547803699,"traceId":"32dfe4e4005e962a"},{"name":"collect-build-traces","duration":120922537,"timestamp":190324294134,"id":8638,"parentId":1,"tags":{},"startTime":1789547803143,"traceId":"32dfe4e4005e962a"},{"name":"telemetry-flush","duration":86,"timestamp":190445225395,"id":8650,"parentId":1,"tags":{},"startTime":1789547924074,"traceId":"32dfe4e4005e962a"},{"name":"next-build","duration":221474982,"timestamp":190223750518,"id":1,"tags":{"buildMode":"default","version":"16.3.1","bundler":"webpack","has-custom-webpack-config":"false","use-build-worker":"true"},"startTime":1789547702599,"traceId":"32dfe4e4005e962a"}]
+[{"name":"run-webpack","duration":65018578,"timestamp":190699472758,"id":14,"parentId":1,"tags":{},"startTime":1789548178322,"traceId":"81cbf31ad746bf67"},{"name":"run-typescript","duration":30170357,"timestamp":190764498160,"id":8510,"parentId":1,"tags":{},"startTime":1789548243347,"traceId":"81cbf31ad746bf67"},{"name":"static-check","duration":4939007,"timestamp":190794693838,"id":8513,"parentId":1,"tags":{},"startTime":1789548273543,"traceId":"81cbf31ad746bf67"},{"name":"static-generation","duration":11735570,"timestamp":190800161532,"id":8641,"parentId":1,"tags":{},"startTime":1789548279010,"traceId":"81cbf31ad746bf67"},{"name":"collect-build-traces","duration":77405971,"timestamp":190799638121,"id":8638,"parentId":1,"tags":{},"startTime":1789548278487,"traceId":"81cbf31ad746bf67"},{"name":"telemetry-flush","duration":87,"timestamp":190877050463,"id":8650,"parentId":1,"tags":{},"startTime":1789548355899,"traceId":"81cbf31ad746bf67"},{"name":"next-build","duration":177770980,"timestamp":190699279583,"id":1,"tags":{"buildMode":"default","version":"16.3.1","bundler":"webpack","has-custom-webpack-config":"false","use-build-worker":"true"},"startTime":1789548178129,"traceId":"81cbf31ad746bf67"}]

package/bin/agent-studio.js

@@ -19,6 +19,8 @@ const fs = require("fs");
 const { getHelpText, parseLaunchOptions } = require("./agent-studio-options");
 // eslint-disable-next-line @typescript-eslint/no-require-imports
 const { wireChildProcessLifecycle } = require("./process-lifecycle");
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const { browserUrl } = require("./browser-url");
 
 let launchOptions;
 try {
@@ -89,7 +91,9 @@ const child = spawn(process.execPath, [nextBin, ...nextArgs], {
 wireChildProcessLifecycle(child);
 
 let browserOpened = false;
-const url = `http://${hostname}:${port}`;
+// A wildcard bind (`-H 0.0.0.0`) answers on loopback too, but browsers refuse
+// to navigate to the wildcard address itself, so open the loopback URL.
+const url = browserUrl(hostname, port);
 
 child.stdout.on("data", (chunk) => {
   const text = chunk.toString();

package/bin/browser-url.js

@@ -0,0 +1,40 @@
+"use strict";
+
+/**
+ * Hostnames a server may bind to but a browser refuses to navigate to.
+ *
+ * `0.0.0.0` and `::` are "any interface" wildcards: the OS answers on every
+ * address including loopback, so `curl`/`Invoke-WebRequest` succeed, but
+ * Chrome/Edge reject the navigation itself (ERR_ADDRESS_INVALID) and show
+ * "This page couldn't load". `pi-web -H 0.0.0.0` used to auto-open exactly
+ * that URL.
+ */
+const WILDCARD_HOSTNAMES = new Set([
+  "",
+  "0.0.0.0",
+  "::",
+  "[::]",
+  "0:0:0:0:0:0:0:0",
+  "[0:0:0:0:0:0:0:0]",
+]);
+
+/** True when the hostname is an "any interface" bind rather than a destination. */
+function isWildcardHostname(hostname) {
+  return WILDCARD_HOSTNAMES.has(String(hostname ?? "").trim());
+}
+
+/**
+ * Build the URL a local browser should open for a server bound to `hostname`.
+ *
+ * A wildcard bind is reachable over loopback, so the browser is pointed at
+ * 127.0.0.1 instead of the wildcard. IPv6 literals are bracketed as URLs
+ * require. Anything else (a hostname, a LAN address) is used verbatim.
+ */
+function browserUrl(hostname, port) {
+  const raw = String(hostname ?? "").trim();
+  const host = isWildcardHostname(raw) ? "127.0.0.1" : raw;
+  const formatted = host.includes(":") && !host.startsWith("[") ? `[${host}]` : host;
+  return `http://${formatted}:${port}`;
+}
+
+module.exports = { browserUrl, isWildcardHostname, WILDCARD_HOSTNAMES };

package/bin/browser-url.test.mjs

@@ -0,0 +1,44 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import { createRequire } from "node:module";
+
+const require = createRequire(import.meta.url);
+const { browserUrl, isWildcardHostname } = require("./browser-url.js");
+
+// Regression: `pi-web -H 0.0.0.0` auto-opened http://0.0.0.0:<port>, which
+// Chrome/Edge refuse to navigate to, so the first thing a user saw was
+// "This page couldn't load" even though the server was healthy.
+test("wildcard binds open the browser on loopback", () => {
+  assert.equal(browserUrl("0.0.0.0", 30141), "http://127.0.0.1:30141");
+  assert.equal(browserUrl("::", 30141), "http://127.0.0.1:30141");
+  assert.equal(browserUrl("[::]", 30141), "http://127.0.0.1:30141");
+  assert.equal(browserUrl("0:0:0:0:0:0:0:0", 30141), "http://127.0.0.1:30141");
+  assert.equal(browserUrl("", 8080), "http://127.0.0.1:8080");
+});
+
+test("concrete hosts are preserved", () => {
+  assert.equal(browserUrl("127.0.0.1", 30141), "http://127.0.0.1:30141");
+  assert.equal(browserUrl("localhost", 30141), "http://localhost:30141");
+  assert.equal(browserUrl("10.10.0.200", 30141), "http://10.10.0.200:30141");
+  assert.equal(browserUrl("pi.example.com", 443), "http://pi.example.com:443");
+});
+
+test("IPv6 literals are bracketed and never treated as wildcards", () => {
+  assert.equal(browserUrl("::1", 30141), "http://[::1]:30141");
+  assert.equal(browserUrl("[::1]", 30141), "http://[::1]:30141");
+  assert.equal(browserUrl("fe80::1", 30141), "http://[fe80::1]:30141");
+});
+
+test("isWildcardHostname only matches any-interface binds", () => {
+  for (const value of ["0.0.0.0", "::", "[::]", "", "   "]) {
+    assert.equal(isWildcardHostname(value), true, `${JSON.stringify(value)} is a wildcard`);
+  }
+  for (const value of ["127.0.0.1", "::1", "localhost", "10.0.0.5"]) {
+    assert.equal(isWildcardHostname(value), false, `${value} is a destination`);
+  }
+});
+
+test("undefined-ish input degrades to loopback instead of crashing", () => {
+  assert.equal(browserUrl(undefined, 30141), "http://127.0.0.1:30141");
+  assert.equal(browserUrl(null, 30141), "http://127.0.0.1:30141");
+});

package/bin/pi-web.js

@@ -19,6 +19,8 @@ const fs = require("fs");
 const { getHelpText, parseLaunchOptions } = require("./pi-web-options");
 // eslint-disable-next-line @typescript-eslint/no-require-imports
 const { wireChildProcessLifecycle } = require("./process-lifecycle");
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const { browserUrl } = require("./browser-url");
 
 let launchOptions;
 try {
@@ -89,7 +91,9 @@ const child = spawn(process.execPath, [nextBin, ...nextArgs], {
 wireChildProcessLifecycle(child);
 
 let browserOpened = false;
-const url = `http://${hostname}:${port}`;
+// A wildcard bind (`-H 0.0.0.0`) answers on loopback too, but browsers refuse
+// to navigate to the wildcard address itself, so open the loopback URL.
+const url = browserUrl(hostname, port);
 
 child.stdout.on("data", (chunk) => {
   const text = chunk.toString();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goldenlustre/agent-studio",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "AgentStudio - AI Agent Development Environment",
   "homepage": "https://github.com/agegr/agent-studio#readme",
   "repository": {
@@ -38,7 +38,7 @@
     "lint": "eslint .",
     "test:terminal": "node e2e/terminal.mjs",
     "test:e2e": "node e2e/run.mjs",
-    "test": "node --experimental-strip-types --test \"app/**/*.test.mjs\" \"components/**/*.test.mjs\" \"hooks/**/*.test.mjs\" \"lib/**/*.test.mjs\" \"public/**/*.test.mjs\"",
+    "test": "node --experimental-strip-types --test \"app/**/*.test.mjs\" \"components/**/*.test.mjs\" \"hooks/**/*.test.mjs\" \"lib/**/*.test.mjs\" \"public/**/*.test.mjs\" \"bin/**/*.test.mjs\"",
     "release": "npm version patch --no-git-tag-version && npm run build && npm publish --access public",
     "postbuild": "node scripts/postbuild-fix-paths.js"
   },