@qt4798/pi-web 0.14.4 → 0.14.6

package/.next/trace-build

@@ -1 +1 @@
-[{"name":"run-webpack","duration":16322663,"timestamp":87065397042,"id":14,"parentId":1,"tags":{},"startTime":1787935240121,"traceId":"c607530efec7c7b6"},{"name":"run-typescript","duration":3020795,"timestamp":87081721948,"id":869,"parentId":1,"tags":{},"startTime":1787935256446,"traceId":"c607530efec7c7b6"},{"name":"static-check","duration":1663427,"timestamp":87084750021,"id":872,"parentId":1,"tags":{},"startTime":1787935259474,"traceId":"c607530efec7c7b6"},{"name":"static-generation","duration":2207359,"timestamp":87086531616,"id":1056,"parentId":1,"tags":{},"startTime":1787935261256,"traceId":"c607530efec7c7b6"},{"name":"collect-build-traces","duration":16065660,"timestamp":87086414991,"id":1053,"parentId":1,"tags":{},"startTime":1787935261139,"traceId":"c607530efec7c7b6"},{"name":"output-standalone","duration":1490491,"timestamp":87102483639,"id":1064,"parentId":1,"tags":{},"startTime":1787935277208,"traceId":"c607530efec7c7b6"},{"name":"telemetry-flush","duration":15,"timestamp":87103975796,"id":1067,"parentId":1,"tags":{},"startTime":1787935278700,"traceId":"c607530efec7c7b6"},{"name":"next-build","duration":39200258,"timestamp":87064775557,"id":1,"tags":{"buildMode":"default","version":"16.3.1","bundler":"webpack","has-custom-webpack-config":"false","use-build-worker":"true"},"startTime":1787935239500,"traceId":"c607530efec7c7b6"}]
+[{"name":"run-webpack","duration":24268003,"timestamp":171763054687,"id":14,"parentId":1,"tags":{},"startTime":1788019938891,"traceId":"b5c8c000f3d6e835"},{"name":"run-typescript","duration":4511201,"timestamp":171787325982,"id":865,"parentId":1,"tags":{},"startTime":1788019963163,"traceId":"b5c8c000f3d6e835"},{"name":"static-check","duration":3333877,"timestamp":171791847649,"id":868,"parentId":1,"tags":{},"startTime":1788019967685,"traceId":"b5c8c000f3d6e835"},{"name":"static-generation","duration":3878156,"timestamp":171795418581,"id":1052,"parentId":1,"tags":{},"startTime":1788019971256,"traceId":"b5c8c000f3d6e835"},{"name":"collect-build-traces","duration":26656691,"timestamp":171795184673,"id":1049,"parentId":1,"tags":{},"startTime":1788019971022,"traceId":"b5c8c000f3d6e835"},{"name":"output-standalone","duration":2424887,"timestamp":171821843015,"id":1060,"parentId":1,"tags":{},"startTime":1788019997681,"traceId":"b5c8c000f3d6e835"},{"name":"telemetry-flush","duration":34,"timestamp":171824273008,"id":1063,"parentId":1,"tags":{},"startTime":1788020000111,"traceId":"b5c8c000f3d6e835"},{"name":"next-build","duration":62105703,"timestamp":171762167345,"id":1,"tags":{"buildMode":"default","version":"16.3.1","bundler":"webpack","has-custom-webpack-config":"false","use-build-worker":"true"},"startTime":1788019938003,"traceId":"b5c8c000f3d6e835"}]

package/bin/memory-watchdog.js

