@marimo-team/frontend 0.23.10-dev3 → 0.23.10-dev5

package/dist/index.html

@@ -66,7 +66,7 @@
     <marimo-server-token data-token="{{ server_token }}" hidden></marimo-server-token>
     <!-- /TODO -->
     <title>{{ title }}</title>
-    <script type="module" crossorigin src="./assets/index-DIc0rD1S.js"></script>
+    <script type="module" crossorigin src="./assets/index-v6_0iwpH.js"></script>
     <link rel="modulepreload" crossorigin href="./assets/preload-helper-BPPi7vOr.js">
     <link rel="modulepreload" crossorigin href="./assets/chunk-LvLJmgfZ.js">
     <link rel="modulepreload" crossorigin href="./assets/react-Bj1aDYRI.js">
@@ -133,7 +133,7 @@
     <link rel="modulepreload" crossorigin href="./assets/debounce-DhnxH9Rh.js">
     <link rel="modulepreload" crossorigin href="./assets/database-zap-kIkTfzTX.js">
     <link rel="modulepreload" crossorigin href="./assets/main-B0OX4z33.js">
-    <link rel="modulepreload" crossorigin href="./assets/cells-BRMYB7Wo.js">
+    <link rel="modulepreload" crossorigin href="./assets/cells-BFTdVtl5.js">
     <link rel="modulepreload" crossorigin href="./assets/ErrorBoundary-DyYDV0HI.js">
     <link rel="modulepreload" crossorigin href="./assets/kbd-CTUAEnEx.js">
     <link rel="modulepreload" crossorigin href="./assets/useInstallPackage-DUF4IRRI.js">
@@ -147,35 +147,35 @@
     <link rel="modulepreload" crossorigin href="./assets/usePress-DQ_tAz5W.js">
     <link rel="modulepreload" crossorigin href="./assets/input-C3Hrdlqq.js">
     <link rel="modulepreload" crossorigin href="./assets/ImperativeModal-B3Th7k4R.js">
-    <link rel="modulepreload" crossorigin href="./assets/cell-link-Gwsf-bMV.js">
+    <link rel="modulepreload" crossorigin href="./assets/cell-link-xJoNOwtA.js">
     <link rel="modulepreload" crossorigin href="./assets/multi-map-CUuNtzHt.js">
     <link rel="modulepreload" crossorigin href="./assets/alert-yTS3WpF4.js">
     <link rel="modulepreload" crossorigin href="./assets/chevron-right-CG5QYXYk.js">
     <link rel="modulepreload" crossorigin href="./assets/dropdown-menu-CR7cnzLX.js">
     <link rel="modulepreload" crossorigin href="./assets/links-D1JoyKTt.js">
-    <link rel="modulepreload" crossorigin href="./assets/useRunCells-2n74L4TT.js">
+    <link rel="modulepreload" crossorigin href="./assets/useRunCells-D8uIzoGA.js">
     <link rel="modulepreload" crossorigin href="./assets/copy-LK56fFow.js">
     <link rel="modulepreload" crossorigin href="./assets/copy-BwrPA9zQ.js">
     <link rel="modulepreload" crossorigin href="./assets/copy-icon-OjtDb4gO.js">
-    <link rel="modulepreload" crossorigin href="./assets/RenderHTML-B_YU1hMw.js">
-    <link rel="modulepreload" crossorigin href="./assets/datasource-CHSjxFZg.js">
-    <link rel="modulepreload" crossorigin href="./assets/state-BZoBW9cM.js">
+    <link rel="modulepreload" crossorigin href="./assets/RenderHTML-CGw415hl.js">
+    <link rel="modulepreload" crossorigin href="./assets/datasource-BEIQWqLs.js">
+    <link rel="modulepreload" crossorigin href="./assets/state-0417Fmr2.js">
     <link rel="modulepreload" crossorigin href="./assets/package-Tv6ztuzw.js">
     <link rel="modulepreload" crossorigin href="./assets/sparkles-lWUAsPhp.js">
-    <link rel="modulepreload" crossorigin href="./assets/MarimoErrorOutput-HYZo17DD.js">
+    <link rel="modulepreload" crossorigin href="./assets/MarimoErrorOutput-DYkyRvbE.js">
     <link rel="modulepreload" crossorigin href="./assets/spinner-Bhir8k53.js">
-    <link rel="modulepreload" crossorigin href="./assets/html-to-image-GGmvROsd.js">
-    <link rel="modulepreload" crossorigin href="./assets/focus-B2t0WTxH.js">
+    <link rel="modulepreload" crossorigin href="./assets/html-to-image-DWnBdIxw.js">
+    <link rel="modulepreload" crossorigin href="./assets/focus-Ch00G7oo.js">
     <link rel="modulepreload" crossorigin href="./assets/useAsyncData-bgszE9F0.js">
     <link rel="modulepreload" crossorigin href="./assets/LazyAnyLanguageCodeMirror-BEvXb3VX.js">
     <link rel="modulepreload" crossorigin href="./assets/micromark-factory-space-BygYYKhs.js">
     <link rel="modulepreload" crossorigin href="./assets/chunk-5FQGJX7Z-D9iBG0F7.js">
