chore(biome): enable noUnusedImports rule

biome.json

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

package.json

@@ -20,7 +20,7 @@
     "codecov": "codecov",
     "fix": "run-s 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",

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,6 +1,5 @@
 import * as Sentry from '@sentry/browser';
-// eslint-disable-next-line no-unused-vars
-import * as _ from '@sentry/tracing';
+import '@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/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';
 

packages/core/src/metrics/utils.ts

@@ -1,6 +1,6 @@
 import type { MeasurementUnit, MetricBucketItem } from '@sentry/types';
 import { dropUndefinedKeys } from '@sentry/utils';
-import type { MetricType, SimpleMetricBucket } from './types';
+import type { MetricType } from './types';
 
 /**
  * Generate bucket key from metric properties.

packages/core/src/scope.ts

@@ -23,7 +23,7 @@ import type {
   Transaction,
   User,
 } from '@sentry/types';
-import { arrayify, dateTimestampInSeconds, isPlainObject, uuid4 } from '@sentry/utils';
+import { dateTimestampInSeconds, isPlainObject, uuid4 } from '@sentry/utils';
 
 import { getGlobalEventProcessors, notifyEventProcessors } from './eventProcessors';
 import { updateSession } from './session';

packages/deno/src/integrations/deno-cron.ts

@@ -1,6 +1,5 @@
 import { withMonitor } from '@sentry/core';
 import type { Integration } from '@sentry/types';
-import type { DenoClient } from '../client';
 import { parseScheduleToString } from './deno-cron-format';
 
 type CronOptions = { backoffSchedule?: number[]; signal?: AbortSignal };

packages/deno/src/integrations/globalhandlers.ts

@@ -1,7 +1,5 @@
 import type { ServerRuntimeClient } from '@sentry/core';
-import { captureEvent } from '@sentry/core';
-import { getClient } from '@sentry/core';
-import { flush } from '@sentry/core';
+import { captureEvent, flush, getClient } from '@sentry/core';
 import type { Client, Event, Integration, Primitive, StackParser } from '@sentry/types';
 import { eventFromUnknownInput, isPrimitive } from '@sentry/utils';
 

packages/e2e-tests/test-applications/create-remix-app-v2/app/entry.server.tsx

@@ -6,7 +6,7 @@
 
 import { PassThrough } from 'node:stream';
 
-import type { AppLoadContext, DataFunctionArgs, EntryContext } from '@remix-run/node';
+import type { AppLoadContext, EntryContext } from '@remix-run/node';
 import { createReadableStreamFromReadable } from '@remix-run/node';
 import { installGlobals } from '@remix-run/node';
 import { RemixServer } from '@remix-run/react';

packages/e2e-tests/test-applications/create-remix-app/app/entry.server.tsx

@@ -6,7 +6,7 @@
 
 import { PassThrough } from 'node:stream';
 
-import type { AppLoadContext, DataFunctionArgs, EntryContext } from '@remix-run/node';
+import type { AppLoadContext, EntryContext } from '@remix-run/node';
 import { Response } from '@remix-run/node';
 import { RemixServer } from '@remix-run/react';
 import * as Sentry from '@sentry/remix';

packages/e2e-tests/test-applications/generic-ts3.8/index.ts

@@ -1,13 +1,22 @@
-/* eslint-disable @typescript-eslint/no-unused-vars */
-// we need to import the SDK to ensure tsc check the types
+// biome-ignore lint/nursery/noUnusedImports: we need to import the SDK to ensure tsc check the types
 import * as _SentryBrowser from '@sentry/browser';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryCore from '@sentry/core';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryHub from '@sentry/hub';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryIntegrations from '@sentry/integrations';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryNode from '@sentry/node';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryOpentelemetry from '@sentry/opentelemetry-node';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryReplay from '@sentry/replay';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryTracing from '@sentry/tracing';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryTypes from '@sentry/types';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryUtils from '@sentry/utils';
+// biome-ignore lint/nursery/noUnusedImports:
 import * as _SentryWasm from '@sentry/wasm';

packages/e2e-tests/test-applications/nextjs-app-dir/tests/async-context-edge.test.ts

@@ -1,5 +1,5 @@
 import { expect, test } from '@playwright/test';
