fix(core): Ignore query and hash in filepaths for release injection (#272)

Author: Luca Forstner

packages/bundler-plugin-core/src/index.ts

@@ -16,6 +16,7 @@ import {
   getPackageJson,
   parseMajorVersion,
   stringToUUID,
+  stripQueryAndHashFromPath,
 } from "./utils";
 
 interface SentryUnpluginFactoryOptions {
@@ -278,11 +279,18 @@ export function createRollupReleaseInjectionHooks(injectionCode: string) {
         return null;
       }
 
-      if (id.match(/\\node_modules\\|\/node_modules\//)) {
+      // id may contain query and hash which will trip up our file extension logic below
+      const idWithoutQueryAndHash = stripQueryAndHashFromPath(id);
+
+      if (idWithoutQueryAndHash.match(/\\node_modules\\|\/node_modules\//)) {
         return null;
       }
 
-      if (![".js", ".ts", ".jsx", ".tsx", ".mjs"].some((ending) => id.endsWith(ending))) {
+      if (
+        ![".js", ".ts", ".jsx", ".tsx", ".mjs"].some((ending) =>
+          idWithoutQueryAndHash.endsWith(ending)
+        )
+      ) {
         return null;
       }
 

packages/bundler-plugin-core/src/utils.ts

@@ -271,3 +271,8 @@ function getBuildInformation() {
     nodeVersion: parseMajorVersion(process.version),
   };
 }
+
+export function stripQueryAndHashFromPath(path: string): string {
+  // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+  return path.split("?")[0]!.split("#")[0]!;
+}