@marimo-team/islands 0.22.1-dev9 → 0.22.2

package/src/core/codemirror/reactive-references/__tests__/analyzer.test.ts

@@ -45,9 +45,8 @@ def foo(x):
   });
 
   test("should handle lambda parameters", () => {
-    expect(
-      runHighlight(["a", "b"], "result = lambda a: a + b"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["a", "b"], "result = lambda a: a + b"))
+      .toMatchInlineSnapshot(`
       "
       result = lambda a: a + b
                              ^
@@ -56,9 +55,8 @@ def foo(x):
   });
 
   test("should handle comprehension variables", () => {
-    expect(
-      runHighlight(["a", "data"], "result = [a for a in data]"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["a", "data"], "result = [a for a in data]"))
+      .toMatchInlineSnapshot(`
       "
       result = [a for a in data]
                            ^^^^
@@ -146,9 +144,8 @@ def factorial(n):
   });
 
   test("function param vs global", () => {
-    expect(
-      runHighlight(["a", "b"], "def foo(a): return a + b"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["a", "b"], "def foo(a): return a + b"))
+      .toMatchInlineSnapshot(`
       "
       def foo(a): return a + b
                              ^
@@ -157,9 +154,8 @@ def factorial(n):
   });
 
   test("lambda param vs global", () => {
-    expect(
-      runHighlight(["x", "b"], "func = lambda x: x + b"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["x", "b"], "func = lambda x: x + b"))
+      .toMatchInlineSnapshot(`
       "
       func = lambda x: x + b
                            ^
@@ -168,9 +164,8 @@ def factorial(n):
   });
 
   test("lambda with multiple params", () => {
-    expect(
-      runHighlight(["x", "y", "z"], "f = lambda x, y: x + y + z"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["x", "y", "z"], "f = lambda x, y: x + y + z"))
+      .toMatchInlineSnapshot(`
       "
       f = lambda x, y: x + y + z
                                ^
@@ -187,9 +182,8 @@ def factorial(n):
   });
 
   test("nested comprehension", () => {
-    expect(
-      runHighlight(["a", "b"], "[(a + b) for a, b in [(1,2), (3,4)]]"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["a", "b"], "[(a + b) for a, b in [(1,2), (3,4)]]"))
+      .toMatchInlineSnapshot(`
       "
       [(a + b) for a, b in [(1,2), (3,4)]]
       "
@@ -225,9 +219,8 @@ def factorial(n):
   });
 
   test("class body using globals", () => {
-    expect(
-      runHighlight(["a", "b"], "class MyClass:\n  value = a + b"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["a", "b"], "class MyClass:\n  value = a + b"))
+      .toMatchInlineSnapshot(`
       "
       class MyClass:
         value = a + b
@@ -237,9 +230,8 @@ def factorial(n):
   });
 
   test("decorator using global", () => {
-    expect(
-      runHighlight(["logger"], "@logger\ndef decorated(): pass"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["logger"], "@logger\ndef decorated(): pass"))
+      .toMatchInlineSnapshot(`
       "
       @logger
        ^^^^^^
@@ -317,9 +309,8 @@ ${" ".repeat(8)}
   });
 
   test("multiple assignment", () => {
-    expect(
-      runHighlight(["x", "y", "z", "a"], "x = y = z + a"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["x", "y", "z", "a"], "x = y = z + a"))
+      .toMatchInlineSnapshot(`
       "
       x = y = z + a
               ^   ^
@@ -353,9 +344,8 @@ ${" ".repeat(8)}
   });
 
   test("global in return", () => {
-    expect(
-      runHighlight(["config"], "def get_config(): return config"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["config"], "def get_config(): return config"))
+      .toMatchInlineSnapshot(`
       "
       def get_config(): return config
                                ^^^^^^
@@ -410,9 +400,8 @@ class Configurable:
   });
 
   test("comprehension shadows global 2", () => {
-    expect(
-      runHighlight(["i"], "squares = [i**2 for i in range(10)]"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["i"], "squares = [i**2 for i in range(10)]"))
+      .toMatchInlineSnapshot(`
       "
       squares = [i**2 for i in range(10)]
       "
@@ -420,9 +409,8 @@ class Configurable:
   });
 
   test("comprehension with global in condition", () => {
-    expect(
-      runHighlight(["x", "z"], "filtered = [x for x in [] if x > z]"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["x", "z"], "filtered = [x for x in [] if x > z]"))
+      .toMatchInlineSnapshot(`
       "
       filtered = [x for x in [] if x > z]
                                        ^
@@ -459,9 +447,8 @@ def make_adder(x):
   });
 
   test("rebinding in list comprehension", () => {
-    expect(
-      runHighlight(["x"], "rebinding = [x for x in range(5)]"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["x"], "rebinding = [x for x in range(5)]"))
+      .toMatchInlineSnapshot(`
       "
       rebinding = [x for x in range(5)]
       "
@@ -644,9 +631,8 @@ def run(polars):
   });
 
   test("mixed comprehension and outer globals", () => {
-    expect(
-      runHighlight(["y", "z"], "values = [y + z for y in range(5)]"),
-    ).toMatchInlineSnapshot(`
+    expect(runHighlight(["y", "z"], "values = [y + z for y in range(5)]"))
+      .toMatchInlineSnapshot(`
       "
       values = [y + z for y in range(5)]
                     ^

package/src/core/codemirror/rtc/loro/awareness.ts

@@ -185,7 +185,7 @@ export const createSelectionLayer = (): Extension =>
         .flatMap(([, state]) => {
           const selectionRange = EditorSelection.range(
             state.cursor.anchor,
-            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+            // oxlint-disable-next-line typescript/no-non-null-assertion
             state.cursor.head!,
           );
           const markers = RectangleMarker.forRange(

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

@@ -70,9 +70,7 @@ test("default UserConfig - empty", () => {
         "reference_highlighting": true,
         "theme": "light",
       },
-      "experimental": {
-        "storage_inspector": true,
-      },
+      "experimental": {},
       "formatting": {
         "line_length": 79,
       },
@@ -142,9 +140,7 @@ test("default UserConfig - one level", () => {
         "reference_highlighting": true,
         "theme": "light",
       },
-      "experimental": {
-        "storage_inspector": true,
-      },
+      "experimental": {},
       "formatting": {
         "line_length": 79,
       },

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

@@ -186,7 +186,6 @@ export const UserConfigSchema = z
       .looseObject({
         markdown: z.boolean().optional(),
         rtc: z.boolean().optional(),
-        storage_inspector: z.boolean().prefault(true),
         // Add new experimental features here
       })
       // Pass through so that we don't remove any extra keys that the user has added.

package/src/core/config/feature-flag.tsx

@@ -4,14 +4,13 @@ import { repl } from "@/utils/repl";
 import { getRequestClient } from "../network/requests";
 import { getResolvedMarimoConfig } from "./config";
 
-// eslint-disable-next-line @typescript-eslint/no-empty-interface
+// oxlint-disable-next-line typescript/no-empty-object-type
 export interface ExperimentalFeatures {
   markdown: boolean; // Used in playground (community cloud)
   wasm_layouts: boolean; // Used in playground (community cloud)
   rtc_v2: boolean;
   cache_panel: boolean;
   external_agents: boolean;
-  storage_inspector: boolean;
   // Add new feature flags here
 }
 
@@ -21,7 +20,6 @@ const defaultValues: ExperimentalFeatures = {
   rtc_v2: false,
   cache_panel: false,
   external_agents: import.meta.env.DEV,
-  storage_inspector: true,
 };
 
 export function getFeatureFlag<T extends keyof ExperimentalFeatures>(

package/src/core/datasets/data-source-connections.ts

@@ -32,8 +32,10 @@ const initialConnections: ConnectionsMap = new Map([
 ]);
 
 // Extend the backend type but override the name property with the strongly typed ConnectionName
-export interface DataSourceConnection
-  extends Omit<DataSourceConnectionType, "name"> {
+export interface DataSourceConnection extends Omit<
+  DataSourceConnectionType,
+  "name"
+> {
   name: ConnectionName;
 }
 

package/src/core/datasets/state.ts

@@ -66,7 +66,7 @@ const {
       return true;
     });
 
-    const sortedTables = dedupedTables.sort((a, b) => {
+    const sortedTables = dedupedTables.toSorted((a, b) => {
       return a.name.localeCompare(b.name);
     });
 

package/src/core/dom/__tests__/htmlUtils.test.ts

@@ -6,20 +6,14 @@ import { UIElementRegistry } from "../uiregistry";
 const registry = UIElementRegistry.INSTANCE;
 
 describe("htmlUtils", () => {
-  it.each([
-    false,
-    { a: 1 },
-    true,
-    0,
-    1,
-    [{ a: 1 }, { b: 2 }],
-    "hello",
-    "",
-  ])("can parse element.dataset.initialValue", (initialValue) => {
-    const div = document.createElement("div");
-    div.dataset.initialValue = JSON.stringify(initialValue);
-    expect(parseInitialValue(div, registry)).toEqual(initialValue);
-  });
+  it.each([false, { a: 1 }, true, 0, 1, [{ a: 1 }, { b: 2 }], "hello", ""])(
+    "can parse element.dataset.initialValue",
+    (initialValue) => {
+      const div = document.createElement("div");
+      div.dataset.initialValue = JSON.stringify(initialValue);
+      expect(parseInitialValue(div, registry)).toEqual(initialValue);
+    },
+  );
 
   it("can parse an element with no initialValue", () => {
     const div = document.createElement("div");

package/src/core/dom/__tests__/outline.test.ts

@@ -439,9 +439,8 @@ const OUTLINE_2: Outline = {
 };
 
 it("mergeOutlines", () => {
-  expect(
-    mergeOutlines([OUTLINE_1, null, OUTLINE_2, null]),
-  ).toMatchInlineSnapshot(`
+  expect(mergeOutlines([OUTLINE_1, null, OUTLINE_2, null]))
+    .toMatchInlineSnapshot(`
     {
       "items": [
         {

package/src/core/dom/outline.ts

@@ -39,7 +39,7 @@ function getOutline(html: string): Outline | null {
   const document = parser.parseFromString(html, "text/html");
 
   const headings = document.querySelectorAll(HEADER_TAGS);
-  // eslint-disable-next-line unicorn/prefer-spread
+  // oxlint-disable-next-line unicorn/prefer-spread
   for (const heading of Array.from(headings)) {
     const name = heading.textContent;
     // Check if the heading is within any of the excluded tags

package/src/core/dom/uiregistry.ts

@@ -10,6 +10,23 @@ import {
 } from "./events";
 import { parseInitialValue } from "./htmlUtils";
 
+/**
+ * Kernel-initiated UI value update, sent via the existing
+ * `send-ui-element-message` channel when `set_ui_value` (code_mode)
+ * changes a widget's value so the frontend can reflect the new state.
+ */
+interface UIValueUpdateMessage {
+  type: "marimo-ui-value-update";
+  value: ValueType;
+}
+
+function isUIValueUpdateMessage(msg: unknown): msg is UIValueUpdateMessage {
+  if (typeof msg !== "object" || msg === null) {
+    return false;
+  }
+  return "type" in msg && msg.type === "marimo-ui-value-update";
+}
+
 interface UIElementEntry {
   objectId: string;
   value: ValueType;
@@ -141,21 +158,38 @@ export class UIElementRegistry {
     const entry = this.entries.get(objectId);
     if (entry === undefined) {
       Logger.warn("UIElementRegistry missing entry", objectId);
-    } else {
+      return;
+    }
+
+    // Kernel-initiated value update — push into DOM elements without
+    // dispatching MarimoValueReadyEvent to avoid a round-trip.
+    if (isUIValueUpdateMessage(message)) {
+      entry.value = message.value;
       entry.elements.forEach((element) => {
         element.dispatchEvent(
-          MarimoIncomingMessageEvent.create({
-            bubbles: false, // only the intended target gets the message
+          MarimoValueUpdateEvent.create({
+            bubbles: false,
             composed: true,
-            detail: {
-              objectId: objectId,
-              message: message,
-              buffers: buffers,
-            },
+            detail: { value: message.value, element: element },
           }),
         );
       });
+      return;
     }
+
+    entry.elements.forEach((element) => {
+      element.dispatchEvent(
+        MarimoIncomingMessageEvent.create({
+          bubbles: false, // only the intended target gets the message
+          composed: true,
+          detail: {
+            objectId: objectId,
+            message: message,
+            buffers: buffers,
+          },
+        }),
+      );
+    });
   }
 
   /**

package/src/core/edit-app.tsx

@@ -1,7 +1,10 @@
 /* Copyright 2026 Marimo. All rights reserved. */
 
 import { usePrevious } from "@dnd-kit/utilities";
-import { TooltipProvider } from "@radix-ui/react-tooltip";
+import { Tooltip } from "radix-ui";
+
+const TooltipProvider = Tooltip.Provider;
+
 import { useAtomValue, useSetAtom } from "jotai";
 import { useEffect } from "react";
 import { NotStartedConnectionAlert } from "@/components/editor/alerts/connecting-alert";

package/src/core/export/__tests__/hooks.test.ts

@@ -55,7 +55,7 @@ describe("useEnrichCellOutputs", () => {
 
   // Helper to set the mocked atom (cast to any to work around type mismatch)
   const setCellsRuntime = (value: Record<CellId, CellRuntimeState>) => {
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    // oxlint-disable-next-line typescript/no-explicit-any
     store.set(cellsRuntimeAtom as any, value);
   };
 

package/src/core/hotkeys/__tests__/hotkeys.test.ts

@@ -15,7 +15,7 @@ function createHotkeys(
   keys: Partial<Record<HotkeyAction, Hotkey>>,
 ): Record<HotkeyAction, Hotkey> {
   return new Proxy(keys as Record<HotkeyAction, Hotkey>, {
-    // biome-ignore lint: ok to have three arguments here (It's a web API)
+    // oxlint-ignore-next-line -- ok to have three arguments here (It's a web API)
     get(target, p, receiver) {
       const key = Reflect.get(target, p, receiver);
       if (key === "undefined") {

package/src/core/hotkeys/shortcuts.ts

@@ -73,7 +73,7 @@ function areKeysPressed(keys: string[], e: IKeyboardEvent): boolean {
         // - US keyboards: "&"
         // - Some layouts: "7"
         // Using e.code (physical key) instead of e.key (produced character)
-        // eslint-disable-next-line unicorn/prefer-ternary
+        // oxlint-disable-next-line unicorn/prefer-ternary
         if (/^\d$/.test(key) && e.shiftKey) {
           satisfied &&= e.code === `Digit${key}`;
         } else {

package/src/core/islands/__tests__/bridge.test.ts

@@ -84,7 +84,7 @@ describe("IslandsPyodideBridge", () => {
   beforeEach(() => {
     vi.clearAllMocks();
     // Reset the singleton by clearing the window property
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    // oxlint-disable-next-line typescript/no-explicit-any
     delete (window as any)._marimo_private_IslandsPyodideBridge;
     // Access the singleton - creates a fresh instance
     bridge = IslandsPyodideBridge.INSTANCE;
@@ -92,7 +92,7 @@ describe("IslandsPyodideBridge", () => {
 
   afterEach(() => {
     // Clean up singleton
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    // oxlint-disable-next-line typescript/no-explicit-any
     delete (window as any)._marimo_private_IslandsPyodideBridge;
   });
 

package/src/core/islands/__tests__/parse.test.ts

@@ -78,11 +78,12 @@ describe("parseIslandCode", () => {
 
   codes = [...codes, ...codes.map(encodeURIComponent)];
 
-  it.each(
-    codes,
-  )("should return the code without leading or trailing whitespace", (code) => {
-    const result = parseIslandCode(code);
-    const expected = 'def __():\n    print("Hello, World!")\n    return';
-    expect(result).toBe(expected);
-  });
+  it.each(codes)(
+    "should return the code without leading or trailing whitespace",
+    (code) => {
+      const result = parseIslandCode(code);
+      const expected = 'def __():\n    print("Hello, World!")\n    return';
+      expect(result).toBe(expected);
+    },
+  );
 });

package/src/core/islands/bridge.ts

@@ -1,5 +1,5 @@
 /* Copyright 2026 Marimo. All rights reserved. */
-/* eslint-disable @typescript-eslint/no-explicit-any */
+/* oxlint-disable typescript/no-explicit-any */
 
 import { getWorkerRPC } from "@/core/wasm/rpc";
 import { Deferred } from "@/utils/Deferred";
@@ -47,7 +47,7 @@ export class IslandsPyodideBridge implements RunRequests, EditRequests {
     const blob = new Blob([js], { type: "application/javascript" });
     const objURL = URL.createObjectURL(blob);
     const worker = new Worker(
-      // eslint-disable-next-line unicorn/relative-url-style
+      // oxlint-disable-next-line unicorn/relative-url-style
       objURL,
       {
         type: "module",

package/src/core/islands/components/output-wrapper.tsx

@@ -65,7 +65,7 @@ export const MarimoOutputWrapper: React.FC<Props> = ({
   // Lint still wants use to have the same event listeners per instance (which
   // makes sense), so noop is used.
   const maybeNoop = (fn: (e: KeyboardEvent) => void) =>
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
+    // oxlint-disable-next-line typescript/no-empty-function
     alwaysShowRun ? () => {} : fn;
 
   useEventListener(

package/src/core/islands/parse.ts

@@ -72,7 +72,7 @@ export function parseMarimoIslandApps(): MarimoIslandApp[] {
     if (!apps.has(id)) {
       apps.set(id, { id, cells: [] });
     }
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    // oxlint-disable-next-line typescript/no-non-null-assertion
     const app = apps.get(id)!;
     const idx = app.cells.length;
     app.cells.push({

package/src/core/lsp/__tests__/transport.test.ts

@@ -1,5 +1,5 @@
 /* Copyright 2026 Marimo. All rights reserved. */
-/* eslint-disable @typescript-eslint/no-explicit-any */
+/* oxlint-disable typescript/no-explicit-any */
 
 import { WebSocketTransport } from "@open-rpc/client-js";
 import { beforeEach, describe, expect, it, vi } from "vitest";

package/src/core/network/CachingRequestRegistry.ts

@@ -27,7 +27,7 @@ export class CachingRequestRegistry<REQ, RES> {
       if (typeof o !== "object" || o === null) {
         return String(o);
       }
-      return JSON.stringify(o, Object.keys(o).sort(), 2);
+      return JSON.stringify(o, Object.keys(o).toSorted(), 2);
     };
   }
 

package/src/core/network/DeferredRequestRegistry.ts

@@ -1,5 +1,5 @@
 /* Copyright 2026 Marimo. All rights reserved. */
-/* eslint-disable @typescript-eslint/no-redeclare */
+/* oxlint-disable typescript/no-redeclare */
 import type { components } from "@marimo-team/marimo-api";
 import { Deferred } from "@/utils/Deferred";
 import { generateUUID } from "@/utils/uuid";

package/src/core/network/__tests__/requests-network.test.ts

@@ -1,6 +1,6 @@
 /* Copyright 2026 Marimo. All rights reserved. */
 
-/* eslint-disable @typescript-eslint/no-explicit-any */
+/* oxlint-disable typescript/no-explicit-any */
 
 import { beforeEach, describe, expect, it, vi } from "vitest";
 import * as apiModule from "../api";

package/src/core/network/api.ts

@@ -108,7 +108,7 @@ export const API = {
     response: Response;
   }): Promise<T> => {
     if (response.error) {
-      // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
+      // oxlint-disable-next-line typescript/prefer-promise-reject-errors
       return Promise.reject(response.error);
     }
     return Promise.resolve(response.data as T);
@@ -118,7 +118,7 @@ export const API = {
     response: Response;
   }): Promise<null> => {
     if (response.error) {
-      // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
+      // oxlint-disable-next-line typescript/prefer-promise-reject-errors
       return Promise.reject(response.error);
     }
     return Promise.resolve(null);

package/src/core/network/requests-lazy.ts

@@ -138,7 +138,7 @@ export function createLazyRequests(
     await runtimeManager.init();
   });
 
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  // oxlint-disable-next-line typescript/no-explicit-any
   function wrapRequest<T extends (...args: any[]) => Promise<any>>(
     request: T,
     key: keyof AllRequests,

package/src/core/network/requests-toasting.tsx

@@ -1,5 +1,5 @@
 /* Copyright 2026 Marimo. All rights reserved. */
-/* eslint-disable @typescript-eslint/no-explicit-any */
+/* oxlint-disable typescript/no-explicit-any */
 
 import { useAtomValue } from "jotai";
 import { Spinner } from "@/components/icons/spinner";

package/src/core/runtime/runtime.ts

@@ -22,6 +22,7 @@ export class RuntimeManager {
     } catch (error) {
       throw new Error(
         `Invalid runtime URL: ${this.config.url}. ${error instanceof Error ? error.message : "Unknown error"}`,
+        { cause: error },
       );
     }
 

package/src/core/saving/__tests__/filename.test.ts

@@ -5,12 +5,13 @@ import { EDGE_CASE_FILENAMES } from "../../../__tests__/mocks";
 import { Paths } from "../../../utils/paths";
 
 describe("filename handling logic", () => {
-  it.each(
-    EDGE_CASE_FILENAMES,
-  )("should extract basename correctly for document title: %s", (filename) => {
-    const basename = Paths.basename(filename);
-    expect(basename).toBe(filename); // Since no path separator
-  });
+  it.each(EDGE_CASE_FILENAMES)(
+    "should extract basename correctly for document title: %s",
+    (filename) => {
+      const basename = Paths.basename(filename);
+      expect(basename).toBe(filename); // Since no path separator
+    },
+  );
 
   it("should handle full paths with unicode filenames", () => {
     EDGE_CASE_FILENAMES.forEach((filename) => {

package/src/core/static/__tests__/download-html.test.ts

@@ -44,23 +44,24 @@ describe("updateAssetUrl", () => {
 });
 
 describe("filename handling for downloads", () => {
-  it.each(
-    EDGE_CASE_FILENAMES,
-  )("should handle edge case filenames in download operations: %s", (filename) => {
-    // Test that basename extraction works correctly for downloads
-    const basename = Paths.basename(filename);
-    expect(basename).toBe(filename);
+  it.each(EDGE_CASE_FILENAMES)(
+    "should handle edge case filenames in download operations: %s",
+    (filename) => {
+      // Test that basename extraction works correctly for downloads
+      const basename = Paths.basename(filename);
+      expect(basename).toBe(filename);
 
-    // Test filename conversion for HTML downloads
-    const htmlFilename = Filenames.toHTML(filename);
-    expect(htmlFilename).toMatch(/\.html$/);
-    expect(htmlFilename).toContain(Filenames.withoutExtension(filename));
+      // Test filename conversion for HTML downloads
+      const htmlFilename = Filenames.toHTML(filename);
+      expect(htmlFilename).toMatch(/\.html$/);
+      expect(htmlFilename).toContain(Filenames.withoutExtension(filename));
 
-    // Ensure unicode and spaces are preserved in basename
-    const withoutExt = Filenames.withoutExtension(filename);
-    expect(withoutExt).not.toBe("");
-    expect(typeof withoutExt).toBe("string");
-  });
+      // Ensure unicode and spaces are preserved in basename
+      const withoutExt = Filenames.withoutExtension(filename);
+      expect(withoutExt).not.toBe("");
+      expect(typeof withoutExt).toBe("string");
+    },
+  );
 
   it("should handle blob download filename generation", () => {
     // Mock URL.createObjectURL for blob testing