@frockbot/plugin-shell 0.3.22 → 0.3.23

package/src/client/index.ts

@@ -16,6 +16,7 @@ import {
 import { clientSurfaceRegistryKey } from "@frockbot/client-core";
 import { COMPACTED_ANNOUNCEMENT_TEXT_V1 } from "../compaction.js";
 import { voiceCaptureSupportedV1 } from "./voice-microphone.js";
+import { deploymentStaleV1 } from "./deployment.js";
 import { readViewerFocusV1, shouldNotifyForBotV1 } from "../focus.js";
 // Connection mutations use the provider-neutral hosted command contract.
 import type {
@@ -1433,6 +1434,19 @@ export const shellClientPlugin: ClientPlugin = (ctx) => {
     voiceAvailable:
       typeof ctx.transport.openVoiceDictation === "function" &&
       voiceCaptureSupportedV1(),
+    deploymentStale: false,
+    reloadHolds: 0,
+    holdReload: () => {
+      web.value.reloadHolds += 1;
+      let released = false;
+      return () => {
+        // Idempotent: a Package that lets go twice must not leave the count
+        // below zero, which would read as "another holder released early".
+        if (released) return;
+        released = true;
+        web.value.reloadHolds -= 1;
+      };
+    },
     activeBotId: undefined,
     composerContext: undefined,
     transcripts: {
@@ -3161,6 +3175,43 @@ export const shellClientPlugin: ClientPlugin = (ctx) => {
     { immediate: true },
   );
 
+  /*
+   * Following a release in a page that is already open.
+   *
+   * Every answer names the application that produced it and the request layer
+   * reports it here, so a page whose own application has stopped answering is
+   * running code the backend has moved past. Nothing polls for this: the
+   * reads the app already makes carry the answer.
+   */
+  const stopDeploymentWatch = ctx.transport.observeDeployment?.((answered) => {
+    if (deploymentStaleV1(ctx.transport.servedDeployment, answered)) {
+      web.value.deploymentStale = true;
+    }
+  });
+  /*
+   * A tab that has been in the background for a day should not have to wait
+   * for the next scheduled read to find out. Coming back asks for the
+   * manifest the client already reads at start-up rather than adding a poll
+   * of its own, and only while the answer is still unknown.
+   */
+  const readManifest = ctx.transport.readApplicationManifest?.bind(
+    ctx.transport,
+  );
+  const followDeploymentOnReturn = () => {
+    if (document.visibilityState !== "visible") return;
+    if (web.value.deploymentStale || !readManifest) return;
+    void readManifest().catch(() => {
+      // A read that failed says nothing about the deployment. The next one,
+      // or the next answer of any kind, will.
+    });
+  };
+  // The Plugin is also installed without a document — the unit tests drive it
+  // headless — and there a tab can neither be hidden nor come back.
+  const inDocument = typeof document !== "undefined";
+  if (inDocument) {
+    document.addEventListener("visibilitychange", followDeploymentOnReturn);
+  }
+
   return [
     ctx.provide(clientSurfaceRegistryKey, surfaces),
     // The shared client projection is updated by the contracts lane. This cast
@@ -3177,6 +3228,13 @@ export const shellClientPlugin: ClientPlugin = (ctx) => {
       component: PackageIframeSettings,
     }),
     () => {
+      stopDeploymentWatch?.();
+      if (inDocument) {
+        document.removeEventListener(
+          "visibilitychange",
+          followDeploymentOnReturn,
+        );
+      }
       stopEntrySync();
       stopRunFollow();
       stopRunChannelWatch();

package/src/client/styles.css

@@ -644,6 +644,45 @@
   cursor: pointer;
 }
 
+/*
+ * The release bar. It sits where the error banner sits and only one shows at
+ * a time, so it borrows that geometry and none of its colour: nothing here is
+ * wrong, there is just newer code waiting.
+ */
+.update-banner {
+  position: absolute;
+  right: 20px;
+  bottom: 84px;
+  left: 20px;
+  z-index: 5;
+  display: flex;
+  min-height: 38px;
+  align-items: center;
+  justify-content: space-between;
+  gap: 12px;
+  padding: 7px 12px;
+  border: 1px solid var(--frock-border);
+  border-radius: 12px;
+  color: var(--frock-text);
+  background: var(--frock-surface-raised);
+  box-shadow: var(--frock-shadow-card);
+  font-size: var(--frock-text-sm);
+}
+
+.update-banner button {
+  flex: 0 0 auto;
+  padding: 5px 10px;
+  border-radius: 999px;
+  color: var(--frock-on-accent);
+  background: var(--frock-action-primary);
+  font-weight: 600;
+  cursor: pointer;
+}
+
+.update-banner button:hover {
+  background: var(--frock-action-primary-hover);
+}
+
 .banner-enter-active,
 .banner-leave-active {
   transition:
@@ -732,25 +771,73 @@
   left: 0;
   display: inline-flex;
   max-width: 100%;
-  height: var(--frock-control-md);
+  min-height: var(--frock-control-md);
   align-items: center;
-  gap: 6px;
-  padding: 0 10px;
+  gap: 8px;
+  padding: 6px 10px;
   border: 1px solid var(--frock-border-strong);
-  border-radius: 999px;
+  border-radius: var(--frock-radius-card);
   color: var(--frock-text);
   background: var(--frock-surface-raised);
   box-shadow: var(--frock-shadow-control);
   font-size: var(--frock-text-sm);
+  text-align: left;
   cursor: pointer;
 }
 
+.applet-chip-text {
+  display: flex;
+  min-width: 0;
+  flex-direction: column;
+}
+
 .applet-chip-name {
   overflow: hidden;
   white-space: nowrap;
   text-overflow: ellipsis;
 }
 
+/*
+ * What the Bot is doing to it, so the phone says the same thing the canvas
+ * beside a wide conversation says without opening anything.
+ */
+.applet-chip-status {
+  display: flex;
+  overflow: hidden;
+  align-items: center;
+  gap: 6px;
+  color: var(--frock-text-muted);
+  font-size: var(--frock-text-xs);
+  white-space: nowrap;
+  text-overflow: ellipsis;
+}
+
+.applet-chip-dot {
+  width: 6px;
+  height: 6px;
+  flex: 0 0 auto;
+  border-radius: 999px;
+  background: var(--frock-action-primary);
+  animation: applet-chip-pulse 1.4s ease-in-out infinite;
+}
+
+@keyframes applet-chip-pulse {
+  0%,
+  100% {
+    opacity: 0.35;
+  }
+
+  50% {
+    opacity: 1;
+  }
+}
+
+@media (prefers-reduced-motion: reduce) {
+  .applet-chip-dot {
+    animation: none;
+  }
+}
+
 .applet-chip-action {
   flex: 0 0 auto;
   color: var(--frock-action-primary);

package/src/run-protocol.ts

@@ -45,6 +45,16 @@ const MAX_SESSION_ID_LENGTH = 320;
 const MAX_TASK_DESCRIPTION_BYTES = 800;
 const MAX_TASK_MODEL_BYTES = 512;
 export const CLIENT_RUN_PAGE_LIMIT = 32;
+/**
+ * How many run-index candidates one transcript page may walk past.
+ *
+ * The index is global and the transcript is one conversation, so filling a
+ * page can mean stepping over Turns of other conversations and over
+ * automation firings. Scanning is cheap — a run record, not its journal — but
+ * it is not free, so it stops at a budget and hands back a cursor rather than
+ * walking a Bot's whole history inside one request.
+ */
+export const CLIENT_RUN_SCAN_LIMIT = CLIENT_RUN_PAGE_LIMIT * 8;
 export const CLIENT_RUN_LIST_MAX_BYTES = 512_000;
 
 export type ClientRunStatusV1 =

package/src/shared.ts

@@ -269,6 +269,20 @@ export interface FrockBotWebData {
    * changes shape and nothing about it moves.
    */
   voiceAvailable: boolean;
+  /**
+   * The application answering is no longer the one this page was served, so
+   * the code running here is behind a release. FrockBot ships several times a
+   * day and a tab stays open for days, so this is ordinary rather than rare.
+   */
+  deploymentStale: boolean;
+  /**
+   * Live work a reload would throw away, held by whichever Package holds the
+   * work. Counted rather than flagged so two holders at once do not release
+   * each other. The shell reloads on its own only at zero.
+   */
+  reloadHolds: number;
+  /** Keeps the shell from reloading. Call the result to let go. */
+  holdReload(): () => void;
   activeBotId?: string;
   composerContext?: unknown;
   messages: WebChatMessage[];