@opengeni/react 1.4.4 → 1.4.5-canary.37458084490001

package/src/timeline/platform-activity-presentation.ts

@@ -0,0 +1,122 @@
+import type { SandboxItem, StartupPhaseItem, WorkerItem } from "./types";
+
+/* Pure titles and durations for platform activity rows (startup phases and
+   sandbox operations), shared by DOM and non-DOM renderers. */
+
+export function sandboxRowTitle(item: SandboxItem, displayName: (name: string) => string): string {
+  if (item.name === "sandbox.provision") {
+    return startupPhaseTitle(
+      "sandbox",
+      item.status === "cancelled" ? "complete" : item.status,
+      item.origin ?? null,
+    );
+  }
+  return displayName(item.name);
+}
+
+export function startupPhaseTitle(
+  phase: StartupPhaseItem["phase"],
+  status: StartupPhaseItem["status"],
+  outcome: StartupPhaseItem["outcome"],
+): string {
+  if (status === "complete" && phase === "sandbox" && outcome && outcome !== "skipped") {
+    return `Sandbox ${outcome === "resumed" ? "reattached" : outcome}`;
+  }
+  if (status === "complete" && phase === "rig" && outcome === "skipped") {
+    return "Sandbox Environment already ready";
+  }
+  const statusIndex =
+    status === "running" ? 0 : status === "failed" ? 1 : status === "cancelled" ? 2 : 3;
+  return STARTUP_PHASE_TITLES[phase][statusIndex];
+}
+
+export const STARTUP_WAIT_TITLES: Record<NonNullable<StartupPhaseItem["blockedReason"]>, string> = {
+  capture_in_progress: "Waiting for sandbox checkpoint",
+  rotation_in_progress: "Waiting for sandbox rotation",
+  provider_recovery_in_progress: "Waiting for sandbox recovery",
+};
+
+const STARTUP_PHASE_TITLES: Record<
+  StartupPhaseItem["phase"],
+  readonly [string, string, string, string]
+> = {
+  queue: [
+    "Waiting for a worker",
+    "Worker startup failed",
+    "Worker wait interrupted",
+    "Worker started",
+  ],
+  sandbox: [
+    "Starting sandbox",
+    "Sandbox didn’t start",
+    "Sandbox startup interrupted",
+    "Sandbox ready",
+  ],
+  rig: [
+    "Setting up sandbox environment",
+    "Sandbox Environment setup failed",
+    "Sandbox Environment setup interrupted",
+    "Sandbox Environment ready",
+  ],
+  repository: [
+    "Preparing repository",
+    "Repository preparation failed",
+    "Repository preparation interrupted",
+    "Repository ready",
+  ],
+  files: [
+    "Preparing files",
+    "File preparation failed",
+    "File preparation interrupted",
+    "Files ready",
+  ],
+  tools: [
+    "Connecting tools",
+    "Tool connection failed",
+    "Tool connection interrupted",
+    "Tools ready",
+  ],
+  model_preparation: [
+    "Preparing runtime and model request",
+    "Runtime/model preparation failed",
+    "Runtime/model preparation interrupted",
+    "Model request dispatched",
+  ],
+  provider_first_byte: [
+    "Waiting for model",
+    "Model didn’t respond",
+    "Model wait interrupted",
+    "Model started responding",
+  ],
+};
+
+export function startupDuration(durationMs: number | null): string | null {
+  if (durationMs === null || !Number.isFinite(durationMs) || durationMs < 0) return null;
+  if (durationMs < 1_000) return `${Math.round(durationMs)}ms`;
+  if (durationMs < 60_000) return `${(durationMs / 1_000).toFixed(1)}s`;
+  const minutes = Math.floor(durationMs / 60_000);
+  const seconds = Math.round((durationMs % 60_000) / 1_000);
+  return `${minutes}m ${String(seconds).padStart(2, "0")}s`;
+}
+
+/** The worker step title for a spawn or message, by status. */
+export function workerRowTitle(item: Pick<WorkerItem, "action" | "status">): string {
+  const running = item.status === "running";
+  const failed = item.status === "failed";
+  const cancelled = item.status === "cancelled";
+  return item.action === "spawn"
+    ? running
+      ? "Spawning worker"
+      : failed
+        ? "Worker spawn failed"
+        : cancelled
+          ? "Worker interrupted"
+          : "Worker spawned"
+    : running
+      ? "Messaging worker"
+      : failed
+        ? "Worker message failed"
+        : cancelled
+          ? "Worker interrupted"
+          : "Worker messaged";
+}

