@seqyuan/annovibe 0.8.134 → 0.8.136

package/bin/pi-web.js

@@ -3,27 +3,16 @@
 
 process.title = "annovibe";
 
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const { spawn, spawnSync } = require("child_process");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const path = require("path");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const fs = require("fs");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const os = require("os");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const http = require("http");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const net = require("net");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const readline = require("readline");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const { parseArgs } = require("util");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const bcrypt = require("bcryptjs");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const https = require("https");
-// eslint-disable-next-line @typescript-eslint/no-require-imports
 const { randomBytes } = require("crypto");
 
 const pkgDir = path.join(__dirname, "..");
@@ -258,9 +247,70 @@ function staleStateRemoved() {
   return false;
 }
 
+const QUOTA_ERROR_MARKER = "disk quota exceeded (EDQUOT, errno 122)";
+
+function isQuotaExceededError(error) {
+  if (!error) return false;
+  const code = typeof error === "object" ? error.code : undefined;
+  const errno = typeof error === "object" ? error.errno : undefined;
+  const message = error instanceof Error ? error.message : String(error);
+  return (
+    code === "EDQUOT" ||
+    errno === -122 ||
+    errno === 122 ||
+    /\bEDQUOT\b/i.test(message) ||
+    /unknown system error -122\b/i.test(message)
+  );
+}
+
+// errno 122 is EDQUOT on Linux. Some Node/libuv builds do not know the name,
+// so they print "Unknown system error -122" instead, which looks like a bug
+// in the missing file. Turn it into an actionable message.
+function formatFsError(error, action) {
+  const raw = error instanceof Error ? error.message : String(error);
+  if (isQuotaExceededError(error)) {
+    return [
+      `${action}: ${QUOTA_ERROR_MARKER}`,
+      "Free space/quota in your home directory, then retry.",
+      "Useful checks: `quota -s`, `df -h ~`, `df -i ~`, `du -sh ~/.pi ~/.npm ~/.cache`.",
+      "If a previous `annovibe start` left a server running, run `annovibe stop` first.",
+    ].join("\n");
+  }
+  return `${action}: ${raw}`;
+}
+
+function printCommandError(prefix, error) {
+  const message = error instanceof Error ? error.message : String(error);
+  if (message.includes(QUOTA_ERROR_MARKER)) {
+    console.error(`${prefix}: ${message}`);
+    return;
+  }
+  if (isQuotaExceededError(error)) {
+    console.error(formatFsError(error, prefix));
+    return;
+  }
+  console.error(`${prefix}: ${message}`);
+}
+
 function writeState(state) {
   ensureAgentDir();
-  fs.writeFileSync(stateFile, JSON.stringify({ ...state, updatedAt: nowIso() }, null, 2));
+  try {
+    fs.writeFileSync(stateFile, JSON.stringify({ ...state, updatedAt: nowIso() }, null, 2));
+  } catch (error) {
+    throw new Error(formatFsError(error, `cannot write ${stateFile}`));
+  }
+}
+
+function assertAgentDirWritable() {
+  ensureAgentDir();
+  const probeFile = path.join(agentDir, `.annovibe-write-check-${process.pid}`);
+  try {
+    fs.writeFileSync(probeFile, "ok");
+  } catch (error) {
+    throw new Error(formatFsError(error, `cannot write to ${agentDir}`));
+  } finally {
+    try { fs.rmSync(probeFile, { force: true }); } catch { /* ignore */ }
+  }
 }
 
 function findListeningPids(listenPort) {
@@ -631,7 +681,7 @@ async function startManagedServer({ quiet = false, effectivePort: optPort, effec
     }
     removeState();
   }
-  ensureAgentDir();
+  assertAgentDirWritable();
   const child = spawnDetachedServer({}, effectivePort, effectiveHost);
   if (!child.pid) {
     throw new Error("failed to start annovibe server");
@@ -650,7 +700,15 @@ async function startManagedServer({ quiet = false, effectivePort: optPort, effec
     stateFile,
     startedAt: nowIso(),
   };
-  writeState(state);
+  try {
+    writeState(state);
+  } catch (error) {
+    // The supervisor has already been spawned. Stop it and drop any partial
+    // state file so a failed start does not leave an untracked process behind.
+    try { process.kill(child.pid, "SIGTERM"); } catch { /* already gone */ }
+    removeState();
+    throw error;
+  }
   if (!quiet) {
     console.log(`annovibe start: pid ${child.pid}`);
     console.log(`url: ${state.url}`);
@@ -735,7 +793,7 @@ if (firstPos === "stop") {
   stopManagedServer()
     .then((code) => process.exit(code))
     .catch((error) => {
-      console.error(`annovibe stop: ${error instanceof Error ? error.message : String(error)}`);
+      printCommandError("annovibe stop", error);
       process.exit(1);
     });
   return;
@@ -745,7 +803,7 @@ if (firstPos === "start") {
   startManagedServer()
     .then((code) => process.exit(code))
     .catch((error) => {
-      console.error(`annovibe start: ${error instanceof Error ? error.message : String(error)}`);
+      printCommandError("annovibe start", error);
       process.exit(1);
     });
   return;
@@ -759,7 +817,7 @@ if (firstPos === "restart") {
     if (code !== 0) process.exit(code);
     process.exit(await startManagedServerWithPort(restartPort, restartHost));
   }).catch((error) => {
-    console.error(`annovibe restart: ${error instanceof Error ? error.message : String(error)}`);
+    printCommandError("annovibe restart", error);
     process.exit(1);
   });
   return;
@@ -1073,6 +1131,7 @@ async function runServerProcess() {
       ANNOVIBE_INTERNAL_TOKEN: internalToken,
       ANNOVIBE_SUPERVISOR_PID: String(process.pid),
       ANNOVIBE_NEXT_WRAPPER: "1",
+      ...(hostname ? { ANNOVIBE_HOSTNAME: String(hostname) } : {}),
     },
   });
   let restarting = false;
@@ -1101,22 +1160,28 @@ async function runServerProcess() {
 
   const writeCurrentState = (patch = {}) => {
     const current = readState() ?? {};
-    writeState({
-      schemaVersion: 1,
-      version: VERSION,
-      command: getServerCommand(),
-      supervisorPid: process.pid,
-      nextPid: child.pid ?? null,
-      imGatewayPid: imGatewayChild?.pid ?? null,
-      imGatewayLogFile,
-      port: String(port),
-      hostname,
-      url,
-      logFile,
-      stateFile,
-      startedAt: current.startedAt ?? nowIso(),
-      ...patch,
-    });
+    try {
+      writeState({
+        schemaVersion: 1,
+        version: VERSION,
+        command: getServerCommand(),
+        supervisorPid: process.pid,
+        nextPid: child.pid ?? null,
+        imGatewayPid: imGatewayChild?.pid ?? null,
+        imGatewayLogFile,
+        port: String(port),
+        hostname,
+        url,
+        logFile,
+        stateFile,
+        startedAt: current.startedAt ?? nowIso(),
+        ...patch,
+      });
+    } catch (error) {
+      // Keep the server alive, but say that status/stop may not find it.
+      console.error(error instanceof Error ? error.message : String(error));
+      console.error("annovibe: failed to persist supervisor state; `annovibe status` / `annovibe stop` may not work until this is fixed.");
+    }
   };
 
   writeCurrentState();
@@ -1159,6 +1224,8 @@ async function runServerProcess() {
 
 function runNextProcess() {
   setStableProcessTitle(NEXT_PROCESS_TITLE);
+  // The proxy allow-lists the hostname the operator explicitly bound.
+  if (hostname) process.env.ANNOVIBE_HOSTNAME = String(hostname);
   if (!fs.existsSync(nextDir)) {
     console.error("Build artifacts not found. Please report this issue.");
     process.exit(1);
@@ -1167,27 +1234,26 @@ function runNextProcess() {
   const nextArgs = ["start", "-p", String(port)];
   if (hostname) nextArgs.push("-H", String(hostname));
   process.argv = [process.execPath, nextBin, ...nextArgs];
-  // eslint-disable-next-line @typescript-eslint/no-require-imports
   require(nextBin);
 }
 
 if (firstPos === "__server") {
   runServerProcess().catch((error) => {
-    console.error(`annovibe server: ${error instanceof Error ? error.message : String(error)}`);
+    printCommandError("annovibe server", error);
     process.exit(1);
   });
 } else if (firstPos === "__next") {
   try {
     runNextProcess();
   } catch (error) {
-    console.error(`annovibe next: ${error instanceof Error ? error.message : String(error)}`);
+    printCommandError("annovibe next", error);
     process.exit(1);
   }
 } else {
   startManagedServer()
     .then((code) => process.exit(code))
     .catch((error) => {
-      console.error(`annovibe start: ${error instanceof Error ? error.message : String(error)}`);
+      printCommandError("annovibe start", error);
       process.exit(1);
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@seqyuan/annovibe",
-  "version": "0.8.134",
+  "version": "0.8.136",
   "description": "AI-native bioinformatics workspace by Annoroad",
   "license": "MIT",
   "bin": {
@@ -31,12 +31,13 @@
     "pack:check": "node scripts/check-npm-pack.mjs",
     "smoke:generative-ui": "python3 scripts/smoke_generative_ui.py",
     "prepublishOnly": "node scripts/check-npm-pack.mjs",
-    "release": "npm version patch --no-git-tag-version && npm run build && npm run pack:check && npm publish --access public"
+    "release": "npm version patch --no-git-tag-version && npm run build && npm run pack:check && npm publish --access public",
+    "test": "node --experimental-strip-types --test \"lib/**/*.test.mjs\""
   },
   "dependencies": {
-    "@earendil-works/pi-agent-core": "0.80.2",
-    "@earendil-works/pi-ai": "0.80.2",
-    "@earendil-works/pi-coding-agent": "0.80.2",
+    "@earendil-works/pi-agent-core": "0.84.2",
+    "@earendil-works/pi-ai": "0.84.2",
+    "@earendil-works/pi-coding-agent": "0.84.2",
     "@seqyuan/kernel-lite-core": "^0.2.1",
     "@seqyuan/pi-kernel-plot": "^0.2.1",
     "bcryptjs": "^3.0.3",