Make hoistConstantJsx as an option

lxsmnsyc · Aug 5, 2023 · 6068e69 · 6068e69
1 parent edf596c
commit 6068e69
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/packages/forgetti/src/core/presets.ts b/packages/forgetti/src/core/presets.ts
@@ -39,6 +39,9 @@ export interface Preset {
     hooks: HookRegistration[];
     hocs: ImportDefinition[];
   };
+  optimizations?: {
+    hoistConstantJsx?: boolean;
+  };
 }
 
 export interface Options {
@@ -125,6 +128,9 @@ export const PRESETS = {
         },
       ],
     },
+    optimizations: {
+      hoistConstantJsx: true,
+    },
   }),
   preact: createPreset({
     filters: {
@@ -219,5 +225,8 @@ export const PRESETS = {
         },
       ],
     },
+    optimizations: {
+      hoistConstantJsx: true,
+    },
   }),
 };
diff --git a/packages/forgetti/src/index.ts b/packages/forgetti/src/index.ts
@@ -275,13 +275,15 @@ export default function forgettiPlugin(): babel.PluginObj<State> {
           },
         };
 
-        // Register all import specifiers
         programPath.traverse({
+          // Register all import specifiers
           ImportDeclaration(path) {
             extractImportIdentifiers(ctx, path);
           },
           JSXElement(path) {
-            hoistConstantJSX(ctx, path);
+            if (preset.optimizations?.hoistConstantJsx) {
+              hoistConstantJSX(ctx, path);
+            }
           },
         });