package/src/timeline/platform-activity-row.tsx

@@ -1,3 +1,9 @@
+import {
+  sandboxRowTitle,
+  startupDuration,
+  startupPhaseTitle,
+  STARTUP_WAIT_TITLES,
+} from "./platform-activity-presentation";
 import type { ReasoningItem, SandboxItem, StartupPhaseItem } from "./types";
 
 type RowJsx = typeof import("react/jsx-runtime").jsx;
@@ -118,99 +124,3 @@ function lucideIcon(j: RowJsx, s: RowJsxs, name: string, children: ReturnType<Ro
     children,
   });
 }
-
-function sandboxRowTitle(item: SandboxItem, displayName: (name: string) => string): string {
-  if (item.name === "sandbox.provision") {
-    return startupPhaseTitle(
-      "sandbox",
-      item.status === "cancelled" ? "complete" : item.status,
-      item.origin ?? null,
-    );
-  }
-  return displayName(item.name);
-}
-
-function startupPhaseTitle(
-  phase: StartupPhaseItem["phase"],
-  status: StartupPhaseItem["status"],
-  outcome: StartupPhaseItem["outcome"],
-): string {
-  if (status === "complete" && phase === "sandbox" && outcome && outcome !== "skipped") {
-    return `Sandbox ${outcome === "resumed" ? "reattached" : outcome}`;
-  }
-  if (status === "complete" && phase === "rig" && outcome === "skipped") {
-    return "Sandbox Environment already ready";
-  }
-  const statusIndex =
-    status === "running" ? 0 : status === "failed" ? 1 : status === "cancelled" ? 2 : 3;
-  return STARTUP_PHASE_TITLES[phase][statusIndex];
-}
-
-const STARTUP_WAIT_TITLES: Record<NonNullable<StartupPhaseItem["blockedReason"]>, string> = {
-  capture_in_progress: "Waiting for sandbox checkpoint",
-  rotation_in_progress: "Waiting for sandbox rotation",
-  provider_recovery_in_progress: "Waiting for sandbox recovery",
-};
-
-const STARTUP_PHASE_TITLES: Record<
-  StartupPhaseItem["phase"],
-  readonly [string, string, string, string]
-> = {
-  queue: [
-    "Waiting for a worker",
-    "Worker startup failed",
-    "Worker wait interrupted",
-    "Worker started",
-  ],
-  sandbox: [
-    "Starting sandbox",
-    "Sandbox didn’t start",
-    "Sandbox startup interrupted",
-    "Sandbox ready",
-  ],
-  rig: [
-    "Setting up sandbox environment",
-    "Sandbox Environment setup failed",
-    "Sandbox Environment setup interrupted",
-    "Sandbox Environment ready",
-  ],
-  repository: [
-    "Preparing repository",
-    "Repository preparation failed",
-    "Repository preparation interrupted",
-    "Repository ready",
-  ],
-  files: [
-    "Preparing files",
-    "File preparation failed",
-    "File preparation interrupted",
-    "Files ready",
-  ],
-  tools: [
-    "Connecting tools",
-    "Tool connection failed",
-    "Tool connection interrupted",
-    "Tools ready",
-  ],
-  model_preparation: [
-    "Preparing runtime and model request",
-    "Runtime/model preparation failed",
-    "Runtime/model preparation interrupted",
-    "Model request dispatched",
-  ],
-  provider_first_byte: [
-    "Waiting for model",
-    "Model didn’t respond",
-    "Model wait interrupted",
-    "Model started responding",
-  ],
-};
-
-function startupDuration(durationMs: number | null): string | null {
-  if (durationMs === null || !Number.isFinite(durationMs) || durationMs < 0) return null;
-  if (durationMs < 1_000) return `${Math.round(durationMs)}ms`;
-  if (durationMs < 60_000) return `${(durationMs / 1_000).toFixed(1)}s`;
-  const minutes = Math.floor(durationMs / 60_000);
-  const seconds = Math.round((durationMs % 60_000) / 1_000);
-  return `${minutes}m ${String(seconds).padStart(2, "0")}s`;
-}

