Remove nullish check from api.caller

nicolo-ribaudo · Sep 19, 2023 · e0cb7ba · e0cb7ba
1 parent 4bf5932
commit e0cb7ba
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/packages/babel-preset-env/src/index.ts b/packages/babel-preset-env/src/index.ts
@@ -355,7 +355,7 @@ option \`forceAllTransforms: true\` instead.
 
   const compatData = getPluginList(shippedProposals, bugfixes);
   const shouldSkipExportNamespaceFrom =
-    (modules === "auto" && api.caller?.(supportsExportNamespaceFrom)) ||
+    (modules === "auto" && api.caller(supportsExportNamespaceFrom)) ||
     (modules === false &&
       !isRequired("transform-export-namespace-from", transformTargets, {
         compatData,
@@ -365,11 +365,9 @@ option \`forceAllTransforms: true\` instead.
   const modulesPluginNames = getModulesPluginNames({
     modules,
     transformations: moduleTransformations,
-    // TODO: Remove the 'api.caller' check eventually. Just here to prevent
-    // unnecessary breakage in the short term for users on older betas/RCs.
-    shouldTransformESM: modules !== "auto" || !api.caller?.(supportsStaticESM),
+    shouldTransformESM: modules !== "auto" || !api.caller(supportsStaticESM),
     shouldTransformDynamicImport:
-      modules !== "auto" || !api.caller?.(supportsDynamicImport),
+      modules !== "auto" || !api.caller(supportsDynamicImport),
     shouldTransformExportNamespaceFrom: !shouldSkipExportNamespaceFrom,
   });