ref(nextjs): Replace multiplexer with conditional exports (#11442)

Author: Luca Forstner

packages/nextjs/package.json

@@ -11,28 +11,48 @@
   },
   "main": "build/cjs/index.server.js",
   "module": "build/esm/index.server.js",
-  "browser": "build/esm/index.client.js",
   "types": "build/types/index.types.d.ts",
   "exports": {
     "./package.json": "./package.json",
     ".": {
+      "edge": {
+        "import": "./build/esm/edge/index.js",
+        "require": "./build/cjs/edge/index.js",
+        "default": "./build/esm/edge/index.js"
+      },
+      "edge-light": {
+        "import": "./build/esm/edge/index.js",
+        "require": "./build/cjs/edge/index.js",
+        "default": "./build/esm/edge/index.js"
+      },
+      "worker": {
+        "import": "./build/esm/edge/index.js",
+        "require": "./build/cjs/edge/index.js",
+        "default": "./build/esm/edge/index.js"
+      },
+      "workerd": {
+        "import": "./build/esm/edge/index.js",
+        "require": "./build/cjs/edge/index.js",
+        "default": "./build/esm/edge/index.js"
+      },
       "browser": {
         "import": "./build/esm/index.client.js",
         "require": "./build/cjs/index.client.js"
       },
       "node": "./build/cjs/index.server.js",
+      "import": "./build/esm/index.server.js",
       "types": "./build/types/index.types.d.ts"
     },
     "./register": {
       "import": {
         "default": "./build/register.mjs"
       }
     },
-"./hook": {
+    "./hook": {
       "import": {
         "default": "./build/hook.mjs"
       }
-}
+    }
   },
   "typesVersions": {
     "<4.9": {

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

@@ -1,4 +1,3 @@
 export { default as valueInjectionLoader } from './valueInjectionLoader';
 export { default as prefixLoader } from './prefixLoader';
 export { default as wrappingLoader } from './wrappingLoader';
-export { default as sdkMultiplexerLoader } from './sdkMultiplexerLoader';

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

@@ -24,12 +24,6 @@ import type {
 } from './types';
 import { getWebpackPluginOptions } from './webpackPluginOptions';
 
-const RUNTIME_TO_SDK_ENTRYPOINT_MAP = {
-  client: './client',
-  server: './server',
-  edge: './edge',
-} as const;
-
 // Next.js runs webpack 3 times, once for the client, the server, and for edge. Because we don't want to print certain
 // warnings 3 times, we keep track of them here.
 let showedMissingGlobalErrorWarningMsg = false;
@@ -79,18 +73,6 @@ export function constructWebpackConfigFunction(
     // Add a loader which will inject code that sets global values
     addValueInjectionLoader(newConfig, userNextConfig, userSentryOptions, buildContext);
 
-    newConfig.module.rules.push({
-      test: /node_modules[/\\]@sentry[/\\]nextjs/,
-      use: [
-        {
-          loader: path.resolve(__dirname, 'loaders', 'sdkMultiplexerLoader.js'),
-          options: {
-            importTarget: RUNTIME_TO_SDK_ENTRYPOINT_MAP[runtime],
-          },
-        },
-      ],
-    });
-
     let pagesDirPath: string | undefined;
     const maybePagesDirPath = path.join(projectDir, 'pages');
     const maybeSrcPagesDirPath = path.join(projectDir, 'src', 'pages');

packages/nextjs/src/config/webpack.ts

@@ -2,9 +2,3 @@ export * from './client';
 
 // This file is the main entrypoint for non-Next.js build pipelines that use
 // the package.json's "browser" field or the Edge runtime (Edge API routes and middleware)
-
-/**
- * This const serves no purpose besides being an identifier for this file that the SDK multiplexer loader can use to
- * determine that this is in fact a file that wants to be multiplexed.
- */
-export const _SENTRY_SDK_MULTIPLEXER = true;

packages/nextjs/src/index.client.ts

@@ -1,10 +1,2 @@
 export * from './config';
 export * from './server';
-
-// This file is the main entrypoint on the server and/or when the package is `require`d
-
-/**
- * This const serves no purpose besides being an identifier for this file that the SDK multiplexer loader can use to
- * determine that this is in fact a file that wants to be multiplexed.
- */
-export const _SENTRY_SDK_MULTIPLEXER = true;