feat: Remove glob dependency

Author: timfish

packages/bundler-plugin-core/package.json

@@ -57,7 +57,6 @@
     "@sentry/cli": "^2.57.0",
     "dotenv": "^16.3.1",
     "find-up": "^5.0.0",
-    "glob": "^10.5.0",
     "magic-string": "0.30.8"
   },
   "devDependencies": {

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

@@ -5,7 +5,6 @@ import componentNameAnnotatePlugin, {
 import SentryCli from "@sentry/cli";
 import { logger } from "@sentry/utils";
 import * as fs from "fs";
-import { glob } from "glob";
 import { CodeInjection, containsOnlyImports, stripQueryAndHashFromPath } from "./utils";
 
 /**
@@ -63,19 +62,6 @@ export function shouldSkipCodeInjection(
   return false;
 }
 
-export function globFiles(outputDir: string): Promise<string[]> {
-  return glob(
-    ["/**/*.js", "/**/*.mjs", "/**/*.cjs", "/**/*.js.map", "/**/*.mjs.map", "/**/*.cjs.map"].map(
-      (q) => `${q}?(\\?*)?(#*)`
-    ), // We want to allow query and hashes strings at the end of files
-    {
-      root: outputDir,
-      absolute: true,
-      nodir: true,
-    }
-  );
-}
-
 // eslint-disable-next-line @typescript-eslint/explicit-function-return-type
 export function createComponentNameAnnotateHooks(
   ignoredComponents: string[],

packages/rollup-plugin/src/index.ts

@@ -9,7 +9,6 @@ import {
   stringToUUID,
   COMMENT_USE_STRICT_REGEX,
   createDebugIdUploadFunction,
-  globFiles,
   createComponentNameAnnotateHooks,
   replaceBooleanFlagsInCode,
   CodeInjection,
@@ -206,18 +205,8 @@ export function _rollupPluginInternal(
       await sentryBuildPluginManager.createRelease();
 
       if (sourcemapsEnabled && options.sourcemaps?.disable !== "disable-upload") {
-        if (outputOptions.dir) {
-          const outputDir = outputOptions.dir;
-          const buildArtifacts = await globFiles(outputDir);
-          await upload(buildArtifacts);
-        } else if (outputOptions.file) {
-          await upload([outputOptions.file]);
-        } else {
-          const buildArtifacts = Object.keys(bundle).map((asset) =>
-            path.join(path.resolve(), asset)
-          );
-          await upload(buildArtifacts);
-        }
+        const buildArtifacts = Object.keys(bundle).map((asset) => path.join(path.resolve(), asset));
+        await upload(buildArtifacts);
       }
     } finally {
       freeGlobalDependencyOnBuildArtifacts();

yarn.lock

@@ -7164,7 +7164,7 @@ glob@8.0.3:
     minimatch "^5.0.1"
     once "^1.3.0"
 
-glob@^10.2.2, glob@^10.5.0:
+glob@^10.2.2:
   version "10.5.0"
   resolved "https://registry.npmjs.org/glob/-/glob-10.5.0.tgz#8ec0355919cd3338c28428a23d4f24ecc5fe738c"
   integrity sha512-DfXN8DfhJ7NH3Oe7cFmu3NCu1wKbkReJ8TorzSAFbSKrlNaQSKfIzqYqVY8zlbs2NLBbWpRiU52GX2PbaBVNkg==