@marimo-team/islands 0.25.3-dev189 → 0.25.3-dev191

package/dist/{swimlanes-42K2YHIH-ogImlOmp.js → swimlanes-42K2YHIH-Bj3hiOY5.js}

@@ -9,7 +9,7 @@ import "./chunk-4HAMMTFA-D4eFPyFh.js";
 import "./chunk-GVQU2GXP-Cwbt-lF0.js";
 import "./chunk-L3NEJ4N5-n-zgIVSy.js";
 import { o as n } from "./chunk-OSK3NFVY-DXa-Cxbn.js";
-import { c as r } from "./mermaid-CZ6FrClS.js";
+import { c as r } from "./mermaid-DU9ZRmkG.js";
 //#region ../node_modules/.pnpm/mermaid@11.17.2/node_modules/mermaid/dist/chunks/mermaid.core/swimlanes-42K2YHIH.mjs
 var i = 5, a = 1e-5, o = 1e-6;
 function s(e) {

package/dist/{swimlanesDiagram-VR7AAH4N-D6x-zsci.js → swimlanesDiagram-VR7AAH4N-BzEo2r_N.js}

@@ -9,11 +9,11 @@ import "./chunk-4HAMMTFA-D4eFPyFh.js";
 import "./chunk-GVQU2GXP-Cwbt-lF0.js";
 import "./chunk-L3NEJ4N5-n-zgIVSy.js";
 import "./chunk-OSK3NFVY-DXa-Cxbn.js";
-import "./mermaid-CZ6FrClS.js";
+import "./mermaid-DU9ZRmkG.js";
 import "./chunk-F27PBJKO-ufseIn5y.js";
 import "./chunk-XXDRQBXY-Dnl5VBqG.js";
 import "./chunk-POPQ4Y6H-hYOXbYH_.js";
-import { r as t, t as n } from "./chunk-SHT3W25Y-mSNp-WpO.js";
+import { r as t, t as n } from "./chunk-SHT3W25Y-DFrEYG54.js";
 //#region ../node_modules/.pnpm/mermaid@11.17.2/node_modules/mermaid/dist/chunks/mermaid.core/swimlanesDiagram-VR7AAH4N.mjs
 var r = n({
 	defaultLayout: "swimlane",

package/dist/{timeline-definition-24CTP7MA-DQpLtsHD.js → timeline-definition-24CTP7MA-CrPQk42H.js}

@@ -3,7 +3,7 @@ import { p as t } from "./chunk-75Z2AOVW-D7gxIs2Q.js";
 import { n, t as r } from "./src-CA_m4ONl.js";
 import { n as i, t as a } from "./chunk-Y2CYZVJY-DKKS1VyV.js";
 import { $ as o, J as s, a as c, b as l, et as u, nt as d, o as f, x as p } from "./chunk-DU6HZSFF-BwiyaNpg.js";
-import { p as m } from "./mermaid-CZ6FrClS.js";
+import { p as m } from "./mermaid-DU9ZRmkG.js";
 //#region ../node_modules/.pnpm/mermaid@11.17.2/node_modules/mermaid/dist/chunks/mermaid.core/timeline-definition-24CTP7MA.mjs
 var h = (function() {
 	var e = /* @__PURE__ */ i(function(e, t, n, r) {

package/dist/{toDate-CDzoofJo.js → toDate-Cu2pIZcC.js}

@@ -2,7 +2,7 @@ import { t as e } from "./rolldown-runtime-Cb76FRpK.js";
 import { g as t } from "./button-BQRdqQi1.js";
 import { t as n } from "./compiler-runtime-CldRdSsv.js";
 import { m as r } from "./tooltip-s4zZxwKe.js";
-import { D as i, _ as a, a as o, w as s, x as c } from "./useTheme-DMfQYmlG.js";
+import { D as i, _ as a, a as o, w as s, x as c } from "./useTheme-CoEtr4mK.js";
 import { t as l } from "./invariant-DLmrl-hA.js";
 var u = r("circle-question-mark", [
 	["circle", {

package/dist/{useAsyncData-Bn8SxUqM.js → useAsyncData-DEhLdRIu.js}

@@ -1,7 +1,7 @@
 import { a as e } from "./rolldown-runtime-Cb76FRpK.js";
 import { t } from "./react-sHtZdjeP.js";
 import { t as n } from "./compiler-runtime-CldRdSsv.js";
-import { A as r } from "./useTheme-DMfQYmlG.js";
+import { A as r } from "./useTheme-CoEtr4mK.js";
 import { t as i } from "./invariant-DLmrl-hA.js";
 //#region src/hooks/useAsyncData.ts
 var a = n(), o = /* @__PURE__ */ e(t(), 1), s = {

package/dist/{useDeepCompareMemoize-CC2tnPCk.js → useDeepCompareMemoize-RjZGVhBV.js}

@@ -1,6 +1,6 @@
 import { a as e } from "./rolldown-runtime-Cb76FRpK.js";
 import { t } from "./react-sHtZdjeP.js";
-import { b as n } from "./useTheme-DMfQYmlG.js";
+import { b as n } from "./useTheme-CoEtr4mK.js";
 //#region src/hooks/useDeepCompareMemoize.ts
 var r = /* @__PURE__ */ e(t(), 1);
 function i(e) {

package/dist/{useLifecycle-BJlxHqWa.js → useLifecycle-DtcB2s1z.js}

@@ -4,7 +4,7 @@ import { t as i } from "./react-sHtZdjeP.js";
 import { t as a } from "./compiler-runtime-CldRdSsv.js";
 import { m as o } from "./tooltip-s4zZxwKe.js";
 import { t as s } from "./jsx-runtime-3BD1kT0s.js";
-import { D as c, T as l } from "./useTheme-DMfQYmlG.js";
+import { D as c, T as l } from "./useTheme-CoEtr4mK.js";
 var u = o("calendar", [
 	["path", {
 		d: "M8 2v4",

package/dist/{useTheme-DMfQYmlG.js → useTheme-CoEtr4mK.js}

@@ -681,7 +681,7 @@ var ze = [
 		enabled: p().prefault(!0),
 		rules: u().prefault(""),
 		max_tokens: d().int().positive().nullable().optional(),
-		mode: m(We).prefault("manual"),
+		mode: m(We).transform((e) => e === "manual" ? "ask" : e).prefault("ask"),
 		inline_tooltip: p().prefault(!1),
 		allow_provider_config: p().prefault(!0),
 		open_ai: J.optional(),

package/dist/{vega-component-CovWHi60.js → vega-component-BhjTrP9-.js}

@@ -3,20 +3,20 @@ import { g as t, h as n, m as r, o as i } from "./button-BQRdqQi1.js";
 import { t as a } from "./react-sHtZdjeP.js";
 import { t as o } from "./compiler-runtime-CldRdSsv.js";
 import { t as s } from "./tooltip-s4zZxwKe.js";
-import { C as c, a as l } from "./toDate-CDzoofJo.js";
+import { C as c, a as l } from "./toDate-Cu2pIZcC.js";
 import { t as u } from "./jsx-runtime-3BD1kT0s.js";
 import { n as d } from "./error-banner-DxJ3s8YO.js";
 import { t as f } from "./debounce-C3jyATpc.js";
-import { A as p, n as ee } from "./useTheme-DMfQYmlG.js";
+import { A as p, n as ee } from "./useTheme-CoEtr4mK.js";
 import { c as m, t as h } from "./arrays-R6JsImiT.js";
 import { t as te } from "./objects-DCmMvZtn.js";
-import { t as ne } from "./useAsyncData-Bn8SxUqM.js";
-import { a as g, i as re, n as ie, t as ae } from "./formats-DI7HEVv7.js";
+import { t as ne } from "./useAsyncData-DEhLdRIu.js";
+import { a as g, i as re, n as ie, t as ae } from "./formats-DbXnijtH.js";
 import { t as _ } from "./semaphore-89jsoHtS.js";
 import { D as oe, n as se } from "./vega-loader.browser-B9Pz0EGP.js";
 import { t as ce } from "./react-vega-C4T9Rb_w.js";
 import { a as le, n as v } from "./loader-DOVB0Syl.js";
-import { t as ue } from "./useDeepCompareMemoize-CC2tnPCk.js";
+import { t as ue } from "./useDeepCompareMemoize-RjZGVhBV.js";
 import { t as de } from "./use-vega-container-remeasure-Bz4PZMtt.js";
 //#region src/plugins/impl/vega/fix-relative-url.ts
 var fe = /* @__PURE__ */ e(a(), 1), pe = o();

package/dist/{vennDiagram-4TSXK5OY-BD-SL_vq.js → vennDiagram-4TSXK5OY-BaYMoIpj.js}

@@ -3,7 +3,7 @@ import { t } from "./src-CA_m4ONl.js";
 import { n } from "./chunk-Y2CYZVJY-DKKS1VyV.js";
 import { $ as r, H as i, K as a, U as o, a as s, b as c, c as l, et as u, f as d, nt as f, tt as p, v as m, w as h, y as g } from "./chunk-DU6HZSFF-BwiyaNpg.js";
 import { t as _ } from "./rough.esm-CtV9AnV1.js";
-import { p as v } from "./mermaid-CZ6FrClS.js";
+import { p as v } from "./mermaid-DU9ZRmkG.js";
 //#region ../node_modules/.pnpm/khroma@2.1.0/node_modules/khroma/dist/methods/transparentize.js
 var y = (e, t) => p(e, "a", -t), b = 1e-10;
 function x(e, t) {

package/dist/{wardleyDiagram-VM6X3IG4-BDgJV4ah.js → wardleyDiagram-VM6X3IG4-DjHeuj8p.js}

@@ -2,7 +2,7 @@ import { i as e } from "./chunk-75Z2AOVW-D7gxIs2Q.js";
 import { n as t } from "./src-CA_m4ONl.js";
 import { n } from "./chunk-Y2CYZVJY-DKKS1VyV.js";
 import { D as r, H as i, K as a, U as o, a as s, b as c, c as l, v as u, w as d, x as f, y as p } from "./chunk-DU6HZSFF-BwiyaNpg.js";
-import { p as m } from "./mermaid-CZ6FrClS.js";
+import { p as m } from "./mermaid-DU9ZRmkG.js";
 import { n as h } from "./mermaid-parser.core-DxuSfYwJ.js";
 import { t as g } from "./chunk-JWPE2WC7-DJvv1t8U.js";
 //#region ../node_modules/.pnpm/mermaid@11.17.2/node_modules/mermaid/dist/chunks/mermaid.core/wardleyDiagram-VM6X3IG4.mjs

package/dist/{xychartDiagram-S5SC5T6Z-1hIHN6Rl.js → xychartDiagram-S5SC5T6Z-Bc_hcuXS.js}

@@ -9,7 +9,7 @@ import { n as s } from "./chunk-Y2CYZVJY-DKKS1VyV.js";
 import { D as c, H as l, K as u, U as d, a as f, b as p, c as m, f as h, v as g, w as _, y as v, z as y } from "./chunk-DU6HZSFF-BwiyaNpg.js";
 import "./chunk-PWAF6VOD-CQw5qATZ.js";
 import { t as b } from "./chunk-GMAD6QVW-BhaJx0X3.js";
-import { p as x } from "./mermaid-CZ6FrClS.js";
+import { p as x } from "./mermaid-DU9ZRmkG.js";
 //#region ../node_modules/.pnpm/d3-scale@4.0.2/node_modules/d3-scale/src/band.js
 function S() {
 	var e = t().unknown(void 0), r = e.domain, i = e.range, o = 0, s = 1, c, l, u = !1, d = 0, f = 0, p = .5;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@marimo-team/islands",
-  "version": "0.25.3-dev189",
+  "version": "0.25.3-dev191",
   "main": "dist/main.js",
   "types": "dist/index.d.ts",
   "type": "module",

package/src/components/chat/__tests__/notebook-access-selector.test.tsx

@@ -0,0 +1,64 @@
+/* Copyright 2026 Marimo. All rights reserved. */
+
+import { fireEvent, render, screen } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import type { CopilotMode } from "@/core/config/config-schema";
+import { NotebookAccessSelector } from "../notebook-access-selector";
+
+Element.prototype.scrollIntoView = vi.fn();
+
+describe("NotebookAccessSelector", () => {
+  it("shows legacy Manual as Read only and offers exactly three levels", async () => {
+    const onValueChange = vi.fn();
+    render(
+      <NotebookAccessSelector value="manual" onValueChange={onValueChange} />,
+    );
+
+    fireEvent.keyDown(
+      screen.getByRole("combobox", { name: "Notebook access: Read only" }),
+      { key: "ArrowDown" },
+    );
+    const options = await screen.findAllByRole("option");
+    expect(options.map((option) => option.getAttribute("data-state"))).toEqual([
+      "checked",
+      "unchecked",
+      "unchecked",
+    ]);
+    expect(options.map((option) => option.textContent)).toEqual([
+      "Read onlyRead notebook code, outputs, and variable previews.",
+      "Suggest editsRead the notebook and propose cell changes for review.",
+      "Full accessRead, edit, and run cells, with Python scratchpad access.",
+    ]);
+    expect(onValueChange).not.toHaveBeenCalled();
+  });
+
+  it.each<{ mode: CopilotMode; label: string }>([
+    { mode: "ask", label: "Read only" },
+    { mode: "agent", label: "Suggest edits" },
+    { mode: "code_mode", label: "Full access" },
+  ])("maps $label to the existing $mode setting", async ({ mode, label }) => {
+    const onValueChange = vi.fn();
+    const { rerender } = render(
+      <NotebookAccessSelector value={mode} onValueChange={onValueChange} />,
+    );
+
+    expect(
+      screen.getByRole("combobox", { name: `Notebook access: ${label}` }),
+    ).toHaveTextContent(label);
+    expect(onValueChange).not.toHaveBeenCalled();
+
+    rerender(
+      <NotebookAccessSelector
+        value={mode === "ask" ? "agent" : "ask"}
+        onValueChange={onValueChange}
+      />,
+    );
+    fireEvent.keyDown(screen.getByRole("combobox"), { key: "ArrowDown" });
+    const option = await screen.findByRole("option", {
+      name: new RegExp(`^${label} `),
+    });
+    fireEvent.keyDown(option, { key: "Enter" });
+
+    expect(onValueChange).toHaveBeenCalledExactlyOnceWith(mode);
+  });
+});

package/src/components/chat/chat-panel.tsx

@@ -13,28 +13,15 @@ import {
 import { useAtom, useAtomValue, useSetAtom, useStore } from "jotai";
 import {
   BotMessageSquareIcon,
-  HatGlasses,
   ArrowRightIcon,
   Loader2,
-  type LucideIcon,
-  MessageCircleIcon,
-  NotebookText,
   PlusIcon,
   SparklesIcon,
   SettingsIcon,
-  CodeIcon,
 } from "lucide-react";
 import { memo, useEffect, useRef, useState } from "react";
 import useEvent from "react-use-event-hook";
 import { Button } from "@/components/ui/button";
-import {
-  Select,
-  SelectContent,
-  SelectGroup,
-  SelectItem,
-  SelectLabel,
-  SelectTrigger,
-} from "@/components/ui/select";
 import { replaceMessagesInChat } from "@/core/ai/chat-utils";
 import { useAIConfigActions } from "@/core/ai/config";
 import { AI_SDK_UI_THROTTLE_MS } from "@/core/ai/constants";
@@ -49,10 +36,7 @@ import {
   pendingAiPromptAtom,
 } from "@/core/ai/state";
 import type { ToolNotebookContext } from "@/core/ai/tools/base";
-import {
-  type CopilotMode,
-  FRONTEND_TOOL_REGISTRY,
-} from "@/core/ai/tools/registry";
+import { FRONTEND_TOOL_REGISTRY } from "@/core/ai/tools/registry";
 import { useCellActions } from "@/core/cells/actions";
 import { aiAtom, aiModelConfiguredAtom } from "@/core/config/config";
 import { DEFAULT_AI_MODEL } from "@/core/config/config-schema";
@@ -97,6 +81,7 @@ import {
 import { renderUIMessage } from "./chat-display";
 import { ChatHistoryPopover } from "./chat-history-popover";
 import { CapabilitiesPopover } from "./capabilities-popover";
+import { NotebookAccessSelector } from "./notebook-access-selector";
 import {
   type ChatMessagePart,
   convertToFileUIPart,
@@ -117,7 +102,7 @@ import ScrollToBottomButton from "./acp/scroll-to-bottom-button";
 import { useChatControllerId } from "./use-chat-controller-id";
 
 // Default mode for the AI
-const DEFAULT_MODE = "manual";
+const DEFAULT_MODE = "ask";
 
 const QueuedMessageDisplay = memo(
   ({ message }: { message: QueuedUserMessage }) => {
@@ -321,82 +306,17 @@ const ChatInputFooter: React.FC<ChatInputFooterProps> = memo(
 
     const { saveModeChange } = useAIConfigActions();
 
-    const modeOptions: {
-      value: CopilotMode;
-      label: string;
-      subtitle: string;
-      Icon: LucideIcon;
-    }[] = [
-      {
-        value: "manual",
-        label: "Manual",
-        subtitle: "Pure chat, no tool usage",
-        Icon: MessageCircleIcon,
-      },
-      {
-        value: "ask",
-        label: "Ask",
-        subtitle: "AI with access to read-only tools like documentation search",
-        Icon: NotebookText,
-      },
-      {
-        value: "agent",
-        label: "Agent",
-        subtitle: "AI with access to read and write tools",
-        Icon: HatGlasses,
-      },
-      {
-        value: "code_mode",
-        label: "Code Mode (beta)",
-        subtitle:
-          "AI with access to the notebook's kernel. Potentially destructive.",
-        Icon: CodeIcon,
-      },
-    ];
-
     const isAttachmentSupported =
       PROVIDERS_THAT_SUPPORT_ATTACHMENTS.has(currentProvider);
 
-    const currentModeOption = modeOptions.find((o) => o.value === currentMode);
-    const CurrentModeIcon = currentModeOption?.Icon;
-    const CurrentModeLabel = currentModeOption?.label;
-
     return (
       <TooltipProvider>
         <div className="px-3 py-2 border-t border-border/20 flex flex-wrap items-center gap-1">
           <div className="flex flex-wrap items-center gap-1">
-            <Select value={currentMode} onValueChange={saveModeChange}>
-              <SelectTrigger className="h-6 text-xs border-border shadow-none! ring-0! bg-muted hover:bg-muted/30 py-0 px-2 gap-1.5 shrink-0">
-                {CurrentModeIcon && <CurrentModeIcon className="h-3 w-3" />}
-                <span className="text-nowrap">{CurrentModeLabel}</span>
-              </SelectTrigger>
-              <SelectContent>
-                <SelectGroup>
-                  <SelectLabel className="text-xs uppercase tracking-wider text-muted-foreground/70 font-medium">
-                    AI Mode
-                  </SelectLabel>
-                  {modeOptions.map((option) => (
-                    <SelectItem
-                      key={option.value}
-                      value={option.value}
-                      className="text-xs py-1"
-                    >
-                      <div className="flex items-start gap-2.5">
-                        <span className="mt-1 text-muted-foreground">
-                          <option.Icon className="h-3 w-3" />
-                        </span>
-                        <div className="flex flex-col gap-0.5">
-                          <span className="font-semibold">{option.label}</span>
-                          <span className="text-muted-foreground">
-                            {option.subtitle}
-                          </span>
-                        </div>
-                      </div>
-                    </SelectItem>
-                  ))}
-                </SelectGroup>
-              </SelectContent>
-            </Select>
+            <NotebookAccessSelector
+              value={currentMode}
+              onValueChange={saveModeChange}
+            />
             <AIModelDropdown
               placeholder="Model"
               triggerClassName="h-6 text-xs shadow-none! ring-0! bg-muted hover:bg-muted/30 rounded-sm max-w-[200px]"
@@ -573,7 +493,7 @@ const ChatPanelBody = () => {
     setAbortNotice(null);
   });
   const runtimeManager = useRuntimeManager();
-  const { invokeAiTool, sendRun } = useRequestClient();
+  const { invokeAiTool } = useRequestClient();
   const { openModal, closeModal } = useImperativeModal();
 
   const activeChatId = activeChat?.id;
@@ -582,12 +502,10 @@ const ChatPanelBody = () => {
   const store = useStore();
 
   const { addStagedCell } = useStagedAICellsActions();
-  const { prepareForRun, createCell } = useCellActions();
+  const { createCell } = useCellActions();
   const toolContext: ToolNotebookContext = {
     addStagedCell,
     createCell,
-    prepareForRun,
-    sendRun,
     store,
   };
 

package/src/components/chat/notebook-access-selector.stories.tsx

@@ -0,0 +1,22 @@
+/* Copyright 2026 Marimo. All rights reserved. */
+
+import type { Meta, StoryObj } from "@storybook/react-vite";
+import { useState } from "react";
+import { NotebookAccessSelector } from "./notebook-access-selector";
+
+const meta = {
+  title: "Chat/Notebook access",
+  component: NotebookAccessSelector,
+  args: { value: "ask", onValueChange: () => undefined },
+  render: function Render(args) {
+    const [value, setValue] = useState(args.value);
+    return <NotebookAccessSelector value={value} onValueChange={setValue} />;
+  },
+} satisfies Meta<typeof NotebookAccessSelector>;
+
+export default meta;
+type Story = StoryObj<typeof meta>;
+
+export const ReadOnly: Story = { args: { value: "ask" } };
+export const SuggestEdits: Story = { args: { value: "agent" } };
+export const FullAccess: Story = { args: { value: "code_mode" } };

package/src/components/chat/notebook-access-selector.tsx

@@ -0,0 +1,100 @@
+/* Copyright 2026 Marimo. All rights reserved. */
+
+import {
+  EyeIcon,
+  type LucideIcon,
+  PencilIcon,
+  TerminalIcon,
+} from "lucide-react";
+import {
+  Select,
+  SelectContent,
+  SelectGroup,
+  SelectItem,
+  SelectLabel,
+  SelectTrigger,
+} from "@/components/ui/select";
+import type { CopilotMode } from "@/core/config/config-schema";
+
+const ACCESS_LEVELS = {
+  ask: {
+    label: "Read only",
+    description: "Read notebook code, outputs, and variable previews.",
+    Icon: EyeIcon,
+  },
+  agent: {
+    label: "Suggest edits",
+    description: "Read the notebook and propose cell changes for review.",
+    Icon: PencilIcon,
+  },
+  code_mode: {
+    label: "Full access",
+    description: "Read, edit, and run cells, with Python scratchpad access.",
+    Icon: TerminalIcon,
+  },
+} satisfies Record<
+  Exclude<CopilotMode, "manual">,
+  { label: string; description: string; Icon: LucideIcon }
+>;
+
+const ACCESS_MODES = ["ask", "agent", "code_mode"] as const;
+
+interface NotebookAccessSelectorProps {
+  value: CopilotMode;
+  onValueChange: (value: CopilotMode) => void;
+}
+
+export const NotebookAccessSelector = ({
+  value,
+  onValueChange,
+}: NotebookAccessSelectorProps) => {
+  const mode = value === "manual" ? "ask" : value;
+  const selected = ACCESS_LEVELS[mode];
+
+  return (
+    <Select
+      value={mode}
+      onValueChange={(value) => {
+        const mode = ACCESS_MODES.find((mode) => mode === value);
+        if (mode) {
+          onValueChange(mode);
+        }
+      }}
+    >
+      <SelectTrigger
+        aria-label={`Notebook access: ${selected.label}`}
+        title={`Notebook access: ${selected.description}`}
+        className="h-6 text-xs border-border shadow-none! ring-0! bg-muted hover:bg-muted/30 py-0 px-2 gap-1.5 shrink-0"
+      >
+        <selected.Icon className="h-3 w-3" />
+        <span className="text-nowrap">{selected.label}</span>
+      </SelectTrigger>
+      <SelectContent className="max-w-[calc(100vw-2rem)]">
+        <SelectGroup>
+          <SelectLabel className="text-xs uppercase tracking-wider text-muted-foreground/70 font-medium">
+            Notebook access
+          </SelectLabel>
+          {ACCESS_MODES.map((mode) => {
+            const { label, description, Icon } = ACCESS_LEVELS[mode];
+            return (
+              <SelectItem
+                key={mode}
+                value={mode}
+                textValue={label}
+                className="text-xs py-1"
+              >
+                <div className="flex items-start gap-2.5">
+                  <Icon className="h-3 w-3 mt-1 shrink-0 text-muted-foreground" />
+                  <div className="flex flex-col gap-0.5">
+                    <span className="font-semibold">{label}</span>
+                    <span className="text-muted-foreground">{description}</span>
+                  </div>
+                </div>
+              </SelectItem>
+            );
+          })}
+        </SelectGroup>
+      </SelectContent>
+    </Select>
+  );
+};

package/src/core/ai/tools/__tests__/edit-notebook-tool.test.ts

@@ -65,8 +65,6 @@ describe("EditNotebookTool", () => {
   let toolContext: {
     addStagedCell: ReturnType<typeof vi.fn>;
     createCell: ReturnType<typeof vi.fn>;
-    prepareForRun: ReturnType<typeof vi.fn>;
-    sendRun: ReturnType<typeof vi.fn>;
     store: ReturnType<typeof getDefaultStore>;
   };
 
@@ -82,8 +80,6 @@ describe("EditNotebookTool", () => {
         store.set(stagedAICellsAtom, new Map(current).set(cellId, edit));
       }),
       createCell: vi.fn(),
-      prepareForRun: vi.fn(),
-      sendRun: vi.fn().mockResolvedValue(null),
       store,
     };
     tool = new EditNotebookTool();
@@ -100,7 +96,7 @@ describe("EditNotebookTool", () => {
     it("should have correct metadata", () => {
       expect(tool.name).toBe("edit_notebook_tool");
       expect(tool.description).toBeDefined();
-      expect(tool.description.baseDescription).toContain("editing operations");
+      expect(tool.description.baseDescription).toContain("Propose cell edits");
       expect(tool.schema).toBeDefined();
       expect(tool.outputSchema).toBeDefined();
       expect(tool.mode).toEqual(["agent"]);

package/src/core/ai/tools/__tests__/registry.test.ts

@@ -10,6 +10,23 @@ import { TestFrontendTool } from "../sample-tool";
 const INVALID_KEYS_IN_SCHEMA_PARAMS = new Set(["anyOf", "oneOf"]);
 
 describe("FrontendToolRegistry", () => {
+  it("only offers cell proposals with Suggest edits access", () => {
+    expect(
+      Object.fromEntries(
+        COPILOT_MODES.map((mode) => [
+          mode,
+          FRONTEND_TOOL_REGISTRY.getToolSchemas(mode).map((tool) => tool.name),
+        ]),
+      ),
+    ).toEqual({
+      manual: [],
+      ask: [],
+      agent: ["edit_notebook_tool"],
+      code_mode: [],
+    });
+    expect(FRONTEND_TOOL_REGISTRY.has("run_stale_cells_tool")).toBe(false);
+  });
+
   it("registers tools via constructor and supports has()", () => {
     const registry = new FrontendToolRegistry([new TestFrontendTool()]);
     expect(registry.has("test_frontend_tool")).toBe(true);

package/src/core/ai/tools/base.ts

@@ -3,7 +3,6 @@
 import { z } from "zod";
 import type { CreateCell } from "@/core/cells/actions";
 import type { CellId } from "@/core/cells/ids";
-import type { ExecuteCellsRequest } from "@/core/network/types";
 import type { JotaiStore } from "@/core/state/jotai";
 import type { Edit } from "../staged-cells";
 import type { CopilotMode } from "./registry";
@@ -67,9 +66,6 @@ export class ToolExecutionError extends Error {
   }
 }
 
-// Can be any input
-export type EmptyToolInput = unknown;
-
 /**
  * Base interface for tool output responses.
  * Mirrors the SuccessResult dataclass from marimo/_ai/_tools/types.py
@@ -114,8 +110,6 @@ export interface ToolNotebookContext {
   store: JotaiStore;
   addStagedCell: (payload: { cellId: CellId; edit: Edit }) => void;
   createCell: CreateCell;
-  prepareForRun: (payload: { cellId: CellId }) => void;
-  sendRun: (request: ExecuteCellsRequest) => Promise<null>;
 }
 
 export interface AiTool<TIn, TOut> {

package/src/core/ai/tools/edit-notebook-tool.ts

@@ -21,7 +21,7 @@ import type { CopilotMode } from "./registry";
 
 const description: ToolDescription = {
   baseDescription:
-    "Perform editing operations on the current notebook. You should prefer to create new cells unless you need to edit existing cells. Call this tool multiple times to perform multiple edits. Separate code into logical individual cells to take advantage of the notebook's reactive execution model.",
+    "Propose cell edits for the user to review in the current notebook. You should prefer to create new cells unless you need to edit existing cells. Call this tool multiple times to perform multiple edits. Separate code into logical individual cells to take advantage of the notebook's reactive execution model.",
   prerequisites: [
     "If you are updating existing cells, you need the cellIds. If they are not known, call the lightweight_cell_map_tool to find out.",
   ],
@@ -254,7 +254,6 @@ export class EditNotebookTool implements AiTool<
       next_steps: [
         "If you need to perform more edits, call this tool again.",
         "You should use the lint notebook tool to check for errors and lint issues. Fix them by editing the notebook.",
-        "You should use the run stale cells tool to run the cells that have been edited or newly added. This allows you to see the output of the cells and fix any errors.",
       ],
     };
   };

package/src/core/ai/tools/registry.ts

@@ -9,7 +9,6 @@ import {
   type ToolNotebookContext,
 } from "./base";
 import { EditNotebookTool } from "./edit-notebook-tool";
-import { RunStaleCellsTool } from "./run-cells-tool";
 import { formatToolDescription } from "./utils";
 
 export type AnyZodObject = ZodObject<z.ZodRawShape>;
@@ -148,5 +147,4 @@ export class FrontendToolRegistry {
 
 export const FRONTEND_TOOL_REGISTRY = new FrontendToolRegistry([
   new EditNotebookTool(),
-  new RunStaleCellsTool(),
 ]);

package/src/core/config/__tests__/config-schema.test.ts

@@ -16,6 +16,11 @@ import {
   UserConfigSchema,
 } from "../config-schema";
 
+test("legacy Manual config resolves to Read only", () => {
+  const config = UserConfigSchema.parse({ ai: { mode: "manual" } });
+  expect(config).toEqual(UserConfigSchema.parse({ ai: { mode: "ask" } }));
+});
+
 test("default AppConfig", () => {
   const defaultConfig = AppConfigSchema.parse({});
   expect(defaultConfig).toMatchInlineSnapshot(`
@@ -51,7 +56,7 @@ test("default UserConfig - empty", () => {
         "custom_providers": {},
         "enabled": true,
         "inline_tooltip": false,
-        "mode": "manual",
+        "mode": "ask",
         "models": {
           "custom_models": [],
           "displayed_models": [],
@@ -128,7 +133,7 @@ test("default UserConfig - one level", () => {
         "custom_providers": {},
         "enabled": true,
         "inline_tooltip": false,
-        "mode": "manual",
+        "mode": "ask",
         "models": {
           "custom_models": [],
           "displayed_models": [],