Rebase changes to makeAST onto #5546

Author: Luca Forstner

packages/nextjs/src/config/loaders/dataFetchersLoader.ts

@@ -123,7 +123,7 @@ export default function wrapDataFetchersLoader(this: LoaderThis<LoaderOptions>,
   // we know we're in a proxied file and do not need to proxy again.
 
   if (!this.resourceQuery.includes('sentry-proxy-loader')) {
-    const ast = makeAST(userCode, true); // is there a reason to ever parse without typescript?
+    const ast = makeAST(userCode); // is there a reason to ever parse without typescript?
 
     const exportedIdentifiers = getExportIdentifierNames(ast);
 

packages/nextjs/test/config/ast.test.ts

@@ -35,7 +35,7 @@ test.each([
   ['export { default } from "module-name";', true],
   ['export { default, name1 } from "module-name";', true],
 ])('hasDefaultExport(%s) should return %p', (program, expectedResult) => {
-  const ast = makeAST(program, true);
+  const ast = makeAST(program);
   expect(hasDefaultExport(ast)).toBe(expectedResult);
 });
 
@@ -95,6 +95,6 @@ test.each([
   ['export { default } from "module-name";', []],
   ['export { default, name1 } from "module-name";', ['name1']],
 ])('getExportIdentifiers(%s) should return %p', (program, expectedIdentifiers) => {
-  const ast = makeAST(program, true);
+  const ast = makeAST(program);
   expect(getExportIdentifierNames(ast)).toStrictEqual(expectedIdentifiers);
 });