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

package/src/components/tracing/tracing.test.tsx

@@ -5,36 +5,36 @@ import { formatChartTime } from "./utils";
 describe("formatChartTime", () => {
   beforeAll(() => {
     // Mock Date to always use UTC
-    vi.spyOn(global.Date.prototype, "getFullYear").mockImplementation(function (
-      this: Date,
-    ) {
-      return this.getUTCFullYear();
-    });
-    vi.spyOn(global.Date.prototype, "getMonth").mockImplementation(function (
-      this: Date,
-    ) {
-      return this.getUTCMonth();
-    });
-    vi.spyOn(global.Date.prototype, "getDate").mockImplementation(function (
-      this: Date,
-    ) {
-      return this.getUTCDate();
-    });
-    vi.spyOn(global.Date.prototype, "getHours").mockImplementation(function (
-      this: Date,
-    ) {
-      return this.getUTCHours();
-    });
-    vi.spyOn(global.Date.prototype, "getMinutes").mockImplementation(function (
-      this: Date,
-    ) {
-      return this.getUTCMinutes();
-    });
-    vi.spyOn(global.Date.prototype, "getSeconds").mockImplementation(function (
-      this: Date,
-    ) {
-      return this.getUTCSeconds();
-    });
+    vi.spyOn(global.Date.prototype, "getFullYear").mockImplementation(
+      function (this: Date) {
+        return this.getUTCFullYear();
+      },
+    );
+    vi.spyOn(global.Date.prototype, "getMonth").mockImplementation(
+      function (this: Date) {
+        return this.getUTCMonth();
+      },
+    );
+    vi.spyOn(global.Date.prototype, "getDate").mockImplementation(
+      function (this: Date) {
+        return this.getUTCDate();
+      },
+    );
+    vi.spyOn(global.Date.prototype, "getHours").mockImplementation(
+      function (this: Date) {
+        return this.getUTCHours();
+      },
+    );
+    vi.spyOn(global.Date.prototype, "getMinutes").mockImplementation(
+      function (this: Date) {
+        return this.getUTCMinutes();
+      },
+    );
+    vi.spyOn(global.Date.prototype, "getSeconds").mockImplementation(
+      function (this: Date) {
+        return this.getUTCSeconds();
+      },
+    );
     vi.spyOn(global.Date.prototype, "getMilliseconds").mockImplementation(
       function (this: Date) {
         return this.getUTCMilliseconds();

package/src/components/ui/badge.tsx

@@ -29,7 +29,8 @@ const badgeVariants = cva(
 );
 
 export interface BadgeProps
-  extends React.HTMLAttributes<HTMLDivElement>,
+  extends
+    React.HTMLAttributes<HTMLDivElement>,
     VariantProps<typeof badgeVariants> {}
 
 const Badge = ({ className, variant, ...props }: BadgeProps) => {

package/src/components/ui/button.tsx

@@ -92,7 +92,8 @@ const buttonVariants = cva(
 );
 
 export interface ButtonProps
-  extends React.ButtonHTMLAttributes<HTMLButtonElement>,
+  extends
+    React.ButtonHTMLAttributes<HTMLButtonElement>,
     Omit<VariantProps<typeof buttonVariants>, "disabled"> {
   asChild?: boolean;
   keyboardShortcut?: string;

package/src/components/ui/calendar.tsx

@@ -183,8 +183,9 @@ const Calendar = <T extends AriaDateValue>({
   );
 };
 
-interface RangeCalendarProps<T extends AriaDateValue>
-  extends AriaRangeCalendarProps<T> {
+interface RangeCalendarProps<
+  T extends AriaDateValue,
+> extends AriaRangeCalendarProps<T> {
   errorMessage?: string;
 }
 

package/src/components/ui/combobox.tsx

@@ -230,7 +230,8 @@ interface ComboboxItemOptions<TValue> {
 }
 
 export interface ComboboxItemProps<TValue>
-  extends ComboboxItemOptions<TValue>,
+  extends
+    ComboboxItemOptions<TValue>,
     Omit<
       React.ComponentProps<typeof CommandItem>,
       keyof ComboboxItemOptions<TValue> | "onSelect" | "role"

package/src/components/ui/date-input.tsx

@@ -41,8 +41,7 @@ const DateSegment = ({ className, ...props }: AriaDateSegmentProps) => {
 };
 
 interface DateInputProps
-  extends AriaDateInputProps,
-    VariantProps<typeof fieldGroupVariants> {}
+  extends AriaDateInputProps, VariantProps<typeof fieldGroupVariants> {}
 
 const DateInput = ({
   className,
@@ -61,8 +60,9 @@ const DateInput = ({
   );
 };
 
-interface DateFieldProps<T extends AriaDateValue>
-  extends AriaDateFieldProps<T> {
+interface DateFieldProps<
+  T extends AriaDateValue,
+> extends AriaDateFieldProps<T> {
   label?: string;
   description?: string;
   errorMessage?: string | ((validation: AriaValidationResult) => string);
@@ -94,8 +94,9 @@ const DateField = <T extends AriaDateValue>({
   );
 };
 
-interface TimeFieldProps<T extends AriaTimeValue>
-  extends AriaTimeFieldProps<T> {
+interface TimeFieldProps<
+  T extends AriaTimeValue,
+> extends AriaTimeFieldProps<T> {
   label?: string;
   description?: string;
   errorMessage?: string | ((validation: AriaValidationResult) => string);

package/src/components/ui/date-picker.tsx

@@ -51,8 +51,9 @@ const DatePickerContent = ({
   </Popover>
 );
 
-interface DatePickerProps<T extends AriaDateValue>
-  extends AriaDatePickerProps<T> {
+interface DatePickerProps<
+  T extends AriaDateValue,
+> extends AriaDatePickerProps<T> {
   label?: string;
   description?: string;
   errorMessage?: string | ((validation: AriaValidationResult) => string);
@@ -115,8 +116,9 @@ const DatePicker = <T extends AriaDateValue>({
   );
 };
 
-interface DateRangePickerProps<T extends AriaDateValue>
-  extends AriaDateRangePickerProps<T> {
+interface DateRangePickerProps<
+  T extends AriaDateValue,
+> extends AriaDateRangePickerProps<T> {
   label?: string;
   description?: string;
   errorMessage?: string | ((validation: AriaValidationResult) => string);

package/src/components/ui/field.tsx

@@ -66,8 +66,7 @@ const fieldGroupVariants = cva("", {
 });
 
 interface GroupProps
-  extends AriaGroupProps,
-    VariantProps<typeof fieldGroupVariants> {}
+  extends AriaGroupProps, VariantProps<typeof fieldGroupVariants> {}
 
 const FieldGroup = ({ className, variant, ...props }: GroupProps) => {
   return (

package/src/components/ui/progress.tsx

@@ -5,8 +5,9 @@ import * as React from "react";
 
 import { cn } from "@/utils/cn";
 
-interface ProgressProps
-  extends React.ComponentPropsWithoutRef<typeof ProgressPrimitive.Root> {
+interface ProgressProps extends React.ComponentPropsWithoutRef<
+  typeof ProgressPrimitive.Root
+> {
   /**
    * When true, shows an indeterminate animated progress bar.
    */

package/src/components/ui/query-param-preserving-link.tsx

@@ -3,8 +3,10 @@
 import React, { useRef } from "react";
 import { usePress } from "react-aria";
 
-interface QueryParamPreservingLinkProps
-  extends Omit<React.HTMLAttributes<HTMLAnchorElement>, "href"> {
+interface QueryParamPreservingLinkProps extends Omit<
+  React.HTMLAttributes<HTMLAnchorElement>,
+  "href"
+> {
   href: string;
   children: React.ReactNode;
 }

package/src/components/ui/sheet.tsx

@@ -51,7 +51,8 @@ const sheetVariants = cva(
 );
 
 interface SheetContentProps
-  extends React.ComponentPropsWithoutRef<typeof SheetPrimitive.Content>,
+  extends
+    React.ComponentPropsWithoutRef<typeof SheetPrimitive.Content>,
     VariantProps<typeof sheetVariants> {}
 
 const SheetContent = React.forwardRef<

package/src/components/ui/textarea.tsx

@@ -6,8 +6,7 @@ import { useDebounceControlledState } from "@/hooks/useDebounce";
 import { cn } from "@/utils/cn";
 import { Events } from "@/utils/events";
 
-export interface TextareaProps
-  extends React.TextareaHTMLAttributes<HTMLTextAreaElement> {
+export interface TextareaProps extends React.TextareaHTMLAttributes<HTMLTextAreaElement> {
   bottomAdornment?: React.ReactNode;
 }
 

package/src/core/ai/context/providers/cell-output.ts

@@ -34,8 +34,7 @@ export interface CellOutputData {
 // For the context provider
 // Currently, we enforce that cellOutput is present.
 interface CellOutputContextData
-  extends CellOutputData,
-    Record<string, unknown> {
+  extends CellOutputData, Record<string, unknown> {
   cellOutput: BaseOutput;
 }
 

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

@@ -72,9 +72,10 @@ type EditNotebookInput = z.infer<typeof editNotebookSchema>;
 type EditOperation = EditNotebookInput["edit"];
 export type EditType = EditOperation["type"];
 
-export class EditNotebookTool
-  implements AiTool<EditNotebookInput, ToolOutputBase>
-{
+export class EditNotebookTool implements AiTool<
+  EditNotebookInput,
+  ToolOutputBase
+> {
   readonly name = "edit_notebook_tool";
   readonly description = description;
   readonly schema = editNotebookSchema;

package/src/core/ai/tools/run-cells-tool.ts

@@ -50,9 +50,10 @@ const filePartSchema = z.object({
   mediaType: z.string(),
 }) satisfies z.ZodType<FileUIPart>;
 
-export class RunStaleCellsTool
-  implements AiTool<EmptyToolInput, RunStaleCellsOutput>
-{
+export class RunStaleCellsTool implements AiTool<
+  EmptyToolInput,
+  RunStaleCellsOutput
+> {
   readonly name = "run_stale_cells_tool";
   readonly description = description;
   readonly schema = z.object({});

package/src/core/cells/__tests__/add-missing-import.test.ts

@@ -48,28 +48,29 @@ describe("maybeAddMissingImport", () => {
     "import marimo as mo\nimport marimo as mo",
     "import   marimo    as   mo",
   ];
-  it.each(
-    VALID_IMPORTS,
-  )("should not add an import if the import statement already exists in the notebook", (code) => {
-    const appStore = createStore();
-    appStore.set(variablesAtom, {} as Variables);
-    appStore.set(
-      notebookAtom,
-      MockNotebook.notebookState({
-        cellData: {
-          [Cell1]: { code: code },
-        },
-      }),
-    );
-    const onAddImport = vi.fn();
-    maybeAddMissingImport({
-      moduleName: "marimo",
-      variableName: "mo",
-      onAddImport,
-      appStore,
-    });
-    expect(onAddImport).not.toHaveBeenCalled();
-  });
+  it.each(VALID_IMPORTS)(
+    "should not add an import if the import statement already exists in the notebook",
+    (code) => {
+      const appStore = createStore();
+      appStore.set(variablesAtom, {} as Variables);
+      appStore.set(
+        notebookAtom,
+        MockNotebook.notebookState({
+          cellData: {
+            [Cell1]: { code: code },
+          },
+        }),
+      );
+      const onAddImport = vi.fn();
+      maybeAddMissingImport({
+        moduleName: "marimo",
+        variableName: "mo",
+        onAddImport,
+        appStore,
+      });
+      expect(onAddImport).not.toHaveBeenCalled();
+    },
+  );
 
   it("should add an import if the variable is not in the variables state and the import statement does not exist in the notebook", () => {
     const appStore = createStore();

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

@@ -43,19 +43,20 @@ describe("outputIsLoading", () => {
 });
 
 describe("outputIsStale", () => {
-  it.each(
-    STATUSES,
-  )("should return true if the cell is edited and status is %s", (status) => {
-    const cell = {
-      status: status,
-      staleInputs: true,
-      output: null,
-      runStartTimestamp: null,
-      interrupted: false,
-    };
-    const edited = true;
-    expect(outputIsStale(cell, edited)).toBe(true);
-  });
+  it.each(STATUSES)(
+    "should return true if the cell is edited and status is %s",
+    (status) => {
+      const cell = {
+        status: status,
+        staleInputs: true,
+        output: null,
+        runStartTimestamp: null,
+        interrupted: false,
+      };
+      const edited = true;
+      expect(outputIsStale(cell, edited)).toBe(true);
+    },
+  );
 
   it("should return true if the cell is loading", () => {
     const cell = {

package/src/core/cells/logs.ts

@@ -98,7 +98,7 @@ const CellLogLogger = {
           ? "red"
           : "orange";
     const status = payload.level.toUpperCase();
-    /** biome-ignore lint/suspicious/noConsole: for debugging */
+    // oxlint-disable-next-line no-console -- intentional debug logging
     console.log(
       `%c[${status}]`,
       `color:${color}; padding:2px 0; border-radius:2px; font-weight:bold`,

package/src/core/codemirror/cells/__tests__/extensions.test.ts

@@ -18,24 +18,22 @@ function createTransaction(spec: TransactionSpec) {
 }
 
 describe("shouldAutorunMarkdownUpdate", () => {
-  it.each([
-    "input.type",
-    "delete.backward",
-    "undo",
-    "redo",
-  ])("accepts local %s transactions", (userEvent) => {
-    const transaction = createTransaction({
-      changes: { from: 0, insert: "#" },
-      annotations: [Transaction.userEvent.of(userEvent)],
-    });
+  it.each(["input.type", "delete.backward", "undo", "redo"])(
+    "accepts local %s transactions",
+    (userEvent) => {
+      const transaction = createTransaction({
+        changes: { from: 0, insert: "#" },
+        annotations: [Transaction.userEvent.of(userEvent)],
+      });
 
-    expect(
-      shouldAutorunMarkdownUpdate({
-        docChanged: transaction.docChanged,
-        transactions: [transaction],
-      }),
-    ).toBe(true);
-  });
+      expect(
+        shouldAutorunMarkdownUpdate({
+          docChanged: transaction.docChanged,
+          transactions: [transaction],
+        }),
+      ).toBe(true);
+    },
+  );
 
   it("ignores formatting changes", () => {
     const transaction = createTransaction({

package/src/core/codemirror/copilot/transport.ts

@@ -169,8 +169,7 @@ export class LazyWebsocketTransport extends Transport {
           // Show error toast on final retry
           this.options.showError(
             "GitHub Copilot Connection Error",
-            "Failed to connect to GitHub Copilot. Please check your settings and try again.\n\n" +
-              prettyError(error),
+            `Failed to connect to GitHub Copilot. Please check your settings and try again.\n\n${prettyError(error)}`,
           );
           throw error;
         }

package/src/core/codemirror/language/languages/markdown.ts

@@ -38,9 +38,7 @@ export const MARKDOWN_INITIAL_HIDE_CODE = true;
 /**
  * Language adapter for Markdown.
  */
-export class MarkdownLanguageAdapter
-  implements LanguageAdapter<MarkdownLanguageAdapterMetadata>
-{
+export class MarkdownLanguageAdapter implements LanguageAdapter<MarkdownLanguageAdapterMetadata> {
   private parser = new MarkdownParser();
 
   readonly type = "markdown";

package/src/core/codemirror/language/languages/python.ts

@@ -45,6 +45,7 @@ import type { LanguageAdapter } from "../types";
 
 const pylspClient = once((lspConfig: LSPConfig) => {
   // Create a mutable reference for the resync callback
+  // oxlint-disable-next-line prefer-const -- reassigned after closure capture
   let resyncCallback: (() => Promise<void>) | undefined;
 
   const transport = createTransport("pylsp", async () => {
@@ -151,6 +152,7 @@ const pylspClient = once((lspConfig: LSPConfig) => {
 });
 
 const tyLspClient = once((_: LSPConfig) => {
+  // oxlint-disable-next-line prefer-const -- reassigned after closure capture
   let resyncCallback: (() => Promise<void>) | undefined;
 
   const transport = createTransport("ty", async () => {
@@ -181,6 +183,7 @@ const tyLspClient = once((_: LSPConfig) => {
 
 const pyreflyClient = once(
   (lspConfig: LSPConfig & { diagnostics: DiagnosticsConfig }) => {
+    // oxlint-disable-next-line prefer-const -- reassigned after closure capture
     let resyncCallback: (() => Promise<void>) | undefined;
 
     const transport = createTransport("pyrefly", async () => {
@@ -218,6 +221,7 @@ const pyreflyClient = once(
 );
 
 const pyrightClient = once((_: LSPConfig) => {
+  // oxlint-disable-next-line prefer-const -- reassigned after closure capture
   let resyncCallback: (() => Promise<void>) | undefined;
 
   const transport = createTransport("basedpyright", async () => {

package/src/core/codemirror/language/languages/sql/completion-sources.tsx

@@ -91,13 +91,11 @@ export function customKeywordCompletionSource(): CompletionSource {
 
 // e.g. lazily load keyword docs
 const getKeywordDocs = once(async (): Promise<Record<string, unknown>> => {
-  const keywords = await import(
-    "@marimo-team/codemirror-sql/data/common-keywords.json"
-  );
+  const keywords =
+    await import("@marimo-team/codemirror-sql/data/common-keywords.json");
   // Include DuckDB for now, but we can remove this once we have a better way to handle dialect-specific keywords
-  const duckdbKeywords = await import(
-    "@marimo-team/codemirror-sql/data/duckdb-keywords.json"
-  );
+  const duckdbKeywords =
+    await import("@marimo-team/codemirror-sql/data/duckdb-keywords.json");
   return {
     ...keywords.default.keywords,
     ...duckdbKeywords.default.keywords,

package/src/core/codemirror/language/languages/sql/sql.ts

@@ -77,9 +77,7 @@ function getLatestEngine(): ConnectionName {
 /**
  * Language adapter for SQL.
  */
-export class SQLLanguageAdapter
-  implements LanguageAdapter<SQLLanguageAdapterMetadata>
-{
+export class SQLLanguageAdapter implements LanguageAdapter<SQLLanguageAdapterMetadata> {
   private parser = new SQLParser();
   readonly type = "sql";
   sqlModeEnabled: boolean;