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

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

@@ -8,24 +8,24 @@ vi.mock("@/utils/Logger", () => ({
 }));
 
 class FakeWebSocket {
-  static readonly OPEN = 1;
-  static readonly CLOSED = 3;
-  static readonly instances: FakeWebSocket[] = [];
+  public static readonly OPEN = 1;
+  public static readonly CLOSED = 3;
+  public static readonly instances: FakeWebSocket[] = [];
 
-  readonly sent: string[] = [];
-  readyState = 0;
-  readonly url: string;
+  public readonly sent: string[] = [];
+  public readyState = 0;
+  public readonly url: string;
   private readonly listeners = new Map<
     string,
     Set<(event: Record<string, never>) => void>
   >();
 
-  constructor(url: string) {
+  public constructor(url: string) {
     this.url = url;
     FakeWebSocket.instances.push(this);
   }
 
-  addEventListener(
+  public addEventListener(
     type: string,
     listener: (event: Record<string, never>) => void,
   ) {
@@ -34,21 +34,21 @@ class FakeWebSocket {
     this.listeners.set(type, listeners);
   }
 
-  send(frame: string) {
+  public send(frame: string) {
     this.sent.push(frame);
   }
 
-  close() {
+  public close() {
     this.readyState = FakeWebSocket.CLOSED;
     this.emit("close");
   }
 
-  open() {
+  public open() {
     this.readyState = FakeWebSocket.OPEN;
     this.emit("open");
   }
 
-  fail() {
+  public fail() {
     this.emit("error");
   }
 

package/src/core/codemirror/copilot/language-server.ts

@@ -117,7 +117,7 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
   private lastOpenedDocument: DidOpenTextDocumentParams | undefined;
   private copilotSettings: Record<string, unknown> = {};
 
-  constructor(
+  public constructor(
     options: ConstructorParameters<typeof LanguageServerClient>[0] & {
       copilotSettings?: Record<string, unknown>;
     },
@@ -140,7 +140,7 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
    * Re-run the LSP initialize handshake and send configuration.
    * Called by the transport's onReconnect callback after reconnecting.
    */
-  async reInitialize(): Promise<void> {
+  public async reInitialize(): Promise<void> {
     logger.log("#reInitialize: Re-initializing LSP connection");
     this.initializePromise = this.initialize();
     await this.initializePromise;
@@ -186,7 +186,7 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
     return this.notify(method, params);
   }
 
-  override getInitializationOptions() {
+  public override getInitializationOptions() {
     const info = {
       name: "marimo",
       version: "0.1.0",
@@ -208,7 +208,7 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
     return !isCopilotEnabled();
   }
 
-  override async textDocumentDidOpen(
+  public override async textDocumentDidOpen(
     params: DidOpenTextDocumentParams,
   ): Promise<boolean> {
     if (this.isDisabled()) {
@@ -223,7 +223,7 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
     return super.textDocumentDidOpen(params);
   }
 
-  override async textDocumentDidClose(
+  public override async textDocumentDidClose(
     params: DidCloseTextDocumentParams,
   ): Promise<void> {
     if (this.openDocumentCount === 0) {
@@ -233,14 +233,14 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
     return super.textDocumentDidClose(params);
   }
 
-  override async textDocumentCompletion(
+  public override async textDocumentCompletion(
     params: CompletionParams,
   ): Promise<CompletionList | CompletionItem[]> {
     // Not used in Copilot
     return [];
   }
 
-  override async textDocumentDidChange(
+  public override async textDocumentDidChange(
     params: DidChangeTextDocumentParams,
   ): Promise<void> {
     if (this.isDisabled()) {
@@ -304,17 +304,17 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
     });
   }
 
-  override textDocumentHover(params: HoverParams): Promise<Hover> {
+  public override textDocumentHover(params: HoverParams): Promise<Hover> {
     // Not used in Copilot
     return Promise.resolve({ contents: [] });
   }
 
   // AUTH
-  signOut() {
+  public signOut() {
     return this._request("signOut", {});
   }
 
-  async signInInitiate() {
+  public async signInInitiate() {
     logger.log("#signInInitiate: Starting sign-in flow");
     try {
       const result = await this._request("signIn", {});
@@ -326,7 +326,7 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
     }
   }
 
-  async signInConfirm(params: GitHubCopilotSignInConfirmParams) {
+  public async signInConfirm(params: GitHubCopilotSignInConfirmParams) {
     logger.log("#signInConfirm: Confirming sign-in");
     try {
       const result = await this._request("signInConfirm", params);
@@ -338,7 +338,7 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
     }
   }
 
-  async signedIn() {
+  public async signedIn() {
     try {
       const { status } = await this._request("checkStatus", {});
       logger.log("#checkStatus: Status check completed", { status });
@@ -375,7 +375,7 @@ export class CopilotLanguageServerClient extends LanguageServerClient {
     200,
   );
 
-  async getCompletion(
+  public async getCompletion(
     params: InlineCompletionParams,
   ): Promise<InlineCompletionList | InlineCompletionItem[] | null> {
     if (this.isDisabled()) {

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

@@ -38,7 +38,7 @@ export interface LazyWebsocketTransportOptions {
  * Copilot-specific readiness checks, retries, and user-facing errors.
  */
 export class LazyWebsocketTransport extends ReconnectingWebSocketTransport {
-  constructor(options: LazyWebsocketTransportOptions) {
+  public constructor(options: LazyWebsocketTransportOptions) {
     super({
       getWsUrl: options.getWsUrl,
       waitForConnection: options.waitForReady,

package/src/core/codemirror/find-replace/search-highlight.ts

@@ -90,15 +90,15 @@ const HighlightMargin = 250;
 
 export const searchHighlighter = ViewPlugin.fromClass(
   class {
-    decorations: DecorationSet;
-    readonly view: EditorView;
+    public decorations: DecorationSet;
+    public readonly view: EditorView;
 
-    constructor(view: EditorView) {
+    public constructor(view: EditorView) {
       this.view = view;
       this.decorations = this.highlight(view.state.field(searchState));
     }
 
-    update(update: ViewUpdate) {
+    public update(update: ViewUpdate) {
       const state = update.state.field(searchState);
       if (
         state !== update.startState.field(searchState) ||
@@ -110,7 +110,7 @@ export const searchHighlighter = ViewPlugin.fromClass(
       }
     }
 
-    highlight(query: QueryType) {
+    public highlight(query: QueryType) {
       if (!query.spec.valid) {
         return Decoration.none;
       }

package/src/core/codemirror/go-to-definition/underline.ts

@@ -57,7 +57,7 @@ class MetaUnderlineVariablePlugin {
   private hoveredRange: { from: number; to: number; position: number } | null;
   private onClick: (view: EditorView) => void;
 
-  constructor(view: EditorView, onClick: (view: EditorView) => void) {
+  public constructor(view: EditorView, onClick: (view: EditorView) => void) {
     this.view = view;
     this.commandClickMode = false;
     this.hoveredRange = null;
@@ -69,13 +69,13 @@ class MetaUnderlineVariablePlugin {
     window.addEventListener("mouseleave", this.windowBlur);
   }
 
-  update(_update: ViewUpdate) {
+  public update(_update: ViewUpdate) {
     // We cannot add any transactions here (e.g. clearing underlines),
     // otherwise CM fails with
     // "Calls to EditorView.update are not allowed while an update is in progress"
   }
 
-  destroy() {
+  public destroy() {
     window.removeEventListener("keydown", this.keydown);
     window.removeEventListener("keyup", this.keyup);
     window.removeEventListener("blur", this.windowBlur);

package/src/core/codemirror/keymaps/vim.ts

@@ -346,7 +346,7 @@ class CodeMirrorVimSync {
     // noop
   }
 
-  addInstance(instance: EditorView) {
+  public addInstance(instance: EditorView) {
     this.instances.add(instance);
 
     const cm = getCM(instance);
@@ -376,11 +376,11 @@ class CodeMirrorVimSync {
     });
   }
 
-  removeInstance(instance: EditorView) {
+  public removeInstance(instance: EditorView) {
     this.instances.delete(instance);
   }
 
-  broadcastModeChange(
+  public broadcastModeChange(
     originInstance: EditorView,
     mode: string,
     subMode?: string,

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

@@ -41,23 +41,23 @@ export const MARKDOWN_INITIAL_HIDE_CODE = true;
 export class MarkdownLanguageAdapter implements LanguageAdapter<MarkdownLanguageAdapterMetadata> {
   private parser = new MarkdownParser();
 
-  readonly type = "markdown";
-  readonly defaultCode = this.parser.defaultCode;
-  readonly defaultMetadata: MarkdownLanguageAdapterMetadata =
+  public readonly type = "markdown";
+  public readonly defaultCode = this.parser.defaultCode;
+  public readonly defaultMetadata: MarkdownLanguageAdapterMetadata =
     this.parser.defaultMetadata;
 
-  static fromMarkdown(markdown: string) {
+  public static fromMarkdown(markdown: string) {
     return MarkdownParser.fromMarkdown(markdown);
   }
 
-  transformIn(
+  public transformIn(
     pythonCode: string,
   ): [string, number, MarkdownLanguageAdapterMetadata] {
     const result = this.parser.transformIn(pythonCode);
     return [result.code, result.offset, result.metadata];
   }
 
-  transformOut(
+  public transformOut(
     code: string,
     metadata: MarkdownLanguageAdapterMetadata,
   ): [string, number] {
@@ -65,11 +65,11 @@ export class MarkdownLanguageAdapter implements LanguageAdapter<MarkdownLanguage
     return [result.code, result.offset];
   }
 
-  isSupported(pythonCode: string): boolean {
+  public isSupported(pythonCode: string): boolean {
     return this.parser.isSupported(pythonCode);
   }
 
-  getExtension(
+  public getExtension(
     _cellId: CellId,
     _completionConfig: CompletionConfig,
     hotkeys: HotkeyProvider,

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

@@ -261,25 +261,25 @@ const pyrightClient = once((_: LSPConfig) => {
 export class PythonLanguageAdapter implements LanguageAdapter<{}> {
   private parser = new PythonParser();
 
-  readonly type = "python";
-  readonly defaultCode = this.parser.defaultCode;
-  readonly defaultMetadata = this.parser.defaultMetadata;
+  public readonly type = "python";
+  public readonly defaultCode = this.parser.defaultCode;
+  public readonly defaultMetadata = this.parser.defaultMetadata;
 
-  transformIn(code: string): [string, number, {}] {
+  public transformIn(code: string): [string, number, {}] {
     const result = this.parser.transformIn(code);
     return [result.code, result.offset, result.metadata];
   }
 
-  transformOut(code: string, metadata: {}): [string, number] {
+  public transformOut(code: string, metadata: {}): [string, number] {
     const result = this.parser.transformOut(code, metadata);
     return [result.code, result.offset];
   }
 
-  isSupported(code: string): boolean {
+  public isSupported(code: string): boolean {
     return this.parser.isSupported(code);
   }
 
-  getExtension(
+  public getExtension(
     cellId: CellId,
     completionConfig: CompletionConfig,
     hotkeys: HotkeyProvider,

package/src/core/codemirror/language/languages/sql/completion-builder.ts

@@ -25,7 +25,7 @@ export class CompletionBuilder {
   /**
    * Add a table with its columns at the specified path
    */
-  addTable(path: string[], table: DataTable): this {
+  public addTable(path: string[], table: DataTable): this {
     const tableNamespace: SQLNamespace = {
       self: tableToCompletion({
         table: table,
@@ -44,7 +44,7 @@ export class CompletionBuilder {
   /**
    * Add a schema at the specified path
    */
-  addSchema(path: string[], schema: DatabaseSchema): this {
+  public addSchema(path: string[], schema: DatabaseSchema): this {
     const schemaObject: SQLNamespace = {
       self: schemaToCompletion({
         namespace: schema,
@@ -60,7 +60,7 @@ export class CompletionBuilder {
   /**
    * Add a database at the specified path
    */
-  addDatabase(path: string[], database: Database): this {
+  public addDatabase(path: string[], database: Database): this {
     const databaseObject: SQLNamespace = {
       self: databaseToCompletion({
         namespace: database,
@@ -90,14 +90,14 @@ export class CompletionBuilder {
   /**
    * Build the final schema
    */
-  build(): SQLNamespace {
+  public build(): SQLNamespace {
     return this.schema;
   }
 
   /**
    * Reset for reuse
    */
-  reset(): this {
+  public reset(): this {
     this.schema = {};
     return this;
   }

package/src/core/codemirror/language/languages/sql/completion-store.ts

@@ -28,7 +28,7 @@ const datasetTableCompletionsAtom = atom((get) => {
 class SQLCompletionStore {
   private cache: LRUCache<DataSourceConnection, CachedSchema>;
 
-  constructor() {
+  public constructor() {
     this.cache = new LRUCache(10, {
       create: (connection) => this.getConnectionSchema(connection),
     });
@@ -121,7 +121,7 @@ class SQLCompletionStore {
    * Returns the raw dialect of the connection passed from the backend,
    * or null if the connection is not found
    */
-  getInternalDialect(connectionName: ConnectionName): string | null {
+  public getInternalDialect(connectionName: ConnectionName): string | null {
     const connection = this.getConnection(connectionName);
     if (!connection) {
       return null;
@@ -133,7 +133,7 @@ class SQLCompletionStore {
    * Get the inferred SQL dialect for a connection
    * If the connection is not found, return the standard SQL dialect.
    */
-  getDialect(connectionName: ConnectionName): SQLDialect {
+  public getDialect(connectionName: ConnectionName): SQLDialect {
     const connection = this.getConnection(connectionName);
     if (!connection) {
       return ModifiedStandardSQL;
@@ -141,7 +141,7 @@ class SQLCompletionStore {
     return guessDialect(connection);
   }
 
-  getCompletionSource(connectionName: ConnectionName): SQLConfig | null {
+  public getCompletionSource(connectionName: ConnectionName): SQLConfig | null {
     const connection = this.getConnection(connectionName);
     if (!connection) {
       return null;

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

@@ -99,21 +99,21 @@ function getLatestEngine(): ConnectionName {
  */
 export class SQLLanguageAdapter implements LanguageAdapter<SQLLanguageAdapterMetadata> {
   private parser = new SQLParser();
-  readonly type = "sql";
-  sqlModeEnabled: boolean;
+  public readonly type = "sql";
+  public sqlModeEnabled: boolean;
 
-  constructor() {
+  public constructor() {
     this.sqlModeEnabled = true;
   }
 
-  get defaultMetadata(): SQLLanguageAdapterMetadata {
+  public get defaultMetadata(): SQLLanguageAdapterMetadata {
     return {
       ...this.parser.defaultMetadata,
       engine: getLatestEngine() || DUCKDB_ENGINE,
     };
   }
 
-  get defaultCode(): string {
+  public get defaultCode(): string {
     const engine = getLatestEngine();
     if (engine && engine !== DUCKDB_ENGINE) {
       return `_df = mo.sql(f"""SELECT * FROM """, engine=${engine})`;
@@ -121,9 +121,9 @@ export class SQLLanguageAdapter implements LanguageAdapter<SQLLanguageAdapterMet
     return this.parser.defaultCode;
   }
 
-  static fromQuery = (query: string) => SQLParser.fromQuery(query);
+  public static fromQuery = (query: string) => SQLParser.fromQuery(query);
 
-  transformIn(
+  public transformIn(
     pythonCode: string,
   ): [
     sqlQuery: string,
@@ -143,7 +143,7 @@ export class SQLLanguageAdapter implements LanguageAdapter<SQLLanguageAdapterMet
     return [result.code, result.offset, metadata];
   }
 
-  transformOut(
+  public transformOut(
     code: string,
     metadata: SQLLanguageAdapterMetadata,
   ): [string, number] {
@@ -151,11 +151,11 @@ export class SQLLanguageAdapter implements LanguageAdapter<SQLLanguageAdapterMet
     return [result.code, result.offset];
   }
 
-  isSupported(pythonCode: string): boolean {
+  public isSupported(pythonCode: string): boolean {
     return this.parser.isSupported(pythonCode);
   }
 
-  getExtension(
+  public getExtension(
     _cellId: CellId,
     _completionConfig: CompletionConfig,
     _hotkeys: HotkeyProvider,
@@ -309,19 +309,19 @@ class DialectAwareQueryContextAnalyzer extends QueryContextAnalyzer {
   private readonly analyzers = new Map<ParserDialects, QueryContextAnalyzer>();
   private readonly analysisParser: NodeSqlParser;
 
-  constructor(analysisParser: NodeSqlParser) {
+  public constructor(analysisParser: NodeSqlParser) {
     super(analysisParser);
     this.analysisParser = analysisParser;
   }
 
-  override getContext(
+  public override getContext(
     sql: string,
     opts: { state: EditorState },
   ): ReturnType<QueryContextAnalyzer["getContext"]> {
     return this.getAnalyzer(opts.state).getContext(sql, opts);
   }
 
-  override clearCache(): void {
+  public override clearCache(): void {
     for (const analyzer of this.analyzers.values()) {
       analyzer.clearCache();
     }
@@ -342,25 +342,25 @@ class DialectAwareSqlStructureAnalyzer extends SqlStructureAnalyzer {
   private readonly analyzers = new Map<ParserDialects, SqlStructureAnalyzer>();
   private readonly analysisParser: NodeSqlParser;
 
-  constructor(analysisParser: NodeSqlParser) {
+  public constructor(analysisParser: NodeSqlParser) {
     super(analysisParser);
     this.analysisParser = analysisParser;
   }
 
-  override analyzeDocument(
+  public override analyzeDocument(
     state: EditorState,
   ): ReturnType<SqlStructureAnalyzer["analyzeDocument"]> {
     return this.getAnalyzer(state).analyzeDocument(state);
   }
 
-  override getStatementAtPosition(
+  public override getStatementAtPosition(
     state: EditorState,
     position: number,
   ): ReturnType<SqlStructureAnalyzer["getStatementAtPosition"]> {
     return this.getAnalyzer(state).getStatementAtPosition(state, position);
   }
 
-  override getStatementsInRange(
+  public override getStatementsInRange(
     state: EditorState,
     from: number,
     to: number,
@@ -368,7 +368,7 @@ class DialectAwareSqlStructureAnalyzer extends SqlStructureAnalyzer {
     return this.getAnalyzer(state).getStatementsInRange(state, from, to);
   }
 
-  override clearCache(): void {
+  public override clearCache(): void {
     for (const analyzer of this.analyzers.values()) {
       analyzer.clearCache();
     }
@@ -392,7 +392,7 @@ class CustomSqlParser extends NodeSqlParser {
   private readonly VALIDATION_DELAY_MS = 300; // Wait 300ms after user stops typing
   private isFocused = false; // Only validate if the editor is focused
 
-  setFocusState(focused: boolean) {
+  public setFocusState(focused: boolean) {
     this.isFocused = focused;
   }
 
@@ -446,7 +446,7 @@ class CustomSqlParser extends NodeSqlParser {
     });
   }
 
-  override async validateSql(
+  public override async validateSql(
     sql: string,
     opts: { state: EditorState },
   ): Promise<SqlParseError[]> {
@@ -470,7 +470,7 @@ class CustomSqlParser extends NodeSqlParser {
     );
   }
 
-  override async parse(
+  public override async parse(
     sql: string,
     opts: { state: EditorState },
   ): Promise<NodeSqlParserResult> {

package/src/core/codemirror/lsp/__tests__/transports.test.ts

@@ -17,38 +17,38 @@ vi.mock("../../../runtime/config", () => ({
 
 /** Minimal stand-in for the browser WebSocket used by `WebSocketTransport`. */
 class FakeWebSocket {
-  static readonly instances: FakeWebSocket[] = [];
-  static readonly OPEN = 1;
+  public static readonly instances: FakeWebSocket[] = [];
+  public static readonly OPEN = 1;
 
-  readyState = 0;
-  readonly url: string;
+  public readyState = 0;
+  public readonly url: string;
   private readonly listeners = new Map<string, Set<(event: unknown) => void>>();
 
-  constructor(url: string) {
+  public constructor(url: string) {
     this.url = url;
     FakeWebSocket.instances.push(this);
   }
 
-  addEventListener(type: string, listener: (event: unknown) => void) {
+  public addEventListener(type: string, listener: (event: unknown) => void) {
     const listeners = this.listeners.get(type) ?? new Set();
     listeners.add(listener);
     this.listeners.set(type, listeners);
   }
 
-  send() {
+  public send() {
     // no-op
   }
 
-  close() {
+  public close() {
     this.readyState = 3;
   }
 
-  open() {
+  public open() {
     this.readyState = FakeWebSocket.OPEN;
     this.emit("open", {});
   }
 
-  receive(message: unknown) {
+  public receive(message: unknown) {
     this.emit("message", { data: JSON.stringify(message) });
   }