@@ -0,0 +1,135 @@
+"use strict";
+
+// Idle memory watchdog for the next-server process. V8 only returns freed
+// heap to the OS on process exit, so a long-running server's RSS settles at
+// its high-water mark no matter how much GC reclaims internally. This module
+// watches RSS and exits the process once it has stayed above the threshold
+// while no agent session is running; the supervisor (launchd KeepAlive) then
+// starts a fresh process.
+
+const DEFAULT_RESTART_MB = 1229;
+const DEFAULT_CHECK_MINUTES = 5;
+const CONSECUTIVE_OVER_LIMIT = 2;
+const STARTUP_GRACE_TICKS = 3;
+const RUNNING_CHECK_TIMEOUT_MS = 5000;
+
+function parsePositiveInt(raw, fallback, minimum) {
+  const n = Number.parseInt(raw ?? "", 10);
+  if (!Number.isFinite(n) || n < minimum) return fallback;
+  return n;
+}
+
+/**
+ * Resolve the watchdog config from env. Returns null when the watchdog must
+ * not run: explicitly disabled (PI_WEB_RSS_RESTART_MB=0), running on Windows
+ * (no launchd-style supervisor to restart us, and auto-restart is not wanted
+ * there), or no port known (without a port we cannot ask /api/agent/running
+ * and must never exit).
+ */
+function resolveConfig(env) {
+  if (process.platform === "win32") return null;
+  const rawRestart = env.PI_WEB_RSS_RESTART_MB;
+  if (rawRestart !== undefined && rawRestart !== "" && Number(rawRestart) === 0) {
+    return null;
+  }
+  const port = Number.parseInt(env.PI_WEB_PORT ?? "", 10);
+  if (!Number.isFinite(port) || port <= 0) return null;
+  return {
+    port,
+    hostname: env.PI_WEB_HOSTNAME || "127.0.0.1",
+    restartMb: parsePositiveInt(rawRestart, DEFAULT_RESTART_MB, 1),
+    checkMinutes: parsePositiveInt(env.PI_WEB_RSS_CHECK_MINUTES, DEFAULT_CHECK_MINUTES, 1),
+  };
+}
+
+function basicAuthHeaders(env) {
+  // Next.js loads PI_WEB_PASSWORD from .env in production, and the proxy gate
+  // rejects unauthenticated /api calls. Reuse the credential this process
+  // already holds so the loopback check passes without weakening auth.
+  const password = env.PI_WEB_PASSWORD;
+  if (!password) return undefined;
+  return { authorization: `Basic ${Buffer.from(`pi:${password}`).toString("base64")}` };
+}
+
+async function isServerIdle(config, env) {
+  const res = await fetch(`http://${config.hostname}:${config.port}/api/agent/running`, {
+    headers: basicAuthHeaders(env),
+    signal: AbortSignal.timeout(RUNNING_CHECK_TIMEOUT_MS),
+  });
+  if (!res.ok) return null;
+  const data = await res.json();
+  const ids = Array.isArray(data && data.runningSessionIds) ? data.runningSessionIds : [];
+  return ids.length === 0;
+}
+
+/**
+ * Start the watchdog loop in the current process. Never throws; failures to
+ * check are skipped conservatively (streak reset) so a transient error can
+ * never cause an exit.
+ */
+function startMemoryWatchdog(log = defaultLog) {
+  const config = resolveConfig(process.env);
+  if (!config) return null;
+
+  let ticks = 0;
+  let overLimitStreak = 0;
+  let loggedNonIdle = false;
+  log(`watching RSS every ${config.checkMinutes}min, restart at >= ${config.restartMb}MB when idle (grace: ${STARTUP_GRACE_TICKS} ticks)`);
+
+  const timer = setInterval(() => {
+    ticks += 1;
+    const rssMb = process.memoryUsage().rss / (1024 * 1024);
+    if (ticks <= STARTUP_GRACE_TICKS || rssMb < config.restartMb) {
+      overLimitStreak = 0;
+      return;
+    }
+    isServerIdle(config, process.env)
+      .then((idle) => {
+        if (idle !== true) {
+          // Sessions running, or the check failed — never exit on unknown state.
+          if (!loggedNonIdle) {
+            loggedNonIdle = true;
+            log(`over limit (${Math.round(rssMb)}MB) but idle check says ${idle === null ? "unavailable (check failed)" : "sessions running"}; not restarting`);
+          }
+          overLimitStreak = 0;
+          return;
+        }
+        overLimitStreak += 1;
+        log(`RSS ${Math.round(rssMb)}MB >= ${config.restartMb}MB and idle (${overLimitStreak}/${CONSECUTIVE_OVER_LIMIT})`);
+        if (overLimitStreak >= CONSECUTIVE_OVER_LIMIT) {
+          clearInterval(timer);
+          log("exiting so the supervisor can restart us and reclaim memory");
+          process.exit(0);
+        }
+      })
+      .catch((err) => {
+        if (!loggedNonIdle) {
+          loggedNonIdle = true;
+          log(`over limit but idle check errored: ${err && err.message}; not restarting`);
+        }
+        overLimitStreak = 0;
+      });
+  }, config.checkMinutes * 60 * 1000);
+  // The server itself keeps the process alive; the watchdog must never be the
+  // reason a process stays up.
+  timer.unref();
+  return timer;
+}
+
+function defaultLog(message) {
+  console.log(`[memory-watchdog] ${message}`);
+}
+
+module.exports = { startMemoryWatchdog, resolveConfig };
+
+// CLI: `node bin/memory-watchdog.js` runs the loop in-process (smoke tests).
+// The interval is unref'd for server use, so re-ref it to keep this process
+// alive.
+if (require.main === module) {
+  const timer = startMemoryWatchdog();
+  if (!timer) {
+    console.log("[memory-watchdog] disabled (PI_WEB_RSS_RESTART_MB=0 or no PI_WEB_PORT)");
+  } else {
+    timer.ref();
+  }
+}