package/src/timeline/presented-image.ts

@@ -20,7 +20,15 @@ export function isRetainedImageContentType(contentType: string): boolean {
   ].includes(contentType);
 }
 
-/** Deliberately published images are primary output, not incidental screenshots. */
+/** A published patch, applied locally with `git apply`. */
+export function isPatchFilename(filename: string): boolean {
+  return /\.(patch|diff)$/iu.test(filename.trim());
+}
+
+/**
+ * Deliberately published images and patches are primary output, not
+ * incidental screenshots or work files.
+ */
 export function activityPresentsImage(items: readonly ActivityItem[]): boolean {
   return items.some((item) => {
     if (item.kind !== "tool-call") return false;
@@ -30,7 +38,13 @@ export function activityPresentsImage(items: readonly ActivityItem[]): boolean {
     const output = normalizeMcpOutput(item.output);
     if (output.isError) return false;
     const receipt = parseSandboxFileArtifactReceipt(output.text);
-    return receipt !== null && isRetainedImageContentType(receipt.artifact.contentType);
+    return (
+      receipt !== null &&
+      (isRetainedImageContentType(receipt.artifact.contentType) ||
+        // A published patch is the deliverable when there's no pull request:
+        // keep its card (with the apply command) visible like an image.
+        isPatchFilename(receipt.filename))
+    );
   });
 }
 

package/src/timeline/question-nav-model.ts

@@ -0,0 +1,28 @@
+/**
+ * "Back to your message": which prompt the reader is reading the answer to, and
+ * whether that prompt has scrolled out of view. Renderer-neutral geometry so
+ * the web timeline (DOM rects) and native timelines (layout offsets) agree.
+ */
+
+/** How far above the viewport a prompt must be before the pill appears. */
+export const QUESTION_NAV_HIDDEN_PX = 24;
+/** Breathing room above the question when the reader jumps back to it. */
+export const QUESTION_NAV_MARGIN_PX = 12;
+
+export type QuestionNavPrompt = { key: string; top: number; bottom: number };
+
+/**
+ * The last prompt before the reading position (the viewport's middle) owns the
+ * response in view; it is a target only once it has scrolled fully above the
+ * viewport. Prompts are in document order; coordinates share one axis.
+ */
+export function questionNavTarget(
+  prompts: readonly QuestionNavPrompt[],
+  view: { top: number; height: number },
+): string | null {
+  const middle = view.top + view.height / 2;
+  const prompt = [...prompts].reverse().find((candidate) => candidate.top <= middle);
+  if (!prompt) return null;
+  if (prompt.top >= view.top - QUESTION_NAV_HIDDEN_PX || prompt.bottom > view.top) return null;
+  return prompt.key;
+}

package/src/timeline/registry.ts

@@ -35,7 +35,17 @@ export type RetainedScreenshotLoader = (
 export type RetainedArtifactLoader = (
   artifact: RetainedArtifactReference,
   signal: AbortSignal,
-) => Promise<Uint8Array | { url: string } | null>;
+  options?: RetainedArtifactLoadOptions,
+) => Promise<Uint8Array | { url: string; expiresAt?: string | undefined } | null>;
+
+export type RetainedArtifactLoadOptions = {
+  /**
+   * Ask for a short-lived download URL instead of bytes, for example to build a
+   * copyable `curl … | git apply` command. A loader that cannot mint one may
+   * still return bytes; callers must handle that.
+   */
+  prefer?: "url" | undefined;
+};
 
 /** Mint an expiring source for native browser playback without loading video bytes in JS. */
 export type VideoArtifactPlaybackLoader = (

package/src/timeline/retained-loaders.ts

@@ -1,5 +1,6 @@
 import type { OpenGeniClient, RetainedArtifactReference } from "@opengeni/sdk";
 import type {
+  RetainedArtifactLoadOptions,
   RetainedArtifactLoader,
   RetainedScreenshotLoader,
   VideoArtifactPlaybackLoader,
@@ -21,21 +22,25 @@ export type RetainedVideoLoaderClient = Pick<OpenGeniClient, "createVideoArtifac
  * Bind permanent retained-artifact retrieval (generated images, published
  * files) to one authenticated workspace, for `MessageTimeline`'s
  * `loadRetainedArtifact`. Images get a short-lived URL; files are assembled
- * from verified byte ranges.
+ * from verified byte ranges unless the caller asks for a URL.
  */
 export function createWorkspaceRetainedArtifactLoader(
   client: RetainedArtifactLoaderClient,
   workspaceId: string,
 ): RetainedArtifactLoader {
-  return async (artifact: RetainedArtifactReference, signal: AbortSignal) => {
-    if (artifact.kind === "file") {
+  return async (
+    artifact: RetainedArtifactReference,
+    signal: AbortSignal,
+    options?: RetainedArtifactLoadOptions,
+  ) => {
+    if (artifact.kind === "file" && options?.prefer !== "url") {
       const download = await client.downloadRetainedArtifact(workspaceId, artifact, { signal });
       return download.bytes;
     }
     const download = await client.createRetainedArtifactDownloadUrl(workspaceId, artifact, {
       signal,
     });
-    return { url: download.url };
+    return { url: download.url, expiresAt: download.expiresAt };
   };
 }
 

package/src/timeline/rolling-activity.tsx

@@ -7,6 +7,7 @@ import { defaultToolRegistry } from "./tool-renderers";
 import type { ToolRegistry } from "./registry";
 import { toolDisplayName } from "./tool-display-name";
 import type { ActivityItem } from "./types";
+import { rollingActivityItem } from "./work-presentation";
 
 /** One stable viewport; updates replace its content without moving the conversation. */
 export function RollingActivity({
@@ -25,21 +26,11 @@ export function RollingActivity({
   const reduced = useReducedMotion();
   const [mounted, setMounted] = useState(false);
   useEffect(() => setMounted(true), []);
-  // Progress notes have their own line in the status row; the reel shows steps.
-  const work = items.filter(
-    (item) => item.kind !== "startup-phase" && item.kind !== "agent-message",
-  );
-  const active = work.filter((item) =>
-    item.kind === "reasoning" ? item.streaming : "status" in item && item.status === "running",
-  );
-  // Advance with the event order; finishing a parallel tool must not replay an older one.
-  const item =
-    !mounted && previousItem && previousItem.kind !== "agent-message" ? previousItem : work.at(-1);
-  if (!item) return null;
-  const earlierCount = Math.max(
-    0,
-    work.findIndex((entry) => entry.id === item.id),
-  );
+  // Progress notes have their own line in the status row; the reel shows steps, advancing
+  // with the event order so finishing a parallel tool never replays an older one.
+  const selected = rollingActivityItem(items, previousItem, mounted);
+  if (!selected) return null;
+  const { item, earlierCount } = selected;
   const fallback = (
     <span className="og-rolling-label">
       <WrenchIcon className="size-3.5" />
@@ -49,10 +40,7 @@ export function RollingActivity({
     </span>
   );
   return (
-    <span
-      className="og-rolling-status"
-      data-running={active.some((entry) => entry.id === item.id) ? "true" : undefined}
-    >
+    <span className="og-rolling-status" data-running={selected.running ? "true" : undefined}>
       <span className="sr-only">
         {item.kind === "tool-call"
           ? toolDisplayName(item.name, item.display)