arisa 4.0.10 → 4.0.14

package/README.md

@@ -162,7 +162,7 @@ Notes:
 
 - interactive bootstrap remains unchanged when no CLI overrides are provided
 - `--bootstrap` can be combined with overrides to regenerate config non-interactively
-- when `--pi.apiKey` is omitted and the provider supports OAuth, Arisa starts a temporary web page on `PORT` (default `10000`) where you can complete authentication from any browser
+- Arisa's HTTP server listens on `ARISA_HTTP_PORT` (default `11970`); bootstrap OAuth pages and tool-provided web routes use this server
 - unknown `--pi.provider` or `--pi.model` values are ignored and replaced by safe defaults
 
 Telegram bot tokens can be created with:

package/package.json

@@ -1,12 +1,17 @@
 {
   "name": "arisa",
-  "version": "4.0.10",
+  "version": "4.0.14",
   "description": "Telegram + Pi Agent modular assistant",
   "type": "module",
   "main": "src/index.js",
   "bin": {
     "arisa": "bin/arisa.js"
   },
+  "scripts": {
+    "start": "node src/index.js",
+    "bootstrap": "node src/index.js --bootstrap",
+    "test": "node --test"
+  },
   "keywords": [
     "telegram",
     "pi-agent",
@@ -36,14 +41,10 @@
     "url": "https://github.com/clasen/Arisa/issues"
   },
   "homepage": "https://github.com/clasen/Arisa#readme",
+  "packageManager": "pnpm@11.3.0+sha512.2c403d6594527287672b1f7056343a1f7c3634036a67ffabfcc2b3d7595d843768f8787148d1b57cf7956c90606bbd192857c363af19e96d2d0ec9ec5741d215",
   "dependencies": {
     "@earendil-works/pi-coding-agent": "^0.79.9",
     "@sinclair/typebox": "^0.34.41",
     "grammy": "^1.42.0"
-  },
-  "scripts": {
-    "start": "node src/index.js",
-    "bootstrap": "node src/index.js --bootstrap",
-    "test": "node --test"
   }
-}
+}

package/src/index.js

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
 
 import { createServer } from "node:http";
+import { execFile } from "node:child_process";
 import { bootstrapIfNeeded } from "./runtime/bootstrap.js";
 import { createApp } from "./runtime/create-app.js";
 import { createLogger } from "./runtime/logger.js";
@@ -22,20 +23,126 @@ const logger = createLogger({ verbose });
 let activeApp = null;
 let shuttingDown = false;
 
-const httpPort = Number(process.env.PORT);
+const defaultHttpPort = 11970;
+const httpPort = Number(process.env.ARISA_HTTP_PORT || defaultHttpPort);
+const shouldStartHttpServer = Boolean(httpPort && !["stop", "status", "flush"].includes(command));
 let httpRequestHandler = null;
+let httpServerListening = false;
+let httpServer = null;
 
 function setHttpRequestHandler(handler) {
   httpRequestHandler = handler;
 }
 
