@marimo-team/frontend 0.23.10 → 0.23.11-dev10

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-fNBoXCyz.js"></script>
+    <script type="module" crossorigin src="./assets/index-CuhY66ZR.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-DOA0Gew8.js">
+    <link rel="modulepreload" crossorigin href="./assets/cells-D-v2lBet.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-GI6gu7_S.js">
@@ -147,38 +147,38 @@
     <link rel="modulepreload" crossorigin href="./assets/usePress-DQ_tAz5W.js">
     <link rel="modulepreload" crossorigin href="./assets/input-DA8xlX70.js">
     <link rel="modulepreload" crossorigin href="./assets/ImperativeModal-BW1dmntO.js">
-    <link rel="modulepreload" crossorigin href="./assets/cell-link-Bj4-yOIx.js">
+    <link rel="modulepreload" crossorigin href="./assets/cell-link-DY95GSb7.js">
     <link rel="modulepreload" crossorigin href="./assets/multi-map-C4OANz8X.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-CZ4XbE-c.js">
     <link rel="modulepreload" crossorigin href="./assets/links-D1JoyKTt.js">
-    <link rel="modulepreload" crossorigin href="./assets/useRunCells-cSZpNqnR.js">
+    <link rel="modulepreload" crossorigin href="./assets/useRunCells-BClkx9Pq.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-BuRdHNPA.js">
-    <link rel="modulepreload" crossorigin href="./assets/RenderHTML-BQ1PO4Wd.js">
-    <link rel="modulepreload" crossorigin href="./assets/datasource-CR6RRpTi.js">
-    <link rel="modulepreload" crossorigin href="./assets/state-BWM3qRkR.js">
+    <link rel="modulepreload" crossorigin href="./assets/RenderHTML-DAt48X-F.js">
+    <link rel="modulepreload" crossorigin href="./assets/datasource-BQGI7c_u.js">
+    <link rel="modulepreload" crossorigin href="./assets/state-BSm0OM93.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-BH6hs0Ir.js">
+    <link rel="modulepreload" crossorigin href="./assets/MarimoErrorOutput-DzoKyXWR.js">
     <link rel="modulepreload" crossorigin href="./assets/eye-off-Cm6DdkeR.js">
     <link rel="modulepreload" crossorigin href="./assets/refresh-cw-DHwG4Mac.js">
-    <link rel="modulepreload" crossorigin href="./assets/tree-actions-1KQDSu1H.js">
+    <link rel="modulepreload" crossorigin href="./assets/tree-actions-BvrKFU0g.js">
     <link rel="modulepreload" crossorigin href="./assets/spinner-Bhir8k53.js">
-    <link rel="modulepreload" crossorigin href="./assets/html-to-image-CyAtzePO.js">
-    <link rel="modulepreload" crossorigin href="./assets/focus-Ldqh99xE.js">
+    <link rel="modulepreload" crossorigin href="./assets/html-to-image-IPNqWX7V.js">
+    <link rel="modulepreload" crossorigin href="./assets/focus-BfFAxl9X.js">
     <link rel="modulepreload" crossorigin href="./assets/useAsyncData-bgszE9F0.js">
     <link rel="modulepreload" crossorigin href="./assets/LazyAnyLanguageCodeMirror-C7ALwTI_.js">
     <link rel="modulepreload" crossorigin href="./assets/micromark-factory-space-bqhKsQDn.js">
     <link rel="modulepreload" crossorigin href="./assets/chunk-5FQGJX7Z-D91x7QHS.js">
-    <link rel="modulepreload" crossorigin href="./assets/markdown-renderer-C9Ujvj0b.js">
+    <link rel="modulepreload" crossorigin href="./assets/markdown-renderer-DWRPo52L.js">
     <link rel="modulepreload" crossorigin href="./assets/command-BmOGG6-2.js">
     <link rel="modulepreload" crossorigin href="./assets/field-DCdxnSKm.js">
     <link rel="modulepreload" crossorigin href="./assets/popover-CNCYAwbC.js">
     <link rel="modulepreload" crossorigin href="./assets/errors-D3F5g1xA.js">
