@marimo-team/islands 0.25.1-dev27 → 0.25.1-dev28

package/src/core/wasm/worker/message-buffer.ts

@@ -8,12 +8,12 @@ export class MessageBuffer<T> {
   private buffer: T[];
   private started = false;
   private onMessage: (data: T) => void;
-  constructor(onMessage: (data: T) => void) {
+  public constructor(onMessage: (data: T) => void) {
     this.onMessage = onMessage;
     this.buffer = [];
   }
 
-  push = (data: T) => {
+  public push = (data: T) => {
     if (this.started) {
       this.onMessage(data);
     } else {
@@ -24,7 +24,7 @@ export class MessageBuffer<T> {
   /**
    * Start processing messages
    */
-  start = () => {
+  public start = () => {
     this.started = true;
     // Flush the buffer
     this.buffer.forEach((data) => this.onMessage(data));

package/src/core/websocket/transports/__tests__/ws.test.ts

@@ -7,20 +7,20 @@ let innerListeners: Record<string, ((e: unknown) => void)[]>;
 
 vi.mock("partysocket/ws", () => {
   class FakeReconnectingWebSocket {
-    retryCount = 0;
-    readyState = WebSocket.CONNECTING;
-    constructor() {
+    public retryCount = 0;
+    public readyState = WebSocket.CONNECTING;
+    public constructor() {
       innerListeners = { open: [], close: [], message: [], error: [] };
     }
-    addEventListener(event: string, cb: (e: unknown) => void) {
+    public addEventListener(event: string, cb: (e: unknown) => void) {
       innerListeners[event].push(cb);
     }
-    removeEventListener(event: string, cb: (e: unknown) => void) {
+    public removeEventListener(event: string, cb: (e: unknown) => void) {
       innerListeners[event] = innerListeners[event].filter((c) => c !== cb);
     }
-    reconnect() {}
-    close() {}
-    send() {}
+    public reconnect() {}
+    public close() {}
+    public send() {}
   }
   return { default: FakeReconnectingWebSocket };
 });

package/src/core/websocket/transports/basic.ts

@@ -13,11 +13,13 @@ export class BasicTransport implements IConnectionTransport {
   private subscriptions = new ConnectionSubscriptions();
   private producer?: MessageProducer;
 
-  static withProducerCallback(producer: MessageProducer): IConnectionTransport {
+  public static withProducerCallback(
+    producer: MessageProducer,
+  ): IConnectionTransport {
     return new BasicTransport(producer);
   }
 
-  static empty(): IConnectionTransport {
+  public static empty(): IConnectionTransport {
     return new BasicTransport();
   }
 
@@ -39,21 +41,26 @@ export class BasicTransport implements IConnectionTransport {
     });
   }
 
-  get readyState(): WebSocket["readyState"] {
+  public get readyState(): WebSocket["readyState"] {
     return WebSocket.OPEN;
   }
 
-  reconnect(_code?: number | undefined, _reason?: string | undefined): void {
+  public reconnect(
+    _code?: number | undefined,
+    _reason?: string | undefined,
+  ): void {
     this.close();
     this.connect();
     return;
   }
 
-  close(): void {
+  public close(): void {
     this.subscriptions.notify("close", new Event("close"));
   }
 
-  send(data: string | ArrayBuffer | Blob | ArrayBufferView): Promise<void> {
+  public send(
+    data: string | ArrayBuffer | Blob | ArrayBufferView,
+  ): Promise<void> {
     this.subscriptions.notify(
       "message",
       new MessageEvent("message", {
@@ -63,7 +70,7 @@ export class BasicTransport implements IConnectionTransport {
     return Promise.resolve();
   }
 
-  addEventListener<T extends ConnectionEvent>(
+  public addEventListener<T extends ConnectionEvent>(
     event: T,
     callback: ConnectionTransportCallback<T>,
   ): void {
@@ -83,7 +90,7 @@ export class BasicTransport implements IConnectionTransport {
     }
   }
 
-  removeEventListener<T extends ConnectionEvent>(
+  public removeEventListener<T extends ConnectionEvent>(
     event: T,
     callback: ConnectionTransportCallback<T>,
   ): void {

package/src/core/websocket/transports/sse.ts

@@ -52,7 +52,7 @@ export class SseTransport implements IConnectionTransport {
   private userClosed = false;
   private state: WebSocket["readyState"] = WebSocket.CLOSED;
 
-  constructor(
+  public constructor(
     urlProvider: () => string,
     headersProvider: () => Record<string, string> = () => ({}),
   ) {
@@ -60,11 +60,11 @@ export class SseTransport implements IConnectionTransport {
     this.headersProvider = headersProvider;
   }
 
-  get readyState(): WebSocket["readyState"] {
+  public get readyState(): WebSocket["readyState"] {
     return this.state;
   }
 
-  reconnect(_code?: number, _reason?: string): void {
+  public reconnect(_code?: number, _reason?: string): void {
     this.userClosed = false;
     this.retryCount = 0;
     this.clearRetryTimer();
@@ -72,7 +72,7 @@ export class SseTransport implements IConnectionTransport {
     this.connect();
   }
 
-  close(): void {
+  public close(): void {
     this.userClosed = true;
     this.clearRetryTimer();
     this.state = WebSocket.CLOSED;
@@ -80,13 +80,13 @@ export class SseTransport implements IConnectionTransport {
     this.abortController = null;
   }
 
-  send(_data: string | ArrayBuffer | Blob | ArrayBufferView): void {
+  public send(_data: string | ArrayBuffer | Blob | ArrayBufferView): void {
     // The kernel connection is receive-only; all client requests go over
     // HTTP POST endpoints.
     Logger.warn("SseTransport does not support send(); dropping message");
   }
 
-  addEventListener<T extends ConnectionEvent>(
+  public addEventListener<T extends ConnectionEvent>(
     event: T,
     callback: ConnectionTransportCallback<T>,
   ): void {
@@ -96,7 +96,7 @@ export class SseTransport implements IConnectionTransport {
     );
   }
 
-  removeEventListener<T extends ConnectionEvent>(
+  public removeEventListener<T extends ConnectionEvent>(
     event: T,
     callback: ConnectionTransportCallback<T>,
   ): void {

package/src/core/websocket/transports/transport.ts

@@ -33,7 +33,7 @@ export class ConnectionSubscriptions {
     Set<ConnectionTransportCallback<ConnectionEvent>>
   >();
 
-  addSubscription(
+  public addSubscription(
     event: ConnectionEvent,
     callback: ConnectionTransportCallback<ConnectionEvent>,
   ): void {
@@ -43,14 +43,14 @@ export class ConnectionSubscriptions {
     this.subscriptions.get(event)?.add(callback);
   }
 
-  removeSubscription(
+  public removeSubscription(
     event: ConnectionEvent,
     callback: ConnectionTransportCallback<ConnectionEvent>,
   ): void {
     this.subscriptions.get(event)?.delete(callback);
   }
 
-  notify(
+  public notify(
     event: ConnectionEvent,
     data: IConnectionTransportMap[ConnectionEvent],
   ): void {

package/src/core/websocket/transports/ws.ts

@@ -20,7 +20,7 @@ export class WsTransport implements IConnectionTransport {
     ConnectionTransportCallback<"close">
   >();
 
-  constructor(urlProvider: () => string) {
+  public constructor(urlProvider: () => string) {
     this.inner = new ReconnectingWebSocket(urlProvider, undefined, {
       maxRetries: MAX_RETRIES,
       debug: false,
@@ -30,23 +30,25 @@ export class WsTransport implements IConnectionTransport {
     });
   }
 
-  get readyState(): WebSocket["readyState"] {
+  public get readyState(): WebSocket["readyState"] {
     return this.inner.readyState as WebSocket["readyState"];
   }
 
-  reconnect(code?: number, reason?: string): void {
+  public reconnect(code?: number, reason?: string): void {
     this.inner.reconnect(code, reason);
   }
 
-  close(): void {
+  public close(): void {
     this.inner.close();
   }
 
-  send(data: string | ArrayBuffer | Blob | ArrayBufferView<ArrayBuffer>): void {
+  public send(
+    data: string | ArrayBuffer | Blob | ArrayBufferView<ArrayBuffer>,
+  ): void {
     this.inner.send(data);
   }
 
-  addEventListener<T extends ConnectionEvent>(
+  public addEventListener<T extends ConnectionEvent>(
     event: T,
     callback: ConnectionTransportCallback<T>,
   ): void {
@@ -78,7 +80,7 @@ export class WsTransport implements IConnectionTransport {
     this.inner.addEventListener(event, callback as never);
   }
 
-  removeEventListener<T extends ConnectionEvent>(
+  public removeEventListener<T extends ConnectionEvent>(
     event: T,
     callback: ConnectionTransportCallback<T>,
   ): void {

package/src/hooks/__tests__/useOverflowDetection.test.tsx

@@ -10,23 +10,23 @@ const mockObserve = vi.fn();
 const mockDisconnect = vi.fn();
 
 global.ResizeObserver = class MockResizeObserver {
-  callback: ResizeObserverCallback;
+  public callback: ResizeObserverCallback;
 
-  constructor(callback: ResizeObserverCallback) {
+  public constructor(callback: ResizeObserverCallback) {
     this.callback = callback;
   }
 
-  observe(target: Element) {
+  public observe(target: Element) {
     mockObserve(target);
     // Simulate initial observation callback
     this.callback([], this);
   }
 
-  unobserve() {
+  public unobserve() {
     // noop
   }
 
-  disconnect() {
+  public disconnect() {
     mockDisconnect();
   }
 };

package/src/hooks/__tests__/useResizeObserver.test.tsx

@@ -16,9 +16,9 @@ describe("useResizeObserver", () => {
   it("should call observe when ref is provided", () => {
     const observeSpy = vi.fn();
     global.ResizeObserver = class MockedResizeObserver {
-      observe = observeSpy;
-      unobserve = vi.fn();
-      disconnect = vi.fn();
+      public observe = observeSpy;
+      public unobserve = vi.fn();
+      public disconnect = vi.fn();
     };
 
     renderHook(() =>
@@ -33,9 +33,9 @@ describe("useResizeObserver", () => {
   it("should not call observe when ref is not provided", () => {
     const observeSpy = vi.fn();
     global.ResizeObserver = class MockedResizeObserver {
-      observe = observeSpy;
-      unobserve = vi.fn();
-      disconnect = vi.fn();
+      public observe = observeSpy;
+      public unobserve = vi.fn();
+      public disconnect = vi.fn();
     };
 
     renderHook(() =>
@@ -50,9 +50,9 @@ describe("useResizeObserver", () => {
   it("should not call observe when skipped", () => {
     const observeSpy = vi.fn();
     global.ResizeObserver = class MockedResizeObserver {
-      observe = observeSpy;
-      unobserve = vi.fn();
-      disconnect = vi.fn();
+      public observe = observeSpy;
+      public unobserve = vi.fn();
+      public disconnect = vi.fn();
     };
 
     renderHook(() =>
@@ -68,9 +68,9 @@ describe("useResizeObserver", () => {
   it("disconnect should be called once unmounted", () => {
     const disconnectSpy = vi.fn();
     global.ResizeObserver = class MockedResizeObserver {
-      observe = vi.fn();
-      unobserve = vi.fn();
-      disconnect = disconnectSpy;
+      public observe = vi.fn();
+      public unobserve = vi.fn();
+      public disconnect = disconnectSpy;
     };
 
     const { unmount } = renderHook(() =>

package/src/plugins/core/registerReactComponent.tsx

@@ -336,7 +336,7 @@ export function registerReactComponent<T>(plugin: IPlugin<T, unknown>): void {
     private pluginRef = createRef<PluginSlotHandle>();
     protected __type__ = customElementLocator;
 
-    constructor() {
+    public constructor() {
       super();
       // Create a shadow root so we can store the React tree on the shadow root, while the original
       // element's children are still on the DOM
@@ -349,7 +349,7 @@ export function registerReactComponent<T>(plugin: IPlugin<T, unknown>): void {
       });
     }
 
-    connectedCallback() {
+    public connectedCallback() {
       // Skip mounting if this element is in the light DOM of another
       // marimo custom element. The parent element's shadow DOM will
       // re-create this element via getChildren() -> renderHTML(), so
@@ -383,7 +383,7 @@ export function registerReactComponent<T>(plugin: IPlugin<T, unknown>): void {
       });
     }
 
-    disconnectedCallback() {
+    public disconnectedCallback() {
       this.observer.disconnect();
       this.root?.unmount();
       if (this.mounted) {
@@ -419,7 +419,7 @@ export function registerReactComponent<T>(plugin: IPlugin<T, unknown>): void {
      * Reset the plugin initial value and data.
      * And then re-render the plugin.
      */
-    reset() {
+    public reset() {
       this.dispatchEvent(
         createInputEvent(parseAttrValue(this.dataset.initialValue), this),
       );
@@ -429,7 +429,7 @@ export function registerReactComponent<T>(plugin: IPlugin<T, unknown>): void {
     /**
      * Re-render the plugin.
      */
-    rerender() {
+    public rerender() {
       this.pluginRef.current?.reset();
     }
 

package/src/plugins/core/sidebar-element.tsx

@@ -19,7 +19,7 @@ export function initializeSidebarElement(): void {
     private observer?: MutationObserver;
     private uniqueId = Symbol(createId());
 
-    connectedCallback() {
+    public connectedCallback() {
       // Render the component for the first time
       this.mountReactComponent();
 
@@ -39,7 +39,7 @@ export function initializeSidebarElement(): void {
       });
     }
 
-    disconnectedCallback() {
+    public disconnectedCallback() {
       if (this.observer) {
         this.observer.disconnect();
         this.unmountReactComponent();

package/src/plugins/impl/ButtonPlugin.tsx

@@ -20,9 +20,9 @@ interface Data {
 }
 
 export class ButtonPlugin implements IPlugin<number, Data> {
-  tagName = "marimo-button";
+  public tagName = "marimo-button";
 
-  validator = z.object({
+  public validator = z.object({
     label: z.string(),
     kind: zodIntent,
     disabled: z.boolean().default(false),
@@ -31,7 +31,7 @@ export class ButtonPlugin implements IPlugin<number, Data> {
     keyboardShortcut: z.string().optional(),
   });
 
-  render(props: IPluginProps<number, Data>): JSX.Element {
+  public render(props: IPluginProps<number, Data>): JSX.Element {
     const {
       data: { disabled, kind, label, fullWidth, tooltip, keyboardShortcut },
     } = props;

package/src/plugins/impl/CheckboxPlugin.tsx

@@ -18,15 +18,15 @@ interface Data {
 }
 
 export class CheckboxPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-checkbox";
+  public tagName = "marimo-checkbox";
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.boolean(),
     label: z.string().nullable(),
     disabled: z.boolean().optional(),
   });
 
-  render(props: IPluginProps<T, Data>): JSX.Element {
+  public render(props: IPluginProps<T, Data>): JSX.Element {
     return <CheckboxComponent {...props} />;
   }
 }

package/src/plugins/impl/CodeEditorPlugin.tsx

@@ -25,9 +25,9 @@ interface Data {
 }
 
 export class CodeEditorPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-code-editor";
+  public tagName = "marimo-code-editor";
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.string(),
     language: z.string().default("python"),
     placeholder: z.string(),
@@ -40,7 +40,7 @@ export class CodeEditorPlugin implements IPlugin<T, Data> {
     debounce: z.union([z.boolean(), z.number()]).default(false),
   });
 
-  render(props: IPluginProps<T, Data>): JSX.Element {
+  public render(props: IPluginProps<T, Data>): JSX.Element {
     return (
       <CodeEditorComponent
         {...props.data}

package/src/plugins/impl/DatePickerPlugin.tsx

@@ -19,9 +19,9 @@ interface Data {
 }
 
 export class DatePickerPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-date";
+  public tagName = "marimo-date";
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.string(),
     label: z.string().nullable(),
     start: z.string(),
@@ -31,7 +31,7 @@ export class DatePickerPlugin implements IPlugin<T, Data> {
     disabled: z.boolean().optional(),
   });
 
-  render(props: IPluginProps<T, Data>): JSX.Element {
+  public render(props: IPluginProps<T, Data>): JSX.Element {
     return (
       <DatePickerComponent
         {...props.data}

package/src/plugins/impl/DateRangePlugin.tsx

@@ -19,9 +19,9 @@ interface Data {
 }
 
 export class DateRangePickerPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-date-range";
+  public tagName = "marimo-date-range";
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.tuple([z.string(), z.string()]),
     label: z.string().nullable(),
     start: z.string(),
@@ -31,7 +31,7 @@ export class DateRangePickerPlugin implements IPlugin<T, Data> {
     disabled: z.boolean().optional(),
   });
 
-  render(props: IPluginProps<T, Data>): JSX.Element {
+  public render(props: IPluginProps<T, Data>): JSX.Element {
     return (
       <DateRangePickerComponent
         {...props.data}

package/src/plugins/impl/DateTimePickerPlugin.tsx

@@ -20,9 +20,9 @@ interface Data {
 }
 
 export class DateTimePickerPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-datetime";
+  public tagName = "marimo-datetime";
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.string(),
     label: z.string().nullable(),
     start: z.string(),
@@ -33,7 +33,7 @@ export class DateTimePickerPlugin implements IPlugin<T, Data> {
     disabled: z.boolean().optional(),
   });
 
-  render(props: IPluginProps<T, Data>): JSX.Element {
+  public render(props: IPluginProps<T, Data>): JSX.Element {
     return (
       <DateTimePickerComponent
         {...props.data}

package/src/plugins/impl/DictPlugin.tsx

@@ -18,14 +18,14 @@ interface Data {
 }
 
 export class DictPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-dict";
+  public tagName = "marimo-dict";
 
-  validator = z.object({
+  public validator = z.object({
     label: z.string().nullable(),
     elementIds: z.record(z.string(), z.string()),
   });
 
-  render(props: IPluginProps<T, Data>): JSX.Element {
+  public render(props: IPluginProps<T, Data>): JSX.Element {
     return (
       <Dict {...props.data} value={props.value} setValue={props.setValue}>
         {props.children}

package/src/plugins/impl/DropdownPlugin.tsx

@@ -17,9 +17,9 @@ interface Data {
 }
 
 export class DropdownPlugin implements IPlugin<string[], Data> {
-  tagName = "marimo-dropdown";
+  public tagName = "marimo-dropdown";
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.array(z.string()),
     label: z.string().nullable(),
     options: z.array(z.string()),
@@ -29,7 +29,7 @@ export class DropdownPlugin implements IPlugin<string[], Data> {
     disabled: z.boolean().default(false),
   });
 
-  render(props: IPluginProps<string[], Data>): JSX.Element {
+  public render(props: IPluginProps<string[], Data>): JSX.Element {
     if (props.data.searchable) {
       const value = props.value.length > 0 ? props.value[0] : null;
       const setValue = (newValue: string | null) =>

package/src/plugins/impl/FileUploadPlugin.tsx

@@ -34,9 +34,9 @@ interface Data {
 type T = [string, string][];
 
 export class FileUploadPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-file";
+  public tagName = "marimo-file";
 
-  validator = z.object({
+  public validator = z.object({
     filetypes: z.array(z.string()),
     multiple: z.boolean(),
     kind: z.enum(["button", "area"]),
@@ -44,7 +44,7 @@ export class FileUploadPlugin implements IPlugin<T, Data> {
     max_size: z.number(),
   });
 
-  render(props: IPluginProps<T, Data>): JSX.Element {
+  public render(props: IPluginProps<T, Data>): JSX.Element {
     return (
       <FileUpload
         label={props.data.label}

package/src/plugins/impl/MatrixPlugin.tsx

@@ -23,11 +23,11 @@ interface Data {
 }
 
 export class MatrixPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-matrix";
+  public tagName = "marimo-matrix";
 
-  cssStyles = [matrixCss];
+  public cssStyles = [matrixCss];
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.array(z.array(z.number())),
     label: z.string().nullable(),
     minValue: z.array(z.array(z.number())).nullish(),
@@ -42,7 +42,7 @@ export class MatrixPlugin implements IPlugin<T, Data> {
     disabled: z.array(z.array(z.boolean())),
   });
 
-  render(props: IPluginProps<T, Data>): JSX.Element {
+  public render(props: IPluginProps<T, Data>): JSX.Element {
     return (
       <MatrixComponent
         {...props.data}

package/src/plugins/impl/MicrophonePlugin.tsx

@@ -18,13 +18,13 @@ interface Data {
 }
 
 export class MicrophonePlugin implements IPlugin<Value, Data> {
-  tagName = "marimo-microphone";
+  public tagName = "marimo-microphone";
 
-  validator = z.object({
+  public validator = z.object({
     label: z.string().nullish(),
   });
 
-  render(props: IPluginProps<Value, Data>): JSX.Element {
+  public render(props: IPluginProps<Value, Data>): JSX.Element {
     return <Microphone {...props} />;
   }
 }

package/src/plugins/impl/MultiselectPlugin.tsx

@@ -17,9 +17,9 @@ interface Data {
 type T = string[];
 
 export class MultiselectPlugin implements IPlugin<T, Data> {
-  tagName = "marimo-multiselect";
+  public tagName = "marimo-multiselect";
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.array(z.string()),
     label: z.string().nullable(),
     options: z.array(z.string()),
@@ -28,7 +28,7 @@ export class MultiselectPlugin implements IPlugin<T, Data> {
     disabled: z.boolean().default(false),
   });
 
-  render(props: IPluginProps<string[], Data>): JSX.Element {
+  public render(props: IPluginProps<string[], Data>): JSX.Element {
     return (
       <Multiselect
         {...props.data}

package/src/plugins/impl/NumberPlugin.tsx

@@ -20,9 +20,9 @@ interface Data {
 }
 
 export class NumberPlugin implements IPlugin<T | null, Data> {
-  tagName = "marimo-number";
+  public tagName = "marimo-number";
 
-  validator = z.object({
+  public validator = z.object({
     initialValue: z.number().nullish(),
     label: z.string().nullable(),
     start: z.number().nullish(),
@@ -33,7 +33,7 @@ export class NumberPlugin implements IPlugin<T | null, Data> {
     disabled: z.boolean().optional(),
   });
 
-  render(props: IPluginProps<T | null, Data>): JSX.Element {
+  public render(props: IPluginProps<T | null, Data>): JSX.Element {
     return (
       <NumberComponent
         {...props.data}