-    <link rel="modulepreload" crossorigin href="./assets/markdown-renderer-CHlWY8PG.js">
+    <link rel="modulepreload" crossorigin href="./assets/markdown-renderer-DMWX12-G.js">
     <link rel="modulepreload" crossorigin href="./assets/command-KARR7KMq.js">
     <link rel="modulepreload" crossorigin href="./assets/field-zLmMOSA4.js">
     <link rel="modulepreload" crossorigin href="./assets/popover-Bz_0Vkyf.js">
     <link rel="modulepreload" crossorigin href="./assets/errors-vr57w7Ul.js">
-    <link rel="modulepreload" crossorigin href="./assets/download-cs3tezQD.js">
+    <link rel="modulepreload" crossorigin href="./assets/download-DwObWxor.js">
     <link rel="modulepreload" crossorigin href="./assets/table-BGPSHfig.js">
     <link rel="modulepreload" crossorigin href="./assets/useIframeCapabilities-CcI1zSdn.js">
     <link rel="modulepreload" crossorigin href="./assets/error-banner-LdWZDbqd.js">
@@ -196,25 +196,25 @@
     <link rel="modulepreload" crossorigin href="./assets/plus-BgB18UzY.js">
     <link rel="modulepreload" crossorigin href="./assets/trash-2-rVklqqFF.js">
     <link rel="modulepreload" crossorigin href="./assets/react-resizable-panels.browser.esm-CV8-hvjx.js">
-    <link rel="modulepreload" crossorigin href="./assets/JsonOutput-40Fp2jEG.js">
+    <link rel="modulepreload" crossorigin href="./assets/JsonOutput-fyYubT1B.js">
     <link rel="modulepreload" crossorigin href="./assets/chart-no-axes-column-nqk474t8.js">
     <link rel="modulepreload" crossorigin href="./assets/square-function-uY_yJr5g.js">
     <link rel="modulepreload" crossorigin href="./assets/spec-CPQR_o92.js">
     <link rel="modulepreload" crossorigin href="./assets/ellipsis-vertical-CkwWkOQL.js">
     <link rel="modulepreload" crossorigin href="./assets/refresh-cw-DHwG4Mac.js">
     <link rel="modulepreload" crossorigin href="./assets/tree-actions-BM_EJr3E.js">
-    <link rel="modulepreload" crossorigin href="./assets/components-C_z0Dsug.js">
-    <link rel="modulepreload" crossorigin href="./assets/column-preview-Bd7jDAiw.js">
+    <link rel="modulepreload" crossorigin href="./assets/components-zJe4_t36.js">
+    <link rel="modulepreload" crossorigin href="./assets/column-preview-COP8lRZk.js">
     <link rel="modulepreload" crossorigin href="./assets/icons-Ol38nIbL.js">
-    <link rel="modulepreload" crossorigin href="./assets/radio-group-DcUXH-59.js">
-    <link rel="modulepreload" crossorigin href="./assets/floating-outline-C_E8vH7G.js">
+    <link rel="modulepreload" crossorigin href="./assets/radio-group-Cc9ma3vN.js">
+    <link rel="modulepreload" crossorigin href="./assets/floating-outline-DQaphys-.js">
     <link rel="modulepreload" crossorigin href="./assets/objectWithoutPropertiesLoose-DfWeGRFv.js">
     <link rel="modulepreload" crossorigin href="./assets/esm-CqWdmSnV.js">
-    <link rel="modulepreload" crossorigin href="./assets/readonly-python-code-D6Ca_8n6.js">
+    <link rel="modulepreload" crossorigin href="./assets/readonly-python-code-CT0pldK0.js">
     <link rel="modulepreload" crossorigin href="./assets/file-headphone-BrQspHac.js">
     <link rel="modulepreload" crossorigin href="./assets/file-BrdxGLRX.js">
     <link rel="modulepreload" crossorigin href="./assets/image-DQHXdEQn.js">
-    <link rel="modulepreload" crossorigin href="./assets/file-icons-CNegz5Wt.js">
+    <link rel="modulepreload" crossorigin href="./assets/file-icons-BqkEAg4n.js">
     <link rel="modulepreload" crossorigin href="./assets/switch-BiU_sAcn.js">
     <link rel="modulepreload" crossorigin href="./assets/events-CPoJAfgx.js">
     <link rel="modulepreload" crossorigin href="./assets/globals-7JnRMGd4.js">
