@frockbot/plugin-shell 0.3.15 → 0.3.17

package/src/client/index.ts

@@ -363,6 +363,13 @@ function assistantMessage(
   run: ClientRun,
   notification: ClientNotificationIntent | undefined,
 ): WebChatMessage {
+  const syncNotice = run.events.find(
+    (event) => event.type === "computer/sync" && event.message,
+  )?.message;
+  const notice = (primary?: string): string | undefined =>
+    [primary, syncNotice]
+      .filter((part): part is string => Boolean(part))
+      .join(" ") || undefined;
   if (run.status === "running") {
     // A streaming Turn carries only the text the model has produced. Until
     // there is any, the thread shows the animated avatar and no bubble.
@@ -375,6 +382,7 @@ function assistantMessage(
       // A Turn that has not started shows nothing of its own: the greyed user
       // message is the whole of what the thread says about it.
       ...(run.queued ? { pending: true } : {}),
+      ...(syncNotice ? { notice: syncNotice } : {}),
       tools: toolsFrom(run.events),
       sends: [],
       tasks: tasksFrom(run.events),
@@ -391,6 +399,7 @@ function assistantMessage(
       role: "assistant",
       text: visibleAssistantText(run),
       status: "aborted",
+      ...(syncNotice ? { notice: syncNotice } : {}),
       tools: toolsFrom(run.events),
       sends: [],
       tasks: tasksFrom(run.events),
@@ -409,7 +418,7 @@ function assistantMessage(
        * exactly like the Bot speaking.
        */
       text: visibleAssistantText(run),
-      notice: "This reply stopped partway. Try again to continue it.",
+      notice: notice("This reply stopped partway. Try again to continue it."),
       status: "reconciliation-required",
       tools: toolsFrom(run.events),
       sends: [],
@@ -422,7 +431,7 @@ function assistantMessage(
       runId: run.runId,
       role: "assistant",
       text: visibleAssistantText(run),
-      notice: "You stopped this.",
+      notice: notice("You stopped this."),
       status: "aborted",
       tools: toolsFrom(run.events),
       sends: [],
@@ -447,7 +456,7 @@ function assistantMessage(
       // wire, so this keeps whatever that chose — the model-deadline copy says
       // something the outcome alone cannot — and falls back to the same line a
       // reply-less failure gets.
-      notice: knownFailureCopyV1(run.failure),
+      notice: notice(knownFailureCopyV1(run.failure)),
       status: "error",
       tools: toolsFrom(run.events),
       sends: [],
@@ -465,8 +474,10 @@ function assistantMessage(
     // Why the Turn ends there, under whatever it had already said — never as
     // the bubble's own text, which reads as the Bot saying it.
     ...(run.status === "failed"
-      ? { notice: knownFailureCopyV1(run.failure) }
-      : {}),
+      ? { notice: notice(knownFailureCopyV1(run.failure)) }
+      : syncNotice
+        ? { notice: syncNotice }
+        : {}),
     status: run.status === "failed" ? "error" : "completed",
     tools: toolsFrom(run.events),
     sends: [],
@@ -561,6 +572,7 @@ export function projectDurableRuns(
           ? { at: existingUser.at }
           : {}),
       status: "completed",
+      ...(run.via ? { via: run.via } : {}),
       // Greyed while its Turn waits, ordinary the moment it is running. The
       // flag comes from durable run state, so a reload draws the same thing.
       ...(run.queued ? { pending: true } : {}),
@@ -3070,6 +3082,14 @@ export const shellClientPlugin: ClientPlugin = (ctx) => {
    * and the one-bubble-per-send contract is untouched.
    */
   let stopRunChannel: (() => void) | undefined;
+  // The channel's own health, for the observation below: `fallback` means the
+  // socket is gone and the transcript is flying blind, and each return to
+  // `open` means it was gone for a while. Either is a reason to read the
+  // running Turn from authority rather than trust a POST that may have died
+  // with the same connection — which is how a phone kept drawing the working
+  // trail for a Turn the server had already settled (2026-09-04).
+  const channelFallback = ref(false);
+  const channelReconnects = ref(0);
   const stopRunChannelWatch = watch(
     () => web.value.activeBotId,
     (botId) => {
@@ -3096,10 +3116,14 @@ export const shellClientPlugin: ClientPlugin = (ctx) => {
           restoredWithoutRead = undefined;
           await deliverNotifications(botId, generation);
         },
-        status() {
+        status(status) {
           // The channel's health is not the transcript's: an unavailable
           // socket falls back to the observation below, which is what a
           // client without one uses anyway.
+          if (generation !== selectionGeneration) return;
+          const wasDown = channelFallback.value;
+          channelFallback.value = status === "fallback";
+          if (status === "open" && wasDown) channelReconnects.value += 1;
         },
       });
     },
@@ -3107,12 +3131,21 @@ export const shellClientPlugin: ClientPlugin = (ctx) => {
   );
 
   const stopRunObservation = watch(
-    () => [web.value.activeBotId, web.value.activeRunId] as const,
-    ([botId, runId]) => {
+    () =>
+      [
+        web.value.activeBotId,
+        web.value.activeRunId,
+        channelFallback.value,
+        channelReconnects.value,
+      ] as const,
+    ([botId, runId, fallback]) => {
       // The send path owns the run it started: its POST is the observation,
       // and `stopRun` starts its own. This is for every other way a client
-      // finds itself watching a Turn it is not holding open.
-      if (!botId || !runId || activeRequest || runObserver) return;
+      // finds itself watching a Turn it is not holding open — and, once the
+      // state channel has dropped or come back, for the Turn it *is* holding
+      // open, because that POST shared the connection that just failed.
+      if (!botId || !runId || runObserver) return;
+      if (activeRequest && !fallback && channelReconnects.value === 0) return;
       const generation = selectionGeneration;
       const observer = new AbortController();
       runObserver = observer;

package/src/client/mobile-safe-area.test.ts

@@ -0,0 +1,25 @@
+import { describe, expect, test } from "bun:test";
+
+const styles = await Bun.file(new URL("./styles.css", import.meta.url)).text();
+
+describe("phone safe-area layout", () => {
+  test("keeps the conversation header and thread below the status bar", () => {
+    expect(styles).toContain(
+      "height: calc(var(--frock-titlebar-height) + var(--frock-safe-top));",
+    );
+    expect(styles).toContain("padding-top: calc(var(--frock-safe-top) + 0px);");
+    expect(styles).toContain(
+      "top: calc(var(--frock-titlebar-height) + var(--frock-safe-top));",
+    );
+  });
+
+  test("keeps drawers, Settings, and the composer clear of both bars", () => {
+    expect(styles).toContain(
+      "padding-bottom: calc(10px + var(--frock-safe-bottom));",
+    );
+    expect(styles).toContain(
+      "padding-bottom: calc(16px + var(--frock-safe-bottom));",
+    );
+    expect(styles).toContain("bottom: calc(10px + var(--frock-safe-bottom));");
+  });
+});

package/src/client/styles.css

@@ -499,6 +499,17 @@
   background: var(--frock-action-primary);
 }
 
+.message-user-column {
+  display: grid;
+  justify-items: end;
+  gap: 4px;
+}
+
+.message-via {
+  color: var(--frock-text-muted);
+  font-size: var(--frock-text-xs);
+}
+
 /*
  * User-facing sends. They stack under the Bot's avatar in the same column the
  * text bubble occupies, so a Turn that only sent a widget still reads as the
@@ -1118,6 +1129,7 @@
     /* `calc` so the declaration is a length the checker can type; `env` alone
        is not one it knows. */
     padding-top: calc(var(--frock-safe-top) + 0px);
+    padding-bottom: calc(10px + var(--frock-safe-bottom));
     box-shadow: var(--frock-shadow-panel);
     transform: translateX(-100%);
     visibility: hidden;
@@ -1189,12 +1201,27 @@
   /* No window chrome to clear, but the panel toggle still sits at the
      trailing edge, so the row ends before it rather than under it. */
   .topbar {
+    height: calc(var(--frock-titlebar-height) + var(--frock-safe-top));
     gap: 8px;
     padding: 0 52px 0 4px;
+    padding-top: calc(var(--frock-safe-top) + 0px);
   }
 
   .window-actions {
+    height: calc(var(--frock-titlebar-height) + var(--frock-safe-top));
     padding: 0 8px;
+    padding-top: calc(var(--frock-safe-top) + 0px);
+  }
+
+  .right-panel-header,
+  .panel-surface-header {
+    height: calc(var(--frock-titlebar-height) + var(--frock-safe-top));
+    padding-top: calc(var(--frock-safe-top) + 0px);
+  }
+
+  .right-panel-body,
+  .panel-surface-content {
+    padding-bottom: calc(16px + var(--frock-safe-bottom));
   }
 
   .brand-mark {
@@ -1211,6 +1238,7 @@
    * vertical anchors all clear the home indicator.
    */
   .thread {
+    top: calc(var(--frock-titlebar-height) + var(--frock-safe-top));
     bottom: calc(76px + var(--frock-safe-bottom));
     padding: 12px 12px 20px;
   }

package/src/compaction.test.ts

@@ -15,6 +15,7 @@ import {
   parseCompactionSummaryV1,
   PRUNED_TOOL_RESULT_V1,
   pruneToolOutputsV1,
+  renderCompactionSummaryV1,
   runCompactionV1,
   COMPACTION_TRIGGER_RATIO_V1,
 } from "./compaction.js";
@@ -385,6 +386,31 @@ describe("injecting a compaction", () => {
 });
 
 describe("reading the summariser's answer", () => {
+  test("renders the validated fields into the durable heading format", () => {
+    expect(
+      renderCompactionSummaryV1({
+        summary: "Work on the Applet.",
+        decisions: ["Keep the first design."],
+        openItems: [],
+        identifiers: ["applet-9f2c"],
+      }),
+    ).toBe(
+      [
+        "## Summary",
+        "Work on the Applet.",
+        "",
+        "## Decisions",
+        "- Keep the first design.",
+        "",
+        "## Open items",
+        "- none",
+        "",
+        "## Identifiers mentioned",
+        "- applet-9f2c",
+      ].join("\n"),
+    );
+  });
+
   test("lifts the identifiers out of their heading", () => {
     const parsed = parseCompactionSummaryV1(
       [

package/src/compaction.ts

@@ -26,6 +26,7 @@ import {
   type ModelBindingSnapshot,
   type Session,
   type SessionEvent,
+  type StructuredOutputSchemaV1,
 } from "@frockbot/kernel-contracts";
 
 /**
@@ -262,27 +263,58 @@ export function assessCompactionV1(input: {
  * ids in passing, and the list is what the event stores.
  */
 export const COMPACTION_SYSTEM_PROMPT_V1 = [
-  "You are compressing the earlier part of a conversation so it can be carried forward in a smaller prompt. Write the summary, and nothing else.",
+  "You are compressing the earlier part of a conversation so it can be carried forward in a smaller prompt.",
   "",
   "CRITICAL: You MUST preserve ALL opaque identifiers exactly as they appear. That includes UUIDs, hashes, full URLs with their query parameters, file and Workspace paths, Package ids, Applet ids, Bot ids, Session ids, tool call ids, model names and version strings. Do NOT paraphrase, abbreviate, or generalise an identifier. Copy it exactly.",
   "",
-  "Use exactly these headings, in this order, and omit none of them:",
-  "",
-  "## Summary",
-  "What the conversation is about and what has happened, in a few short paragraphs or bullets.",
-  "",
-  "## Decisions",
-  "Decisions made and the reason for each. Where a decision was later changed, keep only the latest and say it superseded an earlier one.",
-  "",
-  "## Open items",
-  "Work that is pending, promised, or unfinished. Be specific about what is owed and by whom.",
-  "",
-  "## Identifiers mentioned",
-  "A bullet list of every opaque identifier that appeared, one per line, copied exactly. Write `- none` if there were none.",
+  "Put the gist in `summary`, decisions and their reasons in `decisions`, pending work in `openItems`, and every opaque identifier copied exactly in `identifiers`. Keep only the latest decision where one superseded another.",
   "",
   "Leave out pleasantries, repetition, and superseded detail. Do not invent anything that is not in the transcript. Do not address the user.",
 ].join("\n");
 
+export interface CompactionSummaryPayloadV1 {
+  summary: string;
+  decisions: string[];
+  openItems: string[];
+  identifiers: string[];
+}
+
+/** The actual production consumer of the shared structured-output seam. */
+export const COMPACTION_RESPONSE_SCHEMA_V1 = {
+  type: "object",
+  properties: {
+    summary: { type: "string" },
+    decisions: { type: "array", items: { type: "string" } },
+    openItems: { type: "array", items: { type: "string" } },
+    identifiers: { type: "array", items: { type: "string" } },
+  },
+  required: ["summary", "decisions", "openItems", "identifiers"],
+  additionalProperties: false,
+} as const satisfies StructuredOutputSchemaV1;
+
+/** Keeps the durable summary format readable while model I/O stays typed. */
+export function renderCompactionSummaryV1(
+  payload: CompactionSummaryPayloadV1,
+): string {
+  const bullets = (values: readonly string[]) =>
+    values.length > 0
+      ? values.map((value) => `- ${value}`).join("\n")
+      : "- none";
+  return [
+    "## Summary",
+    payload.summary,
+    "",
+    "## Decisions",
+    bullets(payload.decisions),
+    "",
+    "## Open items",
+    bullets(payload.openItems),
+    "",
+    "## Identifiers mentioned",
+    bullets(payload.identifiers),
+  ].join("\n");
+}
+
 /** The transcript one summariser call is given, flattened to plain text. */
 export function compactionTranscriptV1(
   messages: readonly LlmMessage[],

package/src/debug-protocol.ts

@@ -20,9 +20,10 @@ export const BOT_DEBUG_RUN_LIMIT_V1 = 20;
 export const BOT_DEBUG_DEFAULT_RUN_LIMIT_V1 = 5;
 export const BOT_DEBUG_GENERATION_LIMIT_V1 = 5;
 /**
- * The event budget one snapshot spends. Session events carry whole prompts, so
- * a handful of runs can be megabytes; past this the *oldest* events of a run
- * are dropped, because a failure is described by the tail of its log.
+ * The event budget one snapshot spends. Large durable events are already
+ * bounded diagnostic projections, but a handful of runs can still accumulate
+ * many of them; past this the *oldest* events of a run are dropped, because a
+ * failure is described by the tail of its log.
  */
 export const BOT_DEBUG_EVENT_BYTES_V1 = 512_000;
 

package/src/history.ts

@@ -175,7 +175,10 @@ export function chatWindowV1(
 ): ChatWindowV1 {
   const turns = messageTurnsV1(events);
   const types = turnTypesByTurnV1(events);
-  const chat = (turn: number) => (types.get(turn) ?? "chat") === "chat";
+  const chat = (turn: number) => {
+    const type = types.get(turn) ?? "chat";
+    return type === "chat" || type === "agent";
+  };
   const state = compactionStateV1(events);
   const current = currentTurnV1(events);
   // A compaction never covers the Turn being assembled, whatever the log says:
@@ -229,7 +232,10 @@ export function turnScopedMessagesV1(
   }
   const types = turnTypesByTurnV1(input.events);
   const current = currentTurnV1(input.events);
-  const chatTurn = (turn: number) => (types.get(turn) ?? "chat") === "chat";
+  const chatTurn = (turn: number) => {
+    const type = types.get(turn) ?? "chat";
+    return type === "chat" || type === "agent";
+  };
   if (chatTurn(current)) {
     const window = chatWindowV1(input.events, input.messages);
     // Tier 1 of ADR 0030, and the only one that costs nothing: a tool result

package/src/run-failure-copy.test.ts

@@ -1,3 +1,4 @@
+import { STEP_LIMIT_REASON_V1 } from "@frockbot/kernel-agent-loop";
 import { describe, expect, test } from "bun:test";
 import type { SessionEvent } from "@frockbot/kernel-contracts";
 import { MODEL_FIRST_BYTE_DEADLINE_REASON_V1 } from "@frockbot/kernel-contracts";
@@ -157,3 +158,13 @@ describe("runFailureCopyV1", () => {
     expect(projected.outcome.message).toBe(RUN_FAILURE_COPY_V1.interrupted);
   });
 });
+
+test("a reply that ran out of steps says so in plain words", () => {
+  // `kernel-do` wraps the loop's reason; the sentence survives the wrapper.
+  expect(
+    runFailureCopyV1({
+      failure: `Bot turn ended with outcome interrupted: ${STEP_LIMIT_REASON_V1}`,
+    }),
+  ).toBe(STEP_LIMIT_REASON_V1);
+  expect(knownFailureCopyV1(STEP_LIMIT_REASON_V1)).toBe(STEP_LIMIT_REASON_V1);
+});

package/src/run-failure-copy.ts

@@ -7,7 +7,10 @@ import {
   MODEL_FIRST_BYTE_DEADLINE_REASON_V1,
   MODEL_IDLE_DEADLINE_REASON_V1,
 } from "@frockbot/kernel-contracts";
-import { TURN_DEADLINE_REASON_V1 } from "@frockbot/kernel-agent-loop";
+import {
+  STEP_LIMIT_REASON_V1,
+  TURN_DEADLINE_REASON_V1,
+} from "@frockbot/kernel-agent-loop";
 import { UNRECONCILABLE_RUN_FAILURE_V1 } from "@frockbot/kernel-do";
 
 /**
@@ -47,6 +50,7 @@ export const USER_FACING_FAILURE_REASONS_V1: readonly string[] = [
   MODEL_FIRST_BYTE_DEADLINE_REASON_V1,
   MODEL_IDLE_DEADLINE_REASON_V1,
   TURN_DEADLINE_REASON_V1,
+  STEP_LIMIT_REASON_V1,
   UNRECONCILABLE_RUN_FAILURE_V1,
 ];
 

package/src/run-protocol.test.ts

@@ -24,6 +24,7 @@ import {
   projectClientRunLookupV1,
   projectClientRunListV1,
   projectClientRunV1,
+  isVisibleRunV1,
   projectClientRunOrDegradedV1,
   projectClientTurnV1,
   UNRECORDED_TOOL_RESULT_TEXT_V1,
@@ -88,6 +89,37 @@ function storedRun(
 }
 
 describe("client run protocol v1", () => {
+  test("projects an agent Turn with its Bot origin marker", () => {
+    const agent = {
+      ...storedRun([]),
+      admission: {
+        schemaVersion: 1 as const,
+        turnType: "agent" as const,
+        origin: {
+          kind: "bot" as const,
+          fromBotId: "researcher",
+          fromBotName: "Researcher",
+          messageId: "message-1",
+        },
+      },
+    };
+
+    expect(isVisibleRunV1(agent)).toBe(true);
+    const projected = projectClientRunV1(agent);
+    expect(projected).toMatchObject({
+      schemaVersion: 3,
+      input: "continue",
+      via: { kind: "bot", name: "Researcher", botId: "researcher" },
+    });
+    expect(
+      decodeClientRunListV1({
+        schemaVersion: 1,
+        runs: [projected],
+        page: { truncated: false },
+      })[0]?.via,
+    ).toEqual(projected.via);
+  });
+
   test("rejects durable runs missing current admission fields", () => {
     const complete = storedRun([], "running");
     for (const field of [
@@ -728,7 +760,7 @@ describe("client run protocol v1", () => {
       schemaVersion: 1,
       runs: [
         {
-          schemaVersion: 2,
+          schemaVersion: 3,
           runId: "run-1",
           admittedAt: timestamp,
           input: "continue",
@@ -977,9 +1009,8 @@ describe("client run protocol v1", () => {
       ]),
     );
 
-    // Version 2 is what carries the two new event types; a client pinned to 1
-    // still decodes the body it produces.
-    expect(projected.schemaVersion).toBe(2);
+    // Version 3 carries agent-origin markers; older bodies still decode.
+    expect(projected.schemaVersion).toBe(3);
     expect(projected.events).toEqual([
       { type: "send/to-user", payload: { type: "text", text: "On it." } },
       { type: "tool/call", call: { id: "tool-1", name: "lookup" } },
@@ -1614,3 +1645,41 @@ describe("dispatched subagents in the run projection", () => {
     );
   });
 });
+
+describe("Computer sync degradation in the run projection", () => {
+  const degraded: SessionEvent = {
+    type: "computer/sync",
+    seq: 4,
+    timestamp,
+    turn: 1,
+    reason: "open",
+    status: "degraded",
+    detail: "Excluded 2 reproducible Workspace items from sync.",
+    pulled: 0,
+    pushed: 1,
+    restored: 0,
+    removed: 0,
+    adopted: 0,
+    conflicts: 0,
+    failures: 0,
+    ignored: 2,
+    omitted: 0,
+  };
+
+  test("projects one plain-language event and round-trips it", () => {
+    const projected = projectClientRunV1(
+      storedRun([degraded, { ...degraded, seq: 5, reason: "turn-end" }]),
+    );
+    expect(projected.events).toEqual([
+      {
+        type: "computer/sync",
+        status: "degraded",
+        message: "Excluded 2 reproducible Workspace items from sync.",
+      },
+    ]);
+    const page = createClientRunListV1([projected], { truncated: false });
+    expect(
+      decodeClientRunPageV1(structuredClone(page)).runs[0]?.events,
+    ).toEqual(projected.events);
+  });
+});