dsh-phone-bridge 0.2.5 → 0.2.7

package/index.js

@@ -943,36 +943,106 @@ export function apply(ctx, config = {}) {
         }
         const sessionController = ctx.get("sessionController");
         if (!sessionController) throw new Error("sessionController is unavailable");
-        if (typeof sessionController.page !== "function") {
-          throw new Error("this DSH build does not expose sessionController.page");
-        }
 
         const signal = new AbortController().signal;
-        // throughSeq: -1 means "the newest page"; maxMessages stays well above the
-        // requested count because assistant and tool events sit between the user's.
-        const page = await sessionController.page({
-          address: { kind: "session", sessionId },
-          throughSeq: -1,
-          maxMessages: 200,
-        }, signal);
+        const traces = {};
+
+        // Two ways in, tried in order.
+        //
+        // inspect() hands back the event list outright (SessionInspection.events),
+        // so it needs no cursor. page() does need one: throughSeq is an absolute
+        // sequence, and -1 is not "newest" but "nothing", since paginate starts at
+        // end = throughSeq + 1 = 0 and walks backwards from -1. That misreading is
+        // what made the first version answer ok with zero messages.
+        let events = [];
+        let used = null;
+
+        if (typeof sessionController.inspect === "function") {
+          try {
+            const inspection = await sessionController.inspect(sessionId, signal);
+            const list = Array.isArray(inspection?.events) ? inspection.events : [];
+            traces.inspect = {
+              eventCount: list.length,
+              firstSeq: list[0]?.seq ?? null,
+              lastSeq: list[list.length - 1]?.seq ?? null,
+            };
+            if (list.length) {
+              events = list;
+              used = "inspect";
+            }
+          } catch (error) {
+            traces.inspect = { error: String(error?.message ?? error) };
+          }
+        }
+
+        if (!events.length && typeof sessionController.page === "function") {
+          try {
+            // Page from whatever cursor inspect gave us, else from the very start.
+            const cursor = Number(traces.inspect?.lastSeq ?? 0);
+            const page = await sessionController.page({
+              address: { kind: "session", sessionId },
+              throughSeq: Number.isSafeInteger(cursor) && cursor >= 0 ? cursor : 0,
+              maxMessages: 400,
+            }, signal);
+            const list = (page?.records ?? []).map((record) => record?.event).filter(Boolean);
+            traces.page = {
+              cursor,
+              recordCount: list.length,
+              hasMore: page?.hasMore ?? null,
+              firstSeq: list[0]?.seq ?? null,
+              lastSeq: list[list.length - 1]?.seq ?? null,
+            };
+            if (list.length) {
+              events = list;
+              used = "page";
+            }
+          } catch (error) {
+            traces.page = { error: String(error?.message ?? error) };
+          }
+        }
 
         const texts = [];
-        for (const record of page?.records ?? []) {
-          const event = record?.event;
-          if (event?.type !== "user/message") continue;
+        const typeTally = {};
+        let firstUserEvent = null;
+        let userEventCount = 0;
+        for (const event of events) {
+          if (!event) continue;
+          const key = event.type ?? "(none)";
+          typeTally[key] = (typeTally[key] ?? 0) + 1;
+          if (event.type !== "user/message") continue;
+          userEventCount += 1;
+          if (!firstUserEvent) firstUserEvent = event;
           if (event?.data?.source?.kind !== "user") continue;
           const text = textFromContentBlocks(event.data.content);
           if (!text) continue;
           texts.push({ at: event.time ?? null, text });
         }
 
-        sendJson(res, 200, {
+        const payload = {
           ok: true,
           sessionId,
+          source: used,
           total: texts.length,
           // Newest last, so the caller can slice from the end.
           messages: texts.slice(-limit),
-        });
+        };
+
+        // Opt-in shape report. The history event format is not a documented
+        // surface, so when a read comes back empty the only quick way to find out
+        // why is to look at what actually arrived. Loopback-only, and truncated.
+        if ((url.searchParams.get("debug") ?? "") === "1") {
+          payload.diagnostics = {
+            used,
+            eventCount: events.length,
+            userEventCount,
+            eventTypes: typeTally,
+            traces,
+            firstEventKeys: events[0] ? Object.keys(events[0]) : [],
+            firstUserEventSample: firstUserEvent ? JSON.stringify(firstUserEvent).slice(0, 1500) : null,
+          };
+        }
+
+        sendJson(res, 200, payload);
       } catch (error) {
         sendJson(res, 500, { ok: false, error: String(error?.message ?? error) });
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-phone-bridge",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "Drive an existing DSH desktop session from a chat app, plus a sidebar delete button and trash panel",
   "type": "module",
   "main": "index.js",