@tangle-network/ui 11.51.7 → 11.51.9

package/src/editor/markdown-document-editor.test.tsx

@@ -1,126 +0,0 @@
-import { render, screen, waitFor } from "@testing-library/react";
-import { Component, type ReactNode } from "react";
-import { afterEach, describe, expect, it, vi } from "vitest";
-import { MarkdownDocumentEditor } from "./markdown-document-editor";
-import type * as EditorPeers from "./editor-peers";
-
-class ErrorTrap extends Component<{ children: ReactNode }, { message: string | null }> {
-  state = { message: null as string | null };
-
-  static getDerivedStateFromError(error: Error) {
-    return { message: error.message };
-  }
-
-  render() {
-    return this.state.message ?? this.props.children;
-  }
-}
-
-/**
- * Imports the editor with a fresh module graph, so its module-scope `lazy`
- * has not resolved yet. `loadDocumentEditorPeers` is replaced by `load`.
- */
-async function freshEditor(load: () => Promise<EditorPeers.DocumentEditorPeers>) {
-  vi.resetModules();
-  vi.doMock("./editor-peers", async () => ({
-    ...(await vi.importActual<typeof EditorPeers>("./editor-peers")),
-    loadDocumentEditorPeers: load,
-  }));
-  return (await import("./markdown-document-editor")).MarkdownDocumentEditor;
-}
-
-// The first render imports tiptap for real. A cold import on a busy runner
-// takes several seconds, well past waitFor's 1s and a test's 5s defaults.
-const PEERS_LOADED = { timeout: 15_000 };
-const LOADS_PEERS = 30_000;
-
-afterEach(() => {
-  vi.doUnmock("./editor-peers");
-  vi.resetModules();
-});
-
-describe("MarkdownDocumentEditor", () => {
-  it("renders the real tiptap surface once the peers load", async () => {
-    const { container } = render(
-      <MarkdownDocumentEditor value={"# Title\n\nBody text."} />,
-    );
-
-    await waitFor(() => {
-      expect(container.querySelector(".ProseMirror")).not.toBeNull();
-    }, PEERS_LOADED);
-
-    const surface = container.querySelector(".ProseMirror");
-    expect(surface?.querySelector("h1")?.textContent).toBe("Title");
-    expect(surface?.textContent).toContain("Body text.");
-    // StarterKit arrives through `peers.starterKit.default`. A broken hand-off
-    // leaves the document without the heading node that StarterKit configures.
-    expect(screen.getByTitle(/^Bold/)).toBeInTheDocument();
-  }, LOADS_PEERS);
-
-  it("holds a placeholder while the peers are still loading", async () => {
-    let releasePeers: (() => void) | null = null;
-    const gate = new Promise<void>((resolve) => {
-      releasePeers = resolve;
-    });
-    const HeldEditor = await freshEditor(async () => {
-      await gate;
-      const actual = await vi.importActual<typeof EditorPeers>("./editor-peers");
-      return actual.loadDocumentEditorPeers();
-    });
-
-    const { container } = render(<HeldEditor value="# Title" />);
-
-    // A live region, so a screen reader hears that the editor is still coming.
-    expect(screen.getByRole("status")).toHaveTextContent("Loading editor…");
-    expect(container.querySelector(".ProseMirror")).toBeNull();
-
-    releasePeers?.();
-
-    await waitFor(() => {
-      expect(container.querySelector(".ProseMirror h1")?.textContent).toBe("Title");
-    });
-    expect(screen.queryByText("Loading editor…")).toBeNull();
-  });
-
-  it("applies a new value prop to the loaded editor without reporting a change", async () => {
-    const onChange = vi.fn();
-    const { container, rerender } = render(
-      <MarkdownDocumentEditor value="start" onChange={onChange} />,
-    );
-
-    await waitFor(() => {
-      expect(container.querySelector(".ProseMirror")).not.toBeNull();
-    }, PEERS_LOADED);
-
-    // The editor reports once when it parses the initial content. What must
-    // not grow is the count across a value-prop write: pushing that back
-    // through onChange makes a controlled parent echo its own write.
-    const callsAfterMount = onChange.mock.calls.length;
-
-    rerender(<MarkdownDocumentEditor value="# Changed" onChange={onChange} />);
-
-    await waitFor(() => {
-      expect(container.querySelector(".ProseMirror h1")?.textContent).toBe("Changed");
-    });
-    expect(onChange).toHaveBeenCalledTimes(callsAfterMount);
-  }, LOADS_PEERS);
-
-  it("fails with the install list when the peers cannot be loaded", async () => {
-    const missing = "install @tiptap/react and @tiptap/starter-kit";
-    const LoadFailingEditor = await freshEditor(async () => {
-      throw new Error(missing);
-    });
-
-    render(
-      <ErrorTrap>
-        <LoadFailingEditor value="start" />
-      </ErrorTrap>,
-    );
-
-    // Loud, and only here: the consumer's build already succeeded without the
-    // peers, so this render is the first place the gap can surface.
-    await waitFor(() => {
-      expect(screen.getByText(missing)).toBeInTheDocument();
-    });
-  });
-});