-    <link rel="modulepreload" crossorigin href="./assets/download-DhxnAw14.js">
+    <link rel="modulepreload" crossorigin href="./assets/download-WGU5w_3m.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-D0xvrLog.js">
@@ -198,21 +198,21 @@
     <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-DRNPZOvX.js">
+    <link rel="modulepreload" crossorigin href="./assets/JsonOutput-uEJijGXp.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/column-preview-nu3Qo2OW.js">
+    <link rel="modulepreload" crossorigin href="./assets/column-preview-BzvFdLI5.js">
     <link rel="modulepreload" crossorigin href="./assets/icons-Ol38nIbL.js">
-    <link rel="modulepreload" crossorigin href="./assets/radio-group-D7rh6Zek.js">
-    <link rel="modulepreload" crossorigin href="./assets/floating-outline-BvHFWRoz.js">
+    <link rel="modulepreload" crossorigin href="./assets/radio-group-DV-lQvil.js">
+    <link rel="modulepreload" crossorigin href="./assets/floating-outline-D33Zu-Ad.js">
     <link rel="modulepreload" crossorigin href="./assets/objectWithoutPropertiesLoose-DfWeGRFv.js">
     <link rel="modulepreload" crossorigin href="./assets/esm-xaw4Vy8N.js">
-    <link rel="modulepreload" crossorigin href="./assets/readonly-python-code-CvPx4CU_.js">
+    <link rel="modulepreload" crossorigin href="./assets/readonly-python-code-B0glFTKW.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-D2f3nfbq.js">
+    <link rel="modulepreload" crossorigin href="./assets/file-icons-CF-YT4hq.js">
     <link rel="modulepreload" crossorigin href="./assets/switch-Ac7CgLgh.js">
     <link rel="modulepreload" crossorigin href="./assets/events-CPoJAfgx.js">
     <link rel="modulepreload" crossorigin href="./assets/globals-BivClSmn.js">
@@ -221,11 +221,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-C8yHPSLN.js">
+    <link rel="modulepreload" crossorigin href="./assets/state-CbkvENzk.js">
     <link rel="modulepreload" crossorigin href="./assets/label-C2_YQRio.js">
-    <link rel="modulepreload" crossorigin href="./assets/textarea-bBuxbFm7.js">
+    <link rel="modulepreload" crossorigin href="./assets/textarea-CBbixTxi.js">
     <link rel="modulepreload" crossorigin href="./assets/refresh-ccw-C-n2VFP5.js">
-    <link rel="modulepreload" crossorigin href="./assets/form-CxBAInfg.js">
+    <link rel="modulepreload" crossorigin href="./assets/form-CVBFx2z3.js">
     <link rel="modulepreload" crossorigin href="./assets/renderShortcut--ku7VECI.js">
     <link rel="modulepreload" crossorigin href="./assets/useBoolean-CE6PfxGj.js">
     <link rel="modulepreload" crossorigin href="./assets/useDeepCompareMemoize-CVmg-GkP.js">
@@ -246,7 +246,7 @@
     <link rel="stylesheet" crossorigin href="./assets/cells-pu5xzE_v.css">
     <link rel="stylesheet" crossorigin href="./assets/markdown-renderer-DdDKmWlR.css">
     <link rel="stylesheet" crossorigin href="./assets/JsonOutput-B7vuddcd.css">
-    <link rel="stylesheet" crossorigin href="./assets/index-BAYF7dcV.css">
+    <link rel="stylesheet" crossorigin href="./assets/index-BNWrEIlp.css">
   </head>
   <body>
     <div id="root"></div>

package/package.json

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

package/src/components/datasources/__tests__/column-preview.test.tsx

