chore(biome): enable noUnusedImports rule (#9895)

Adds https://biomejs.dev/linter/rules/no-unused-imports/

Author: AbhiPrasad

biome.json

@@ -20,6 +20,9 @@
       "suspicious": {
         "all": false,
         "noControlCharactersInRegex": "error"
+      },
+      "nursery": {
+        "noUnusedImports": "error"
       }
     },
     "ignore": [".vscode/*", "**/*.json"]

package.json

@@ -18,12 +18,12 @@
     "clean:deps": "lerna clean --yes && rm -rf node_modules && yarn",
     "clean:all": "run-s clean:build clean:caches clean:deps",
     "codecov": "codecov",
-    "fix": "run-s fix:lerna fix:biome",
+    "fix": "run-p fix:lerna fix:biome",
     "fix:lerna": "lerna run fix",
-    "fix:biome": "biome check --apply-unsafe .",
+    "fix:biome": "biome check --apply .",
     "changelog": "ts-node ./scripts/get-commit-list.ts",
     "link:yarn": "lerna exec yarn link",
-    "lint": "run-s lint:lerna lint:biome",
+    "lint": "run-p lint:lerna lint:biome",
     "lint:lerna": "lerna run lint",
     "lint:biome": "biome check .",
     "validate:es5": "lerna run validate:es5",

packages/angular/test/tracing.test.ts

@@ -1,6 +1,5 @@
 import { Component } from '@angular/core';
 import type { ActivatedRouteSnapshot, CanActivate, RouterStateSnapshot } from '@angular/router';
-import type { Hub } from '@sentry/types';
 
 import { TraceClassDecorator, TraceDirective, TraceMethodDecorator, instrumentAngularRouting } from '../src';
 import { getParameterizedRouteFromSnapshot } from '../src/tracing';

packages/astro/test/server/middleware.test.ts

@@ -1,6 +1,6 @@
 import * as SentryNode from '@sentry/node';
 import type { Client } from '@sentry/types';
-import { SpyInstance, vi } from 'vitest';
+import { vi } from 'vitest';
 
 import { handleRequest, interpolateRouteFromUrlAndParams } from '../../src/server/middleware';
 

packages/browser-integration-tests/suites/public-api/startTransaction/init.js

@@ -1,5 +1,6 @@
+/* eslint-disable no-unused-vars */
 import * as Sentry from '@sentry/browser';
-// eslint-disable-next-line no-unused-vars
+// biome-ignore lint/nursery/noUnusedImports: Need to import tracing for side effect
 import * as _ from '@sentry/tracing';
 
 window.Sentry = Sentry;

packages/browser/src/integrations/breadcrumbs.ts

@@ -1,4 +1,3 @@
-import { get } from 'http';
 /* eslint-disable max-lines */
 import { addBreadcrumb, convertIntegrationFnToClass, getClient } from '@sentry/core';
 import type {

packages/browser/src/integrations/trycatch.ts

@@ -1,5 +1,5 @@
 import { convertIntegrationFnToClass } from '@sentry/core';
-import type { Client, IntegrationFn, WrappedFunction } from '@sentry/types';
+import type { IntegrationFn, WrappedFunction } from '@sentry/types';
 import { fill, getFunctionName, getOriginalFunction } from '@sentry/utils';
 
 import { WINDOW, wrap } from '../helpers';

packages/bun/src/index.ts

@@ -70,6 +70,7 @@ export {
   startInactiveSpan,
   startSpanManual,
   continueTrace,
+  metrics,
 } from '@sentry/core';
 export type { SpanStatusType } from '@sentry/core';
 export { autoDiscoverNodePerformanceMonitoringIntegrations } from '@sentry/node';

packages/core/src/integration.ts

@@ -1,14 +1,4 @@
-import type {
-  Client,
-  Event,
-  EventHint,
-  EventProcessor,
-  Hub,
-  Integration,
-  IntegrationClass,
-  IntegrationFn,
-  Options,
-} from '@sentry/types';
+import type { Client, Event, EventHint, EventProcessor, Hub, Integration, IntegrationFn, Options } from '@sentry/types';
 import { arrayify, logger } from '@sentry/utils';
 
 import { DEBUG_BUILD } from './debug-build';

packages/core/src/integrations/metadata.ts

@@ -1,4 +1,4 @@
-import type { Event, EventItem, IntegrationFn } from '@sentry/types';
+import type { EventItem, IntegrationFn } from '@sentry/types';
 import { forEachEnvelopeItem } from '@sentry/utils';
 import { convertIntegrationFnToClass } from '../integration';