Explain target module name

Author: Luca Forstner

packages/nextjs/rollup.npm.config.js

@@ -29,7 +29,7 @@ export default [
           // make it so Rollup calms down about the fact that we're combining default and named exports
           exports: 'named',
         },
-        external: ['@sentry/nextjs', '__SENTRY_WRAPEE__.cjs'],
+        external: ['@sentry/nextjs', '__SENTRY_WRAPPING_TARGET__'],
       },
     }),
   ),

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

@@ -7,8 +7,12 @@ import { rollup } from 'rollup';
 
 import { LoaderThis } from './types';
 
+// Just a simple placeholder to make referencing module consistent
 const SENTRY_WRAPPER_MODULE_NAME = 'sentry-wrapper-module';
-const WRAPPING_TARGET_MODULE_NAME = '__SENTRY_WRAPEE__.cjs';
+
+// needs to end in .cjs in order for the `commonjs` plugin to pick it up - unfortunately the plugin has no option to
+// make this work in combination with the`virtual` plugin
+const WRAPPING_TARGET_MODULE_NAME = '__SENTRY_WRAPPING_TARGET__.cjs';
 
 type LoaderOptions = {
   pagesDir: string;
@@ -59,6 +63,9 @@ export default async function proxyLoader(this: LoaderThis<LoaderOptions>, userC
   // Inject the route and the path to the file we're wrapping into the template
   templateCode = templateCode.replace(/__ROUTE__/g, parameterizedRoute.replace(/\\/g, '\\\\'));
 
+  // Replace the import path of the wrapping target in the template with a path that the `wrapUserCode` function will understand.
+  templateCode = templateCode.replace(/__SENTRY_WRAPPING_TARGET__/g, WRAPPING_TARGET_MODULE_NAME);
+
   // Run the proxy module code through Rollup, in order to split the `export * from '<wrapped file>'` out into
   // individual exports (which nextjs seems to require).
   try {

packages/nextjs/src/config/templates/apiWrapperTemplate.ts

@@ -8,7 +8,7 @@
 
 // @ts-ignore See above
 // eslint-disable-next-line import/no-unresolved
-import * as origModule from '__SENTRY_WRAPEE__.cjs';
+import * as origModule from '__SENTRY_WRAPPING_TARGET__';
 import * as Sentry from '@sentry/nextjs';
 import type { PageConfig } from 'next';
 
@@ -73,4 +73,4 @@ export default exportedHandler;
 // not include anything whose name matchs something we've explicitly exported above.
 // @ts-ignore See above
 // eslint-disable-next-line import/no-unresolved
-export * from '__SENTRY_WRAPEE__.cjs';
+export * from '__SENTRY_WRAPPING_TARGET__';

packages/nextjs/src/config/templates/pageWrapperTemplate.ts

@@ -8,7 +8,7 @@
 
 // @ts-ignore See above
 // eslint-disable-next-line import/no-unresolved
-import * as wrapee from '__SENTRY_WRAPEE__.cjs';
+import * as wrapee from '__SENTRY_WRAPPING_TARGET__';
 import * as Sentry from '@sentry/nextjs';
 import type { GetServerSideProps, GetStaticProps, NextPage as NextPageComponent } from 'next';
 
@@ -53,4 +53,4 @@ export default pageComponent;
 // not include anything whose name matchs something we've explicitly exported above.
 // @ts-ignore See above
 // eslint-disable-next-line import/no-unresolved
-export * from '__SENTRY_WRAPEE__.cjs';
+export * from '__SENTRY_WRAPPING_TARGET__';