@@ -0,0 +1,97 @@
+/* Copyright 2026 Marimo. All rights reserved. */
+
+import { render } from "@testing-library/react";
+import { Provider } from "jotai";
+import type React from "react";
+import { beforeEach, describe, expect, it, vi } from "vitest";
+import { MockRequestClient } from "@/__mocks__/requests";
+import type { SQLTableContext } from "@/core/datasets/data-source-connections";
+import type { DataTable, DataTableColumn } from "@/core/kernel/messages";
+import { requestClientAtom } from "@/core/network/requests";
+import { store } from "@/core/state/jotai";
+import { DatasetColumnPreview } from "../column-preview";
+
+const wrapper = ({ children }: { children: React.ReactNode }) => (
+  <Provider store={store}>{children}</Provider>
+);
+
+const table: DataTable = {
+  name: "users",
+  columns: [],
+  source: "my_engine",
+  // Not "connection"/"catalog", so a preview is requested on mount.
+  source_type: "duckdb",
+  type: "table",
+  engine: "my_engine" as DataTable["engine"],
+  indexes: null,
+  num_columns: null,
+  num_rows: null,
+  variable_name: null,
+  primary_keys: null,
+};
+
+const column = { name: "email" } as DataTableColumn;
+
+function renderPreview(sqlTableContext?: SQLTableContext) {
+  const client = MockRequestClient.create();
+  store.set(requestClientAtom, client);
+  render(
+    <DatasetColumnPreview
+      table={table}
+      column={column}
+      preview={undefined}
+      onAddColumnChart={vi.fn()}
+      sqlTableContext={sqlTableContext}
+    />,
+    { wrapper },
+  );
+  return client;
+}
+
+const ctx = (
+  overrides: Partial<SQLTableContext> & { database: string; schema: string },
+): SQLTableContext => ({
+  engine: "my_engine",
+  dialect: "duckdb",
+  ...overrides,
+});
+
+describe("DatasetColumnPreview fullyQualifiedTableName", () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it("uses just the table name without a SQL context", () => {
+    const client = renderPreview(undefined);
+    expect(client.previewDatasetColumn).toHaveBeenCalledWith(
+      expect.objectContaining({ fullyQualifiedTableName: "users" }),
+    );
+  });
+
+  it("qualifies with database and schema for flat engines", () => {
+    const client = renderPreview(ctx({ database: "db", schema: "public" }));
+    expect(client.previewDatasetColumn).toHaveBeenCalledWith(
+      expect.objectContaining({ fullyQualifiedTableName: "db.public.users" }),
+    );
+  });
+
+  it("does not emit a double dot for schemaless tables", () => {
+    // Regression: previously produced "db..users".
+    const client = renderPreview(ctx({ database: "db", schema: "" }));
+    expect(client.previewDatasetColumn).toHaveBeenCalledWith(
+      expect.objectContaining({ fullyQualifiedTableName: "db.users" }),
+    );
+  });
+
+  it("includes the full schema path for nested namespaces", () => {
+    // Regression: previously dropped schemaPath and emitted "top.deep.users".
+    const client = renderPreview(
+      ctx({ database: "top", schema: "deep", schemaPath: ["nested", "deep"] }),
+    );
+    expect(client.previewDatasetColumn).toHaveBeenCalledWith(
+      expect.objectContaining({
+        fullyQualifiedTableName: "top.nested.deep.users",
+      }),
+    );
+  });
+});

package/src/components/datasources/__tests__/filter-empty.test.ts

@@ -28,11 +28,15 @@ function makeSchema(opts: {
   name: string;
   tables: DataTable[];
   tables_resolved?: boolean;
+  child_schemas?: DatabaseSchema[];
+  child_schemas_resolved?: boolean;
 }): DatabaseSchema {
   return {
     name: opts.name,
     tables: opts.tables,
     tables_resolved: opts.tables_resolved ?? true,
+    child_schemas: opts.child_schemas ?? [],
+    child_schemas_resolved: opts.child_schemas_resolved ?? true,
   };
 }
 
