@ouro.bot/cli 0.1.0-alpha.160 → 0.1.0-alpha.162

package/README.md

@@ -221,3 +221,4 @@ See `skills/configure-dev-tools.md` for the full tool inventory and troubleshoot
 ## A Note To Future Maintainers
 
 If you discover a doc that lies, fix it or remove it. Accuracy is a kindness. A future agent should not have to untangle a fossil record just to understand where their hands are.
+# Production SPA serving

package/changelog.json

@@ -1,6 +1,19 @@
 {
   "_note": "This changelog is maintained as part of the PR/version-bump workflow. Agent-curated, not auto-generated. Agents read this file directly via read_file to understand what changed between versions.",
   "versions": [
+    {
+      "version": "0.1.0-alpha.162",
+      "changes": [
+        "fix(daemon): handle EPIPE in detached daemon \u2014 suppress pipe errors when parent exits after ouro up"
+      ]
+    },
+    {
+      "version": "0.1.0-alpha.161",
+      "changes": [
+        "feat(outlook): serve React SPA at root (/), daemon serves static assets",
+        "fix(daemon): graceful EADDRINUSE handling for Outlook port"
+      ]
+    },
     {
       "version": "0.1.0-alpha.160",
       "changes": [

package/dist/heart/daemon/daemon-entry.js

@@ -183,8 +183,16 @@ process.on("SIGTERM", () => {
     }
     void daemon.stop().then(() => process.exit(0));
 });
+// Suppress EPIPE on stdout/stderr — normal when detached daemon's parent exits
+/* v8 ignore start -- EPIPE suppression: only fires when parent process exits @preserve */
+process.stdout?.on?.("error", () => { });
+process.stderr?.on?.("error", () => { });
+/* v8 ignore stop */
 /* v8 ignore start -- global exception handlers: genuinely untestable in vitest; exercised by real daemon crashes @preserve */
 process.on("uncaughtException", (error) => {
+    // EPIPE is normal for detached daemon processes — parent closed the pipe
+    if (error.code === "EPIPE")
+        return;
     (0, daemon_tombstone_1.writeDaemonTombstone)("uncaughtException", error);
     (0, runtime_1.emitNervesEvent)({
         level: "error",

package/dist/heart/daemon/daemon.js

@@ -295,35 +295,46 @@ class OuroDaemon {
         await this.drainPendingSenseMessages();
         /* v8 ignore start — Outlook server startup, tested via outlook-http.test.ts */
         if (!this.outlookServer) {
-            this.outlookServer = await (0, outlook_http_1.startOutlookHttpServer)({
-                host: "127.0.0.1",
-                port: outlook_types_1.OUTLOOK_DEFAULT_PORT,
-                readMachineState: () => (0, outlook_read_1.readOutlookMachineState)({ bundlesRoot: this.bundlesRoot }),
-                readMachineView: ({ machine }) => {
-                    const overview = this.buildStatusPayload().overview;
-                    return (0, outlook_view_1.buildOutlookMachineView)({
-                        machine,
-                        daemon: {
-                            status: overview.daemon,
-                            health: overview.health,
-                            mode: overview.mode,
-                            socketPath: overview.socketPath,
-                            outlookUrl: overview.outlookUrl,
-                            entryPath: overview.entryPath,
-                            workerCount: overview.workerCount,
-                            senseCount: overview.senseCount,
-                        },
-                    });
-                },
-                readAgentState: (agentName) => (0, outlook_read_1.readOutlookAgentState)(agentName, { bundlesRoot: this.bundlesRoot }),
-                readAgentView: (agentName) => {
-                    const agent = (0, outlook_read_1.readOutlookAgentState)(agentName, { bundlesRoot: this.bundlesRoot });
-                    return (0, outlook_view_1.buildOutlookAgentView)({
-                        agent,
-                        viewer: { kind: "human" },
-                    });
-                },
-            });
+            try {
+                this.outlookServer = await (0, outlook_http_1.startOutlookHttpServer)({
+                    host: "127.0.0.1",
+                    port: outlook_types_1.OUTLOOK_DEFAULT_PORT,
+                    readMachineState: () => (0, outlook_read_1.readOutlookMachineState)({ bundlesRoot: this.bundlesRoot }),
+                    readMachineView: ({ machine }) => {
+                        const overview = this.buildStatusPayload().overview;
+                        return (0, outlook_view_1.buildOutlookMachineView)({
+                            machine,
+                            daemon: {
+                                status: overview.daemon,
+                                health: overview.health,
+                                mode: overview.mode,
+                                socketPath: overview.socketPath,
+                                outlookUrl: overview.outlookUrl,
+                                entryPath: overview.entryPath,
+                                workerCount: overview.workerCount,
+                                senseCount: overview.senseCount,
+                            },
+                        });
+                    },
+                    readAgentState: (agentName) => (0, outlook_read_1.readOutlookAgentState)(agentName, { bundlesRoot: this.bundlesRoot }),
+                    readAgentView: (agentName) => {
+                        const agent = (0, outlook_read_1.readOutlookAgentState)(agentName, { bundlesRoot: this.bundlesRoot });
+                        return (0, outlook_view_1.buildOutlookAgentView)({
+                            agent,
+                            viewer: { kind: "human" },
+                        });
+                    },
+                });
+            }
+            catch (error) {
+                (0, runtime_1.emitNervesEvent)({
+                    level: "warn",
+                    component: "daemon",
+                    event: "daemon.outlook_start_failed",
+                    message: `Outlook server failed to start: ${String(error)}`,
+                    meta: { port: outlook_types_1.OUTLOOK_DEFAULT_PORT },
+                });
+            }
         }
         /* v8 ignore stop */
         if (fs.existsSync(this.socketPath)) {

package/dist/heart/daemon/outlook-http.js

@@ -39,7 +39,6 @@ const http = __importStar(require("http"));
 const path = __importStar(require("path"));
 const runtime_1 = require("../../nerves/runtime");
 const outlook_read_1 = require("./outlook-read");
-const outlook_render_1 = require("./outlook-render");
 function createSseBroadcaster() {
     let nextId = 1;
     const clients = new Set();
@@ -114,10 +113,54 @@ function writeJson(response, statusCode, payload) {
     response.writeHead(statusCode, { "content-type": "application/json; charset=utf-8" });
     response.end(`${JSON.stringify(payload, null, 2)}\n`);
 }
-function writeHtml(response, html) {
-    response.writeHead(200, { "content-type": "text/html; charset=utf-8" });
-    response.end(html);
+/* v8 ignore start — SPA static file serving infrastructure */
+const MIME_TYPES = {
+    ".html": "text/html",
+    ".js": "application/javascript",
+    ".css": "text/css",
+    ".json": "application/json",
+    ".svg": "image/svg+xml",
+    ".png": "image/png",
+    ".ico": "image/x-icon",
+    ".woff2": "font/woff2",
+};
+function resolveSpaDistDir() {
+    // Look for the built SPA relative to this file's location
+    // In production: dist/heart/daemon/outlook-http.js -> ../../packages/outlook-ui/dist/
+    // In dev: packages/outlook-ui/dist/
+    const candidates = [
+        path.resolve(__dirname, "..", "..", "..", "packages", "outlook-ui", "dist"),
+        path.resolve(__dirname, "..", "..", "packages", "outlook-ui", "dist"),
+        path.resolve(__dirname, "..", "..", "..", "..", "packages", "outlook-ui", "dist"),
+        // npm-published layout: the SPA dist is copied to dist/outlook-ui/
+        path.resolve(__dirname, "..", "..", "outlook-ui"),
+        path.resolve(__dirname, "..", "outlook-ui"),
+    ];
+    for (const candidate of candidates) {
+        if (fs.existsSync(path.join(candidate, "index.html")))
+            return candidate;
+    }
+    return null;
+}
+function serveStaticFile(response, filePath) {
+    try {
+        if (!fs.existsSync(filePath))
+            return false;
+        const ext = path.extname(filePath);
+        const contentType = MIME_TYPES[ext] ?? "application/octet-stream";
+        const content = fs.readFileSync(filePath);
+        response.writeHead(200, {
+            "content-type": contentType,
+            "cache-control": ext === ".html" ? "no-cache" : "public, max-age=31536000, immutable",
+        });
+        response.end(content);
+        return true;
+    }
+    catch {
+        return false;
+    }
 }
+/* v8 ignore stop */
 function normalizePath(urlValue = "/") {
     const parsed = new URL(urlValue, "http://127.0.0.1");
     const normalizedPath = parsed.pathname.replace(/\/+$/, "");
@@ -140,7 +183,6 @@ async function startOutlookHttpServer(options = {}) {
     });
     /* v8 ignore stop */
     const readAgentView = options.readAgentView;
-    const renderApp = options.renderApp ?? outlook_render_1.renderOutlookApp;
     /* v8 ignore start — default hook wiring, tested via integration */
     const agentRoot = (agentName) => {
         const base = bundlesRoot ?? "";
@@ -174,10 +216,23 @@ async function startOutlookHttpServer(options = {}) {
             sse.add(response);
             return;
         }
+        /* v8 ignore start — SPA static asset serving */
+        // Serve built SPA static assets: /assets/*
+        if (pathname.startsWith("/assets/")) {
+            const spaDir = resolveSpaDistDir();
+            if (spaDir) {
+                const assetPath = path.join(spaDir, pathname);
+                if (serveStaticFile(response, assetPath))
+                    return;
+            }
+            writeJson(response, 404, { ok: false, error: "asset not found" });
+            return;
+        }
+        /* v8 ignore stop */
+        // Legacy /outlook route — redirect to root
         if (pathname === "/outlook") {
-            const machine = readMachineState();
-            const machineView = readMachineView?.({ origin, machine });
-            writeHtml(response, renderApp({ origin, machine, machineView }));
+            response.writeHead(301, { location: "/" });
+            response.end();
             return;
         }
         if (pathname === "/outlook/api/machine") {
@@ -273,7 +328,14 @@ async function startOutlookHttpServer(options = {}) {
             writeJson(response, 404, { ok: false, error: `unknown agent surface: ${surface}` });
             return;
         }
-        writeJson(response, 404, { ok: false, error: `unknown outlook path: ${pathname}` });
+        /* v8 ignore start — SPA fallback for client-side routing */
+        const spaDir = resolveSpaDistDir();
+        if (spaDir) {
+            if (serveStaticFile(response, path.join(spaDir, "index.html")))
+                return;
+        }
+        /* v8 ignore stop */
+        writeJson(response, 404, { ok: false, error: `not found: ${pathname}` });
     });
     await new Promise((resolve, reject) => {
         server.once("error", reject);