@expo/serve-sim 0.3.1 → 0.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@expo/serve-sim",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "type": "module",
   "author": {
     "name": "Evan Bacon",

package/src/middleware.ts

@@ -1,4 +1,5 @@
-import { execFile, execSync, spawn, type ChildProcess } from "child_process";
+import { openSseStream } from "./sse-stream";
+import { execFile, execSync } from "child_process";
 import { readdirSync, readFileSync, existsSync, unlinkSync, watch, type FSWatcher } from "fs";
 import { readFile, unlink } from "fs/promises";
 import { tmpdir } from "os";
@@ -42,6 +43,11 @@ import {
 import { serveDeviceKitModelAsset } from "./devicekit-model";
 import { validatePanelRoute } from "./panel-route";
 import { createExecWebSocketHandler, type UiRequestHandler } from "./exec-ws";
+import { crashRuntime } from "./crash/runtime";
+import { handleCrashesRequestAfter, handleCrashReportRequest } from "./crash/routes";
+export { handleCrashesRequest, handleCrashReportRequest } from "./crash/routes";
+import { booleanParam } from "./request-params";
+import { logBufferCache, type LogBufferCache, type LogLine } from "./log-buffer";
 import { claimHelperHidSocket, type UpgradeHandlerWebSocket } from "./middleware-utils";
 import { UI_OPTIONS, getUiStatus, normalizeUiValue, setUiOption } from "./ui-settings";
 import { type WebMiddleware } from "./runtime-utils";
@@ -128,10 +134,6 @@ const metricsSamplerCache = createMetricsSamplerCache(
   (udid) => new MetricsSampler({ udid, deviceName: bootedDeviceName(udid) }),
 );
 
-// Hard cap on the SSE line-assembly buffer for child-process stdout.
-// A malformed log entry without a newline can't grow this beyond 1 MB;
-// the partial line is dropped rather than retained indefinitely.
-const SSE_LINE_BUFFER_LIMIT = 1024 * 1024;
 let inspectWebKitBridge: Promise<WebKitBridge> | null = null;
 
 function eventLogLimit(rawUrl: string): number | undefined {
@@ -254,7 +256,8 @@ export function matchInstalledAppByDisplayName(
 
 // Cache simctl's booted-device set briefly so per-request cost stays bounded.
 // The middleware runs inside the user's dev server (Metro etc.) and
-// readServeSimStates() is called on every /api and every page load.
+// The window has to outlast the logs drawer's poll interval.
+const BOOTED_CACHE_TTL_MS = 5_000;
 let bootedSnapshot: {
   at: number;
   booted: Set<string> | null;
@@ -268,7 +271,7 @@ let bootedSnapshot: {
 };
 async function getBootedUdids(): Promise<Set<string> | null> {
   const now = Date.now();
-  if (bootedSnapshot.booted && now - bootedSnapshot.at < 1500) {
+  if (bootedSnapshot.booted && now - bootedSnapshot.at < BOOTED_CACHE_TTL_MS) {
     return bootedSnapshot.booted;
   }
   try {
@@ -995,6 +998,7 @@ export function previewConfigForState(
 ): ServeSimState & {
   basePath: string;
   logsEndpoint: string;
+  crashesEndpoint: string;
   appStateEndpoint: string;
   eventLogEndpoint: string;
   eventLogEventsEndpoint: string;
@@ -1033,6 +1037,7 @@ export function previewConfigForState(
     ...publicState,
     basePath: base,
     logsEndpoint: endpoint(base, "/logs", state.device),
+    crashesEndpoint: endpoint(base, "/crashes", state.device),
     appStateEndpoint: endpoint(base, "/appstate", state.device),
     eventLogEndpoint: endpoint(base, "/api/event-log", state.device),
     eventLogEventsEndpoint: endpoint(base, "/api/event-log/events", state.device),
@@ -1549,13 +1554,103 @@ function httpStreamSettingsFromLegacyCodec(codec: string | undefined): StreamSet
   return undefined;
 }
 
+
+export function handleLogsRequest(
+  req: SimReq,
+  res: SimRes,
+  state: ServeSimState | null,
+  rawUrl: string,
+  cache: LogBufferCache = logBufferCache
+): void {
+  if (!state) {
+    res.writeHead(404, { "Content-Type": "application/json", "Cache-Control": "no-store" });
+    res.end(JSON.stringify({ error: "No serve-sim device" }));
+    return;
+  }
+
+  const params = new URL(rawUrl, "http://127.0.0.1").searchParams;
+  const intQuery = (name: string): number | undefined => {
+    const raw = params.get(name)?.trim();
+    const n = Number(raw);
+    return raw && Number.isSafeInteger(n) && n >= 0 ? n : undefined;
+  };
+  const since = intQuery("since");
+  const limit = intQuery("limit");
+  const snapshot = params.get("snapshot");
+  const wantsJson =
+    snapshot === null
+      ? (req.headers.accept ?? "").includes("application/json")
+      : booleanParam(params, "snapshot");
+  const wantsEnvelope = booleanParam(params, "envelope");
+  const wantsFollow = booleanParam(params, "follow");
+
+  if (wantsJson) {
+    const buffer = wantsFollow ? cache.ensure(state.device) : cache.peek(state.device);
+    res.writeHead(200, { "Content-Type": "application/json", "Cache-Control": "no-store" });
+    res.end(
+      JSON.stringify({
+        device: state.device,
+        latestSeq: buffer?.latestSeq ?? 0,
+        oldestSeq: buffer?.oldestSeq ?? 0,
+        bufferedBytes: buffer?.byteLength ?? 0,
+        status: buffer?.status ?? "stopped",
+        streamError: buffer?.error ?? null,
+        lines: buffer?.read({ since, limit }) ?? [],
+      })
+    );
+    return;
+  }
+
+  const buffer = cache.ensure(state.device);
+  const stream = openSseStream(req, res);
+
+  const frame = (line: LogLine): string =>
+    "data: " +
+    (wantsEnvelope ? JSON.stringify({ seq: line.seq, at: line.at, raw: line.raw }) : line.raw) +
+    "\n\n";
+
+  let lastSent = since ?? 0;
+  for (const line of buffer.read({ since, limit })) {
+    if (!stream.isOpen()) break;
+    stream.write(frame(line));
+    lastSent = line.seq;
+  }
+
+  stream.onClose(
+    buffer.subscribeBatch(
+      (lines) => {
+        let chunk = "";
+        for (const line of lines) {
+          if (line.seq <= lastSent) continue;
+          lastSent = line.seq;
+          chunk += frame(line);
+        }
+        if (chunk) stream.write(chunk);
+      },
+      () => {
+        if (stream.isOpen()) res.end();
+      }
+    )
+  );
+}
+
+async function selectDeviceAndReap(selectedDevice: string | null): Promise<ServeSimState | null> {
+  const states = await readServeSimStates();
+  const state = selectServeSimState(states, selectedDevice);
+  const live = states.map((s) => s.device);
+  crashRuntime.prune(live);
+  logBufferCache.prune(live);
+  return state;
+}
+
 /**
  * Connect-style middleware that serves the simulator preview UI.
  *
  * Routes handled under `basePath` (default `/.sim`):
  *   GET  {basePath}         — the preview HTML page
  *   GET  {basePath}/api     — serve-sim state JSON
- *   GET  {basePath}/logs    — SSE stream of simctl logs
+ *   GET  {basePath}/logs    — simctl logs, JSON snapshot or SSE (bearer token)
+ *   GET  {basePath}/crashes — crash reports (bearer token)
  *   GET  {basePath}/ax      — SSE stream of normalized accessibility snapshots
  */
 export function handleMetricsRequest(
@@ -1615,6 +1710,8 @@ export function simMiddleware(options?: SimMiddlewareOptions): SimMiddleware {
     ? { "Content-Security-Policy": frameAncestorsPolicy(frameAncestors) }
     : {};
 
+  crashRuntime.arm();
+
   // Simulator-settings requests run in-process (just the underlying simctl /
   // ax-tool spawn) instead of round-tripping a full `node <cli>` exec per
   // sidebar interaction.
@@ -2434,50 +2531,51 @@ export function simMiddleware(options?: SimMiddlewareOptions): SimMiddleware {
       return;
     }
 
-    // SSE: simctl log stream
-    if (url === base + "/logs") {
-      const states = await readServeSimStates();
-      const state = selectServeSimState(states, selectedDevice);
-      if (!state) {
-        res.writeHead(404);
-        res.end("No serve-sim device");
+    if (url === base + "/crashes" || url === base + "/crashes/") {
+      const state = await selectDeviceAndReap(selectedDevice);
+      await handleCrashesRequestAfter(
+        () => crashRuntime.start({ deferToRetry: true }).catch(() => {}),
+        req,
+        res,
+        state,
+        rawUrl
+      );
+      return;
+    }
+
+    if (url.startsWith(base + "/crashes/")) {
+      const rawId = url.slice((base + "/crashes/").length);
+      let id: string;
+      try {
+        id = decodeURIComponent(rawId);
+      } catch {
+        res.writeHead(400, { "Content-Type": "application/json" });
+        res.end(
+          JSON.stringify({
+            error:
+              `Malformed percent-escape in the crash id (${rawId}). Copy the id verbatim from ` +
+              "GET {base}/crashes.",
+          })
+        );
         return;
       }
-      const udid = state.device;
-      res.writeHead(200, {
-        "Content-Type": "text/event-stream",
-        "Cache-Control": "no-cache",
-        Connection: "keep-alive",
-        "X-Accel-Buffering": "no",
-      });
-      res.write(":\n\n");
-
-      const child: ChildProcess = spawn("xcrun", [
-        "simctl", "spawn", udid, "log", "stream",
-        "--style", "ndjson",
-        "--level", "info",
-      ], { stdio: ["ignore", "pipe", "ignore"] });
-
-      let buf = "";
-      child.stdout!.on("data", (chunk: Buffer) => {
-        buf += chunk.toString();
-        let nl: number;
-        while ((nl = buf.indexOf("\n")) !== -1) {
-          const line = buf.slice(0, nl).trim();
-          buf = buf.slice(nl + 1);
-          if (line) res.write("data: " + line + "\n\n");
-        }
-        // Drop a runaway partial line so a malformed/never-terminated
-        // log entry can't grow `buf` without bound.
-        if (buf.length > SSE_LINE_BUFFER_LIMIT) buf = "";
-      });
+      const state = await selectDeviceAndReap(selectedDevice);
+      const params = new URL(rawUrl, "http://127.0.0.1").searchParams;
+      await handleCrashReportRequest(
+        req,
+        res,
+        state,
+        id,
+        params.get("occurrence"),
+        undefined,
+        undefined,
+        params.get("key")
+      );
+      return;
+    }
 
-      child.on("error", () => { try { res.end(); } catch {} });
-      child.on("close", () => res.end());
-      req.on("close", () => {
-        child.stdout?.destroy();
-        child.kill();
-      });
+    if (url === base + "/logs") {
+      handleLogsRequest(req, res, await selectDeviceAndReap(selectedDevice), rawUrl);
       return;
     }
 
@@ -2625,6 +2723,7 @@ export function simMiddleware(options?: SimMiddlewareOptions): SimMiddleware {
       `${base}/grid/api/status/events`,
       `${base}/appstate`,
       `${base}/logs`,
+      `${base}/crashes`,
       `${base}/metrics`,
       `${base}/ax`,
     ],