Remove build.minify: true, add v3_lazyRouteDiscovery (#68)

• build: { minify: true } causes variable names to be mangled in the SSR build of the app, which means that Models will have the wrong name and will wind up pointing at non-existent (or wrong) D1 tables • v3_lazyRouteDiscovery is a new default behavior for remix that doesn’t require any other changes to enable
jplhomer · Dec 17, 2024 · f12cb09 · f12cb09
1 parent c46c8d9
commit f12cb09
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/apps/site/vite.config.ts b/apps/site/vite.config.ts
@@ -11,6 +11,7 @@ export default defineConfig({
     remix({
       future: {
         v3_fetcherPersist: true,
+        v3_lazyRouteDiscovery: true,
         v3_relativeSplatPath: true,
         v3_throwAbortReason: true,
       },
@@ -26,7 +27,4 @@ export default defineConfig({
   resolve: {
     mainFields: ["browser", "module", "main"],
   },
-  build: {
-    minify: true,
-  },
 });
diff --git a/examples/remix-cms/vite.config.ts b/examples/remix-cms/vite.config.ts
@@ -10,6 +10,7 @@ export default defineConfig({
     remix({
       future: {
         v3_fetcherPersist: true,
+        v3_lazyRouteDiscovery: true,
         v3_relativeSplatPath: true,
         v3_throwAbortReason: true,
       },
@@ -34,7 +35,4 @@ export default defineConfig({
   resolve: {
     mainFields: ["browser", "module", "main"],
   },
-  build: {
-    minify: true,
-  },
 });
diff --git a/templates/remix/vite.config.ts b/templates/remix/vite.config.ts
@@ -9,6 +9,7 @@ export default defineConfig({
     remix({
       future: {
         v3_fetcherPersist: true,
+        v3_lazyRouteDiscovery: true,
         v3_relativeSplatPath: true,
         v3_throwAbortReason: true,
       },
@@ -23,7 +24,4 @@ export default defineConfig({
   resolve: {
     mainFields: ["browser", "module", "main"],
   },
-  build: {
-    minify: true,
-  },
 });