PR review

Author: timfish

packages/bundler-plugin-core/src/debug-id-upload.ts

@@ -6,6 +6,7 @@ import { promisify } from "util";
 import { SentryBuildPluginManager } from "./build-plugin-manager";
 import { Logger } from "./logger";
 import { ResolveSourceMapHook, RewriteSourcesHook } from "./types";
+import { stripQueryAndHashFromPath } from "./utils";
 
 interface DebugIdUploadPluginOptions {
   sentryBuildPluginManager: SentryBuildPluginManager;
@@ -17,7 +18,7 @@ export function createDebugIdUploadFunction({
   return async (buildArtifactPaths: string[]) => {
     // Webpack and perhaps other bundlers allow you to append query strings to
     // filenames for cache busting purposes. We should strip these before upload.
-    const cleanedPaths = buildArtifactPaths.map((p) => p.split("?")[0] || p);
+    const cleanedPaths = buildArtifactPaths.map(stripQueryAndHashFromPath);
     await sentryBuildPluginManager.uploadSourcemaps(cleanedPaths);
   };
 }