feat(v8/node): Remove LocalVariables class integration (#10558)

Works toward #8844
in a more incremental way.

Author: AbhiPrasad

dev-packages/e2e-tests/test-applications/create-next-app/sentry.server.config.ts

@@ -15,12 +15,7 @@ Sentry.init({
   dsn: process.env.NEXT_PUBLIC_E2E_TEST_DSN,
   // Adjust this value in production, or use tracesSampler for greater control
   tracesSampleRate: 1.0,
-  // ...
-  // Note: if you want to override the automatic release value, do not set a
-  // `release` value here - use the environment variable `SENTRY_RELEASE`, so
-  // that it will also get attached to your source maps
-
-  integrations: [new Sentry.Integrations.LocalVariables()],
+  integrations: [Sentry.localVariablesIntegration()],
 });
 
 Sentry.addEventProcessor(event => {

dev-packages/node-integration-tests/suites/public-api/LocalVariables/local-variables-memory-test.js

@@ -7,7 +7,7 @@ Sentry.init({
   includeLocalVariables: true,
   transport: loggingTransport,
   // Stop the rate limiting from kicking in
-  integrations: [new Sentry.Integrations.LocalVariables({ maxExceptionsPerSecond: 10000000 })],
+  integrations: [Sentry.localVariablesIntegration({ maxExceptionsPerSecond: 10000000 })],
 });
 
 class Some {

packages/bun/src/sdk.ts

@@ -37,7 +37,6 @@ export const defaultIntegrations = [
   // new NodeIntegrations.OnUnhandledRejection(),
   // Event Info
   contextLinesIntegration(),
-  // new NodeIntegrations.LocalVariables(), # does't work with Bun
   nodeContextIntegration(),
   modulesIntegration(),
   // Bun Specific

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

@@ -8,20 +8,10 @@ const {
   ContextLines,
   Context,
   RequestData,
-  LocalVariables,
   // eslint-disable-next-line deprecation/deprecation
 } = NodeIntegrations;
 
-export {
-  Console,
-  OnUncaughtException,
-  OnUnhandledRejection,
-  Modules,
-  ContextLines,
-  Context,
-  RequestData,
-  LocalVariables,
-};
+export { Console, OnUncaughtException, OnUnhandledRejection, Modules, ContextLines, Context, RequestData };
 
 /* eslint-disable deprecation/deprecation */
 export { Express } from './express';

packages/node/src/integrations/index.ts

@@ -7,7 +7,6 @@ export { Modules } from './modules';
 export { ContextLines } from './contextlines';
 export { Context } from './context';
 export { RequestData } from '@sentry/core';
-export { LocalVariables } from './local-variables';
 export { Undici } from './undici';
 export { Spotlight } from './spotlight';
 export { Anr } from './anr';

packages/node/src/integrations/local-variables/index.ts

@@ -1,13 +1,3 @@
-import { LocalVariablesSync, localVariablesSyncIntegration } from './local-variables-sync';
-
-/**
- * Adds local variables to exception frames.
- *
- * @deprecated Use `localVariablesIntegration()` instead.
- */
-// eslint-disable-next-line deprecation/deprecation
-export const LocalVariables = LocalVariablesSync;
-// eslint-disable-next-line deprecation/deprecation
-export type LocalVariables = LocalVariablesSync;
+import { localVariablesSyncIntegration } from './local-variables-sync';
 
 export const localVariablesIntegration = localVariablesSyncIntegration;