@marimo-team/islands 0.15.4 → 0.16.0

package/dist/{time-BtVcKqeD.js → time-Cp5uV45N.js}

@@ -1,7 +1,7 @@
 import { R as q, r as ln, o as P, q as X, C as Z, u as D, v as fn } from "./timer-B0-z63CM.js";
-import { c as hn, a as on } from "./linear-fbJq6cdO.js";
+import { c as hn, a as on } from "./linear-DafxvwEV.js";
 import { i as bn } from "./init-DxzjmxYy.js";
-import { W as gn, X as d, Y as v, Z as k, _ as x, $ as L, a0 as yn, a1 as R, a2 as H, a3 as W, a4 as pn, a5 as mn, a6 as wn, a7 as Mn, a8 as dn, a9 as vn, aa as kn, ab as $, ac as A, ad as B, ae as K, af as _, ag as j, ah as xn } from "./main-HerZgEhd.js";
+import { W as gn, X as d, Y as v, Z as k, _ as x, $ as L, a0 as yn, a1 as R, a2 as H, a3 as W, a4 as pn, a5 as mn, a6 as wn, a7 as Mn, a8 as dn, a9 as vn, aa as kn, ab as $, ac as A, ad as B, ae as K, af as _, ag as j, ah as xn } from "./main-mtVsqZY6.js";
 function Pn(n, t) {
   let i;
   if (t === void 0)

package/dist/{timeline-definition-XQNQX7LJ-DEteLt8D.js → timeline-definition-XQNQX7LJ-BMz0bhXy.js}

@@ -1,4 +1,4 @@
-import { _ as s, c as xt, l as E, d as j, V as kt, W as vt, X as _t, Y as bt, D as wt, $ as St, z as Et } from "./mermaid-DxPYK0KX.js";
+import { _ as s, c as xt, l as E, d as j, V as kt, W as vt, X as _t, Y as bt, D as wt, $ as St, z as Et } from "./mermaid-DUnuxwY7.js";
 import { d as nt } from "./arc-BOhn-m2C.js";
 var Q = (function() {
   var n = /* @__PURE__ */ s(function(x, r, a, c) {

package/dist/{treemap-75Q7IDZK-8S6podme.js → treemap-75Q7IDZK-Dy3GswCD.js}

@@ -2,11 +2,11 @@ var __defProp = Object.defineProperty;
 var __defNormalProp = (obj, key, value) => key in obj ? __defProp(obj, key, { enumerable: true, configurable: true, writable: true, value }) : obj[key] = value;
 var __publicField = (obj, key, value) => __defNormalProp(obj, typeof key !== "symbol" ? key + "" : key, value);
 var _a2, _b, _c2, _d2, _e, _f2, _g, _h2, _i2, _j, _k, _l2, _m2, _n2, _o2, _p2, _q;
-import { dY as Kl, bO as Vl, al as Ma, aT as Hl, u as _t, aO as Wl, ao as Q, bk as jl, bl as zl, k as ql, n as Ni, m as Da, am as Qs, dZ as Fa, aK as Yl, b0 as Ga, d_ as Ua, aM as Xl, b2 as wi, an as Jl, d$ as Zl, bg as Ql, e0 as eu, aP as Ie, q as vt, ar as _i, b3 as tu, e1 as H, e2 as Re } from "./main-HerZgEhd.js";
-import { g as ei, d as bt, k as nu, v as W, l as Ba, m as ru, n as su, a as Ka, c as x, i as qe, r as ae, f as ke, o as z } from "./_baseUniq-CQrhBg_9.js";
-import { m as k, f as Lt, h as I, e as ti, l as Ot, d as iu } from "./min-DBJkhObB.js";
-import { ad as P } from "./mermaid-DxPYK0KX.js";
-import { c as te } from "./clone-DqwV7ges.js";
+import { dY as Kl, bO as Vl, al as Ma, aT as Hl, u as _t, aO as Wl, ao as Q, bk as jl, bl as zl, k as ql, n as Ni, m as Da, am as Qs, dZ as Fa, aK as Yl, b0 as Ga, d_ as Ua, aM as Xl, b2 as wi, an as Jl, d$ as Zl, bg as Ql, e0 as eu, aP as Ie, q as vt, ar as _i, b3 as tu, e1 as H, e2 as Re } from "./main-mtVsqZY6.js";
+import { g as ei, d as bt, k as nu, v as W, l as Ba, m as ru, n as su, a as Ka, c as x, i as qe, r as ae, f as ke, o as z } from "./_baseUniq-DQHjMSO_.js";
+import { m as k, f as Lt, h as I, e as ti, l as Ot, d as iu } from "./min-BLUnrJi8.js";
+import { ad as P } from "./mermaid-DUnuxwY7.js";
+import { c as te } from "./clone-B3q5i6e_.js";
 var au = Object.prototype, ou = au.hasOwnProperty, Ee = Kl(function(n, e) {
   if (Vl(e) || Ma(e)) {
     Hl(e, _t(e), n);

package/dist/{vega-component-D35L45kI.js → vega-component-D-4UjbYh.js}

@@ -1,5 +1,5 @@
-import { D as V, F as K, A as Z, G as z, H as R, I as G, j as d, E as W, J as U, r as k, K as j, M, N as $, L as x, O as _, P as B, Q, S as q, T as J, U as Y, V as X } from "./main-HerZgEhd.js";
-import { M as m, V as ee } from "./index-cz_xaKvT.js";
+import { D as V, F as K, A as Z, G as z, H as R, I as G, j as d, E as W, J as U, r as k, K as j, M, N as $, L as x, O as _, P as B, Q, S as q, T as J, U as Y, V as X } from "./main-mtVsqZY6.js";
+import { M as m, V as ee } from "./index-a9F3H4C8.js";
 function te(e) {
   return e.data && "url" in e.data && (e.data.url = V(e.data.url).href), e;
 }

package/dist/{xychartDiagram-6GGTOJPD-DKwGThyy.js → xychartDiagram-6GGTOJPD-D-zWFjme.js}

@@ -1,9 +1,9 @@
 var _a, _b, _c, _d, _e, _f, _g, _h, _i2, _j;
-import { _ as a, s as ei, g as si, t as Lt, q as ni, a as ai, b as oi, l as Et, K as ri, e as hi, z as li, G as xt, F as It, H as ci, M as ui, i as gi, ac as xi } from "./mermaid-DxPYK0KX.js";
+import { _ as a, s as ei, g as si, t as Lt, q as ni, a as ai, b as oi, l as Et, K as ri, e as hi, z as li, G as xt, F as It, H as ci, M as ui, i as gi, ac as xi } from "./mermaid-DUnuxwY7.js";
 import { i as di } from "./init-DxzjmxYy.js";
 import { o as fi } from "./ordinal-CYN5qNoq.js";
 import { r as pi } from "./range-DdOGybNB.js";
-import { l as Tt } from "./linear-fbJq6cdO.js";
+import { l as Tt } from "./linear-DafxvwEV.js";
 import { l as Dt } from "./timer-B0-z63CM.js";
 function ht() {
   var t = fi().unknown(void 0), i = t.domain, e = t.range, s = 0, n = 1, d, g, m = false, S = 0, D = 0, v = 0.5;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@marimo-team/islands",
-  "version": "0.15.4",
+  "version": "0.16.0",
   "main": "dist/main.js",
   "types": "dist/index.d.ts",
   "type": "module",
@@ -100,9 +100,13 @@
     "@uwdata/flechette": "^1.1.2",
     "@valtown/codemirror-codeium": "^1.1.1",
     "@xterm/addon-attach": "^0.11.0",
+    "@xterm/addon-canvas": "^0.7.0",
     "@xterm/addon-fit": "^0.10.0",
+    "@xterm/addon-search": "^0.15.0",
+    "@xterm/addon-unicode11": "^0.8.0",
+    "@xterm/addon-web-links": "^0.11.0",
     "@xterm/xterm": "^5.5.0",
-    "@zed-industries/agent-client-protocol": "^0.1.2",
+    "@zed-industries/agent-client-protocol": "^0.3.1",
     "ai": "^5.0.30",
     "ansi_up": "^6.0.6",
     "class-variance-authority": "^0.7.1",
@@ -151,7 +155,7 @@
     "remark-gfm": "^4.0.1",
     "rpc-anywhere": "^1.7.0",
     "sql-formatter": "^15.6.6",
-    "streamdown": "^1.2.0",
+    "streamdown": "^1.3.0",
     "string-dedent": "^3.0.2",
     "swiper": "^11.2.10",
     "tailwind-merge": "^2.6.0",
@@ -159,7 +163,7 @@
     "thememirror": "^2.0.1",
     "timestring": "^7.0.0",
     "typescript-memoize": "^1.1.1",
-    "use-acp": "^0.1.5",
+    "use-acp": "^0.2.3",
     "use-resize-observer": "^9.1.0",
     "vega-lite": "^5.23.0",
     "vega-loader": "^4.5.3",

package/src/__tests__/mocks.ts

@@ -0,0 +1,43 @@
+/* Copyright 2024 Marimo. All rights reserved. */
+
+// Edge case filenames for testing unicode, spaces, and special characters
+export const EDGE_CASE_FILENAMES = [
+  // Unicode
+  "tést.py",
+  "café.py",
+  "测试.py",
+  // Emojis
+  "🚀notebook.py",
+  // Spaces
+  "test file.py",
+  "file with spaces.py",
+  // Multiple
+  "café & 测试.py",
+  "café notebook.py",
+  // URL characters
+  "test-file.py",
+  "test_file.py",
+  "test_file.backup.py",
+  "file&with&ampersands.py",
+  "file=with=equals.py",
+  "file?with?questions.py",
+];
+
+// Cell names with unicode and special characters for frontend tests
+export const EDGE_CASE_CELL_NAMES = [
+  "tést_cell",
+  "café_notebook",
+  "测试_cell",
+  "🚀_my_cell",
+  "cell with spaces",
+  "café notebook cell",
+];
+
+// URL test cases with special characters that could break query parameters
+export const URL_SPECIAL_CHAR_FILENAMES = [
+  "file with spaces.py",
+  "file&with&ampersands.py",
+  "file=with=equals.py",
+  "file?with?questions.py",
+  "café & 测试.py",
+];

package/src/components/app-config/user-config-form.tsx

@@ -1283,6 +1283,38 @@ export const UserConfigForm: React.FC = () => {
                 </div>
               )}
             />
+            <FormField
+              control={form.control}
+              name="experimental.external_agents"
+              render={({ field }) => (
+                <div className="flex flex-col gap-y-1">
+                  <FormItem className={formItemClasses}>
+                    <FormLabel className="font-normal">
+                      External Agents
+                    </FormLabel>
+                    <FormControl>
+                      <Checkbox
+                        data-testid="external-agents-checkbox"
+                        checked={field.value === true}
+                        onCheckedChange={field.onChange}
+                      />
+                    </FormControl>
+                  </FormItem>
+                  <IsOverridden
+                    userConfig={config}
+                    name="experimental.external_agents"
+                  />
+                  <FormDescription>
+                    Enable experimental external agents such as Claude Code and
+                    Gemini CLI. Learn more in the{" "}
+                    <ExternalLink href="https://docs.marimo.io/guides/editor_features/agents/">
+                      docs
+                    </ExternalLink>
+                    .
+                  </FormDescription>
+                </div>
+              )}
+            />
           </SettingGroup>
         );
     }

package/src/components/chat/acp/__tests__/__snapshots__/prompt.test.ts.snap

@@ -3,7 +3,7 @@
 exports[`getAgentPrompt > should generate complete agent prompt with default filename 1`] = `
 "
   I am currently editing a marimo notebook.
-  You can read or write to the notebook at @test-notebook.py
+  You can read or write to the notebook at test-notebook.py
 
   If you make edits to the notebook, only edit the contents inside the function decorator with @app.cell.
   marimo will automatically handle adding the parameters and return statement of the function. For example,
@@ -11,7 +11,7 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
 
   \`\`\`
   @app.cell
-  def __():
+  def _():
     <your code here>
     return
   \`\`\`
@@ -37,6 +37,7 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
   7. The last expression in a cell is automatically displayed, just like in Jupyter notebooks.
   8. Don't include comments in markdown cells
   9. Don't include comments in SQL cells
+  10. Never define anything using \`global\`.
 
   ## Reactivity
 
@@ -46,6 +47,7 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
   - UI elements trigger updates when their values change without explicit callbacks
   - UI element values are accessed through \`.value\` attribute
   - You cannot access a UI element's value in the same cell where it's defined
+  - Cells prefixed with an underscore (e.g. _my_var) are local to the cell and cannot be accessed by other cells
 
   ## Best Practices
 
@@ -115,34 +117,50 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
 
   - \`mo.md(text)\` - display markdown
   - \`mo.stop(predicate, output=None)\` - stop execution conditionally
+  - \`mo.output.append(value)\` - append to the output when it is not the last expression
+  - \`mo.output.replace(value)\` - replace the output when it is not the last expression
   - \`mo.Html(html)\` - display HTML
   - \`mo.image(image)\` - display an image
   - \`mo.hstack(elements)\` - stack elements horizontally
   - \`mo.vstack(elements)\` - stack elements vertically
   - \`mo.tabs(elements)\` - create a tabbed interface
 
+
+
   ## Examples
 
+  <example title="Markdown ccell">
+  
+  @app.cell
+  def _():
+      mo.md("""
+      # Hello world
+      This is a _markdown_ **cell**.
+      """)
+      return
+  
+  </example>
+
   <example title="Basic UI with reactivity">
   
   @app.cell
-  def __():
+  def _():
       import marimo as mo
       import matplotlib.pyplot as plt
       import numpy as np
       return
   
   @app.cell
-  def __():
+  def _():
       n_points = mo.ui.slider(10, 100, value=50, label="Number of points")
       n_points
       return
   
   @app.cell
-  def __():
+  def _():
       x = np.random.rand(n_points.value)
       y = np.random.rand(n_points.value)
-    
+
       plt.figure(figsize=(8, 6))
       plt.scatter(x, y, alpha=0.7)
       plt.title(f"Scatter plot with {n_points.value} points")
@@ -156,14 +174,14 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
   <example title="Data explorer">
   
   @app.cell
-  def __():
+  def _():
       import marimo as mo
       import pandas as pd
       from vega_datasets import data
       return
   
   @app.cell
-  def __():
+  def _():
       cars_df = data.cars()
       mo.ui.data_explorer(cars_df)
       return
@@ -173,7 +191,7 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
   <example title="Multiple UI elements">
   
   @app.cell
-  def __():
+  def _():
       import marimo as mo
       import pandas as pd
       import matplotlib.pyplot as plt
@@ -181,12 +199,12 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
       return
   
   @app.cell
-  def __():
+  def _():
       iris = sns.load_dataset('iris')
       return
   
   @app.cell
-  def __():
+  def _():
       species_selector = mo.ui.dropdown(
           options=["All"] + iris["species"].unique().tolist(),
           value="All",
@@ -206,9 +224,9 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
       return
   
   @app.cell
-  def __():
+  def _():
       filtered_data = iris if species_selector.value == "All" else iris[iris["species"] == species_selector.value]
-    
+
       plt.figure(figsize=(10, 6))
       sns.scatterplot(
           data=filtered_data,
@@ -222,17 +240,31 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
   
   </example>
 
+  <example title="Conditional Outputs">
+  
+  @app.cell
+  def _():
+      mo.stop(not data.value, mo.md("No data to display"))
+
+      if mode.value == "scatter":
+        mo.output.replace(render_scatter(data.value))
+      else:
+        mo.output.replace(render_bar_chart(data.value))
+      return
+  
+  </example>
+
   <example title="Interactive chart with Altair">
   
   @app.cell
-  def __():
+  def _():
       import marimo as mo
       import altair as alt
       import pandas as pd
       return
   
   @app.cell
-  def __():
+  def _():
       # Load dataset
       cars_df = pd.read_csv('<https://raw.githubusercontent.com/vega/vega-datasets/master/data/cars.json>')
       _chart = alt.Chart(cars_df).mark_point().encode(
@@ -243,13 +275,13 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
       return
   
   @app.cell
-  def __():
+  def _():
       chart = mo.ui.altair_chart(_chart)
     chart
       return
   
   @app.cell
-  def __():
+  def _():
       # Display the selection
       chart.value
       return
@@ -259,19 +291,19 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
   <example title="Run Button Example">
   
   @app.cell
-  def __():
+  def _():
       import marimo as mo
       return
   
   @app.cell
-  def __():
+  def _():
       first_button = mo.ui.run_button(label="Option 1")
       second_button = mo.ui.run_button(label="Option 2")
       [first_button, second_button]
       return
   
   @app.cell
-  def __():
+  def _():
       if first_button.value:
           print("You chose option 1!")
       elif second_button.value:
@@ -285,18 +317,18 @@ exports[`getAgentPrompt > should generate complete agent prompt with default fil
   <example title="SQL with duckdb">
   
   @app.cell
-  def __():
+  def _():
       import marimo as mo
     import pandas as pd
       return
   
   @app.cell
-  def __():
+  def _():
       cars_df = pd.read_csv('<https://raw.githubusercontent.com/vega/vega-datasets/master/data/cars.json>')
       return
   
   @app.cell
-  def __():
+  def _():
       _df = mo.sql("SELECT * from cars_df WHERE Miles_per_Gallon > 20")
       return
   

package/src/components/chat/acp/__tests__/context-utils.test.ts

@@ -0,0 +1,222 @@
+/* Copyright 2024 Marimo. All rights reserved. */
+
+import { beforeEach, describe, expect, it, type Mocked, vi } from "vitest";
+import {
+  convertFilesToResourceLinks,
+  parseContextFromPrompt,
+} from "../context-utils";
+
+// Mock dependencies
+vi.mock("@/utils/fileToBase64", () => ({
+  blobToString: vi.fn(),
+}));
+
+vi.mock("@/core/ai/context/context", () => ({
+  getAIContextRegistry: vi.fn(),
+}));
+
+vi.mock("@/core/state/jotai", () => ({
+  store: {},
+}));
+
+vi.mock("@/utils/Logger", () => ({
+  Logger: {
+    error: vi.fn(),
+    debug: vi.fn(),
+  },
+}));
+
+import { getAIContextRegistry } from "@/core/ai/context/context";
+import type {
+  AIContextItem,
+  AIContextRegistry,
+  ContextLocatorId,
+} from "@/core/ai/context/registry";
+import { blobToString } from "@/utils/fileToBase64";
+
+const CONTEXT_ID = "context1" as ContextLocatorId;
+
+describe("convertFilesToResourceLinks", () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it("should convert files to resource links", async () => {
+    const mockFile = new File(["content"], "test.txt", { type: "text/plain" });
+    vi.mocked(blobToString).mockResolvedValue(
+      "data:text/plain;base64,Y29udGVudA==",
+    );
+
+    const result = await convertFilesToResourceLinks([mockFile]);
+
+    expect(result).toEqual([
+      {
+        type: "resource_link",
+        uri: "data:text/plain;base64,Y29udGVudA==",
+        mimeType: "text/plain",
+        name: "test.txt",
+      },
+    ]);
+  });
+
+  it("should handle empty file array", async () => {
+    const result = await convertFilesToResourceLinks([]);
+    expect(result).toEqual([]);
+  });
+
+  it("should handle file conversion errors gracefully", async () => {
+    const mockFile = new File(["content"], "test.txt", { type: "text/plain" });
+    vi.mocked(blobToString).mockRejectedValue(new Error("Conversion failed"));
+
+    const result = await convertFilesToResourceLinks([mockFile]);
+
+    expect(result).toEqual([]);
+  });
+
+  it("should process multiple files", async () => {
+    const file1 = new File(["content1"], "test1.txt", { type: "text/plain" });
+    const file2 = new File(["content2"], "test2.txt", { type: "text/plain" });
+
+    vi.mocked(blobToString)
+      .mockResolvedValueOnce("data:text/plain;base64,Y29udGVudDE=")
+      .mockResolvedValueOnce("data:text/plain;base64,Y29udGVudDI=");
+
+    const result = await convertFilesToResourceLinks([file1, file2]);
+
+    expect(result).toHaveLength(2);
+    expect((result[0] as { name: string }).name).toBe("test1.txt");
+    expect((result[1] as { name: string }).name).toBe("test2.txt");
+  });
+});
+
+describe("parseContextFromPrompt", () => {
+  const mockRegistry = {
+    parseAllContextIds: vi.fn(),
+    formatContextForAI: vi.fn(),
+    getAttachmentsForContext: vi.fn(),
+  } as unknown as Mocked<AIContextRegistry<AIContextItem>>;
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+    vi.mocked(getAIContextRegistry).mockReturnValue(mockRegistry);
+  });
+
+  it("should return empty blocks when no @ symbol in prompt", async () => {
+    const result = await parseContextFromPrompt("simple prompt");
+
+    expect(result).toEqual({
+      contextBlocks: [],
+      attachmentBlocks: [],
+    });
+  });
+
+  it("should return empty blocks when no context IDs found", async () => {
+    mockRegistry.parseAllContextIds.mockReturnValue([]);
+
+    const result = await parseContextFromPrompt("prompt with @ but no context");
+
+    expect(result).toEqual({
+      contextBlocks: [],
+      attachmentBlocks: [],
+    });
+  });
+
+  it("should create context blocks when context IDs are found", async () => {
+    mockRegistry.parseAllContextIds.mockReturnValue([CONTEXT_ID]);
+    mockRegistry.formatContextForAI.mockReturnValue("formatted context");
+    mockRegistry.getAttachmentsForContext.mockResolvedValue([]);
+
+    const result = await parseContextFromPrompt("prompt with @context1");
+
+    expect(result.contextBlocks).toHaveLength(1);
+    expect(result.contextBlocks[0]).toEqual({
+      type: "resource",
+      resource: {
+        uri: "context.md",
+        mimeType: "text/markdown",
+        text: "formatted context",
+      },
+    });
+    expect(result.attachmentBlocks).toHaveLength(0);
+  });
+
+  it("should create attachment blocks when attachments are found", async () => {
+    mockRegistry.parseAllContextIds.mockReturnValue([CONTEXT_ID]);
+    mockRegistry.formatContextForAI.mockReturnValue("formatted context");
+    mockRegistry.getAttachmentsForContext.mockResolvedValue([
+      {
+        type: "file",
+        url: "http://example.com/file.pdf",
+        mediaType: "application/pdf",
+        filename: "file.pdf",
+      },
+    ]);
+
+    const result = await parseContextFromPrompt("prompt with @context1");
+
+    expect(result.contextBlocks).toHaveLength(1);
+    expect(result.attachmentBlocks).toHaveLength(1);
+    expect(result.attachmentBlocks[0]).toEqual({
+      type: "resource_link",
+      uri: "http://example.com/file.pdf",
+      mimeType: "application/pdf",
+      name: "file.pdf",
+    });
+  });
+
+  it("should handle empty context string gracefully", async () => {
+    mockRegistry.parseAllContextIds.mockReturnValue([CONTEXT_ID]);
+    mockRegistry.formatContextForAI.mockReturnValue("   ");
+    mockRegistry.getAttachmentsForContext.mockResolvedValue([]);
+
+    const result = await parseContextFromPrompt("prompt with @context1");
+
+    expect(result.contextBlocks).toHaveLength(0);
+    expect(result.attachmentBlocks).toHaveLength(0);
+  });
+
+  it("should handle registry errors gracefully", async () => {
+    vi.mocked(getAIContextRegistry).mockImplementation(() => {
+      throw new Error("Registry error");
+    });
+
+    const result = await parseContextFromPrompt("prompt with @context1");
+
+    expect(result).toEqual({
+      contextBlocks: [],
+      attachmentBlocks: [],
+    });
+  });
+
+  it("should handle attachment errors gracefully", async () => {
+    mockRegistry.parseAllContextIds.mockReturnValue([CONTEXT_ID]);
+    mockRegistry.formatContextForAI.mockReturnValue("formatted context");
+    mockRegistry.getAttachmentsForContext.mockRejectedValue(
+      new Error("Attachment error"),
+    );
+
+    const result = await parseContextFromPrompt("prompt with @context1");
+
+    expect(result.contextBlocks).toHaveLength(1);
+    expect(result.attachmentBlocks).toHaveLength(0);
+  });
+
+  it("should use url as name when filename is not provided", async () => {
+    mockRegistry.parseAllContextIds.mockReturnValue([CONTEXT_ID]);
+    mockRegistry.formatContextForAI.mockReturnValue("formatted context");
+    mockRegistry.getAttachmentsForContext.mockResolvedValue([
+      {
+        type: "file",
+        url: "http://example.com/file.pdf",
+        mediaType: "application/pdf",
+        filename: undefined,
+      },
+    ]);
+
+    const result = await parseContextFromPrompt("prompt with @context1");
+
+    expect((result.attachmentBlocks[0] as { name: string }).name).toBe(
+      "http://example.com/file.pdf",
+    );
+  });
+});

package/src/components/chat/acp/__tests__/prompt.test.ts

@@ -1,4 +1,4 @@
-/* Copyright 2025 Marimo. All rights reserved. */
+/* Copyright 2024 Marimo. All rights reserved. */
 
 import { describe, expect, it } from "vitest";
 import { getAgentPrompt } from "../prompt";

package/src/components/chat/acp/__tests__/state.test.ts

@@ -604,17 +604,13 @@ describe("state utility functions", () => {
   describe("getAgentConnectionCommand", () => {
     it("should return correct command for claude", () => {
       expect(getAgentConnectionCommand("claude")).toMatchInlineSnapshot(`
-        "npx supergateway --stdio\\
-          "npx @zed-industries/claude-code-acp" \\
-           --outputTransport ws --port 3017 "
+        "npx stdio-to-ws "npx @zed-industries/claude-code-acp" --port 3017"
       `);
     });
 
     it("should return correct command for gemini", () => {
       expect(getAgentConnectionCommand("gemini")).toMatchInlineSnapshot(`
-        "npx supergateway --stdio\\
-          "npx @google/gemini-cli --experimental-acp" \\
-           --outputTransport ws --port 3019 "
+        "npx stdio-to-ws "npx @google/gemini-cli --experimental-acp" --port 3019"
       `);
     });
   });