code review changes

packages/nextjs/src/config/webpack.ts

@@ -17,6 +17,8 @@ import { SERVER_SDK_INIT_PATH, storeServerConfigFileLocation } from './utils';
 
 export { SentryWebpackPlugin };
 
+logger.enable();
+
 // TODO: merge default SentryWebpackPlugin ignore with their SentryWebpackPlugin ignore or ignoreFile
 // TODO: merge default SentryWebpackPlugin include with their SentryWebpackPlugin include
 // TODO: drop merged keys from override check? `includeDefaults` option?
@@ -54,7 +56,7 @@ export function constructWebpackConfigFunction(
 ): WebpackConfigFunction {
   // Will be called by nextjs and passed its default webpack configuration and context data about the build (whether
   // we're building server or client, whether we're in dev, what version of webpack we're using, etc). Note that
-  // `currentWebpackConfig` and `buildContext` are referred to as `config` and `options` in the nextjs docs.
+  // `incomingConfig` and `buildContext` are referred to as `config` and `options` in the nextjs docs.
   const newWebpackFunction = (incomingConfig: WebpackConfigObject, buildContext: BuildContext): WebpackConfigObject => {
     let newConfig = { ...incomingConfig };
 
@@ -203,7 +205,7 @@ function addFileToExistingEntryPoint(
     newEntryPoint = [...currentEntryPoint, filepath];
   }
   // descriptor object (webpack 5+)
-  else {
+  else if (typeof currentEntryPoint === 'object' && 'import' in currentEntryPoint) {
     const currentImportValue = currentEntryPoint.import;
     let newImportValue: string | string[];
 
@@ -217,6 +219,13 @@ function addFileToExistingEntryPoint(
       ...currentEntryPoint,
       import: newImportValue,
     };
+  } else {
+    logger.warn(
+      `Could not inject SDK initialization code into entry point ${entryPointName}, as it is not a recognized format.\n`,
+      `Expected: string | Array<string> | { [key:string]: any, import: string | Array<string> }\n`,
+      `Got: ${currentEntryPoint}`,
+    );
+    return;
   }
 
   entryProperty[entryPointName] = newEntryPoint;