@marimo-team/islands 0.22.1-dev23 → 0.22.1-dev27

package/src/utils/__tests__/cell-urls.test.ts

@@ -92,27 +92,30 @@ describe("cell-urls utilities", () => {
   });
 
   describe("edge case cell names with unicode and special characters", () => {
-    it.each(
-      EDGE_CASE_CELL_NAMES,
-    )("should handle unicode cell names in createCellLink: %s", (cellName) => {
-      const url = createCellLink(cellName);
-      expect(url).toContain("scrollTo=");
-      expect(url).toContain(encodeURIComponent(cellName));
-    });
-
-    it.each(
-      EDGE_CASE_CELL_NAMES,
-    )("should round-trip unicode cell names correctly: %s", (cellName) => {
-      const url = createCellLink(cellName);
-      const hash = url.split("#")[1];
-      const extracted = extractCellNameFromHash(`#${hash}`);
-      expect(extracted).toBe(cellName);
-    });
+    it.each(EDGE_CASE_CELL_NAMES)(
+      "should handle unicode cell names in createCellLink: %s",
+      (cellName) => {
+        const url = createCellLink(cellName);
+        expect(url).toContain("scrollTo=");
+        expect(url).toContain(encodeURIComponent(cellName));
+      },
+    );
+
+    it.each(EDGE_CASE_CELL_NAMES)(
+      "should round-trip unicode cell names correctly: %s",
+      (cellName) => {
+        const url = createCellLink(cellName);
+        const hash = url.split("#")[1];
+        const extracted = extractCellNameFromHash(`#${hash}`);
+        expect(extracted).toBe(cellName);
+      },
+    );
 
-    it.each(
-      EDGE_CASE_CELL_NAMES,
-    )("should allow linking to unicode cell names: %s", (cellName) => {
-      expect(canLinkToCell(cellName)).toBe(true);
-    });
+    it.each(EDGE_CASE_CELL_NAMES)(
+      "should allow linking to unicode cell names: %s",
+      (cellName) => {
+        expect(canLinkToCell(cellName)).toBe(true);
+      },
+    );
   });
 });

package/src/utils/__tests__/filenames.test.ts

@@ -35,22 +35,23 @@ describe("Filenames", () => {
     expect(Filenames.withoutExtension("test.foo.txt")).toEqual("test.foo");
   });
 
