@sentry/bundler-plugin-core 4.6.1 → 4.6.2

package/dist/esm/index.mjs

@@ -8299,7 +8299,7 @@ function createSentryInstance(options, shouldSendTelemetry, buildTool) {
     dsn: "https://4c2bae7d9fbc413e8f7385f55c515d51@o1.ingest.sentry.io/6690737",
     tracesSampleRate: 1,
     sampleRate: 1,
-    release: "4.6.1",
+    release: "4.6.2",
     integrations: [],
     tracePropagationTargets: ["sentry.io/api"],
     stackParser: stackParser,
@@ -8908,7 +8908,7 @@ function createSentryBuildPluginManager(userOptions, bundlerPluginMetaContext) {
   });
 
   // Set the User-Agent that Sentry CLI will use when interacting with Sentry
-  process.env["SENTRY_PIPELINE"] = "".concat(bundlerPluginMetaContext.buildTool, "-plugin/", "4.6.1");
+  process.env["SENTRY_PIPELINE"] = "".concat(bundlerPluginMetaContext.buildTool, "-plugin/", "4.6.2");
 
   // Propagate debug flag to Sentry CLI via environment variable
   // Only set if not already defined to respect user's explicit configuration
@@ -9905,42 +9905,76 @@ var COMMENT_USE_STRICT_REGEX =
  * about type mismatches
  */
 