package/src/editor/tiptap-editor.test.tsx

@@ -1,228 +0,0 @@
-import { render, screen, waitFor } from "@testing-library/react";
-import { Component, type ReactNode } from "react";
-import { afterEach, describe, expect, it, vi } from "vitest";
-import { Doc } from "yjs";
-import type * as EditorPeers from "./editor-peers";
-import { createEditorProvider } from "./editor-provider";
-import { createTiptapEditor } from "./tiptap-editor";
-
-class ErrorTrap extends Component<{ children: ReactNode }, { message: string | null }> {
-  state = { message: null as string | null };
-
-  static getDerivedStateFromError(error: Error) {
-    return { message: error.message };
-  }
-
-  render() {
-    return this.state.message ?? this.props.children;
-  }
-}
-
-class StubHocuspocusProvider {
-  static instances: StubHocuspocusProvider[] = [];
-
-  awareness = {
-    clientID: 1,
-    getStates: () => new Map(),
-    getLocalState: () => ({}),
-    setLocalStateField: vi.fn(),
-    on: vi.fn(),
-    off: vi.fn(),
-  };
-  connect = vi.fn();
-  disconnect = vi.fn();
-  destroy = vi.fn();
-
-  constructor(readonly options: { document: Doc }) {
-    StubHocuspocusProvider.instances.push(this);
-  }
-}
-
-/** A tiptap extension records what the editor configured it with. */
-function stubExtension(name: string) {
-  const calls: Record<string, unknown>[] = [];
-  return {
-    calls,
-    default: {
-      configure(options: Record<string, unknown>) {
-        calls.push(options);
-        return { name, options };
-      },
-    },
-  };
-}
-
-/**
- * Only the members the collaborative editor calls. yjs is the real package,
- * because the provider and the editor must agree on one document.
- */
-function collaborationStubs() {
-  const starterKit = stubExtension("starterKit");
-  const collaboration = stubExtension("collaboration");
-  const collaborationCaret = stubExtension("collaborationCaret");
-  const editorOptions: Record<string, unknown>[] = [];
-  const editor = {
-    setEditable: vi.fn(),
-    isEmpty: true,
-    commands: { setContent: vi.fn() },
-  };
-
-  const peers = {
-    react: {
-      useEditor: (options: Record<string, unknown>) => {
-        editorOptions.push(options);
-        return editor;
-      },
-      EditorContent: () => <div data-testid="editor-content" />,
-    },
-    starterKit,
-    collaboration,
-    collaborationCaret,
-    hocuspocus: { HocuspocusProvider: StubHocuspocusProvider },
-    yjs: { Doc },
-  } as unknown as EditorPeers.CollaborationPeers;
-
-  return { peers, starterKit, collaboration, collaborationCaret, editorOptions, editor };
-}
-
-function renderCollaborativeEditor(stubs: ReturnType<typeof collaborationStubs>) {
-  const EditorProvider = createEditorProvider(stubs.peers);
-  const TiptapEditor = createTiptapEditor(stubs.peers);
-  return render(
-    <EditorProvider
-      websocketUrl="wss://collab.example/ws"
-      documentName="doc:readme"
-      token="jwt"
-      user={{ name: "Ada" }}
-    >
-      <TiptapEditor />
-    </EditorProvider>,
-  );
-}
-
-afterEach(() => {
-  StubHocuspocusProvider.instances = [];
-  vi.doUnmock("./editor-peers");
-  vi.resetModules();
-});
-
-describe("createTiptapEditor", () => {
-  it("renders the editor surface from the loaded namespaces", () => {
-    renderCollaborativeEditor(collaborationStubs());
-
-    expect(screen.getByTestId("editor-content")).toBeInTheDocument();
-    expect(screen.getByText("Connecting...")).toBeInTheDocument();
-  });
-
-  it("gives Collaboration the document fragment the provider shares", () => {
-    const stubs = collaborationStubs();
-    renderCollaborativeEditor(stubs);
-
-    const [provider] = StubHocuspocusProvider.instances;
-    // The extensions rebuild once the transport arrives, so read the last
-    // configuration: the transport and the editor must bind to one Y.Doc, or
-    // remote edits never reach the surface.
-    expect(stubs.collaboration.calls.at(-1)?.fragment).toBe(
-      provider.options.document.getXmlFragment("prosemirror"),
-    );
-  });
-
-  it("gives CollaborationCaret the provider and a named user", () => {
-    const stubs = collaborationStubs();
-    renderCollaborativeEditor(stubs);
-
-    const [provider] = StubHocuspocusProvider.instances;
-    // The caret joins only once the provider carries awareness.
-    expect(stubs.collaborationCaret.calls).toHaveLength(1);
-    const [caretOptions] = stubs.collaborationCaret.calls;
-    expect(caretOptions.provider).toBe(provider);
-    // The stub awareness carries no local user yet, so the editor names the
-    // fallback rather than passing undefined into the caret renderer.
-    expect(caretOptions.user).toEqual({ name: "Anonymous", color: "#808080" });
-  });
-
-  it("passes every configured extension to the editor", () => {
-    const stubs = collaborationStubs();
-    renderCollaborativeEditor(stubs);
-
-    expect(stubs.starterKit.calls.at(-1)).toMatchObject({ history: false });
-    const options = stubs.editorOptions.at(-1);
-    expect(
-      (options?.extensions as { name: string }[]).map((one) => one.name),
-    ).toEqual(["starterKit", "collaboration", "collaborationCaret"]);
-  });
-});
-
-describe("TiptapEditor", () => {
-  it("holds a placeholder while the peers are still loading", async () => {
-    let releasePeers: (() => void) | null = null;
-    const gate = new Promise<void>((resolve) => {
-      releasePeers = resolve;
-    });
-    const stubs = collaborationStubs();
-    vi.resetModules();
-    vi.doMock("./editor-peers", async () => ({
-      ...(await vi.importActual<typeof EditorPeers>("./editor-peers")),
-      loadCollaborationPeers: async () => {
-        await gate;
-        return stubs.peers;
-      },
-    }));
-    const { TiptapEditor } = await import("./tiptap-editor");
-    // Both modules come from the reset graph, so the provider and the editor
-    // share one EditorContext. A provider built from the loaded peers
-    // directly, because the exported EditorProvider waits on the same gate and
-    // would render nothing at all.
-    const freshProvider = await import("./editor-provider");
-    const ResolvedProvider = freshProvider.createEditorProvider(stubs.peers);
-
-    render(
-      <ResolvedProvider
-        websocketUrl="wss://collab.example/ws"
-        documentName="doc:readme"
-        token="jwt"
-        user={{ name: "Ada" }}
-      >
-        <TiptapEditor />
-      </ResolvedProvider>,
-    );
-
-    expect(screen.getByText("Loading editor…")).toBeInTheDocument();
-    expect(screen.queryByTestId("editor-content")).toBeNull();
-
-    releasePeers?.();
-
-    await waitFor(() => {
-      expect(screen.getByTestId("editor-content")).toBeInTheDocument();
-    });
-    expect(screen.queryByText("Loading editor…")).toBeNull();
-  });
-
-  it("fails with the install list when the collaboration peers cannot be loaded", async () => {
-    const missing =
-      "@tangle-network/ui/editor needs its optional collaboration peers. Install yjs.";
-    vi.resetModules();
-    vi.doMock("./editor-peers", async () => {
-      const actual = await vi.importActual<typeof EditorPeers>("./editor-peers");
-      return {
-        ...actual,
-        loadCollaborationPeers: async () => {
-          throw new actual.MissingEditorPeersError(missing);
-        },
-      };
-    });
-    const { TiptapEditor } = await import("./tiptap-editor");
-
-    // The wrapper reaches its loader before it reads the editor context, so
-    // the install list surfaces without a provider around it.
-    render(
-      <ErrorTrap>
-        <TiptapEditor />
-      </ErrorTrap>,
-    );
-
-    await waitFor(() => {
-      expect(screen.getByText(missing)).toBeInTheDocument();
-    });
-  });
-});

