@sentry/bundler-plugin-core 4.6.0 → 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.0",
+    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.0");
+  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
@@ -9116,7 +9116,7 @@ function createSentryBuildPluginManager(userOptions, bundlerPluginMetaContext) {
      */
     createRelease: function createRelease() {
       return _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime().mark(function _callee7() {
-        var freeWriteBundleInvocationDependencyOnSourcemapFiles, cliInstance, normalizedInclude;
+        var freeWriteBundleInvocationDependencyOnSourcemapFiles, cliInstance, releaseOutput, normalizedInclude;
         return _regeneratorRuntime().wrap(function _callee7$(_context7) {
           while (1) switch (_context7.prev = _context7.next) {
             case 0:
@@ -9161,14 +9161,17 @@ function createSentryBuildPluginManager(userOptions, bundlerPluginMetaContext) {
               _context7.prev = 24;
               cliInstance = createCliInstance(options);
               if (!options.release.create) {
-                _context7.next = 29;
+                _context7.next = 31;
                 break;
               }
               _context7.next = 29;
               return cliInstance.releases["new"](options.release.name);
             case 29:
+              releaseOutput = _context7.sent;
+              logger.debug("Release created:", releaseOutput);
+            case 31:
               if (!options.release.uploadLegacySourcemaps) {
-                _context7.next = 33;
+                _context7.next = 35;
                 break;
               }
               normalizedInclude = arrayify(options.release.uploadLegacySourcemaps).map(function (includeItem) {
@@ -9185,7 +9188,7 @@ function createSentryBuildPluginManager(userOptions, bundlerPluginMetaContext) {
                   ignore: includeEntry.ignore ? arrayify(includeEntry.ignore) : undefined
                 });
               });
-              _context7.next = 33;
+              _context7.next = 35;
               return cliInstance.releases.uploadSourceMaps(options.release.name, {
                 include: normalizedInclude,
                 dist: options.release.dist,
@@ -9194,66 +9197,66 @@ function createSentryBuildPluginManager(userOptions, bundlerPluginMetaContext) {
                 // see: https://github.com/getsentry/sentry-cli/pull/2605
                 live: "rejectOnError"
               });
-            case 33:
+            case 35:
               if (!(options.release.setCommits !== false)) {
-                _context7.next = 46;
+                _context7.next = 48;
                 break;
               }
-              _context7.prev = 34;
-              _context7.next = 37;
+              _context7.prev = 36;
+              _context7.next = 39;
               return cliInstance.releases.setCommits(options.release.name,
               // set commits always exists due to the normalize function
               // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
               options.release.setCommits);
-            case 37:
-              _context7.next = 46;
-              break;
             case 39:
-              _context7.prev = 39;
-              _context7.t0 = _context7["catch"](34);
+              _context7.next = 48;
+              break;
+            case 41:
+              _context7.prev = 41;
+              _context7.t0 = _context7["catch"](36);
               if (!(options.release.setCommits && "shouldNotThrowOnFailure" in options.release.setCommits && options.release.setCommits.shouldNotThrowOnFailure)) {
-                _context7.next = 45;
+                _context7.next = 47;
                 break;
               }
               logger.debug("An error occurred setting commits on release (this message can be ignored unless you commits on release are desired):", _context7.t0);
-              _context7.next = 46;
+              _context7.next = 48;
               break;
-            case 45:
+            case 47:
               throw _context7.t0;
-            case 46:
+            case 48:
               if (!options.release.finalize) {
-                _context7.next = 49;
+                _context7.next = 51;
                 break;
               }
-              _context7.next = 49;
+              _context7.next = 51;
               return cliInstance.releases.finalize(options.release.name);
-            case 49:
+            case 51:
               if (!options.release.deploy) {
-                _context7.next = 52;
+                _context7.next = 54;
                 break;
               }
-              _context7.next = 52;
+              _context7.next = 54;
               return cliInstance.releases.newDeploy(options.release.name, options.release.deploy);
-            case 52:
-              _context7.next = 60;
-              break;
             case 54:
-              _context7.prev = 54;
+              _context7.next = 62;
+              break;
+            case 56:
+              _context7.prev = 56;
               _context7.t1 = _context7["catch"](24);
               sentryScope.captureException('Error in "releaseManagementPlugin" writeBundle hook');
-              _context7.next = 59;
+              _context7.next = 61;
               return safeFlushTelemetry(sentryClient);
-            case 59:
+            case 61:
               handleRecoverableError(_context7.t1, false);
-            case 60:
-              _context7.prev = 60;
+            case 62:
+              _context7.prev = 62;
               freeWriteBundleInvocationDependencyOnSourcemapFiles();
-              return _context7.finish(60);
-            case 63:
+              return _context7.finish(62);
+            case 65:
             case "end":
               return _context7.stop();
           }
-        }, _callee7, null, [[24, 54, 60, 63], [34, 39]]);
+        }, _callee7, null, [[24, 56, 62, 65], [36, 41]]);
       }))();
     },
     /*
@@ -9902,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) {
@@ -9948,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 {