fix: rename debug logging here already so we don't have to later

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.
      */
-    debugLogs?: boolean;
+    debugLogging?: boolean;
   };
 
   /**
@@ -535,7 +535,7 @@ export type SentryBuildOptions = {
   /**
    * Tree shakes Sentry SDK logger statements from the bundle.
    *
-   * @deprecated Use `webpack.treeshake.debugLogs` instead.
+   * @deprecated Use `webpack.treeshake.debugLogging` instead.
    */
   disableLogger?: boolean;
 

packages/nextjs/src/config/webpack.ts

@@ -431,7 +431,7 @@ export function constructWebpackConfigFunction({
       }
     }
 
-    if (userSentryOptions.webpack?.treeshake?.debugLogs) {
+    if (userSentryOptions.webpack?.treeshake?.debugLogging) {
       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.debugLogs specially since it's nested
+  // Handle treeshake.debugLogging specially since it's nested
   if (userSentryOptions.disableLogger !== undefined) {
     webpack.treeshake = webpack.treeshake || {};
-    webpack.treeshake.debugLogs = withDeprecatedFallback(
-      webpack.treeshake.debugLogs,
+    webpack.treeshake.debugLogging = withDeprecatedFallback(
+      webpack.treeshake.debugLogging,
       userSentryOptions.disableLogger,
-      deprecatedMessage('disableLogger', 'webpack.treeshake.debugLogs'),
+      deprecatedMessage('disableLogger', 'webpack.treeshake.debugLogging'),
     );
   }
 

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

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