@@ -223,11 +223,11 @@
     <link rel="modulepreload" crossorigin href="./assets/memoize-Tp7rARFe.js">
     <link rel="modulepreload" crossorigin href="./assets/get-C-qh_et5.js">
     <link rel="modulepreload" crossorigin href="./assets/_baseSet-CxV9N1bc.js">
-    <link rel="modulepreload" crossorigin href="./assets/state-CRuZ8deV.js">
+    <link rel="modulepreload" crossorigin href="./assets/state-BXPk8FBd.js">
     <link rel="modulepreload" crossorigin href="./assets/label-xHqFtfdz.js">
-    <link rel="modulepreload" crossorigin href="./assets/textarea-0Z0sU8Go.js">
+    <link rel="modulepreload" crossorigin href="./assets/textarea-DoNsOWV-.js">
     <link rel="modulepreload" crossorigin href="./assets/refresh-ccw-C-n2VFP5.js">
-    <link rel="modulepreload" crossorigin href="./assets/form-BnFjtYoC.js">
+    <link rel="modulepreload" crossorigin href="./assets/form-mMtSITO5.js">
     <link rel="modulepreload" crossorigin href="./assets/renderShortcut-CcFk3m01.js">
     <link rel="modulepreload" crossorigin href="./assets/useBoolean-Bp18o6XG.js">
     <link rel="modulepreload" crossorigin href="./assets/useDeepCompareMemoize-zUHU--0D.js">

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@marimo-team/frontend",
-  "version": "0.23.10-dev3",
+  "version": "0.23.10-dev5",
   "main": "dist/main.js",
   "types": "dist/index.d.ts",
   "type": "module",

package/src/components/editor/cell/cell-context-menu.tsx

@@ -61,6 +61,7 @@ export const CellActionsContextMenu = ({
   });
   const [imageRightClicked, setImageRightClicked] =
     React.useState<HTMLImageElement>();
