@marimo-team/islands 0.25.1-dev4 → 0.25.1-dev40

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/css/globals.css

@@ -55,29 +55,21 @@
     --background: light-dark(hsl(0deg 0% 100%), hsl(150deg 7.7% 10.2%));
     --foreground: light-dark(hsl(222.2deg 47.4% 11.2%), hsl(155deg 7% 93%));
     --muted: light-dark(hsl(210deg 40% 96.1%), hsl(180deg 14% 1%));
-    --muted-foreground: light-dark(
-      hsl(215.4deg 16.3% 46.9%),
-      hsl(155deg 5% 68.3%)
-    );
+    --muted-foreground: light-dark(hsl(215.4deg 16.3% 46.9%),
+        hsl(155deg 5% 68.3%));
     --popover: light-dark(hsl(0deg 0% 100%), hsl(151deg 5.5% 15.2%));
-    --popover-foreground: light-dark(
-      hsl(222.2deg 47.4% 11.2%),
-      hsl(155deg 5% 68.3%)
-    );
+    --popover-foreground: light-dark(hsl(222.2deg 47.4% 11.2%),
+        hsl(155deg 5% 68.3%));
     --card: light-dark(hsl(0deg 0% 100%), hsl(151deg 5.5% 15.2%));
-    --card-foreground: light-dark(
-      hsl(222.2deg 47.4% 11.2%),
-      hsl(155deg 5% 76.3%)
-    );
+    --card-foreground: light-dark(hsl(222.2deg 47.4% 11.2%),
+        hsl(155deg 5% 76.3%));
     --border: light-dark(hsl(214.3deg 31.8% 91.4%), hsl(153deg 3.7% 24.2%));
     --input: light-dark(hsl(0deg 0% 64%), hsl(154deg 3.3% 28.7%));
     --primary: light-dark(hsl(208deg 93.5% 47.4%), hsl(192deg 59.8% 39%));
     --primary-foreground: light-dark(hsl(210deg 40% 98%), hsl(190deg 80% 84%));
     --secondary: light-dark(hsl(210deg 40% 96.1%), hsl(155deg 7% 93%));
-    --secondary-foreground: light-dark(
-      hsl(222.2deg 47.4% 11.2%),
-      hsl(151deg 5.5% 15.2%)
-    );
+    --secondary-foreground: light-dark(hsl(222.2deg 47.4% 11.2%),
+        hsl(151deg 5.5% 15.2%));
     --accent: light-dark(hsl(209deg 100% 96.5%), hsl(192deg 56.6% 26.5%));
     --accent-foreground: light-dark(hsl(211deg 90% 42%), hsl(190deg 80% 84%));
     --ring: hsl(215deg 20.2% 65.1%);
@@ -105,10 +97,8 @@
 
     /* Base shadows. */
     --base-shadow: light-dark(hsl(0deg 0% 85% / 40%), hsla(0deg 0% 36% / 60%));
-    --base-shadow-darker: light-dark(
-      hsl(0deg 0% 50% / 40%),
-      hsl(0deg 0% 50% / 60%)
-    );
+    --base-shadow-darker: light-dark(hsl(0deg 0% 50% / 40%),
+        hsl(0deg 0% 50% / 60%));
     --base-shadow-opacity: 5%;
 
     /* Codemirror editor */
@@ -162,7 +152,14 @@
     animation-duration: 0.01ms !important;
     animation-delay: 0s !important;
     animation-iteration-count: 1 !important;
-    transition-duration: 0.01ms !important;
+
+    /*
+    Set to 0s, because `transition-property` defaults to `all`, so any non-zero
+    duration gives every element a transition. Libraries that change an
+    attribute and measure synchronously (e.g. Plotly hover labels) would read
+    stale geometry.
+    */
+    transition-duration: 0s !important;
     transition-delay: 0s !important;
   }