+/**
+ * Checks if a file is a JavaScript file based on its extension.
+ * Handles query strings and hashes in the filename.
+ */
+function isJsFile(fileName) {
+  var cleanFileName = stripQueryAndHashFromPath(fileName);
+  return [".js", ".mjs", ".cjs"].some(function (ext) {
+    return cleanFileName.endsWith(ext);
+  });
+}
+
+/**
+ * Checks if a chunk should be skipped for code injection
+ *
+ * This is necessary to handle Vite's MPA (multi-page application) mode where
+ * HTML entry points create "facade" chunks that should not contain injected code.
+ * See: https://github.com/getsentry/sentry-javascript-bundler-plugins/issues/829
+ *
+ * @param code - The chunk's code content
+ * @param facadeModuleId - The facade module ID (if any) - HTML files create facade chunks
+ * @returns true if the chunk should be skipped
+ */
+function shouldSkipCodeInjection(code, facadeModuleId) {
+  // Skip empty chunks - these are placeholder chunks that should be optimized away
+  if (code.trim().length === 0) {
+    return true;
+  }
+
+  // Skip HTML facade chunks
+  // They only contain import statements and should not have Sentry code injected
+  if (facadeModuleId && stripQueryAndHashFromPath(facadeModuleId).endsWith(".html")) {
+    return true;
+  }
+  return false;
+}
 function createRollupReleaseInjectionHooks(injectionCode) {
   return {
     renderChunk: function renderChunk(code, chunk) {
-      if (
-      // chunks could be any file (html, md, ...)
-      [".js", ".mjs", ".cjs"].some(function (ending) {
-        return stripQueryAndHashFromPath(chunk.fileName).endsWith(ending);
-      })) {
-        var _code$match;
-        var ms = new MagicString(code, {
-          filename: chunk.fileName
-        });
-        var match = (_code$match = code.match(COMMENT_USE_STRICT_REGEX)) === null || _code$match === void 0 ? void 0 : _code$match[0];
-        if (match) {
-          // Add injected code after any comments or "use strict" at the beginning of the bundle.
-          ms.appendLeft(match.length, injectionCode);
-        } else {
-          // ms.replace() doesn't work when there is an empty string match (which happens if
-          // there is neither, a comment, nor a "use strict" at the top of the chunk) so we
-          // need this special case here.
-          ms.prepend(injectionCode);
-        }
-        return {
-          code: ms.toString(),
-          map: ms.generateMap({
-            file: chunk.fileName,
-            hires: "boundary"
-          })
-        };
-      } else {
+      var _code$match;
+      if (!isJsFile(chunk.fileName)) {
         return null; // returning null means not modifying the chunk at all
       }
+
+      // Skip empty chunks and HTML facade chunks (Vite MPA)
+      if (shouldSkipCodeInjection(code, chunk.facadeModuleId)) {
+        return null;
+      }
+      var ms = new MagicString(code, {
+        filename: chunk.fileName
+      });
+      var match = (_code$match = code.match(COMMENT_USE_STRICT_REGEX)) === null || _code$match === void 0 ? void 0 : _code$match[0];
+      if (match) {
+        // Add injected code after any comments or "use strict" at the beginning of the bundle.
+        ms.appendLeft(match.length, injectionCode);
+      } else {
+        // ms.replace() doesn't work when there is an empty string match (which happens if
+        // there is neither, a comment, nor a "use strict" at the top of the chunk) so we
+        // need this special case here.
+        ms.prepend(injectionCode);
+      }
+      return {
+        code: ms.toString(),
+        map: ms.generateMap({
+          file: chunk.fileName,
+          hires: "boundary"
+        })
+      };
     }
   };
 }
-
 function createRollupBundleSizeOptimizationHooks(replacementValues) {
   return {
     transform: function transform(code) {
@@ -9951,77 +9985,83 @@ function createRollupBundleSizeOptimizationHooks(replacementValues) {
 function createRollupDebugIdInjectionHooks() {
   return {
     renderChunk: function renderChunk(code, chunk) {
-      if (
-      // chunks could be any file (html, md, ...)
-      [".js", ".mjs", ".cjs"].some(function (ending) {
-        return stripQueryAndHashFromPath(chunk.fileName).endsWith(ending);
-      })) {
-        var _code$match2;
-        var debugId = stringToUUID(code); // generate a deterministic debug ID
-        var codeToInject = getDebugIdSnippet(debugId);
-        var ms = new MagicString(code, {
-          filename: chunk.fileName
-        });
-        var match = (_code$match2 = code.match(COMMENT_USE_STRICT_REGEX)) === null || _code$match2 === void 0 ? void 0 : _code$match2[0];
-        if (match) {
-          // Add injected code after any comments or "use strict" at the beginning of the bundle.
-          ms.appendLeft(match.length, codeToInject);
-        } else {
-          // ms.replace() doesn't work when there is an empty string match (which happens if
-          // there is neither, a comment, nor a "use strict" at the top of the chunk) so we
-          // need this special case here.
-          ms.prepend(codeToInject);
-        }
-        return {
-          code: ms.toString(),
-          map: ms.generateMap({
-            file: chunk.fileName,
-            hires: "boundary"
-          })
-        };
-      } else {
+      var _code$match2;
+      if (!isJsFile(chunk.fileName)) {
         return null; // returning null means not modifying the chunk at all
       }
+
+      // Skip empty chunks and HTML facade chunks (Vite MPA)
+      if (shouldSkipCodeInjection(code, chunk.facadeModuleId)) {
+        return null;
+      }
+
+      // Check if a debug ID has already been injected to avoid duplicate injection (e.g. by another plugin or Sentry CLI)
+      var chunkStartSnippet = code.slice(0, 6000);
+      var chunkEndSnippet = code.slice(-500);
+      if (chunkStartSnippet.includes("_sentryDebugIdIdentifier") || chunkEndSnippet.includes("//# debugId=")) {
+        return null; // Debug ID already present, skip injection
+      }
+
+      var debugId = stringToUUID(code); // generate a deterministic debug ID
+      var codeToInject = getDebugIdSnippet(debugId);
+      var ms = new MagicString(code, {
+        filename: chunk.fileName
+      });
+      var match = (_code$match2 = code.match(COMMENT_USE_STRICT_REGEX)) === null || _code$match2 === void 0 ? void 0 : _code$match2[0];
+      if (match) {
+        // Add injected code after any comments or "use strict" at the beginning of the bundle.
+        ms.appendLeft(match.length, codeToInject);
+      } else {
+        // ms.replace() doesn't work when there is an empty string match (which happens if
+        // there is neither, a comment, nor a "use strict" at the top of the chunk) so we
+        // need this special case here.
+        ms.prepend(codeToInject);
+      }
+      return {
+        code: ms.toString(),
+        map: ms.generateMap({
+          file: chunk.fileName,
+          hires: "boundary"
+        })
+      };
     }
   };
 }
-
 function createRollupModuleMetadataInjectionHooks(injectionCode) {
   return {
     renderChunk: function renderChunk(code, chunk) {
-      if (
-      // chunks could be any file (html, md, ...)
-      [".js", ".mjs", ".cjs"].some(function (ending) {
-        return stripQueryAndHashFromPath(chunk.fileName).endsWith(ending);
-      })) {
-        var _code$match3;
-        var ms = new MagicString(code, {
-          filename: chunk.fileName
-        });
-        var match = (_code$match3 = code.match(COMMENT_USE_STRICT_REGEX)) === null || _code$match3 === void 0 ? void 0 : _code$match3[0];
-        if (match) {
-          // Add injected code after any comments or "use strict" at the beginning of the bundle.
-          ms.appendLeft(match.length, injectionCode);
-        } else {
-          // ms.replace() doesn't work when there is an empty string match (which happens if
-          // there is neither, a comment, nor a "use strict" at the top of the chunk) so we
-          // need this special case here.
-          ms.prepend(injectionCode);
-        }
-        return {
-          code: ms.toString(),
-          map: ms.generateMap({
-            file: chunk.fileName,
-            hires: "boundary"
-          })
-        };
-      } else {
+      var _code$match3;
+      if (!isJsFile(chunk.fileName)) {
         return null; // returning null means not modifying the chunk at all
       }
+
+      // Skip empty chunks and HTML facade chunks (Vite MPA)
+      if (shouldSkipCodeInjection(code, chunk.facadeModuleId)) {
+        return null;
+      }
+      var ms = new MagicString(code, {
+        filename: chunk.fileName
+      });
+      var match = (_code$match3 = code.match(COMMENT_USE_STRICT_REGEX)) === null || _code$match3 === void 0 ? void 0 : _code$match3[0];
+      if (match) {
+        // Add injected code after any comments or "use strict" at the beginning of the bundle.
+        ms.appendLeft(match.length, injectionCode);
+      } else {
+        // ms.replace() doesn't work when there is an empty string match (which happens if
+        // there is neither, a comment, nor a "use strict" at the top of the chunk) so we
+        // need this special case here.
+        ms.prepend(injectionCode);
+      }
+      return {
+        code: ms.toString(),
+        map: ms.generateMap({
+          file: chunk.fileName,
+          hires: "boundary"
+        })
+      };
     }
   };
 }
-
 function createRollupDebugIdUploadHooks(upload, _logger, createDependencyOnBuildArtifacts) {
   var freeGlobalDependencyOnDebugIdSourcemapArtifacts = createDependencyOnBuildArtifacts();
   return {