@@ -180,4 +184,81 @@ describe("filterEmptyDatabases", () => {
 
     expect(databases).toEqual(snapshot);
   });
+
+  it("keeps a namespace that has only child namespaces (no own tables)", () => {
+    const databases = [
+      makeDatabase("iceberg", [
+        makeSchema({
+          name: "top",
+          tables: [],
+          child_schemas: [
+            makeSchema({ name: "nested", tables: [makeTable("t1")] }),
+          ],
+        }),
+      ]),
+    ];
+
+    expect(filterEmptyDatabases(databases)).toBe(databases);
+  });
+
+  it("preserves a namespace whose child schemas are deferred", () => {
+    const databases = [
+      makeDatabase("iceberg", [
+        makeSchema({
+          name: "top",
+          tables: [],
+          child_schemas: [],
+          child_schemas_resolved: false,
+        }),
+      ]),
+    ];
+
+    expect(filterEmptyDatabases(databases)).toBe(databases);
+  });
+
+  it("hides a nested namespace that is resolved-empty", () => {
+    const databases = [
+      makeDatabase("iceberg", [
+        makeSchema({
+          name: "top",
+          tables: [makeTable("t1")],
+          child_schemas: [
+            makeSchema({ name: "empty_child", tables: [] }),
+            makeSchema({ name: "full_child", tables: [makeTable("t2")] }),
+          ],
+        }),
+      ]),
+    ];
+
+    expect(filterEmptyDatabases(databases)).toEqual([
+      makeDatabase("iceberg", [
+        makeSchema({
+          name: "top",
+          tables: [makeTable("t1")],
+          child_schemas: [
+            makeSchema({ name: "full_child", tables: [makeTable("t2")] }),
+          ],
+        }),
+      ]),
+    ]);
+  });
+
+  it("hides a parent namespace when all its descendants are empty", () => {
+    const databases = [
+      makeDatabase("iceberg", [
+        makeSchema({
+          name: "top",
+          tables: [],
+          child_schemas: [makeSchema({ name: "empty_child", tables: [] })],
+        }),
+        makeSchema({ name: "other", tables: [makeTable("t1")] }),
+      ]),
+    ];
+
+    expect(filterEmptyDatabases(databases)).toEqual([
+      makeDatabase("iceberg", [
+        makeSchema({ name: "other", tables: [makeTable("t1")] }),
+      ]),
+    ]);
+  });
 });

package/src/components/datasources/__tests__/utils.test.ts

@@ -4,7 +4,7 @@ import { describe, expect, it } from "vitest";
 import type { SQLTableContext } from "@/core/datasets/data-source-connections";
 import { DUCKDB_ENGINE } from "@/core/datasets/engines";
 import type { DataTable, DataTableColumn } from "@/core/kernel/messages";
-import { sqlCode } from "../utils";
+import { sqlCode, tableUniqueId } from "../utils";
 
 describe("sqlCode", () => {
   const mockTable: DataTable = {
@@ -458,3 +458,64 @@ describe("sqlCode", () => {
     });
   });
 });