package/src/files/file-artifact-pane.test.tsx

@@ -1,301 +0,0 @@
-import { describe, expect, it, vi } from "vitest";
-import { fireEvent, render, screen } from "@testing-library/react";
-import { FileArtifactPane } from "./file-artifact-pane";
-import { CSV_PREVIEW_ROW_LIMIT, parseCsv } from "./file-preview";
-import { MissingPdfjsError, loadPdfjs } from "./pdf-loader";
-
-vi.mock("./pdf-loader", async (importOriginal) => ({
-  ...(await importOriginal<typeof import("./pdf-loader")>()),
-  loadPdfjs: vi.fn(),
-}));
-
-const BLOB_URL = "blob:https://app.tangle.tools/1f2e3d4c";
-
-function csvWithRows(rowCount: number): string {
-  const rows = Array.from({ length: rowCount }, (_, i) => `"Ada, Countess ${i}",x,${i}`);
-  return ["name,note,n", ...rows].join("\n");
-}
-
-describe("FileArtifactPane previews", () => {
-  it("renders an image from blobUrl and toggles natural size on click", () => {
-    const { container } = render(
-      <FileArtifactPane filename="chart.png" blobUrl={BLOB_URL} mimeType="image/png" />,
-    );
-
-    const img = container.querySelector("img");
-    expect(img).not.toBeNull();
-    expect(img).toHaveAttribute("src", BLOB_URL);
-    expect(img).toHaveAttribute("alt", "chart.png");
-    expect(img).toHaveClass("object-contain");
-
-    const toggle = screen.getByRole("button", { name: "Show image at natural size" });
-    expect(toggle).toHaveAttribute("aria-pressed", "false");
-    fireEvent.click(toggle);
-    expect(screen.getByRole("button", { name: "Fit image to pane" })).toHaveAttribute(
-      "aria-pressed",
-      "true",
-    );
-    expect(container.querySelector("img")).toHaveClass("max-w-none");
-  });
-
-  it("renders a PDF in PdfViewer, and in an <object> of the blobUrl without pdfjs-dist", async () => {
-    vi.mocked(loadPdfjs).mockRejectedValue(new MissingPdfjsError());
-    const consoleError = vi.spyOn(console, "error").mockImplementation(() => {});
-    const { container } = render(
-      <FileArtifactPane filename="report.pdf" blobUrl={BLOB_URL} mimeType="application/pdf" />,
-    );
-
-    expect(screen.getByRole("status")).toHaveTextContent("Loading report.pdf");
-    expect(await screen.findByText(/does not include the page viewer/)).toBeInTheDocument();
-    const viewer = container.querySelector("object");
-    expect(viewer).toHaveAttribute("data", BLOB_URL);
-    expect(viewer).toHaveAttribute("type", "application/pdf");
-    expect(viewer).toHaveAttribute("title", "report.pdf");
-    consoleError.mockRestore();
-  });
-
-  it("renders video and audio with native controls", () => {
-    const video = render(<FileArtifactPane filename="demo.mp4" blobUrl={BLOB_URL} />);
-    const videoElement = video.container.querySelector("video");
-    expect(videoElement).not.toBeNull();
-    expect(videoElement).toHaveAttribute("src", BLOB_URL);
-    expect(videoElement).toHaveAttribute("controls");
-    video.unmount();
-
-    const audio = render(<FileArtifactPane filename="voice.m4a" blobUrl={BLOB_URL} />);
-    const audioElement = audio.container.querySelector("audio");
-    expect(audioElement).not.toBeNull();
-    expect(audioElement).toHaveAttribute("src", BLOB_URL);
-    expect(audioElement).toHaveAttribute("controls");
-    expect(audio.getAllByText("voice.m4a").length).toBeGreaterThan(1); // header + player label
-  });
-
-  it("parses quoted commas in CSV and caps the table at the row limit", () => {
-    const total = CSV_PREVIEW_ROW_LIMIT + 1;
-    const { container } = render(
-      <FileArtifactPane filename="people.csv" content={csvWithRows(total)} />,
-    );
-
-    const headers = Array.from(container.querySelectorAll("thead th")).map((th) => th.textContent);
-    expect(headers).toEqual(["name", "note", "n"]);
-    expect(container.querySelectorAll("thead th")[0]).toHaveClass("sticky");
-
-    const bodyRows = container.querySelectorAll("tbody tr");
-    expect(bodyRows).toHaveLength(CSV_PREVIEW_ROW_LIMIT);
-    expect(bodyRows[0].querySelectorAll("td")[0]).toHaveTextContent("Ada, Countess 0");
-    expect(bodyRows[0].querySelectorAll("td")).toHaveLength(3);
-
-    expect(
-      screen.getByText(`Showing ${CSV_PREVIEW_ROW_LIMIT} of ${total} rows · 3 columns`),
-    ).toBeInTheDocument();
-  });
-
-  it("reports the row count when the CSV is under the cap", () => {
-    render(<FileArtifactPane filename="people.csv" content={csvWithRows(3)} />);
-    expect(screen.getByText("3 rows · 3 columns")).toBeInTheDocument();
-  });
-
-  it("renders a spreadsheet as a download card and calls onDownload", () => {
-    const onDownload = vi.fn();
-    render(
-      <FileArtifactPane filename="book.xlsx" size={1536} onDownload={onDownload} />,
-    );
-
-    expect(screen.getByText("1.5 KB")).toBeInTheDocument();
-    expect(
-      screen.getByText("Download to open this workbook in a spreadsheet app."),
-    ).toBeInTheDocument();
-
-    fireEvent.click(screen.getByRole("button", { name: "Download" }));
-    expect(onDownload).toHaveBeenCalledTimes(1);
-
-    // The header keeps its own download control.
-    fireEvent.click(screen.getByRole("button", { name: "Download book.xlsx" }));
-    expect(onDownload).toHaveBeenCalledTimes(2);
-  });
-
-  it("renders markdown through the package Markdown renderer", () => {
-    const { container } = render(
-      <FileArtifactPane filename="README.md" content={"# Title\n\nBody with **bold**."} />,
-    );
-    expect(container.querySelector(".tangle-prose")).not.toBeNull();
-    expect(container.querySelector("h1")).toHaveTextContent("Title");
-    expect(container.querySelector("strong")).toHaveTextContent("bold");
-  });
-
-  it("forwards Markdown URL transforms to file and rich editor previews", async () => {
-    const urlTransform = vi.fn((url: string, key: string) => (
-      key === "href" && url === "sibling.md"
-        ? "/app/ws-1/vault?file=docs%2Fsibling.md"
-        : url
-    ));
-    const readOnly = render(
-      <FileArtifactPane
-        filename="README.md"
-        content={"[Sibling](sibling.md)"}
-        urlTransform={urlTransform}
-      />,
-    );
-
-    expect(screen.getByRole("link", { name: "Sibling" })).toHaveAttribute(
-      "href",
-      "/app/ws-1/vault?file=docs%2Fsibling.md",
-    );
-    readOnly.unmount();
-
-    render(
-      <FileArtifactPane
-        path="docs/README.md"
-        filename="README.md"
-        content={"[Sibling](sibling.md)"}
-        urlTransform={urlTransform}
-        editor={{ enabled: true, defaultMode: "preview" }}
-      />,
-    );
-
-    // The rich editor lazy-loads its editor packages; under a loaded host that takes
-    // longer than findBy's 1s default.
-    expect(await screen.findByRole("link", { name: "Sibling" }, { timeout: 10_000 })).toHaveAttribute(
-      "href",
-      "/app/ws-1/vault?file=docs%2Fsibling.md",
-    );
-    expect(urlTransform).toHaveBeenCalledWith(
-      "sibling.md",
-      "href",
-      expect.objectContaining({ tagName: "a" }),
-    );
-  });
-
-  it("renders a binary file with no download link as one muted sentence", () => {
-    const onDownload = vi.fn();
-    const withDownload = render(
-      <FileArtifactPane filename="weights.bin" onDownload={onDownload} />,
-    );
-    expect(
-      withDownload.getByText("This file needs a download link to preview."),
-    ).toBeInTheDocument();
-    fireEvent.click(withDownload.getByRole("button", { name: "Download" }));
-    expect(onDownload).toHaveBeenCalledTimes(1);
-    withDownload.unmount();
-
-    render(<FileArtifactPane filename="weights.bin" />);
-    expect(screen.getByText("This file needs a download link to preview.")).toBeInTheDocument();
-    expect(screen.queryByRole("button", { name: "Download" })).toBeNull();
-  });
-
-  it("renders the same sentence for an image, PDF, or video without a blobUrl", () => {
-    for (const filename of ["chart.png", "report.pdf", "demo.mp4"]) {
-      const { getByText, unmount } = render(<FileArtifactPane filename={filename} />);
-      expect(getByText("This file needs a download link to preview.")).toBeInTheDocument();
-      unmount();
-    }
-  });
-
-  it("renders an unknown file's text content as text", () => {
-    const { container } = render(
-      <FileArtifactPane filename="Makefile.custom" content={"build:\n\tgo build"} />,
-    );
-    expect(container.querySelector("pre")).toHaveTextContent("go build");
-    expect(container.querySelector("code")).toBeNull();
-  });
-
-  it("keeps code on the syntax highlighter and text on a plain pre", () => {
-    const code = render(
-      <FileArtifactPane filename="server.ts" content={"const x = 1;\nexport { x };"} />,
-    );
-    expect(code.container.querySelector("code")).not.toBeNull();
-    expect(code.container.querySelector("table")).toBeNull();
-    expect(code.getByText(/ts · 2 lines/)).toBeInTheDocument();
-    code.unmount();
-
-    const text = render(<FileArtifactPane filename="notes.txt" content="just words" />);
-    expect(text.container.querySelector("pre")).toHaveTextContent("just words");
-    expect(text.container.querySelector("code")).toBeNull();
-  });
-
-  it("renders SVG source as code when the host read it as text", () => {
-    const { container } = render(
-      <FileArtifactPane filename="icon.svg" content={'<svg xmlns="http://www.w3.org/2000/svg"/>'} />,
-    );
-    expect(container.querySelector("code")).not.toBeNull();
-    expect(container.querySelector("img")).toBeNull();
-  });
-});
-
-describe("parseCsv", () => {
-  it("keeps commas, line breaks, and doubled quotes inside quoted cells", () => {
-    expect(parseCsv('a,b\n"x, y","line 1\nline 2"\n"say ""hi""",z')).toEqual([
-      ["a", "b"],
-      ["x, y", "line 1\nline 2"],
-      ['say "hi"', "z"],
-    ]);
-  });
-
-  it("trims unquoted cells and accepts CRLF, blank lines, and a trailing line break", () => {
-    expect(parseCsv("a, b\r\n\r\n1 ,2\r\n")).toEqual([
-      ["a", "b"],
-      ["1", "2"],
-    ]);
-  });
-
-  it("returns no rows for empty input", () => {
-    expect(parseCsv("")).toEqual([]);
-    expect(parseCsv("\n\n")).toEqual([]);
-  });
-});
-
-describe("FileArtifactPane controlled drafts", () => {
-  it("retains a host draft and saves it against the persisted baseline", async () => {
-    const onSave = vi.fn();
-    const draft = "# Unsaved source draft";
-    const props = {
-      filename: "notes.md",
-      content: draft,
-      editor: { enabled: true, savedContent: "# Saved notes", onSave },
-    };
-    const view = render(<FileArtifactPane {...props} />);
-    const save = await screen.findByRole("button", { name: "Save changes" }, { timeout: 10_000 });
-    expect(screen.getByRole("heading", { name: "Unsaved source draft" })).toBeVisible();
-    expect(screen.getByText("Unsaved changes")).toBeVisible();
-    expect(save).toBeEnabled();
-    fireEvent.click(save);
-    expect(onSave).toHaveBeenCalledWith(draft);
-
-    view.rerender(<FileArtifactPane {...props} editor={{ ...props.editor, savedContent: draft }} />);
-    // One save state: "Saved", with no Save button left to press.
-    expect(screen.getByRole("status")).toHaveTextContent("Saved");
-    expect(screen.queryByRole("button", { name: "Save changes" })).toBeNull();
-    expect(screen.queryByText("Local draft")).toBeNull();
-  });
-
-  it("follows the host's dirty flag over its own comparison, and saves on Ctrl+S", async () => {
-    const onSave = vi.fn();
-    const content = "# Same text the host saved";
-    const props = { filename: "notes.md", content, editor: { enabled: true, dirty: true, onSave } };
-    const view = render(<FileArtifactPane {...props} />);
-    const save = await screen.findByRole("button", { name: "Save changes" }, { timeout: 10_000 });
-    expect(save).toBeEnabled();
-    expect(screen.getByRole("status")).toHaveTextContent("Unsaved changes");
-
-    fireEvent.keyDown(save, { key: "s", ctrlKey: true });
-    expect(onSave).toHaveBeenCalledWith(content);
-
-    view.rerender(<FileArtifactPane {...props} editor={{ ...props.editor, dirty: false }} />);
-    expect(screen.getByRole("status")).toHaveTextContent("Saved");
-    fireEvent.keyDown(screen.getByRole("status"), { key: "s", metaKey: true });
-    expect(onSave).toHaveBeenCalledTimes(1);
-  });
-
-  it("shows Saving while the host saves", async () => {
-    render(
-      <FileArtifactPane
-        filename="notes.md"
-        content="# Draft"
-        editor={{ enabled: true, dirty: true, saving: true, onSave: vi.fn() }}
-      />,
-    );
-    const save = await screen.findByRole("button", { name: "Save changes" }, { timeout: 10_000 });
-    expect(save).toBeDisabled();
-    expect(screen.getByRole("status")).toHaveTextContent("Saving…");
-  });
-});