replicas-cli 0.3.217 → 0.3.219

package/dist/index.cjs

@@ -19900,7 +19900,7 @@ var init_visuals = __esm({
     VISUAL_COLUMN_WIDTH = 760;
     VISUAL_PHONE_WIDTH = 390;
     VISUAL_MIN_HEIGHT = 24;
-    VISUAL_MAX_HEIGHT = 4e3;
+    VISUAL_MAX_HEIGHT = 2e4;
     VISUAL_FONT_PATH = "/fonts/GeneralSans-Variable.woff2";
     FONT_SANS = '"General Sans", system-ui, -apple-system, "Segoe UI", sans-serif';
     FONT_MONO = '"Geist Mono", ui-monospace, SFMono-Regular, Menlo, Consolas, monospace';
@@ -29942,6 +29942,33 @@ function readClaudeUserPrompt(payload, content) {
 }
 function parseClaudeEvents(events, parentToolUseId) {
   const messages = [];
+  const indexesById = /* @__PURE__ */ new Map();
+  let indexedCount = 0;
+  const indexesOf = (id) => {
+    for (; indexedCount < messages.length; indexedCount++) {
+      const indexes = indexesById.get(messages[indexedCount].id);
+      if (indexes) indexes.push(indexedCount);
+      else indexesById.set(messages[indexedCount].id, [indexedCount]);
+    }
+    return indexesById.get(id) ?? [];
+  };
+  const replaceMessage = (index, message2) => {
+    const previousId = messages[index].id;
+    messages[index] = message2;
+    if (index >= indexedCount || previousId === message2.id) return;
+    const previous = indexesById.get(previousId);
+    if (!previous) throw new Error(`Chat message ${previousId} is missing from the id index`);
+    previous.splice(previous.indexOf(index), 1);
+    const next = indexesById.get(message2.id);
+    if (!next) indexesById.set(message2.id, [index]);
+    else next.splice(next.filter((other) => other < index).length, 0, index);
+  };
+  const upsertMessage = (message2) => {
+    const [index] = indexesOf(message2.id);
+    if (index === void 0) messages.push(message2);
+    else replaceMessage(index, message2);
+    return message2;
+  };
   const filterValue = parentToolUseId !== void 0 ? parentToolUseId : null;
   const filteredEvents = events.filter((e) => {
     const eventParentToolUseId = e.payload.parent_tool_use_id ?? null;
@@ -29972,13 +29999,14 @@ function parseClaudeEvents(events, parentToolUseId) {
   const stampedUserMessages = /* @__PURE__ */ new Set();
   const findMatchingUserMessageIndex = (indexes, content, timestamp, messageId) => {
     if (typeof messageId === "string") {
-      const byId = messages.findIndex((message2, index) => message2.type === "user" && message2.id === messageId && !supersededUserMessageIndexes.has(index));
-      if (byId !== -1) return byId;
+      const byId = indexesOf(messageId).find((index) => messages[index].type === "user" && !supersededUserMessageIndexes.has(index));
+      if (byId !== void 0) return byId;
     }
-    return [...indexes].find((index) => {
+    for (const index of indexes) {
       const message2 = messages[index];
-      return message2?.type === "user" && !(typeof messageId === "string" && stampedUserMessages.has(message2)) && areUserMessagesWithinMatchWindow(message2, { content, timestamp });
-    });
+      if (message2?.type === "user" && !(typeof messageId === "string" && stampedUserMessages.has(message2)) && areUserMessagesWithinMatchWindow(message2, { content, timestamp })) return index;
+    }
+    return void 0;
   };
   const taskAccumulator = new TaskAccumulator();
   let nestedEventsByParent;
@@ -30007,7 +30035,7 @@ function parseClaudeEvents(events, parentToolUseId) {
         if (canonicalIndex === void 0) {
           acceptedUserMessageIndexes.add(messages.push(message2) - 1);
         } else {
-          messages[canonicalIndex] = message2;
+          replaceMessage(canonicalIndex, message2);
           canonicalUserMessageIndexes.delete(canonicalIndex);
         }
         if (typeof messageId === "string" || canonical && stampedUserMessages.has(canonical)) stampedUserMessages.add(message2);
@@ -30031,7 +30059,7 @@ function parseClaudeEvents(events, parentToolUseId) {
           status: payload.status,
           timestamp: event.timestamp
         };
-        if (existing.reasoning !== void 0) messages[existing.reasoning] = reasoningMessage;
+        if (existing.reasoning !== void 0) replaceMessage(existing.reasoning, reasoningMessage);
         else existing.reasoning = messages.push(reasoningMessage) - 1;
       }
       if (payload.text && !completedStreamIds.has(payload.streamId)) {
@@ -30041,7 +30069,7 @@ function parseClaudeEvents(events, parentToolUseId) {
           content: payload.text,
           timestamp: event.timestamp
         };
-        if (existing.agent !== void 0) messages[existing.agent] = agentMessage;
+        if (existing.agent !== void 0) replaceMessage(existing.agent, agentMessage);
         else existing.agent = messages.push(agentMessage) - 1;
       }
       if (payload.tool) streamingTools.set(payload.streamId, { tool: payload.tool, timestamp: event.timestamp });
@@ -30098,7 +30126,7 @@ function parseClaudeEvents(events, parentToolUseId) {
           acceptedUserMessageIndexes.delete(acceptedIndex);
           canonicalUserMessageIndexes.add(messages.push(message2) - 1);
         } else {
-          messages[acceptedIndex] = message2;
+          replaceMessage(acceptedIndex, message2);
         }
         if (typeof stampedId === "string" || accepted && stampedUserMessages.has(accepted)) stampedUserMessages.add(message2);
       }
@@ -30110,7 +30138,7 @@ function parseClaudeEvents(events, parentToolUseId) {
       const localCommand = event.payload.local_command_run;
       const goalArgs = isRecord(localCommand) && localCommand.command === GOAL_TOOL_NAME && typeof localCommand.args === "string" ? localCommand.args.trim() : "";
       if (goalArgs) {
-        upsertDisplayMessage(messages, {
+        upsertMessage({
           id: `goal-${messageKey}`,
           type: "tool_call",
           server: "claude",
@@ -30146,10 +30174,10 @@ function parseClaudeEvents(events, parentToolUseId) {
           timestamp: event.timestamp
         };
         if (streamRefs?.reasoning !== void 0) {
-          messages[streamRefs.reasoning] = reasoningMessage;
+          replaceMessage(streamRefs.reasoning, reasoningMessage);
           streamRefs.reasoning = void 0;
         } else {
-          upsertDisplayMessage(messages, reasoningMessage);
+          upsertMessage(reasoningMessage);
         }
       }
       contentBlocks.forEach((block) => {
@@ -30163,10 +30191,10 @@ function parseClaudeEvents(events, parentToolUseId) {
             timestamp: event.timestamp
           };
           if (streamRefs?.agent !== void 0) {
-            messages[streamRefs.agent] = agentMessage;
+            replaceMessage(streamRefs.agent, agentMessage);
             streamRefs.agent = void 0;
           } else {
-            upsertDisplayMessage(messages, agentMessage);
+            upsertMessage(agentMessage);
           }
         }
         if (block.type === "tool_use" && block.id) {
@@ -30260,7 +30288,7 @@ function parseClaudeEvents(events, parentToolUseId) {
     if (event.type === "claude-result") {
       const payload = coerceClaudeResultPayload(event.payload);
       const last = messages.at(-1);
-      if (last?.type === "agent") messages[messages.length - 1] = { ...last, endsTurn: true };
+      if (last?.type === "agent") replaceMessage(messages.length - 1, { ...last, endsTurn: true });
       if (isClaudeResultError(payload)) {
         const genuineErrors = stripAgentDiagnosticErrors(payload.errors ?? []);
         messages.push({
@@ -30303,7 +30331,7 @@ function parseClaudeEvents(events, parentToolUseId) {
               nestedEvents: existingWorkflowAgent?.nestedEvents ?? collectNestedEvents(workflowAgent.agentId)
             }
           );
-          workflowAgentMessageMap.set(workflowAgent.agentId, upsertDisplayMessage(messages, nextWorkflowAgent));
+          workflowAgentMessageMap.set(workflowAgent.agentId, upsertMessage(nextWorkflowAgent));
         }
         const existing = taskMessageMap.get(payload.taskId);
         const nextStatus = payload.subtype === "task_notification" ? normalizeBackgroundTaskStatus(payload.status) : payload.subtype === "task_updated" && payload.patch?.status !== void 0 ? normalizeBackgroundTaskStatus(payload.patch?.status) : existing?.status ?? "in_progress";
@@ -30327,7 +30355,7 @@ function parseClaudeEvents(events, parentToolUseId) {
           notifiedAt: payload.subtype === "task_notification" ? event.timestamp : existing?.notifiedAt,
           timestamp: existing?.timestamp ?? event.timestamp
         };
-        taskMessageMap.set(payload.taskId, upsertDisplayMessage(messages, nextMessage));
+        taskMessageMap.set(payload.taskId, upsertMessage(nextMessage));
       }
     }
   });
@@ -30396,7 +30424,7 @@ function parseClaudeEvents(events, parentToolUseId) {
       message2.outputOmitted = outputOmitted;
       message2.status = status;
     } else if (message2.type === "skill" && isError && message2.path) {
-      messages[toolInfo.messageIndex] = { id: message2.id, type: "tool_call", server: "claude", tool: toolInfo.toolName, input: toolInfo.input, output: resultContent, outputOmitted, status, timestamp: message2.timestamp };
+      replaceMessage(toolInfo.messageIndex, { id: message2.id, type: "tool_call", server: "claude", tool: toolInfo.toolName, input: toolInfo.input, output: resultContent, outputOmitted, status, timestamp: message2.timestamp });
     } else if (message2.type === "skill") {
       message2.status = status;
     } else if (message2.type === "subagent") {
@@ -30470,9 +30498,15 @@ function parseDisplayMessages(events, agentType, options = {}) {
   return withoutReplacedMessages(excludeQueuedAcceptedEvents(events, []), (segment) => {
     const parsedEvents = parseProviderEvents(segment, agentType, options.parentToolUseId);
     const reconciledEvents = [];
+    const firstIndexById = /* @__PURE__ */ new Map();
     for (const message2 of parsedEvents) {
-      if (message2.type === "user") upsertDisplayMessage(reconciledEvents, message2);
-      else reconciledEvents.push(message2);
+      const index = message2.type === "user" ? firstIndexById.get(message2.id) : void 0;
+      if (index !== void 0) {
+        reconciledEvents[index] = message2;
+        continue;
+      }
+      if (!firstIndexById.has(message2.id)) firstIndexById.set(message2.id, reconciledEvents.length);
+      reconciledEvents.push(message2);
     }
     if (options.filter === false) return reconciledEvents;
     return applyEngineNotices(
@@ -31326,7 +31360,7 @@ var CLI_VERSION;
 var init_cli_version = __esm({
   "../shared/src/cli-version.ts"() {
     "use strict";
-    CLI_VERSION = "0.3.212";
+    CLI_VERSION = "0.3.219";
   }
 });
 
@@ -38371,7 +38405,7 @@ async function inspect(html, source) {
     const resize = (width, height) => page.send("Emulation.setDeviceMetricsOverride", { width, height, deviceScaleFactor: 1, mobile: false });
     const screenshot = async (appearance, width) => {
       await load(appearance, width, 600);
-      await resize(width, clampVisualHeight(await settledHeight(page)));
+      await resize(width, clampVisualHeight(Math.min(MAX_SCREENSHOT_HEIGHT, await settledHeight(page))));
       const { data } = await page.send("Page.captureScreenshot", { format: "png" });
       import_fs10.default.mkdirSync(SCREENSHOT_DIR, { recursive: true });
       const file2 = import_path9.default.join(SCREENSHOT_DIR, `${import_path9.default.basename(source, import_path9.default.extname(source))}-${appearance}-${width}.png`);
@@ -38411,7 +38445,6 @@ function warnings(inspection) {
   return [
     ...inspection.viewportDependent ? [VIEWPORT_DEPENDENT_FIX] : [],
     ...inspection.overflowWidths.length > 0 ? [`Content overflows horizontally at ${inspection.overflowWidths.join("px, ")}px. Let it wrap or stack at narrow widths.`] : [],
-    ...inspection.heights.some(([, height]) => height > VISUAL_MAX_HEIGHT) ? [`The page is taller than ${VISUAL_MAX_HEIGHT}px at some widths, so chat cuts off the bottom. Make it shorter or split it into several visuals.`] : [],
     ...inspection.background ? [`The page paints its own background (${inspection.background}) on html or body. Leave them transparent so it sits on the chat background.`] : []
   ];
 }
@@ -38469,7 +38502,7 @@ Next: publish with \`replicas visualize ${file2} --title "<short title>"\`.`);
     console.log("\nNext: paste the ![\u2026](\u2026) line on its own line where the visual belongs in your final reply. The user sees the visual itself, so don't introduce or restate it; add only what it doesn't say. The Forge link is for posting outside Replicas chat (PRs, Slack, Linear); leave it out of your chat reply.");
   });
 }
-var import_fs10, import_http3, import_path9, MIB, MAX_IMAGE_BYTES, MAX_PAGE_BYTES, LOAD_TIMEOUT_MS, SCREENSHOT_DIR, LOCAL_IMAGE, webAppUrl, VIEWPORT_DEPENDENT_FIX, consoleErrors;
+var import_fs10, import_http3, import_path9, MIB, MAX_IMAGE_BYTES, MAX_PAGE_BYTES, LOAD_TIMEOUT_MS, MAX_SCREENSHOT_HEIGHT, SCREENSHOT_DIR, LOCAL_IMAGE, webAppUrl, VIEWPORT_DEPENDENT_FIX, consoleErrors;
 var init_visualize = __esm({
   "src/commands/visualize.ts"() {
     "use strict";
@@ -38488,6 +38521,7 @@ var init_visualize = __esm({
     MAX_IMAGE_BYTES = 10 * MIB;
     MAX_PAGE_BYTES = 25 * MIB;
     LOAD_TIMEOUT_MS = 1e4;
+    MAX_SCREENSHOT_HEIGHT = 4e3;
     SCREENSHOT_DIR = "/tmp/replicas-visualize";
     LOCAL_IMAGE = /(["'`])(\/(?!\/)[^"'`\r\n]*?\.(?:png|jpe?g|gif|webp|avif|svg))\1|url\(\s*(\/(?!\/)[^\s"'`()]*?\.(?:png|jpe?g|gif|webp|avif|svg))\s*\)/gi;
     webAppUrl = () => process.env.REPLICAS_WEB_URL || DEFAULT_WEB_APP_URL;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "replicas-cli",
-  "version": "0.3.217",
+  "version": "0.3.219",
   "description": "CLI for managing Replicas workspaces - SSH into cloud dev environments with automatic port forwarding",
   "main": "dist/index.cjs",
   "bin": {