+
+describe("tableUniqueId", () => {
+  const ctx = (
+    overrides: Partial<SQLTableContext> & { database: string; schema: string },
+  ): SQLTableContext => ({
+    engine: "e",
+    dialect: "duckdb",
+    ...overrides,
+  });
+
+  it("returns just the table name without a context", () => {
+    expect(tableUniqueId(undefined, "t")).toBe("t");
+  });
+
+  it("uses database + schema for flat engines", () => {
+    expect(tableUniqueId(ctx({ database: "db", schema: "public" }), "t")).toBe(
+      "db.public.t",
+    );
+  });
+
+  it("does not duplicate the leaf schema for nested namespaces", () => {
+    // Regression: previously produced "top.nested.nested.t".
+    expect(
+      tableUniqueId(
+        ctx({ database: "top", schema: "nested", schemaPath: ["nested"] }),
+        "t",
+      ),
+    ).toBe("top.nested.t");
+  });
+
+  it("includes the full schema path for deeply nested namespaces", () => {
+    expect(
+      tableUniqueId(
+        ctx({
+          database: "top",
+          schema: "deep",
+          schemaPath: ["nested", "deep"],
+        }),
+        "t",
+      ),
+    ).toBe("top.nested.deep.t");
+  });
+
+  it("falls back to the flat schema when schemaPath is empty", () => {
+    expect(
+      tableUniqueId(
+        ctx({ database: "db", schema: "public", schemaPath: [] }),
+        "t",
+      ),
+    ).toBe("db.public.t");
+  });
+
+  it("does not emit a double dot for schemaless tables", () => {
+    expect(
+      tableUniqueId(ctx({ database: "db", schema: "", schemaPath: [] }), "t"),
+    ).toBe("db.t");
+    expect(tableUniqueId(ctx({ database: "db", schema: "" }), "t")).toBe(
+      "db.t",
+    );
+  });
+});

package/src/components/datasources/column-preview.tsx

@@ -29,7 +29,7 @@ import { Button } from "../ui/button";
 import { Tooltip } from "../ui/tooltip";
 import { ColumnPreviewContainer } from "./components";
 import { InstallPackageButton } from "./install-package-button";
-import { convertStatsName, sqlCode } from "./utils";
+import { convertStatsName, sqlCode, tableUniqueId } from "./utils";
 
 export const DatasetColumnPreview: React.FC<{
   table: DataTable;
@@ -48,9 +48,7 @@ export const DatasetColumnPreview: React.FC<{
       tableName: table.name,
       columnName: column.name,
       sourceType: table.source_type,
-      fullyQualifiedTableName: sqlTableContext
-        ? `${sqlTableContext.database}.${sqlTableContext.schema}.${table.name}`
-        : table.name,
+      fullyQualifiedTableName: tableUniqueId(sqlTableContext, table.name),
     });
   };
 

package/src/components/datasources/components.tsx

@@ -14,25 +14,31 @@ export const RotatingChevron: React.FC<{ isExpanded: boolean }> = ({
 export const DatasourceLabel: React.FC<{
   children: React.ReactNode;
   className?: string;
-}> = ({ children, className }) => {
+  style?: CSSProperties;
+}> = ({ children, className, style }) => {
   return (
     <div
       className={cn(
         "flex gap-1.5 items-center font-bold py-1.5 text-muted-foreground bg-(--slate-2) text-sm",
         className,
       )}
+      style={style}
     >
       {children}
     </div>
   );
 };
 
-export const EmptyState: React.FC<{ content: string; className?: string }> = ({
-  content,
-  className,
-}) => {
+export const EmptyState: React.FC<{
+  content: string;
+  className?: string;
+  style?: CSSProperties;
+}> = ({ content, className, style }) => {
   return (
-    <div className={cn("text-sm text-muted-foreground py-1", className)}>
+    <div
+      className={cn("text-sm text-muted-foreground py-1", className)}
+      style={style}
+    >
       {content}
     </div>
   );
@@ -61,13 +67,15 @@ export const ErrorState: React.FC<{
 export const LoadingState: React.FC<{
   message: string;
   className?: string;
-}> = ({ message, className }) => {
+  style?: CSSProperties;
+}> = ({ message, className, style }) => {
   return (
     <div
       className={cn(
         "text-sm bg-blue-50 dark:bg-(--accent) text-blue-500 dark:text-blue-50 flex items-center gap-2 p-2 h-8",
         className,
       )}
+      style={style}
     >
       <LoaderCircle className="h-4 w-4 animate-spin" />
       {message}