fix: stop the OOM/502 outage at the root — PDS restore guard, bounded native memory

Dockerfile

@@ -26,6 +26,8 @@ USER bun
 # Nitro bundles all JS and traces native deps (@takumi-rs/core) into .output/server/node_modules — no bun install needed
 COPY --chown=bun:bun --from=build /usr/src/app/.output ./.output
 COPY --chown=bun:bun --from=build /usr/src/app/server/cluster.ts ./cluster.ts
+# cluster.ts imports this at runtime; it must sit next to it.
+COPY --chown=bun:bun --from=build /usr/src/app/server/worker-exit.ts ./worker-exit.ts
 EXPOSE 8080
 # start-period covers migrations on worker 0 plus the staggered sibling spawn
 HEALTHCHECK --interval=30s --timeout=5s --start-period=60s --retries=3 \

bunfig.toml

@@ -1,2 +1,6 @@
 [test]
-root = "src"
+# Was "src", which silently made server/ untestable — server/worker-exit.ts
+# shipped a bug that mislabelled every OOM kill for a month. Scripts pass the
+# directories explicitly (`bun test src server`) so app/'s React Native suites,
+# which use the jest preset rather than bun's runner, stay out of the way.
+root = "."

compose.yaml

@@ -94,7 +94,11 @@ services:
       PDS_ADMIN_PASSWORD: "CHANGE_ME_admin_password"
       OPEN_OBSERVE_USER: "user@bookhive.buzz"
       OPEN_OBSERVE_PASSWORD: "password"
-      OPEN_OBSERVE_URL: "http://localhost:5080"
+      # Must be container-reachable: inside this container `localhost` is the
+      # app itself, not OpenObserve. Both the logs transport and the OTLP trace
+      # exporter derive their endpoints from this, so a wrong value silently
+      # disables observability rather than failing loudly.
+      OPEN_OBSERVE_URL: "http://openobserve:5080"
       IMGPROXY_URL: "http://imgproxy:8080"
       IMGPROXY_KEY: "CHANGE_ME_imgproxy_key_hex"
       IMGPROXY_SALT: "CHANGE_ME_imgproxy_salt_hex"
@@ -104,7 +108,8 @@ services:
       - data:/data
     # Leave the other ~4 host CPUs for pds/imgproxy/openobserve and future services.
     cpus: "4"
-    # ~1GB shared SQLite mmap page cache + ~300MB private per worker.
+    # ~300-400MB anonymous per worker. The old budget assumed a ~1GB SQLite
+    # mmap on top; DB_MMAP_SIZE now defaults to 0, so that term is gone.
     mem_limit: 3g
 # The commented out section below is an example of how to define a PostgreSQL
 # database that your application can use. `depends_on` tells Docker Compose to

package.json

@@ -11,8 +11,8 @@
     "dev": "bunx --bun vp dev",
     "build": "bun --bun run lexgen && bunx --bun vp build",
     "preview": "bun run .output/server/index.mjs",
-    "test": "bun test src",
-    "test:run": "bun test src --no-watch",
+    "test": "bun test src server",
+    "test:run": "bun test src server --no-watch",
     "lint": "vp lint src --type-aware --type-check && vp fmt --write",
     "typecheck": "vp lint src --type-aware --type-check && vp fmt --write",
     "format": "vp fmt",

server/cluster.ts

@@ -10,9 +10,10 @@
  * must pass /healthcheck before the siblings spawn — that ordering is the
  * migration barrier for the non-primary workers.
  *
- * Not bundled — the Dockerfile copies this file verbatim and Bun runs the TS
- * source directly. Zero dependencies.
+ * Not bundled — the Dockerfile copies this file and ./worker-exit.ts verbatim
+ * and Bun runs the TS source directly. Zero external dependencies.
  */
+import { classifyWorkerExit, readProcessMemoryKb, signalName } from "./worker-exit.ts";
 
 const concurrency = Math.max(1, Number(process.env["WEB_CONCURRENCY"]) || 4);
 const port = process.env["PORT"] ?? "8080";
@@ -27,43 +28,38 @@ function log(message: string) {
 }
 
 /**
- * Worker deaths were invisible: the app logged 82 errors against 171,145
- * user-visible 502s on 2026-08-01 because the failure mode was process death,
- * not an exception. A cgroup OOM kill arrives as signal SIGKILL with a null
- * exit code, so emit it as a JSON line the log pipeline can count.
+ * Last memory sample per worker index. `/proc/<pid>` is gone by the time
+ * `onExit` fires, so a worker killed for using 2 GB would otherwise report no
+ * memory at all — exactly the number an OOM investigation needs.
  */
-// Bun reports signalCode as a number.
-const SIGNAL_NAMES: Record<number, string> = {
-  2: "SIGINT",
-  6: "SIGABRT",
-  9: "SIGKILL",
-  11: "SIGSEGV",
-  15: "SIGTERM",
-};
+const lastMemory = new Map<number, { rss_kb?: number; anon_kb?: number }>();
+const MEMORY_SAMPLE_MS = 15_000;
 