-  it.each(
-    EDGE_CASE_FILENAMES,
-  )("should handle edge case filenames: %s", (filename) => {
-    // Test all filename operations with edge cases
-    const withoutExt = Filenames.withoutExtension(filename);
+  it.each(EDGE_CASE_FILENAMES)(
+    "should handle edge case filenames: %s",
+    (filename) => {
+      // Test all filename operations with edge cases
+      const withoutExt = Filenames.withoutExtension(filename);
 
-    expect(Filenames.toMarkdown(filename)).toEqual(`${withoutExt}.md`);
-    expect(Filenames.toHTML(filename)).toEqual(`${withoutExt}.html`);
-    expect(Filenames.toPNG(filename)).toEqual(`${withoutExt}.png`);
-    expect(Filenames.toPY(filename)).toEqual(`${withoutExt}.py`);
-    expect(Filenames.toIPYNB(filename)).toEqual(`${withoutExt}.ipynb`);
+      expect(Filenames.toMarkdown(filename)).toEqual(`${withoutExt}.md`);
+      expect(Filenames.toHTML(filename)).toEqual(`${withoutExt}.html`);
+      expect(Filenames.toPNG(filename)).toEqual(`${withoutExt}.png`);
+      expect(Filenames.toPY(filename)).toEqual(`${withoutExt}.py`);
+      expect(Filenames.toIPYNB(filename)).toEqual(`${withoutExt}.ipynb`);
 
-    // Ensure operations preserve unicode and special characters in base name
-    expect(withoutExt).not.toEqual("");
-    expect(typeof withoutExt).toBe("string");
-  });
+      // Ensure operations preserve unicode and special characters in base name
+      expect(withoutExt).not.toEqual("");
+      expect(typeof withoutExt).toBe("string");
+    },
+  );
 });
 
 describe("getImageExtension", () => {

package/src/utils/__tests__/json-parser.test.ts

@@ -184,9 +184,8 @@ it("can convert json to markdown - with URLs", () => {
   `);
 
   // Mixed content - URL and text in same cell
-  expect(
-    jsonToMarkdown([{ info: "Visit https://example.com for more" }]),
-  ).toMatchInlineSnapshot(`
+  expect(jsonToMarkdown([{ info: "Visit https://example.com for more" }]))
+    .toMatchInlineSnapshot(`
     "| info |
     |---|
     | Visit [https://example.com](https://example.com) for more |"
@@ -203,9 +202,8 @@ it("can convert json to markdown - with URLs", () => {
 });
 
 it("can convert json to markdown - handles nulls and undefined", () => {
-  expect(
-    jsonToMarkdown([{ a: null, b: undefined, c: 1 }]),
-  ).toMatchInlineSnapshot(`
+  expect(jsonToMarkdown([{ a: null, b: undefined, c: 1 }]))
+    .toMatchInlineSnapshot(`
     "| a | b | c |
     |---|---|---|
     |  |  | 1 |"
@@ -226,18 +224,16 @@ it("can convert json to markdown - handles nulls and undefined", () => {
 
 it("can convert json to markdown - handles special characters", () => {
   // Pipes need to be escaped since they're markdown table delimiters
-  expect(
-    jsonToMarkdown([{ a: "value|with|pipes", b: "normal" }]),
-  ).toMatchInlineSnapshot(`
+  expect(jsonToMarkdown([{ a: "value|with|pipes", b: "normal" }]))
+    .toMatchInlineSnapshot(`
     "| a | b |
     |---|---|
     | value\\|with\\|pipes | normal |"
   `);
 
   // Newlines should be replaced with spaces
-  expect(
-    jsonToMarkdown([{ a: "line1\nline2", b: "normal" }]),
-  ).toMatchInlineSnapshot(`
+  expect(jsonToMarkdown([{ a: "line1\nline2", b: "normal" }]))
+    .toMatchInlineSnapshot(`
     "| a | b |
     |---|---|
     | line1 line2 | normal |"
@@ -252,9 +248,8 @@ it("can convert json to markdown - handles special characters", () => {
 });
 
 it("can convert json to markdown - handles different data types", () => {
-  expect(
-    jsonToMarkdown([{ str: "text", num: 42, bool: true, nil: null }]),
-  ).toMatchInlineSnapshot(`
+  expect(jsonToMarkdown([{ str: "text", num: 42, bool: true, nil: null }]))
+    .toMatchInlineSnapshot(`
     "| str | num | bool | nil |
     |---|---|---|---|
     | text | 42 | true |  |"
@@ -282,9 +277,8 @@ it("can convert json to markdown - handles different data types", () => {
     | [1,2,3] |"
   `);
 
-  expect(
-    jsonToMarkdown([{ data: { nested: "value" } }]),
-  ).toMatchInlineSnapshot(`
+  expect(jsonToMarkdown([{ data: { nested: "value" } }]))
+    .toMatchInlineSnapshot(`
     "| data |
     |---|
     | {"nested":"value"} |"
@@ -293,9 +287,8 @@ it("can convert json to markdown - handles different data types", () => {
 
 it("can convert json to markdown - handles existing markdown links", () => {
   // When input already contains a markdown link, it should be preserved as-is
-  expect(
-    jsonToMarkdown([{ link: "[Google](https://google.com)" }]),
-  ).toMatchInlineSnapshot(`
+  expect(jsonToMarkdown([{ link: "[Google](https://google.com)" }]))
+    .toMatchInlineSnapshot(`
     "| link |
     |---|
     | [Google](https://google.com) |"

package/src/utils/__tests__/path.test.ts

@@ -170,36 +170,39 @@ describe("PathBuilder", () => {
   });
 
   describe("edge case filenames", () => {
-    it.each(
-      EDGE_CASE_FILENAMES,
-    )("should handle unicode and spaces in basename: %s", (filename) => {
-      const basename = Paths.basename(filename);
-      expect(basename).toBe(filename);
-      expect(typeof basename).toBe("string");
-      expect(basename).not.toBe("");
-    });
-
-    it.each(
-      EDGE_CASE_FILENAMES,
-    )("should handle unicode and spaces in dirname: %s", (filename) => {
-      const fullPath = `/path/to/${filename}`;
-      const dirname = Paths.dirname(fullPath);
-      expect(dirname).toBe("/path/to");
-    });
-
-    it.each(
-      EDGE_CASE_FILENAMES,
-    )("should handle unicode and spaces in path operations: %s", (filename) => {
-      const baseName = Paths.basename(filename);
-      const extension = Paths.extension(filename);
-
-      // Should preserve unicode characters in basename
-      expect(baseName).toContain(filename.split(".")[0]);
-
-      // Should correctly extract extension
-      if (filename.includes(".")) {
-        expect(extension).toBe(filename.split(".").pop());
-      }
-    });
+    it.each(EDGE_CASE_FILENAMES)(
+      "should handle unicode and spaces in basename: %s",
+      (filename) => {
+        const basename = Paths.basename(filename);
+        expect(basename).toBe(filename);
+        expect(typeof basename).toBe("string");
+        expect(basename).not.toBe("");
+      },
+    );
+
+    it.each(EDGE_CASE_FILENAMES)(
+      "should handle unicode and spaces in dirname: %s",
+      (filename) => {
+        const fullPath = `/path/to/${filename}`;
+        const dirname = Paths.dirname(fullPath);
+        expect(dirname).toBe("/path/to");
+      },
+    );
+
+    it.each(EDGE_CASE_FILENAMES)(
+      "should handle unicode and spaces in path operations: %s",
+      (filename) => {
+        const baseName = Paths.basename(filename);
+        const extension = Paths.extension(filename);
+
+        // Should preserve unicode characters in basename
+        expect(baseName).toContain(filename.split(".")[0]);
+
+        // Should correctly extract extension
+        if (filename.includes(".")) {
+          expect(extension).toBe(filename.split(".").pop());
+        }
+      },
+    );
   });
 });

package/src/utils/__tests__/urls.test.ts

@@ -14,24 +14,25 @@ describe("isUrl", () => {
 });
 
 describe("URL parameter handling with edge case filenames", () => {
-  it.each(
-    EDGE_CASE_FILENAMES,
-  )("should handle unicode filenames in URL parameters: %s", (filename) => {
-    // Test that updateQueryParams can handle unicode filenames
-    updateQueryParams((params) => {
-      params.set("file", filename);
-    });
-
-    // Verify URL encoding/decoding works with unicode
-    const encoded = encodeURIComponent(filename);
-    const decoded = decodeURIComponent(encoded);
-    expect(decoded).toBe(filename);
-
-    // Verify filename can be safely added to URL parameters
-    const url = new URL("https://example.com");
-    url.searchParams.set("file", filename);
-    expect(url.searchParams.get("file")).toBe(filename);
-  });
+  it.each(EDGE_CASE_FILENAMES)(
+    "should handle unicode filenames in URL parameters: %s",
+    (filename) => {
+      // Test that updateQueryParams can handle unicode filenames
+      updateQueryParams((params) => {
+        params.set("file", filename);
+      });
+
+      // Verify URL encoding/decoding works with unicode
+      const encoded = encodeURIComponent(filename);
+      const decoded = decodeURIComponent(encoded);
+      expect(decoded).toBe(filename);
+
+      // Verify filename can be safely added to URL parameters
+      const url = new URL("https://example.com");
+      url.searchParams.set("file", filename);
+      expect(url.searchParams.get("file")).toBe(filename);
+    },
+  );
 
   it("should preserve unicode in query string round-trip", () => {
     EDGE_CASE_FILENAMES.forEach((filename) => {

package/src/utils/tracer.ts

@@ -82,6 +82,7 @@ export class Tracer {
     }
 
     this.spans.forEach((span) => {
+      // oxlint-disable-next-line no-console -- intentional tracing output
       console.log(`Span: ${span.name}`);
       const childSpans = this.spans.filter(
         (s) =>