package/bin/pi-web.js

@@ -48,6 +48,12 @@ const { port, hostname, openBrowser } = parseLaunchOptions();
 const { mergedNodeOptions } = require("./with-memory-limit");
 process.env.NODE_OPTIONS = mergedNodeOptions(process.env.NODE_OPTIONS);
 
+// Idle memory watchdog, loaded only into the next-server process. Injected as
+// a node CLI arg (not NODE_OPTIONS) so agent-spawned child processes never
+// inherit it. Disabled with PI_WEB_RSS_RESTART_MB=0; the module itself also
+// refuses to run when it doesn't know the port to ask about running sessions.
+const watchdogPreload = path.join(__dirname, "watchdog-preload.js");
+
 if (!fs.existsSync(nextDir)) {
   console.error("Build artifacts not found. Please report this issue.");
   process.exit(1);
@@ -71,10 +77,10 @@ const nextArgs = ["start", "-p", port, "-H", hostname];
 
 // Always run next's JS entry with node directly — avoids .bin symlink issues
 // and path-with-spaces problems on Windows when shell: true is used.
-const child = spawn(process.execPath, [nextBin, ...nextArgs], {
+const child = spawn(process.execPath, ["--require", watchdogPreload, nextBin, ...nextArgs], {
   cwd: pkgDir,
   stdio: ["inherit", "pipe", "inherit"],
-  env: { ...process.env, PI_WEB_HOSTNAME: hostname },
+  env: { ...process.env, PI_WEB_HOSTNAME: hostname, PI_WEB_PORT: String(port) },
 });
 wireChildProcessLifecycle(child);
 

package/bin/watchdog-preload.js

@@ -0,0 +1,20 @@
+"use strict";
+
+// Loaded into the next-server process via `node --require bin/watchdog-preload.js
+// next start ...` (wired up in pi-web.js). Startup is deferred so a problem
+// here can never delay or break server boot — worst case is one warn line and
+// no watchdog.
+
+try {
+  // eslint-disable-next-line @typescript-eslint/no-require-imports
+  const { startMemoryWatchdog } = require("./memory-watchdog");
+  setTimeout(() => {
+    try {
+      startMemoryWatchdog();
+    } catch (err) {
+      console.warn(`[memory-watchdog] failed to start: ${err && err.message}`);
+    }
+  }, 3000).unref();
+} catch (err) {
+  console.warn(`[memory-watchdog] preload failed: ${err && err.message}`);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qt4798/pi-web",
-  "version": "0.14.4",
+  "version": "0.14.6",
   "description": "Web workspace for the pi coding agent",
   "homepage": "https://github.com/t479842598/pi-web-QT#readme",
   "repository": {