Fix formatting

Author: andreiborza

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

@@ -1,6 +1,6 @@
 import fs from "fs";
 import path from "path";
-import * as url from "url"
+import * as url from "url";
 import * as util from "util";
 import { promisify } from "util";
 import { SentryBuildPluginManager } from "./build-plugin-manager";
@@ -120,12 +120,16 @@ export async function determineSourceMapPathFromBundle(
   resolveSourceMapHook: ResolveSourceMapHook | undefined
 ): Promise<string | undefined> {
   const sourceMappingUrlMatch = bundleSource.match(/^\s*\/\/# sourceMappingURL=(.*)$/m);
-  const sourceMappingUrl = sourceMappingUrlMatch ? sourceMappingUrlMatch[1] as string : undefined;
+  const sourceMappingUrl = sourceMappingUrlMatch ? (sourceMappingUrlMatch[1] as string) : undefined;
 
   const searchLocations: string[] = [];
 
   if (resolveSourceMapHook) {
-    logger.debug(`Calling sourcemaps.resolveSourceMap(${JSON.stringify(bundlePath)}, ${JSON.stringify(sourceMappingUrl)})`);
+    logger.debug(
+      `Calling sourcemaps.resolveSourceMap(${JSON.stringify(bundlePath)}, ${JSON.stringify(
+        sourceMappingUrl
+      )})`
+    );
     const customPath = await resolveSourceMapHook(bundlePath, sourceMappingUrl);
     logger.debug(`resolveSourceMap hook returned: ${JSON.stringify(customPath)}`);
 
@@ -148,19 +152,19 @@ export async function determineSourceMapPathFromBundle(
     } else if (parsedUrl) {
       // noop, non-file urls don't translate to a local sourcemap file
     } else if (path.isAbsolute(sourceMappingUrl)) {
-      searchLocations.push(path.normalize(sourceMappingUrl))
+      searchLocations.push(path.normalize(sourceMappingUrl));
     } else {
       searchLocations.push(path.normalize(path.join(path.dirname(bundlePath), sourceMappingUrl)));
     }
   }
 
   // 2. try to find source map at path adjacent to chunk source, but with `.map` appended
-  searchLocations.push(bundlePath + ".map")
+  searchLocations.push(bundlePath + ".map");
 
   for (const searchLocation of searchLocations) {
     try {
       await util.promisify(fs.access)(searchLocation);
-      logger.debug(`Source map found for bundle \`${bundlePath}\`: \`${searchLocation}\``)
+      logger.debug(`Source map found for bundle \`${bundlePath}\`: \`${searchLocation}\``);
       return searchLocation;
     } catch (e) {
       // noop
@@ -170,9 +174,11 @@ export async function determineSourceMapPathFromBundle(
   // This is just a debug message because it can be quite spammy for some frameworks
   logger.debug(
     `Could not determine source map path for bundle \`${bundlePath}\`` +
-    ` with sourceMappingURL=${sourceMappingUrl === undefined ? "undefined" : `\`${sourceMappingUrl}\``}` +
-    ` - Did you turn on source map generation in your bundler?` +
-    ` (Attempted paths: ${searchLocations.map(e => `\`${e}\``).join(", ")})`
+      ` with sourceMappingURL=${
+        sourceMappingUrl === undefined ? "undefined" : `\`${sourceMappingUrl}\``
+      }` +
+      ` - Did you turn on source map generation in your bundler?` +
+      ` (Attempted paths: ${searchLocations.map((e) => `\`${e}\``).join(", ")})`
   );
   return undefined;
 }

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

@@ -377,9 +377,12 @@ export interface Options {
 }
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-export type RewriteSourcesHook = (source: string, map: any) => string
+export type RewriteSourcesHook = (source: string, map: any) => string;
 
-export type ResolveSourceMapHook = (artifactPath: string, sourceMappingUrl: string | undefined) => string | undefined | Promise<string | undefined>
+export type ResolveSourceMapHook = (
+  artifactPath: string,
+  sourceMappingUrl: string | undefined
+) => string | undefined | Promise<string | undefined>;
 
 export interface ModuleMetadata {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any

packages/bundler-plugin-core/test/sentry/resolve-source-maps.test.ts

@@ -15,10 +15,10 @@ const separateBundlePath = path.join(fixtureDir, "separate-directory/bundles/ind
 const separateSourceMapPath = path.join(fixtureDir, "separate-directory/sourcemaps/index.js.map");
 const separateBundleContent = fs.readFileSync(separateBundlePath, "utf-8");
 
-const sourceMapUrl = "https://sourcemaps.example.com/foo/index.js.map"
+const sourceMapUrl = "https://sourcemaps.example.com/foo/index.js.map";
 
 function srcMappingUrl(url: string): string {
-  return `\n//# sourceMappingURL=${url}`
+  return `\n//# sourceMappingURL=${url}`;
 }
 
 describe("Resolve source maps", () => {
@@ -82,7 +82,8 @@ describe("Resolve source maps", () => {
       expect(
         await determineSourceMapPathFromBundle(
           separateBundlePath,
-          separateBundleContent + srcMappingUrl(path.relative(path.dirname(separateBundlePath), separateSourceMapPath)),
+          separateBundleContent +
+            srcMappingUrl(path.relative(path.dirname(separateBundlePath), separateSourceMapPath)),
           logger,
           undefined
         )
@@ -103,7 +104,7 @@ describe("Resolve source maps", () => {
     });
 
     it("should pass the correct values to the resolveSourceMap hook", async () => {
-      const hook = jest.fn(() => separateSourceMapPath)
+      const hook = jest.fn(() => separateSourceMapPath);
       expect(
         await determineSourceMapPathFromBundle(
           separateBundlePath,
@@ -112,11 +113,11 @@ describe("Resolve source maps", () => {
           hook
         )
       ).toEqual(separateSourceMapPath);
-      expect(hook.mock.calls[0]).toEqual([separateBundlePath, sourceMapUrl])
+      expect(hook.mock.calls[0]).toEqual([separateBundlePath, sourceMapUrl]);
     });
 
     it("should pass the correct values to the resolveSourceMap hook when no sourceMappingURL is present", async () => {
-      const hook = jest.fn(() => separateSourceMapPath)
+      const hook = jest.fn(() => separateSourceMapPath);
       expect(
         await determineSourceMapPathFromBundle(
           separateBundlePath,
@@ -125,7 +126,7 @@ describe("Resolve source maps", () => {
           hook
         )
       ).toEqual(separateSourceMapPath);
-      expect(hook.mock.calls[0]).toEqual([separateBundlePath, undefined])
+      expect(hook.mock.calls[0]).toEqual([separateBundlePath, undefined]);
     });
 
     it("should prefer resolveSourceMap result over heuristic results", async () => {
@@ -161,4 +162,4 @@ describe("Resolve source maps", () => {
       ).toEqual(adjacentSourceMapPath);
     });
   });
-});
+});