-if (httpPort) {
-  createServer((req, res) => {
-    if (httpRequestHandler) return httpRequestHandler(req, res);
-    res.writeHead(200, { "Content-Type": "text/plain" });
-    res.end("ok");
-  }).listen(httpPort)
-    .on("listening", () => logger.log("http", `health server on port ${httpPort}`));
+const httpServerReady = shouldStartHttpServer
+  ? startHttpServer()
+  : Promise.resolve(false);
+
+async function runCommand(commandName, args = []) {
+  return new Promise((resolve) => {
+    execFile(commandName, args, (error, stdout = "") => {
+      resolve(error ? "" : stdout.trim());
+    });
+  });
+}
+
+async function getListeningPids(port) {
+  const lsofOutput = await runCommand("lsof", ["-tiTCP", `-sTCP:LISTEN`, `-iTCP:${port}`]);
+  if (lsofOutput) {
+    return lsofOutput.split(/\s+/).map(Number).filter(Number.isFinite);
+  }
+
+  const fuserOutput = await runCommand("fuser", ["-n", "tcp", String(port)]);
+  return fuserOutput.split(/\s+/).map(Number).filter(Number.isFinite);
+}
+
+async function getProcessCommand(pid) {
+  return runCommand("ps", ["-p", String(pid), "-o", "command="]);
+}
+
+function looksLikeArisaProcess(commandText) {
+  return /\barisa\b/.test(commandText) || /\/arisa\/src\/index\.js\b/i.test(commandText);
+}
+
+function isProcessRunning(pid) {
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+async function waitForProcessExit(pid, timeoutMs = 3000) {
+  const deadline = Date.now() + timeoutMs;
+  while (Date.now() < deadline) {
+    if (!isProcessRunning(pid)) return true;
+    await new Promise((resolve) => setTimeout(resolve, 100));
+  }
+  return !isProcessRunning(pid);
+}
+
+async function stopStaleArisaListeners(port) {
+  const pids = await getListeningPids(port);
+  for (const pid of pids) {
+    if (pid === process.pid) continue;
+    const commandText = await getProcessCommand(pid);
+    if (!looksLikeArisaProcess(commandText)) {
+      logger.error("http", `port ${port} is already used by pid ${pid}; not killing non-Arisa process`);
+      continue;
+    }
+
+    logger.log("http", `stopping stale Arisa listener on port ${port} (pid ${pid})`);
+    try {
+      process.kill(pid, "SIGTERM");
+      if (!await waitForProcessExit(pid)) {
+        process.kill(pid, "SIGKILL");
+        await waitForProcessExit(pid, 1000);
+      }
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      logger.error("http", `failed to stop stale Arisa listener pid ${pid}: ${message}`);
+    }
+  }
+}
+
+async function startHttpServer() {
+  await stopStaleArisaListeners(httpPort);
+  return new Promise((resolve) => {
+    const server = createServer((req, res) => {
+      if (httpRequestHandler) return httpRequestHandler(req, res);
+      res.writeHead(200, { "Content-Type": "text/plain" });
+      res.end("ok");
+      return undefined;
+    });
+    httpServer = server;
+    server.on("error", (error) => {
+      const message = error instanceof Error ? error.message : String(error);
+      logger.error("http", `server failed on port ${httpPort}: ${message}`);
+      resolve(false);
+    });
+    server.listen(httpPort)
+      .on("listening", () => {
+        httpServerListening = true;
+        logger.log("http", `health server on port ${httpPort}`);
+        resolve(true);
+      });
+  });
+}
+
+async function getHttpOptions() {
+  await httpServerReady;
+  return httpServerListening ? { httpPort, setHttpRequestHandler } : {};
+}
+
+async function stopHttpServer() {
+  if (!httpServerListening || !httpServer) return;
+  await new Promise((resolve) => {
+    httpServer.close(() => resolve());
+  });
+  httpServer = null;
+  httpServerListening = false;
+  httpRequestHandler = null;
 }
 
 function parseCliArgs(rawArgs) {
@@ -97,15 +204,14 @@ function toServiceRunnerArgs(nestedFlags) {
   return args;
 }
 
-const bootstrapHttpOptions = httpPort ? { httpPort, setHttpRequestHandler } : {};
 const webhookUrl = bootstrapOverrides.webhook?.url || "";
-const appHttpOptions = httpPort ? { webhookUrl, setHttpRequestHandler } : {};
 
 async function shutdown(exitCode = 0) {
   if (shuttingDown) return;
   shuttingDown = true;
   try {
     await activeApp?.stop?.();
+    await stopHttpServer();
   } catch (error) {
     logger.error("app", `shutdown failed: ${error instanceof Error ? error.message : String(error)}`);
     exitCode = exitCode || 1;
@@ -124,8 +230,8 @@ process.once("SIGINT", () => {
   shutdown(0);
 });
 
-async function startRuntimeApp() {
-  const app = await createApp({ logger, runtimeOverrides, ...appHttpOptions });
+async function startRuntimeApp(httpOptions = {}) {
+  const app = await createApp({ logger, runtimeOverrides, webhookUrl, ...httpOptions });
   activeApp = app;
   await app.start();
 }
@@ -137,9 +243,10 @@ async function runForeground() {
     || runtimeOverrides?.pi?.apiKey
   );
   logger.log("app", `starting${verbose ? " in verbose mode" : ""}`);
-  await bootstrapIfNeeded({ force: forceBootstrap, cliConfigOverrides: bootstrapOverrides, ...bootstrapHttpOptions });
+  const httpOptions = await getHttpOptions();
+  await bootstrapIfNeeded({ force: forceBootstrap, cliConfigOverrides: bootstrapOverrides, ...httpOptions });
   try {
-    await startRuntimeApp();
+    await startRuntimeApp(httpOptions);
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     if (message.includes("No auth found")) {
@@ -152,8 +259,8 @@ async function runForeground() {
         throw error;
       }
       console.log("Reopening bootstrap so you can provide a Pi API key or switch to a provider you already authenticated with.\n");
-      await bootstrapIfNeeded({ force: true, cliConfigOverrides: bootstrapOverrides, ...bootstrapHttpOptions });
-      await startRuntimeApp();
+      await bootstrapIfNeeded({ force: true, cliConfigOverrides: bootstrapOverrides, ...httpOptions });
+      await startRuntimeApp(httpOptions);
       return;
     }
     throw error;
@@ -168,7 +275,9 @@ async function main() {
   }
 
   if (command === "start") {
-    await bootstrapIfNeeded({ force: forceBootstrap, cliConfigOverrides: bootstrapOverrides, ...bootstrapHttpOptions });
+    const httpOptions = await getHttpOptions();
+    await bootstrapIfNeeded({ force: forceBootstrap, cliConfigOverrides: bootstrapOverrides, ...httpOptions });
+    await stopHttpServer();
     const result = await startService({ verbose, cliArgs: toServiceRunnerArgs(cli.nestedFlags) });
     if (!result.ok) {
       console.log(`Arisa is already running in background (pid ${result.pid}).`);

package/src/runtime/bootstrap.js

@@ -254,7 +254,7 @@ export async function bootstrapIfNeeded({ force = false, cliConfigOverrides = {}
       }
       if (!httpPort || !setHttpRequestHandler) {
         throw new Error(
-          `No auth found for ${resolvedPi.provider}. Auth relay requires an HTTP server on PORT. Provide --pi.apiKey or set the PORT environment variable.`
+          `No auth found for ${resolvedPi.provider}. Arisa's HTTP server is unavailable; set ARISA_HTTP_PORT or free the configured port.`
         );
       }
       const authRelay = installAuthRelay(httpPort, setHttpRequestHandler);