ref: rename debugLogging to removeDebugLogging for explicitness

Author: logaretm

packages/nextjs/src/config/types.ts

@@ -111,7 +111,7 @@ export type SentryBuildWebpackOptions = {
     /**
      * Tree shakes Sentry SDK logger statements from the bundle. Note that this doesn't affect Sentry Logs.
      */
-    debugLogging?: boolean;
+    removeDebugLogging?: boolean;
   };
 
   /**
@@ -535,7 +535,7 @@ export type SentryBuildOptions = {
   /**
    * Tree shakes Sentry SDK logger statements from the bundle.
    *
-   * @deprecated Use `webpack.treeshake.debugLogging` instead.
+   * @deprecated Use `webpack.treeshake.removeDebugLogging` instead.
    */
   disableLogger?: boolean;
 

packages/nextjs/src/config/webpack.ts

@@ -431,7 +431,7 @@ export function constructWebpackConfigFunction({
       }
     }
 
-    if (userSentryOptions.webpack?.treeshake?.debugLogging) {
+    if (userSentryOptions.webpack?.treeshake?.removeDebugLogging) {
       newConfig.plugins = newConfig.plugins || [];
       newConfig.plugins.push(
         new buildContext.webpack.DefinePlugin({

packages/nextjs/src/config/withSentryConfig.ts

@@ -162,13 +162,13 @@ function migrateDeprecatedWebpackOptions(userSentryOptions: SentryBuildOptions):
     deprecatedMessage('disableSentryWebpackConfig', 'webpack.disableSentryConfig'),
   );
 
-  // Handle treeshake.debugLogging specially since it's nested
+  // Handle treeshake.removeDebugLogging specially since it's nested
   if (userSentryOptions.disableLogger !== undefined) {
     webpack.treeshake = webpack.treeshake || {};
-    webpack.treeshake.debugLogging = withDeprecatedFallback(
-      webpack.treeshake.debugLogging,
+    webpack.treeshake.removeDebugLogging = withDeprecatedFallback(
+      webpack.treeshake.removeDebugLogging,
       userSentryOptions.disableLogger,
-      deprecatedMessage('disableLogger', 'webpack.treeshake.debugLogging'),
+      deprecatedMessage('disableLogger', 'webpack.treeshake.removeDebugLogging'),
     );
   }
 

packages/nextjs/test/config/withSentryConfig.test.ts

@@ -334,14 +334,14 @@ describe('withSentryConfig', () => {
         expect(finalConfig.webpack).toBe(originalWebpackFunction);
       });
 
-      it('merges webpack.treeshake.debugLogging with deprecated disableLogger', () => {
+      it('merges webpack.treeshake.removeDebugLogging with deprecated disableLogger', () => {
         delete process.env.TURBOPACK;
 
-        // New webpack.treeshake.debugLogging should map to disableLogger internally
+        // New webpack.treeshake.removeDebugLogging should map to disableLogger internally
         const sentryOptionsNew = {
           webpack: {
             treeshake: {
-              debugLogging: true,
+              removeDebugLogging: true,
             },
           },
         };
@@ -386,7 +386,7 @@ describe('withSentryConfig', () => {
           expect.stringContaining('[@sentry/nextjs] DEPRECATION WARNING: disableLogger is deprecated'),
         );
         expect(consoleWarnSpy).toHaveBeenCalledWith(
-          expect.stringContaining('Use webpack.treeshake.debugLogging instead'),
+          expect.stringContaining('Use webpack.treeshake.removeDebugLogging instead'),
         );
       });
 
@@ -396,7 +396,7 @@ describe('withSentryConfig', () => {
         const sentryOptions = {
           webpack: {
             treeshake: {
-              debugLogging: true,
+              removeDebugLogging: true,
             },
           },
         };
@@ -413,7 +413,7 @@ describe('withSentryConfig', () => {
           disableLogger: true, // deprecated
           webpack: {
             treeshake: {
-              debugLogging: false, // new path takes precedence
+              removeDebugLogging: false, // new path takes precedence
             },
           },
         };