+  const suppressCloseAutoFocus = React.useRef(false);
 
   const DEFAULT_CONTEXT_MENU_ITEMS: ActionButton[] = [
     {
@@ -166,7 +167,10 @@ export const CellActionsContextMenu = ({
       handle: () => {
         const editorView = getEditorView();
         if (editorView) {
-          goToDefinitionAtCursorPosition(editorView);
+          // Only suppress focus restoration when we actually navigated;
+          // otherwise let Radix return focus to the trigger cell.
+          suppressCloseAutoFocus.current =
+            goToDefinitionAtCursorPosition(editorView);
         }
       },
     },
@@ -194,7 +198,16 @@ export const CellActionsContextMenu = ({
       >
         {children}
       </ContextMenuTrigger>
-      <ContextMenuContent className="w-[300px]" scrollable={true}>
+      <ContextMenuContent
+        className="w-[300px]"
+        scrollable={true}
+        onCloseAutoFocus={(evt) => {
+          if (suppressCloseAutoFocus.current) {
+            evt.preventDefault();
+            suppressCloseAutoFocus.current = false;
+          }
+        }}
+      >
         {visibleActions.map((group, i) => (
           <Fragment key={i}>
             {group.map((action) => {

package/src/core/cells/__tests__/__snapshots__/cells.test.ts.snap

@@ -80,7 +80,6 @@ exports[`cell reducer > can initialize stdin 3`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "queued",
@@ -110,7 +109,6 @@ exports[`cell reducer > can initialize stdin 4`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -147,7 +145,6 @@ exports[`cell reducer > can initialize stdin 5`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -184,7 +181,6 @@ exports[`cell reducer > can initialize stdin 6`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -227,7 +223,6 @@ exports[`cell reducer > can initialize stdin 7`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -271,7 +266,6 @@ exports[`cell reducer > can initialize stdin 8`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -321,7 +315,6 @@ exports[`cell reducer > can initialize stdin 9`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -387,7 +380,6 @@ exports[`cell reducer > can receive console when the cell is idle and will clear
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "queued",
@@ -424,7 +416,6 @@ exports[`cell reducer > can receive console when the cell is idle and will clear
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "queued",
@@ -454,7 +445,6 @@ exports[`cell reducer > can receive console when the cell is idle and will clear
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -491,7 +481,6 @@ exports[`cell reducer > can receive console when the cell is idle and will clear
   "output": null,
   "runElapsedTimeMs": 2000,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -521,7 +510,6 @@ exports[`cell reducer > can run a stale cell 1`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "queued",
@@ -551,7 +539,6 @@ exports[`cell reducer > can run a stale cell 2`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": true,
   "status": "queued",
@@ -581,7 +568,6 @@ exports[`cell reducer > can run a stopped cell 1`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "queued",
@@ -611,7 +597,6 @@ exports[`cell reducer > can run a stopped cell 2`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -652,7 +637,6 @@ exports[`cell reducer > can run a stopped cell 3`] = `
   },
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -693,7 +677,6 @@ exports[`cell reducer > can run a stopped cell 4`] = `
   },
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "queued",
@@ -723,7 +706,6 @@ exports[`cell reducer > can run a stopped cell 5`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 40,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -840,7 +822,6 @@ exports[`cell reducer > can run cell and receive cell messages 4`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "queued",
@@ -870,7 +851,6 @@ exports[`cell reducer > can run cell and receive cell messages 5`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -907,7 +887,6 @@ exports[`cell reducer > can run cell and receive cell messages 6`] = `
   "output": null,
   "runElapsedTimeMs": null,
   "runStartTimestamp": 20,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "running",
@@ -949,7 +928,6 @@ exports[`cell reducer > can run cell and receive cell messages 7`] = `
   },
   "runElapsedTimeMs": 13000,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -992,7 +970,6 @@ import numpy",
   },
   "runElapsedTimeMs": 13000,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -1034,7 +1011,6 @@ exports[`cell reducer > can run cell and receive cell messages 9`] = `
   },
   "runElapsedTimeMs": 13000,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -1077,7 +1053,6 @@ import numpy",
   },
   "runElapsedTimeMs": 13000,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -1127,7 +1102,6 @@ import numpy",
   },
   "runElapsedTimeMs": 11000,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -1175,7 +1149,6 @@ import numpy",
   },
   "runElapsedTimeMs": 11000,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",
@@ -1303,7 +1276,6 @@ exports[`cell reducer > errors reset status to idle 4`] = `
   },
   "runElapsedTimeMs": null,
   "runStartTimestamp": null,
-  "serialization": undefined,
   "serializedEditorState": null,
   "staleInputs": false,
   "status": "idle",

package/src/core/cells/__tests__/cell.test.ts

@@ -1,9 +1,10 @@
 /* Copyright 2026 Marimo. All rights reserved. */
 import { describe, expect, it } from "vitest";
-import type { OutputMessage } from "@/core/kernel/messages";
+import type { CellMessage, OutputMessage } from "@/core/kernel/messages";
 import type { RuntimeState } from "@/core/network/types";
 import type { Seconds } from "@/utils/time";
-import { outputIsLoading, outputIsStale } from "../cell";
+import { outputIsLoading, outputIsStale, transitionCell } from "../cell";
+import { createCellRuntimeState } from "../types";
 
 const STATUSES: RuntimeState[] = [
   "queued",
@@ -191,3 +192,29 @@ describe("outputIsStale", () => {
     expect(outputIsStale(cell, edited)).toBe(false);
   });
 });
+
+describe("transitionCell serialization", () => {
+  function cellMessage(serialization?: string | null): CellMessage {
+    return {
+      cell_id: "cell-1",
+      ...(serialization === undefined ? {} : { serialization }),
+    } as CellMessage;
+  }
+
+  it("leaves the hint unchanged when serialization is omitted", () => {
+    const cell = createCellRuntimeState({ serialization: "Valid" });
+    expect(transitionCell(cell, cellMessage()).serialization).toBe("Valid");
+  });
+
+  it("clears the hint when serialization is null", () => {
+    const cell = createCellRuntimeState({ serialization: "Valid" });
+    expect(transitionCell(cell, cellMessage(null)).serialization).toBeNull();
+  });
+
+  it("sets the hint when serialization is a string", () => {
+    const cell = createCellRuntimeState();
+    expect(transitionCell(cell, cellMessage("Valid")).serialization).toBe(
+      "Valid",
+    );
+  });
+});

package/src/core/cells/cell.ts

@@ -73,7 +73,11 @@ export function transitionCell(
   nextCell.output = message.output ?? nextCell.output;
   nextCell.staleInputs = message.stale_inputs ?? nextCell.staleInputs;
   nextCell.status = message.status ?? nextCell.status;
-  nextCell.serialization = message.serialization;
+  // Tri-state partial update: an omitted field (undefined) leaves the hint
+  // unchanged; null explicitly clears it; a string sets it.
+  if (message.serialization !== undefined) {
+    nextCell.serialization = message.serialization;
+  }
 
   let didInterruptFromThisMessage = false;
 

package/src/core/codemirror/go-to-definition/commands.ts

@@ -28,10 +28,11 @@ interface VariableDeclaration {
 }
 
 function goToPosition(view: EditorView, from: number): void {
-  view.focus();
-  // Wait for the next frame, otherwise codemirror will
-  // add a cursor from a pointer click.
+  // Focus on the next frame: a synchronous focus is a no-op while a Radix
+  // context menu still owns focus, and codemirror would otherwise add a
+  // cursor from the pointer click.
   requestAnimationFrame(() => {
+    view.focus();
     view.dispatch({
       selection: {
         anchor: from,