@marimo-team/frontend 0.16.0-dev96986 → 0.16.0

package/src/utils/localStorage.ts

@@ -12,10 +12,7 @@ interface Storage<T> {
 }
 
 export class TypedLocalStorage<T> implements Storage<T> {
-  private defaultValue: T;
-  constructor(defaultValue: T) {
-    this.defaultValue = defaultValue;
-  }
+  constructor(private defaultValue: T) {}
 
   get(key: string): T {
     try {
@@ -36,16 +33,10 @@ export class TypedLocalStorage<T> implements Storage<T> {
 }
 
 export class ZodLocalStorage<T> implements Storage<T> {
-  private schema: ZodType<T, ZodTypeDef, unknown>;
-  private getDefaultValue: () => T;
-
   constructor(
-    schema: ZodType<T, ZodTypeDef, unknown>,
-    getDefaultValue: () => T,
-  ) {
-    this.schema = schema;
-    this.getDefaultValue = getDefaultValue;
-  }
+    private schema: ZodType<T, ZodTypeDef, unknown>,
+    private getDefaultValue: () => T,
+  ) {}
 
   get(key: string): T {
     try {

package/src/utils/numbers.ts

@@ -1,6 +1,6 @@
 /* Copyright 2024 Marimo. All rights reserved. */
 
-export function prettyNumber(value: unknown, locale: string): string {
+export function prettyNumber(value: unknown): string {
   if (value === undefined || value === null) {
     return "";
   }
@@ -18,7 +18,7 @@ export function prettyNumber(value: unknown, locale: string): string {
   }
 
   if (typeof value === "number" || typeof value === "bigint") {
-    return value.toLocaleString(locale, {
+    return value.toLocaleString(undefined, {
       minimumFractionDigits: 0,
       maximumFractionDigits: 2,
     });
@@ -45,9 +45,9 @@ function scientificSpecialCase(value: number): string | null {
 export function prettyScientificNumber(
   value: number,
   opts: {
-    shouldRound?: boolean; // Default to false
-    locale: string;
-  },
+    // Default to false
+    shouldRound?: boolean;
+  } = {},
 ): string {
   // Handle special cases first
   const specialCase = scientificSpecialCase(value);
@@ -58,7 +58,7 @@ export function prettyScientificNumber(
   // Determine if the number should be in scientific notation
   const absValue = Math.abs(value);
   if (absValue < 1e-2 || absValue >= 1e6) {
-    return new Intl.NumberFormat(opts.locale, {
+    return new Intl.NumberFormat(undefined, {
       minimumFractionDigits: 1,
       maximumFractionDigits: 1,
       notation: "scientific",
@@ -67,18 +67,18 @@ export function prettyScientificNumber(
       .toLowerCase();
   }
 
-  const { shouldRound, locale } = opts;
+  const { shouldRound } = opts;
 
   if (shouldRound) {
     // Number has an integer part, format with 2 decimal places
-    return new Intl.NumberFormat(locale, {
+    return new Intl.NumberFormat(undefined, {
       minimumFractionDigits: 0,
       maximumFractionDigits: 2,
     }).format(value);
   }
 
   // Don't round
-  return value.toLocaleString(locale, {
+  return value.toLocaleString(undefined, {
     minimumFractionDigits: 0,
     maximumFractionDigits: 100,
   });
@@ -104,14 +104,14 @@ const prefixes = {
   "-24": "y",
 };
 
-export function prettyEngineeringNumber(value: number, locale: string): string {
+export function prettyEngineeringNumber(value: number): string {
   // Handle special cases first
   const specialCase = scientificSpecialCase(value);
   if (specialCase !== null) {
     return specialCase;
   }
 
-  const [mant, exp] = new Intl.NumberFormat(locale, {
+  const [mant, exp] = new Intl.NumberFormat("en-us", {
     notation: "engineering",
     maximumSignificantDigits: 3,
   })

package/src/utils/once.ts

@@ -1,7 +1,5 @@
 /* Copyright 2024 Marimo. All rights reserved. */
 
-import { arrayShallowEquals } from "./arrays";
-
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function once<T extends (...args: any[]) => any>(fn: T): T {
   let result: ReturnType<T>;
@@ -17,33 +15,3 @@ export function once<T extends (...args: any[]) => any>(fn: T): T {
     return result;
   } as T;
 }
-
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-export function memoizeLastValue<T extends (...args: any[]) => any>(fn: T): T {
-  let result: ReturnType<T>;
-  let lastArgs: Parameters<T> | undefined;
-  let lastError: unknown;
-  let hasError = false;
-
-  return function (
-    this: ThisParameterType<T>,
-    ...args: Parameters<T>
-  ): ReturnType<T> {
-    if (lastArgs === undefined || !arrayShallowEquals(args, lastArgs)) {
-      try {
-        result = fn.apply(this, args) as ReturnType<T>;
-        hasError = false;
-        lastError = undefined;
-      } catch (error) {
-        hasError = true;
-        lastError = error;
-      }
-      lastArgs = args;
-    }
-
-    if (hasError) {
-      throw lastError;
-    }
-    return result;
-  } as T;
-}

package/src/utils/paths.ts

@@ -32,10 +32,7 @@ export const Paths = {
 };
 
 export class PathBuilder {
-  public readonly deliminator: string;
-  constructor(deliminator: "/" | "\\") {
-    this.deliminator = deliminator;
-  }
+  constructor(public readonly deliminator: "/" | "\\") {}
 
   static guessDeliminator(path: string): PathBuilder {
     return path.includes("/") ? new PathBuilder("/") : new PathBuilder("\\");

package/src/utils/pluralize.ts

@@ -1,13 +1,10 @@
 /* Copyright 2024 Marimo. All rights reserved. */
 
 export class PluralWord {
-  public singular: string;
-  public _plural?: string;
-
-  constructor(singular: string, _plural?: string) {
-    this.singular = singular;
-    this._plural = _plural;
-  }
+  constructor(
+    public singular: string,
+    public _plural?: string,
+  ) {}
 
   public pluralize(count: number) {
     return count === 1 ? this.singular : this.plural;
@@ -19,11 +16,7 @@ export class PluralWord {
 }
 
 export class PluralWords {
-  private words: PluralWord[];
-
-  constructor(words: PluralWord[]) {
-    this.words = words;
-  }
+  constructor(private words: PluralWord[]) {}
 
   static of(...words: Array<PluralWord | string>) {
     return new PluralWords(

package/src/utils/python-poet/poet.ts

@@ -25,10 +25,7 @@ function asString(value: string | PythonCode): string {
 }
 
 export class Variable implements PythonCode {
-  name: string;
-  constructor(name: string) {
-    this.name = name;
-  }
+  constructor(public name: string) {}
 
   toCode(): string {
     return this.name;
@@ -41,12 +38,10 @@ interface LiteralOptions {
 }
 
 export class Literal implements PythonCode {
-  public readonly value: unknown;
-  public readonly opts: LiteralOptions;
-  constructor(value: unknown, opts: LiteralOptions = {}) {
-    this.value = value;
-    this.opts = opts;
-  }
+  constructor(
+    public readonly value: unknown,
+    public readonly opts: LiteralOptions = {},
+  ) {}
 
   static from(value: unknown, opts: LiteralOptions = {}): Literal {
     return new Literal(value, opts);
@@ -119,13 +114,10 @@ export class Literal implements PythonCode {
 }
 
 export class VariableDeclaration implements PythonCode {
-  public name: string;
-  public value: string | PythonCode;
-
-  constructor(name: string, value: string | PythonCode) {
-    this.name = name;
-    this.value = value;
-  }
+  constructor(
+    public name: string,
+    public value: string | PythonCode,
+  ) {}
 
   toCode(): string {
     const right = asString(this.value);
@@ -137,13 +129,10 @@ export class VariableDeclaration implements PythonCode {
 }
 
 export class FunctionArg implements PythonCode {
-  public name: string;
-  public value: string | PythonCode;
-
-  constructor(name: string, value: string | PythonCode) {
-    this.name = name;
-    this.value = value;
-  }
+  constructor(
+    public name: string,
+    public value: string | PythonCode,
+  ) {}
 
   toCode(): string {
     return `${this.name}=${asString(this.value)}`;
@@ -152,16 +141,12 @@ export class FunctionArg implements PythonCode {
 
 export class FunctionCall implements PythonCode {
   private args: PythonCode[];
-  public name: string;
-  public multiLine: boolean;
 
   constructor(
-    name: string,
+    public name: string,
     args: PythonCode[] | Record<string, PythonCode>,
-    multiLine = false,
+    public multiLine = false,
   ) {
-    this.name = name;
-    this.multiLine = multiLine;
     this.args = objectToArgs(args);
   }
 

package/src/utils/time.ts

@@ -5,8 +5,6 @@ export type Milliseconds = number & { __type__: "milliseconds" };
 export type Seconds = number & { __type__: "seconds" };
 
 export class Time {
-  private readonly ms: Milliseconds;
-
   static fromMilliseconds(ms: Milliseconds): Time;
   static fromMilliseconds(ms: Milliseconds | null): Time | null;
   static fromMilliseconds(ms: Milliseconds | null): Time | null {
@@ -29,9 +27,7 @@ export class Time {
     return new Time(Date.now() as Milliseconds);
   }
 
-  private constructor(ms: Milliseconds) {
-    this.ms = ms;
-  }
+  private constructor(private readonly ms: Milliseconds) {}
 
   toMilliseconds(): Milliseconds {
     return this.ms;