-import { waitForError, waitForTransaction } from '../event-proxy-server';
+import { waitForTransaction } from '../event-proxy-server';
 
 test('Should allow for async context isolation in the edge SDK', async ({ request }) => {
   // test.skip(process.env.TEST_ENV === 'development', "Doesn't work in dev mode.");

packages/e2e-tests/test-applications/node-hapi-app/tests/server.test.ts

@@ -1,5 +1,5 @@
 import { expect, test } from '@playwright/test';
-import axios, { AxiosError, AxiosResponse } from 'axios';
+import axios, { AxiosError } from 'axios';
 import { waitForError, waitForTransaction } from '../event-proxy-server';
 
 const authToken = process.env.E2E_TEST_AUTH_TOKEN;

packages/e2e-tests/test-applications/react-create-hash-router/src/pages/Index.tsx

@@ -1,4 +1,5 @@
 import * as Sentry from '@sentry/react';
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 import { Link } from 'react-router-dom';
 

packages/e2e-tests/test-applications/react-create-hash-router/src/pages/User.tsx

@@ -1,3 +1,4 @@
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 
 const User = () => {

packages/e2e-tests/test-applications/react-router-6-use-routes/src/pages/Index.tsx

@@ -1,4 +1,5 @@
 import * as Sentry from '@sentry/react';
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 import { Link } from 'react-router-dom';
 

packages/e2e-tests/test-applications/react-router-6-use-routes/src/pages/User.tsx

@@ -1,3 +1,4 @@
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 
 const User = () => {

packages/e2e-tests/test-applications/standard-frontend-react-tracing-import/src/pages/Index.tsx

@@ -1,4 +1,5 @@
 import * as Sentry from '@sentry/react';
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 import { Link } from 'react-router-dom';
 

packages/e2e-tests/test-applications/standard-frontend-react-tracing-import/src/pages/User.tsx

@@ -1,3 +1,4 @@
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 
 const User = () => {

packages/e2e-tests/test-applications/standard-frontend-react/src/pages/Index.tsx

@@ -1,4 +1,5 @@
 import * as Sentry from '@sentry/react';
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 import { Link } from 'react-router-dom';
 

packages/e2e-tests/test-applications/standard-frontend-react/src/pages/User.tsx

@@ -1,3 +1,4 @@
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 
 const User = () => {

packages/gatsby/test/integration.test.tsx

@@ -2,7 +2,7 @@ import { TextDecoder, TextEncoder } from 'util';
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { render } from '@testing-library/react';
 import { useEffect } from 'react';
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 
 import { onClientEntry } from '../gatsby-browser';

packages/integrations/test/debug.test.ts

@@ -1,4 +1,4 @@
-import type { Client, Event, EventHint, Hub, Integration } from '@sentry/types';
+import type { Client, Event, EventHint } from '@sentry/types';
 
 import { Debug } from '../src/debug';
 

packages/nextjs/src/common/_error.ts

@@ -1,4 +1,4 @@
-import { captureException, getClient, withScope } from '@sentry/core';
+import { captureException, withScope } from '@sentry/core';
 import type { NextPageContext } from 'next';
 import { flushQueue } from './utils/responseEnd';
 

packages/nextjs/src/config/templates/routeHandlerWrapperTemplate.ts

@@ -2,8 +2,10 @@ import * as Sentry from '@sentry/nextjs';
 import type { WebFetchHeaders } from '@sentry/types';
 // @ts-expect-error Because we cannot be sure if the RequestAsyncStorage module exists (it is not part of the Next.js public
 // API) we use a shim if it doesn't exist. The logic for this is in the wrapping loader.
+// biome-ignore lint/nursery/noUnusedImports: Biome doesn't understand declare const
 import { requestAsyncStorage } from '__SENTRY_NEXTJS_REQUEST_ASYNC_STORAGE_SHIM__';
 // @ts-expect-error See above
+// biome-ignore lint/nursery/noUnusedImports: Biome doesn't understand declare const
 import * as routeModule from '__SENTRY_WRAPPING_TARGET_FILE__';
 
 import type { RequestAsyncStorage } from './requestAsyncStorageShim';

packages/nextjs/src/config/templates/serverComponentWrapperTemplate.ts

@@ -2,8 +2,10 @@ import * as Sentry from '@sentry/nextjs';
 import type { WebFetchHeaders } from '@sentry/types';
 // @ts-expect-error Because we cannot be sure if the RequestAsyncStorage module exists (it is not part of the Next.js public
 // API) we use a shim if it doesn't exist. The logic for this is in the wrapping loader.
+// biome-ignore lint/nursery/noUnusedImports: Biome doesn't understand the shim with variable import path
 import { requestAsyncStorage } from '__SENTRY_NEXTJS_REQUEST_ASYNC_STORAGE_SHIM__';
 // @ts-expect-error We use `__SENTRY_WRAPPING_TARGET_FILE__` as a placeholder for the path to the file being wrapped.
+// biome-ignore lint/nursery/noUnusedImports: Biome doesn't understand the shim with variable import path
 import * as serverComponentModule from '__SENTRY_WRAPPING_TARGET_FILE__';
 
 import type { RequestAsyncStorage } from './requestAsyncStorageShim';

packages/nextjs/src/edge/wrapApiHandlerWithSentry.ts

@@ -1,4 +1,4 @@
-import { getCurrentHub, getCurrentScope } from '@sentry/core';
+import { getCurrentScope } from '@sentry/core';
 
 import { withEdgeWrapping } from '../common/utils/edgeWrapperUtils';
 import type { EdgeRouteHandler } from './types';

packages/nextjs/test/integration/components/Layout.tsx

@@ -1,6 +1,6 @@
 import Head from 'next/head';
 import Link from 'next/link';
-import React, { ReactNode } from 'react';
+import { ReactNode } from 'react';
 
 type Props = {
   children?: ReactNode;

packages/nextjs/test/integration/components/List.tsx

@@ -1,3 +1,4 @@
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 import { User } from '../interfaces';
 import ListItem from './ListItem';

packages/nextjs/test/integration/components/ListDetail.tsx

@@ -1,3 +1,4 @@
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 
 import { User } from '../interfaces';

packages/nextjs/test/integration/components/ListItem.tsx

@@ -1,4 +1,5 @@
 import Link from 'next/link';
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import React from 'react';
 
 import { User } from '../interfaces';

packages/node-experimental/src/integrations/http.ts

@@ -4,12 +4,11 @@ import { SpanKind } from '@opentelemetry/api';
 import { registerInstrumentations } from '@opentelemetry/instrumentation';
 import { HttpInstrumentation } from '@opentelemetry/instrumentation-http';
 import { addBreadcrumb, hasTracingEnabled, isSentryRequestUrl } from '@sentry/core';
-import { _INTERNAL, getClient, getCurrentHub, getSpanKind, setSpanMetadata } from '@sentry/opentelemetry';
+import { _INTERNAL, getClient, getSpanKind, setSpanMetadata } from '@sentry/opentelemetry';
 import type { EventProcessor, Hub, Integration } from '@sentry/types';
 import { stringMatchesSomePattern } from '@sentry/utils';
 
 import { getIsolationScope, setIsolationScope } from '../sdk/api';
-import { Scope } from '../sdk/scope';
 import type { NodeExperimentalClient } from '../types';
 import { addOriginToSpan } from '../utils/addOriginToSpan';
 import { getRequestUrl } from '../utils/getRequestUrl';

packages/node-experimental/src/otel/contextManager.ts

@@ -4,7 +4,6 @@ import { setHubOnContext } from '@sentry/opentelemetry';
 import { getCurrentHub } from '../sdk/hub';
 
 import { getCurrentScope, getIsolationScope } from './../sdk/api';
-import { Scope } from './../sdk/scope';
 import type { CurrentScopes } from './../sdk/types';
 import { getScopesFromContext, setScopesOnContext } from './../utils/contextData';
 

packages/node-experimental/src/sdk/api.ts

@@ -6,7 +6,6 @@ import type {
   Breadcrumb,
   BreadcrumbHint,
   CaptureContext,
-  Client,
   Event,
   EventHint,
   EventProcessor,

packages/node-experimental/src/sdk/hub.ts

@@ -5,7 +5,6 @@ import type {
   Hub,
   Integration,
   IntegrationClass,
-  Session,
   Severity,
   SeverityLevel,
   TransactionContext,
@@ -14,8 +13,6 @@ import type {
 import {
   addBreadcrumb,
   captureEvent,
-  captureException,
-  captureMessage,
   configureScope,
   endSession,
   getClient,

packages/node-experimental/test/integration/transactions.test.ts

@@ -8,7 +8,6 @@ import { logger } from '@sentry/utils';
 import * as Sentry from '../../src';
 import { startSpan } from '../../src';
 import type { Http, NodeFetch } from '../../src/integrations';
-import { getIsolationScope } from '../../src/sdk/api';
 import type { NodeExperimentalClient } from '../../src/types';
 import { cleanupOtel, getProvider, mockSdkInit } from '../helpers/mockSdkInit';
 

packages/node/test/integrations/localvariables.test.ts

@@ -1,4 +1,3 @@
-import type { ClientOptions, EventProcessor } from '@sentry/types';
 import type { LRUMap } from '@sentry/utils';
 import type { Debugger, InspectorNotification } from 'inspector';
 

packages/opentelemetry/src/spanExporter.ts

@@ -1,4 +1,3 @@
-import type { Span } from '@opentelemetry/api';
 import { SpanKind } from '@opentelemetry/api';
 import type { ExportResult } from '@opentelemetry/core';
 import { ExportResultCode } from '@opentelemetry/core';
@@ -9,7 +8,6 @@ import type { DynamicSamplingContext, Span as SentrySpan, SpanOrigin, Transactio
 import { logger } from '@sentry/utils';
 
 import { getCurrentHub } from './custom/hub';
-import { OpenTelemetryScope } from './custom/scope';
 import type { OpenTelemetryTransaction } from './custom/transaction';
 import { startTransaction } from './custom/transaction';
 import { DEBUG_BUILD } from './debug-build';

packages/react/test/profiler.test.tsx

@@ -1,6 +1,7 @@
 import type { SpanContext } from '@sentry/types';
 import { render } from '@testing-library/react';
 import { renderHook } from '@testing-library/react-hooks';
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 
 import { REACT_MOUNT_OP, REACT_RENDER_OP, REACT_UPDATE_OP } from '../src/constants';

packages/react/test/reactrouterv4.test.tsx

@@ -1,5 +1,6 @@
 import { act, render } from '@testing-library/react';
 import { createMemoryHistory } from 'history-4';
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 import { Route, Router, Switch, matchPath } from 'react-router-4';
 

packages/react/test/reactrouterv5.test.tsx

@@ -1,5 +1,6 @@
 import { act, render } from '@testing-library/react';
 import { createMemoryHistory } from 'history-4';
+// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
 import * as React from 'react';
 import { Route, Router, Switch, matchPath } from 'react-router-5';
 

packages/react/test/redux.test.ts

@@ -1,5 +1,4 @@
 import * as Sentry from '@sentry/browser';
-import type { Scope } from '@sentry/types';
 import * as Redux from 'redux';
 
 import { createReduxEnhancer } from '../src/redux';

packages/remix/test/integration/app_v2/entry.server.tsx

@@ -1,4 +1,4 @@
-import type { DataFunctionArgs, EntryContext } from '@remix-run/node';
+import type { EntryContext } from '@remix-run/node';
 import { RemixServer } from '@remix-run/react';
 import * as Sentry from '@sentry/remix';
 import { renderToString } from 'react-dom/server';

packages/remix/test/integration/common/routes/server-side-unexpected-errors.$id.tsx

@@ -1,4 +1,4 @@
-import { ActionFunction, LoaderFunction, json, redirect } from '@remix-run/node';
+import { ActionFunction } from '@remix-run/node';
 import { useActionData } from '@remix-run/react';
 
 export const action: ActionFunction = async ({ params: { id } }) => {

packages/replay/test/integration/rrweb.test.ts

@@ -1,4 +1,3 @@
-import type { CanvasManagerInterface } from '../../src/types';
 import { resetSdkMock } from '../mocks/resetSdkMock';
 import { useFakeTimers } from '../utils/use-fake-timers';
 

packages/sveltekit/src/client/handleError.ts

@@ -1,6 +1,6 @@
 import { captureException } from '@sentry/svelte';
 import { consoleSandbox } from '@sentry/utils';
-import type { HandleClientError, NavigationEvent } from '@sveltejs/kit';
+import type { HandleClientError } from '@sveltejs/kit';
 
 // The SvelteKit default error handler just logs the error to the console
 // see: https://github.com/sveltejs/kit/blob/369e7d6851f543a40c947e033bfc4a9506fdc0a8/packages/kit/src/core/sync/write_client_manifest.js#LL127C2-L127C2

packages/sveltekit/test/common/utils.test.ts

@@ -1,5 +1,3 @@
-import { redirect } from '@sveltejs/kit';
-
 import { isHttpError, isRedirect } from '../../src/common/utils';
 
 describe('isRedirect', () => {