build: update biome to v1.5.2

anonrig · anonrig · commit 474548ec84ca · 2024-01-16T10:06:43.000-05:00
diff --git a/biome.json b/biome.json
@@ -3,7 +3,8 @@
   "vcs": {
     "enabled": true,
     "clientKind": "git",
-    "useIgnoreFile": true
+    "useIgnoreFile": true,
+    "defaultBranch": "develop"
   },
   "organizeImports": {
     "enabled": true
@@ -17,13 +18,15 @@
         "noUnusedVariables": "error",
         "noPrecisionLoss": "error"
       },
+      "complexity": {
+        "useRegexLiterals": "error"
+      },
       "suspicious": {
         "all": false,
         "noControlCharactersInRegex": "error"
       },
       "nursery": {
-        "noUnusedImports": "error",
-        "useRegexLiterals": "error"
+        "noUnusedImports": "error"
       },
       "performance": {
         "all": true,
@@ -47,8 +50,8 @@
       "dev-packages/browser-integration-tests/suites/**/*.json",
       "dev-packages/browser-integration-tests/loader-suites/**/*.js",
       "dev-packages/browser-integration-tests/suites/stacktraces/**/*.js",
-      "**/fixtures/*/*.json",
-      "**/*.min.js"
+      "fixtures/*/*.json",
+      "*.min.js"
     ]
   },
   "javascript": {
@@ -66,6 +69,10 @@
   "json": {
     "formatter": {
       "enabled": true
+    },
+    "parser": {
+      "allowComments": true,
+      "allowTrailingCommas": true
     }
   }
 }
diff --git a/dev-packages/browser-integration-tests/utils/helpers.ts b/dev-packages/browser-integration-tests/utils/helpers.ts
@@ -68,13 +68,10 @@ export const countEnvelopes = async (
 
     page.on('request', requestHandler);
 
-    setTimeout(
-      () => {
-        page.off('request', requestHandler);
-        resolve(reqCount);
-      },
-      options?.timeout || 1000,
-    );
+    setTimeout(() => {
+      page.off('request', requestHandler);
+      resolve(reqCount);
+    }, options?.timeout || 1000);
   });
 
   if (options?.url) {
diff --git a/dev-packages/node-integration-tests/utils/index.ts b/dev-packages/node-integration-tests/utils/index.ts
@@ -128,7 +128,10 @@ export class TestEnv {
   private _axiosConfig: AxiosRequestConfig | undefined = undefined;
   private _terminator: HttpTerminator;
 
-  public constructor(public readonly server: http.Server, public readonly url: string) {
+  public constructor(
+    public readonly server: http.Server,
+    public readonly url: string,
+  ) {
     this.server = server;
     this.url = url;
     this._terminator = createHttpTerminator({ server: this.server, gracefulTerminationTimeout: 0 });
@@ -300,19 +303,16 @@ export class TestEnv {
           return false;
         });
 
-      setTimeout(
-        () => {
-          nock.removeInterceptor(mock);
+      setTimeout(() => {
+        nock.removeInterceptor(mock);
 
-          nock.cleanAll();
+        nock.cleanAll();
 
-          // eslint-disable-next-line @typescript-eslint/no-floating-promises
-          this._closeServer().then(() => {
-            resolve(reqCount);
-          });
-        },
-        options.timeout || 1000,
-      );
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
+        this._closeServer().then(() => {
+          resolve(reqCount);
+        });
+      }, options.timeout || 1000);
     });
   }
 