-function signalName(signalCode: number | null): string | null {
-  if (signalCode === null) return null;
-  return SIGNAL_NAMES[signalCode] ?? `SIG${signalCode}`;
+function sampleWorkerMemory() {
+  for (const [index, proc] of children) {
+    const sample = readProcessMemoryKb(proc.pid);
+    if (sample) lastMemory.set(index, sample);
+  }
 }
 
 function logWorkerExit(
   index: number,
+  pid: number | null,
   exitCode: number | null,
-  signalCode: number | null,
+  signalCode: number | string | null | undefined,
   uptimeMs: number,
 ) {
-  const signal = signalName(signalCode);
   console.error(
     JSON.stringify({
-      level: 50,
       time: Date.now(),
-      msg: "worker_exit",
-      worker: index,
-      code: exitCode,
-      signal,
-      // A cgroup OOM kill arrives as SIGKILL with no exit code.
-      likely_oom: signal === "SIGKILL",
-      uptime_ms: uptimeMs,
+      ...classifyWorkerExit({
+        index,
+        pid,
+        exitCode,
+        signalCode,
+        uptimeMs,
+        memory: lastMemory.get(index) ?? null,
+      }),
     }),
   );
 }
@@ -75,11 +71,11 @@ function spawnWorker(index: number) {
     env: { ...process.env, WORKER_INDEX: String(index) },
     stdout: "inherit",
     stderr: "inherit",
-    onExit(_proc, exitCode, signalCode) {
+    onExit(exited, exitCode, signalCode) {
       children.delete(index);
       // A SIGTERM we sent ourselves is not a failure — don't page on it.
       if (shuttingDown) return;
-      logWorkerExit(index, exitCode, signalCode, Date.now() - startedAt);
+      logWorkerExit(index, exited?.pid ?? null, exitCode, signalCode, Date.now() - startedAt);
       const now = Date.now();
       const recent = (restartTimes.get(index) ?? []).filter((t) => now - t < 60_000);
       recent.push(now);
@@ -91,10 +87,11 @@ function spawnWorker(index: number) {
       }
       const backoffMs = Math.min(1000 * 2 ** (recent.length - 1), 15_000);
       const signal = signalName(signalCode);
+      const anon = lastMemory.get(index)?.anon_kb;
       log(
         `worker ${index} exited (code ${exitCode}, signal ${signal ?? "none"}${
-          signal === "SIGKILL" ? ", likely OOM" : ""
-        }), restarting in ${backoffMs}ms`,
+          signal === "SIGKILL" && exitCode === null ? ", likely OOM" : ""
+        }${anon ? `, last anon ${Math.round(anon / 1024)}MB` : ""}), restarting in ${backoffMs}ms`,
       );
       setTimeout(() => spawnWorker(index), backoffMs);
     },
@@ -137,6 +134,9 @@ function shutdown(code: number) {
 process.on("SIGTERM", () => shutdown(0));
 process.on("SIGINT", () => shutdown(0));
 
+const memoryTimer = setInterval(sampleWorkerMemory, MEMORY_SAMPLE_MS);
+memoryTimer.unref?.();
+
 spawnWorker(0);
 if (concurrency > 1) {
   await waitForPrimaryHealthy();

server/plugins/otel-sdk.ts

@@ -1,6 +1,14 @@
 /**
  * Nitro plugin: OpenTelemetry SDK lifecycle.
  * Starts the SDK on boot and shuts it down gracefully on close.
+ *
+ * Traces reach OpenObserve at `${OPEN_OBSERVE_URL}/api/bookhive/v1/traces`, so
+ * `OPEN_OBSERVE_URL` must be **container-reachable** — `http://openobserve:5080`
+ * on the shared `backbone` network. Never `localhost`, which inside this
+ * container is the app itself. The repo's compose.yaml said `localhost` while
+ * the deployment said `openobserve`, and that mismatch is exactly how this
+ * pipeline was once misdiagnosed as dead. It is not: verified 2026-08-02, the
+ * `traces/default` stream holds 13.7M spans and is current.
  */
 import { definePlugin } from "nitro";
 import { NodeSDK } from "@opentelemetry/sdk-node";
@@ -25,7 +33,20 @@ export default definePlugin((nitroApp) => {
   const sdk = new NodeSDK({
     serviceName: "bookhive",
     traceExporter,
-    instrumentations: [getNodeAutoInstrumentations()],
+    instrumentations: [
+      getNodeAutoInstrumentations({
+        // A span per filesystem operation. Every static asset read, every
+        // SQLite-adjacent stat, in every worker — the standard advice is to
+        // leave this off, and an unconfigured `getNodeAutoInstrumentations()`
+        // turns it on.
+        "@opentelemetry/instrumentation-fs": { enabled: false },
+        // Inbound requests already get a root span from request-tracing.ts and
+        // a route span from src/middleware/otel-middleware.ts. A third would be
+        // noise. Outbound stays on: a PDS that stops answering and Goodreads
+        // refusing us are precisely what the incidents were about.
+        "@opentelemetry/instrumentation-http": { ignoreIncomingRequestHook: () => true },
+      }),
+    ],
   });
 
   if (!env.isDev) {

server/worker-exit.test.ts

@@ -0,0 +1,147 @@
+import { describe, it, expect } from "bun:test";
+import { classifyWorkerExit, readProcessMemoryKb, signalName } from "./worker-exit.ts";
+
+describe("signalName", () => {
+  // The bug: Bun passes the signal *name*, its types claim a number, and the
+  // number-keyed lookup fell through to `SIG${code}` — producing "SIGSIGKILL"
+  // in production for every one of 148 OOM kills.
+  it("passes through a name Bun already prefixed", () => {
+    expect(signalName("SIGKILL")).toBe("SIGKILL");
+    expect(signalName("SIGTERM")).toBe("SIGTERM");
+  });
+
+  it("still maps a numeric code, in case Bun's types become honest", () => {
+    expect(signalName(9)).toBe("SIGKILL");
+    expect(signalName(15)).toBe("SIGTERM");
+  });
+
+  it("is null for a clean exit", () => {
+    expect(signalName(null)).toBeNull();
+    expect(signalName(undefined)).toBeNull();
+  });
+
+  it("prefixes a bare name", () => {
+    expect(signalName("KILL")).toBe("SIGKILL");
+  });
+});
+
+describe("classifyWorkerExit", () => {
+  it("flags a cgroup OOM kill", () => {
+    const event = classifyWorkerExit({
+      index: 1,
+      pid: 4242,
+      exitCode: null,
+      signalCode: "SIGKILL",
+      uptimeMs: 775_653,
+    });
+
+    expect(event.signal).toBe("SIGKILL");
+    expect(event.likely_oom).toBe(true);
+    expect(event.msg).toBe("worker_exit");
+    expect(event.level).toBe(50);
+    expect(event.worker).toBe(1);
+    expect(event.pid).toBe(4242);
+    expect(event.uptime_ms).toBe(775_653);
+  });
+
+  it("does not flag a graceful exit as an OOM", () => {
+    const event = classifyWorkerExit({
+      index: 0,
+      exitCode: 0,
+      signalCode: null,
+      uptimeMs: 1_000,
+    });
+    expect(event.signal).toBeNull();
+    expect(event.likely_oom).toBe(false);
+  });
+
+  it("does not flag SIGTERM as an OOM", () => {
+    const event = classifyWorkerExit({
+      index: 0,
+      exitCode: null,
+      signalCode: "SIGTERM",
+      uptimeMs: 1_000,
+    });
+    expect(event.likely_oom).toBe(false);
+  });
+
+  it("does not flag a SIGKILL that carried an exit code", () => {
+    // Something other than the kernel's OOM killer produced this.
+    const event = classifyWorkerExit({
+      index: 0,
+      exitCode: 137,
+      signalCode: "SIGKILL",
+      uptimeMs: 1_000,
+    });
+    expect(event.likely_oom).toBe(false);
+  });
+
+  it("carries the last memory sample so a kill is attributable", () => {
+    const event = classifyWorkerExit({
+      index: 2,
+      exitCode: null,
+      signalCode: "SIGKILL",
+      uptimeMs: 700_000,
+      memory: { rss_kb: 2_580_000, anon_kb: 1_779_772 },
+    });
+    expect(event.anon_kb).toBe(1_779_772);
+    expect(event.rss_kb).toBe(2_580_000);
+  });
+
+  it("omits memory fields entirely when there is no sample", () => {
+    const event = classifyWorkerExit({
+      index: 0,
+      exitCode: null,
+      signalCode: "SIGKILL",
+      uptimeMs: 1,
+      memory: null,
+    });
+    expect("anon_kb" in event).toBe(false);
+    expect("rss_kb" in event).toBe(false);
+  });
+
+  it("serializes to a single JSON log line", () => {
+    const line = JSON.parse(
+      JSON.stringify(
+        classifyWorkerExit({
+          index: 1,
+          exitCode: null,
+          signalCode: "SIGKILL",
+          uptimeMs: 5,
+        }),
+      ),
+    );
+    expect(line.msg).toBe("worker_exit");
+    expect(line.signal).toBe("SIGKILL");
+    expect(line.likely_oom).toBe(true);
+  });
+});
+
+describe("readProcessMemoryKb", () => {
+  const SMAPS = [
+    "Rss:             1624780 kB",
+    "Pss:             1019518 kB",
+    "Shared_Clean:    1028192 kB",
+    "Private_Dirty:    554892 kB",
+    "Anonymous:        554892 kB",
+  ].join("\n");
+
+  it("parses Rss and Anonymous out of smaps_rollup", () => {
+    expect(readProcessMemoryKb(1, () => SMAPS)).toEqual({
+      rss_kb: 1_624_780,
+      anon_kb: 554_892,
+    });
+  });
+
+  it("returns null off Linux, where procfs does not exist", () => {
+    expect(
+      readProcessMemoryKb(1, () => {
+        throw new Error("ENOENT");
+      }),
+    ).toBeNull();
+  });
+
+  it("returns null for output with neither field", () => {
+    expect(readProcessMemoryKb(1, () => "Pss: 12 kB")).toBeNull();
+  });
+});