dsh-phone-bridge 0.2.3 → 0.2.5

package/README.md

@@ -37,6 +37,8 @@ trash panel at the bottom of the sidebar where each entry can be restored or pur
 | `POST /stop` | cancel the session's running turn |
 | `GET /models` | the deployment model catalog |
 | `POST /model` | switch the model a session uses |
+| `GET /health` | whether this DSH build still has every `sessionController` method the plugin calls, and which are missing if not |
+| `GET /recent?sessionId=&limit=` | the user's own last few messages in a session, read through `sessionController.page()` |
 
 Deleting **moves** a session to `~/.dsh/deleted-sessions/` instead of unlinking it,
 so a delete stays recoverable and purging is a separate, explicit step.

package/index.js

@@ -227,6 +227,23 @@ const WORKSPACE_FILE = join(DSH_HOME, "storages", "workspace.json");
 // TRASH_ROOT is assigned in apply() from Config, because it is the one path a
 // deployment may reasonably want to move somewhere else.
 
+// Every sessionController method this plugin calls. DSH is pre-1.0 and this set
+// is an internal surface, so it is checked at load and reported by /health
+// rather than discovered one failing route at a time.
+const REQUIRED_CONTROLLER_METHODS = [
+  "cancel",
+  "create",
+  "follow",
+  "inspect",
+  "list",
+  "modelCatalog",
+  "page",
+  "prompt",
+  "rename",
+  "search",
+  "selectModel",
+];
+
 // A session lives at sessions/<slug-of-cwd>/<sessionId>/, where the slug is a
 // lossy encoding of the working directory. Locate it by scanning instead of
 // trying to re-derive that encoding.
@@ -257,6 +274,27 @@ function listOnDiskSessionIds() {
   return ids;
 }
 
+// Plain text out of a message's content blocks.
+//
+// The block shape is not a stable surface either, so only text-ish fields are
+// read and anything unrecognised is skipped rather than stringified into the
+// output as "[object Object]".
+function textFromContentBlocks(content) {
+  if (typeof content === "string") return content.trim();
+  if (!Array.isArray(content)) return "";
+  const parts = [];
+  for (const block of content) {
+    if (typeof block === "string") {
+      if (block.trim()) parts.push(block.trim());
+      continue;
+    }
+    if (!block || typeof block !== "object") continue;
+    const text = [block.text, block.content].find((v) => typeof v === "string" && v.trim());
+    if (text) parts.push(text.trim());
+  }
+  return parts.join("\n").trim();
+}
+
 // Pull just the title out of a session's projection bag.
 //
 // The bag also carries token counts, context pressure, the turn outline and the
@@ -879,6 +917,89 @@ export function apply(ctx, config = {}) {
     },
   }));
 
+  // What this build of DSH is expected to hand us. The plugin drives sessions
+  // through sessionController, which is an internal surface that can change
+  // between releases; when it does, the symptom would otherwise be a bare 500 on
+  // whichever route happens to touch the missing method, with nothing said about
+  // why. Reporting it up front turns that into one clear line.
+  // The user's own last few messages in a session, so the phone can answer "where
+  // did this conversation get to" without scrolling a chat app.
+  //
+  // Read through sessionController.page(), which is the supported history read and
+  // does not activate a cold session. The event shape is not a stable surface, so
+  // every field access is guarded and an unknown payload degrades to fewer lines
+  // rather than throwing.
+  disposers.push(ctx.webServer.register({
+    kind: "exact",
+    path: `${ROUTE_PATH}/recent`,
+    handler: async (req, res) => {
+      try {
+        const url = new URL(req.url ?? "/", "http://127.0.0.1");
+        const sessionId = (url.searchParams.get("sessionId") ?? "").trim();
+        const limit = Math.min(Math.max(Number(url.searchParams.get("limit")) || 3, 1), 20);
+        if (!sessionId) {
+          sendJson(res, 400, { ok: false, error: "sessionId is required" });
+          return;
+        }
+        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 texts = [];
+        for (const record of page?.records ?? []) {
+          const event = record?.event;
+          if (event?.type !== "user/message") continue;
+          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, {
+          ok: true,
+          sessionId,
+          total: texts.length,
+          // Newest last, so the caller can slice from the end.
+          messages: texts.slice(-limit),
+        });
+      } catch (error) {
+        sendJson(res, 500, { ok: false, error: String(error?.message ?? error) });
+      }
+    },
+  }));
+
+  disposers.push(ctx.webServer.register({
+    kind: "exact",
+    path: `${ROUTE_PATH}/health`,
+    handler: async (_req, res) => {
+      try {
+        const sessionController = ctx.get("sessionController");
+        const missing = sessionController
+          ? REQUIRED_CONTROLLER_METHODS.filter((name) => typeof sessionController[name] !== "function")
+          : REQUIRED_CONTROLLER_METHODS.slice();
+        sendJson(res, 200, {
+          ok: missing.length === 0,
+          controllerAvailable: Boolean(sessionController),
+          required: REQUIRED_CONTROLLER_METHODS,
+          missing,
+        });
+      } catch (error) {
+        sendJson(res, 500, { ok: false, error: String(error?.message ?? error) });
+      }
+    },
+  }));
+
   disposers.push(ctx.webServer.register({
     kind: "exact",
     path: ROUTE_PATH,
@@ -886,6 +1007,17 @@ export function apply(ctx, config = {}) {
   }));
 
   ctx.logger?.info?.(`phone-bridge listening on ${ROUTE_PATH}`);
+  {
+    const controller = ctx.get("sessionController");
+    const missing = controller
+      ? REQUIRED_CONTROLLER_METHODS.filter((name) => typeof controller[name] !== "function")
+      : REQUIRED_CONTROLLER_METHODS.slice();
+    if (missing.length > 0) {
+      ctx.logger?.warn?.(
+        `phone-bridge: this DSH build is missing ${missing.join(", ")} on sessionController - the routes that use them will fail. GET ${ROUTE_PATH}/health reports the same list.`,
+      );
+    }
+  }
 
   return async () => {
     for (const dispose of disposers) {

package/package.json

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