diff --git a/dev-packages/overhead-metrics/src/perf/cpu.ts b/dev-packages/overhead-metrics/src/perf/cpu.ts
@@ -7,15 +7,21 @@ export { CpuUsageSampler, CpuUsage };
 export type CpuUsageSerialized = Partial<{ snapshots: JsonObject<number>; average: number }>;
 
 class CpuUsage {
-  public constructor(public snapshots: TimeBasedMap<number>, public average: number) {}
+  public constructor(
+    public snapshots: TimeBasedMap<number>,
+    public average: number,
+  ) {}
 
   public static fromJSON(data: CpuUsageSerialized): CpuUsage {
     return new CpuUsage(TimeBasedMap.fromJSON<number>(data.snapshots || {}), data.average as number);
   }
 }
 
 class MetricsDataPoint {
-  public constructor(public timestamp: number, public activeTime: number) {}
+  public constructor(
+    public timestamp: number,
+    public activeTime: number,
+  ) {}
 }
 
 class CpuUsageSampler {
diff --git a/dev-packages/overhead-metrics/src/results/analyzer.ts b/dev-packages/overhead-metrics/src/results/analyzer.ts
@@ -100,7 +100,10 @@ export interface AnalyzerItemValues {
 const AnalyzerItemValueNotAvailable = 'n/a';
 
 class AnalyzerItemNumberValues implements AnalyzerItemValues {
-  public constructor(private _unit: AnalyzerItemUnit, private _values: (number | undefined)[]) {}
+  public constructor(
+    private _unit: AnalyzerItemUnit,
+    private _values: (number | undefined)[],
+  ) {}
 
   public value(index: number): string {
     if (!this._has(index)) return AnalyzerItemValueNotAvailable;
diff --git a/dev-packages/overhead-metrics/src/scenarios.ts b/dev-packages/overhead-metrics/src/scenarios.ts
@@ -55,7 +55,10 @@ export class JankTestScenario implements Scenario {
 }
 
 export class BookingAppScenario implements Scenario {
-  public constructor(private _indexFile: string, private _count: number) {}
+  public constructor(
+    private _indexFile: string,
+    private _count: number,
+  ) {}
 
   /**
    *
diff --git a/dev-packages/overhead-metrics/src/vitals/index.ts b/dev-packages/overhead-metrics/src/vitals/index.ts
@@ -7,15 +7,23 @@ import { LCP } from './lcp.js';
 export { WebVitals, WebVitalsCollector };
 
 class WebVitals {
-  public constructor(public lcp: number | undefined, public cls: number | undefined, public fid: number | undefined) {}
+  public constructor(
+    public lcp: number | undefined,
+    public cls: number | undefined,
+    public fid: number | undefined,
+  ) {}
 
   public static fromJSON(data: Partial<WebVitals>): WebVitals {
     return new WebVitals(data.lcp as number, data.cls as number, data.fid as number);
   }
 }
 
 class WebVitalsCollector {
-  private constructor(private _lcp: LCP, private _cls: CLS, private _fid: FID) {}
+  private constructor(
+    private _lcp: LCP,
+    private _cls: CLS,
+    private _fid: FID,
+  ) {}
 
   public static async create(page: playwright.Page): Promise<WebVitalsCollector> {
     const result = new WebVitalsCollector(new LCP(page), new CLS(page), new FID(page));
diff --git a/package.json b/package.json
@@ -85,7 +85,7 @@
     "dev-packages/rollup-utils"
   ],
   "devDependencies": {
-    "@biomejs/biome": "^1.4.0",
+    "@biomejs/biome": "^1.5.2",
     "@rollup/plugin-commonjs": "^21.0.1",
     "@rollup/plugin-node-resolve": "^13.1.3",
     "@rollup/plugin-replace": "^3.0.1",
diff --git a/packages/angular/test/errorhandler.test.ts b/packages/angular/test/errorhandler.test.ts
@@ -23,7 +23,10 @@ class CustomError extends Error {
 }
 
 class ErrorLikeShapedClass implements Partial<Error> {
-  constructor(public name: string, public message: string) {}
+  constructor(
+    public name: string,
+    public message: string,
+  ) {}
 }
 
 function createErrorEvent(message: string, innerError: any): ErrorEvent {
diff --git a/packages/browser/src/integrations/trycatch.ts b/packages/browser/src/integrations/trycatch.ts
@@ -189,7 +189,7 @@ function _wrapEventTarget(target: string): void {
     return;
   }
 
-  fill(proto, 'addEventListener', function (original: VoidFunction,): (
+  fill(proto, 'addEventListener', function (original: VoidFunction): (
     eventName: string,
     fn: EventListenerObject,
     options?: boolean | AddEventListenerOptions,
diff --git a/packages/integrations/scripts/buildBundles.ts b/packages/integrations/scripts/buildBundles.ts
@@ -32,13 +32,10 @@ async function buildBundle(integration: string, jsVersion: string): Promise<void
 
 if (runParallel) {
   // We're building a bundle for each integration and each JavaScript version.
-  const tasks = getIntegrations().reduce(
-    (tasks, integration) => {
-      tasks.push(buildBundle(integration, 'es5'), buildBundle(integration, 'es6'));
-      return tasks;
-    },
-    [] as Promise<void>[],
-  );
+  const tasks = getIntegrations().reduce((tasks, integration) => {
+    tasks.push(buildBundle(integration, 'es5'), buildBundle(integration, 'es6'));
+    return tasks;
+  }, [] as Promise<void>[]);
 
   Promise.all(tasks)
     // eslint-disable-next-line no-console
diff --git a/packages/nextjs/src/common/wrapPageComponentWithSentry.ts b/packages/nextjs/src/common/wrapPageComponentWithSentry.ts
@@ -6,7 +6,9 @@ interface FunctionComponent {
 }
 
 interface ClassComponent {
-  new (...args: unknown[]): {
+  new (
+    ...args: unknown[]
+  ): {
     props?: unknown;
     render(...args: unknown[]): unknown;
   };
diff --git a/packages/nextjs/test/integration/test/server/utils/helpers.ts b/packages/nextjs/test/integration/test/server/utils/helpers.ts
@@ -35,7 +35,10 @@ export const startServer = async (server: Server) => {
 };
 
 export class NextTestEnv extends TestEnv {
-  private constructor(public readonly server: http.Server, public readonly url: string) {
+  private constructor(
+    public readonly server: http.Server,
+    public readonly url: string,
+  ) {
     super(server, url);
   }
 
diff --git a/packages/node/test/integrations/localvariables.test.ts b/packages/node/test/integrations/localvariables.test.ts
@@ -20,7 +20,10 @@ interface ThrowOn {
 class MockDebugSession implements DebugSession {
   private _onPause?: (message: InspectorNotification<Debugger.PausedEventDataType>, callback: () => void) => void;
 
-  constructor(private readonly _vars: Record<string, Record<string, unknown>>, private readonly _throwOn?: ThrowOn) {}
+  constructor(
+    private readonly _vars: Record<string, Record<string, unknown>>,
+    private readonly _throwOn?: ThrowOn,
+  ) {}
 
   public configureAndConnect(
     onPause: (message: InspectorNotification<Debugger.PausedEventDataType>, callback: () => void) => void,
diff --git a/packages/remix/test/integration/test/server/utils/helpers.ts b/packages/remix/test/integration/test/server/utils/helpers.ts
@@ -8,7 +8,10 @@ import { TestEnv } from '../../../../../../../dev-packages/node-integration-test
 export * from '../../../../../../../dev-packages/node-integration-tests/utils';
 
 export class RemixTestEnv extends TestEnv {
-  private constructor(public readonly server: http.Server, public readonly url: string) {
+  private constructor(
+    public readonly server: http.Server,
+    public readonly url: string,
+  ) {
     super(server, url);
   }
 
diff --git a/packages/replay-worker/src/worker.ts b/packages/replay-worker/src/worker.ts
@@ -1,3 +1,3 @@
 // This is replaced at build-time with the content from _worker.ts, wrapped as a string.
 // This is just a placeholder so that types etc. are correct.
-export default ('' as string);
+export default '' as string;
diff --git a/packages/tracing-internal/tsconfig.types.json b/packages/tracing-internal/tsconfig.types.json
@@ -4,11 +4,7 @@
   // the fact that it introduces a dependency on `@sentry/browser` which doesn't exist anywhere else in the SDK, which
   // then prevents us from building that and this at the same time when doing a parallellized build from the repo root
   // level.
-  "exclude": [
-    "src/index.bundle.ts",
-    "src/index.bundle.feedback.ts",
-    "src/index.bundle.replay.ts"
-  ],
+  "exclude": ["src/index.bundle.ts", "src/index.bundle.feedback.ts", "src/index.bundle.replay.ts"],
   "compilerOptions": {
     "declaration": true,
     "declarationMap": true,
diff --git a/packages/utils/src/error.ts b/packages/utils/src/error.ts
@@ -7,7 +7,10 @@ export class SentryError extends Error {
 
   public logLevel: ConsoleLevel;
 
-  public constructor(public message: string, logLevel: ConsoleLevel = 'warn') {
+  public constructor(
+    public message: string,
+    logLevel: ConsoleLevel = 'warn',
+  ) {
     super(message);
 
     this.name = new.target.prototype.constructor.name;
diff --git a/packages/utils/src/object.ts b/packages/utils/src/object.ts
@@ -102,9 +102,7 @@ export function urlEncode(object: { [key: string]: any }): string {
  * @returns An Event or Error turned into an object - or the value argurment itself, when value is neither an Event nor
  *  an Error.
  */
-export function convertToPlainObject<V>(
-  value: V,
-):
+export function convertToPlainObject<V>(value: V):
   | {
       [ownProps: string]: unknown;
       type: string;
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,10 @@ export class JankTestScenario implements Scenario {`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`export class BookingAppScenario implements Scenario {`
`58`		`- public constructor(private _indexFile: string, private _count: number) {}`
	`58`	`+ public constructor(`
	`59`	`+ private _indexFile: string,`
	`60`	`+ private _count: number,`
	`61`	`+ ) {}`
`59`	`62`
`60`	`63`	`/**`
`61`	`64`	`*`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,10 @@ class CustomError extends Error {`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`class ErrorLikeShapedClass implements Partial<Error> {`
`26`		`- constructor(public name: string, public message: string) {}`
	`26`	`+ constructor(`
	`27`	`+ public name: string,`
	`28`	`+ public message: string,`
	`29`	`+ ) {}`
`27`	`30`	`}`
`28`	`31`
`29`	`32`	`function createErrorEvent(message: string, innerError: any): ErrorEvent {`
Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,7 @@ function _wrapEventTarget(target: string): void {`
`189`	`189`	`return;`
`190`	`190`	`}`
`191`	`191`
`192`		`- fill(proto, 'addEventListener', function (original: VoidFunction,): (`
	`192`	`+ fill(proto, 'addEventListener', function (original: VoidFunction): (`
`193`	`193`	`eventName: string,`
`194`	`194`	`fn: EventListenerObject,`
`195`	`195`	`options?: boolean \| AddEventListenerOptions,`