Remove memoizeOne dependency

fiberplane · Oct 18, 2024 · 328241e · 328241e
1 parent b101f82
commit 328241e
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 11 deletions.
diff --git a/packages/types/memoize-one.d.ts b/packages/types/memoize-one.d.ts
diff --git a/packages/types/package.json b/packages/types/package.json
@@ -15,7 +15,6 @@
     "build": "tsc"
   },
   "devDependencies": {
-    "zod": "^3.23.8",
-    "memoize-one": "^6.0.0"
+    "zod": "^3.23.8"
   }
 }
diff --git a/packages/types/src/schemas.ts b/packages/types/src/schemas.ts
@@ -7,7 +7,13 @@
 // =========================================== //
 
 import { z } from "zod";
-import memoizeOne from "memoize-one";
+
+// TODO: Polyfill memoizeOne for now, seems like this breaks our TypeScript setup
+// See: https://github.com/alexreardon/memoize-one/issues/267
+// Couldn't get it to work reliably on both the CI and locally with custom module type definitions
+function memoizeOne<T>(schema: T) {
+  return schema;
+}
 
 export const AppStateSchema = memoizeOne(() =>
   z.object({ workspace: z.union([z.lazy(WorkspaceSchema), z.null()]) }),

diff --git a/xtask/src/commands/schemas.rs b/xtask/src/commands/schemas.rs
@@ -80,7 +80,13 @@ fn generate_zod_schemas(schemas: Vec<RootSchema>) -> Result<Vec<u8>> {
 // =========================================== //
 
 import { z } from "zod";
-import memoizeOne from "memoize-one";
+
+// TODO: Polyfill memoizeOne for now, seems like this breaks our TypeScript setup
+// See: https://github.com/alexreardon/memoize-one/issues/267
+// Couldn't get it to work reliably on both the CI and locally with custom module type definitions
+function memoizeOne<T>(schema: T) {
+  return schema;
+}
     "#,
     );