lint: fix unused vars by prefixing _

getsentry · logaretm · Mar 10, 2026 · Feb 2, 2026 · Feb 2, 2026 · Feb 19, 2026
commit d840f01286043ba40084a249aa17c0b2ea2d5852
diff --git a/.oxlintrc.json b/.oxlintrc.json
@@ -9,6 +9,8 @@
   ],
   "categories": {},
   "rules": {
+    "no-unused-vars": ["warn", { "argsIgnorePattern": "^_", "varsIgnorePattern": "^_", "caughtErrorsIgnorePattern": "^_" }],
+
     // === Base rules from eslint-config-sdk/base.js ===
     "no-console": "error",
     "no-alert": "error",

@@ -364,7 +364,7 @@ export function replayEnvelopeIsCompressed(resOrReq: Request | Response): boolea
   const lines: boolean[] = envelopeString.split('\n').map(line => {
     try {
       JSON.parse(line);
-    } catch (error) {
+    } catch (_error) {
-    } catch (_error) {
+    } {
-    } catch (_error) {
+    } {
       // If we fail to parse a line, we _might_ have found a compressed payload,
       // so let's check if this is actually the case.
       // This is quite hacky but we can't go through `line` because the prior operations
@@ -394,7 +394,7 @@ export const replayEnvelopeParser = (request: Request | null): unknown[] => {
   const lines = envelopeString.split('\n').map(line => {
     try {
       return JSON.parse(line);
-    } catch (error) {
+    } catch (_error) {
       // If we fail to parse a line, we _might_ have found a compressed payload,
       // so let's check if this is actually the case.
       // This is quite hacky but we can't go through `line` because the prior operations

@@ -102,7 +102,7 @@ async function run() {
           },
         ],
       });
-    } catch (error) {
+    } catch (_error) {
       // Expected error
     }
   });

diff --git a/dev-packages/node-overhead-gh-action/index.mjs b/dev-packages/node-overhead-gh-action/index.mjs
@@ -157,7 +157,7 @@ async function run() {
             body,
           });
         }
-      } catch (error) {
+      } catch (_error) {
         core.error(
           "Error updating comment. This can happen for PR's originating from a fork without write permissions.",
         );

diff --git a/dev-packages/size-limit-gh-action/index.mjs b/dev-packages/size-limit-gh-action/index.mjs
@@ -171,7 +171,7 @@ async function run() {
             body,
           });
         }
-      } catch (error) {
+      } catch (_error) {
         core.error(
           "Error updating comment. This can happen for PR's originating from a fork without write permissions.",
         );

@@ -27,7 +27,7 @@ export function initUnique<T>(identityObj: object, ClassObj: new () => T): T {
       instanceMap.set(identityObj, new ClassObj());
     }
     return instanceMap.get(identityObj)! as T;
-  } catch (e) {
+  } catch (_e) {
     // --- START Sentry-custom code (try/catch wrapping) ---
     // Fix for cases where identityObj is not a valid key for WeakMap (sometimes a problem in Safari)
     // Just return a new instance without caching it in instanceMap

@@ -614,7 +614,7 @@ export function startJSSelfProfile(): JSSelfProfiler | undefined {
   // as we risk breaking the user's application, so just disable profiling and log an error.
   try {
     return new JSProfilerConstructor({ sampleInterval: samplingIntervalMS, maxBufferSize: maxSamples });
-  } catch (e) {
+  } catch (_e) {
     if (DEBUG_BUILD) {
       debug.log(
         "[Profiling] Failed to initialize the Profiling constructor, this is likely due to a missing 'Document-Policy': 'js-profiling' header.",

@@ -10,7 +10,7 @@ const https = require('https');
 const installScriptUrl = 'https://bun.sh/install';
 
 // Check if bun is installed
-exec('bun --version', (error, version) => {
+exec('bun --version', (error, _version) => {
   if (error) {
     console.error('bun is not installed. Installing...');
     installLatestBun();

@@ -162,7 +162,7 @@ export function wrapRequestHandler(
                 statusText: res.statusText,
                 headers: res.headers,
               });
-            } catch (e) {
+            } catch (_e) {
               // tee() failed (e.g stream already locked) - fall back to non-streaming handling
               span.end();
               waitUntil?.(flushAndDispose(client));

@@ -26,7 +26,7 @@ const mockStep: WorkflowStep = {
             } else {
               return await (maybeCallback ? maybeCallback() : Promise.resolve());
             }
-          } catch (error) {
+          } catch (_error) {
             await new Promise(resolve => setTimeout(resolve, 1000));
           }
         }

diff --git a/packages/core/src/integrations/mcp-server/handlers.ts b/packages/core/src/integrations/mcp-server/handlers.ts
@@ -121,7 +121,7 @@ function captureHandlerError(error: Error, methodName: keyof MCPServerInstance,
       extraData.prompt_name = handlerName;
       captureError(error, 'prompt_execution', extraData);
     }
-  } catch (captureErr) {
+  } catch (_captureErr) {
     // noop
   }
 }

diff --git a/packages/core/src/utils/exports.ts b/packages/core/src/utils/exports.ts
@@ -21,7 +21,7 @@ export function replaceExports(
   // Replace the named export - handle read-only properties
   try {
     exports[exportName] = wrappedConstructor;
-  } catch (error) {
+  } catch (_error) {
     // If direct assignment fails, override the property descriptor
     Object.defineProperty(exports, exportName, {
       value: wrappedConstructor,
@@ -35,7 +35,7 @@ export function replaceExports(
   if (exports.default === original) {
     try {
       exports.default = wrappedConstructor;
-    } catch (error) {
+    } catch (_error) {
       Object.defineProperty(exports, 'default', {
         value: wrappedConstructor,
         writable: true,

diff --git a/packages/core/test/lib/transports/base.test.ts b/packages/core/test/lib/transports/base.test.ts
@@ -339,7 +339,7 @@ describe('createTransport', () => {
 
         try {
           await transport.send(CLIENT_REPORT_ENVELOPE);
-        } catch (e) {
+        } catch (_e) {
           // Expected to throw
         }
 
@@ -383,7 +383,7 @@ describe('createTransport', () => {
 
         try {
           await transport.send(ERROR_ENVELOPE);
-        } catch (e) {
+        } catch (_e) {
           // Expected to throw
         }
 

@@ -66,7 +66,7 @@ export async function streamResponse(span: Span, res: Response): Promise<Respons
         headers: res.headers,
       },
     );
-  } catch (e) {
+  } catch (_e) {
     // tee() failed - handle without streaming
     span.end();
     return res;

@@ -63,7 +63,7 @@ describe('GCPFunction', () => {
 
       try {
         fn(req, res);
-      } catch (error) {
+      } catch (_error) {
         res.end();
       }
     });

@@ -48,7 +48,7 @@ export function makeNodeTransport(options: NodeTransportOptions): Transport {
 
   try {
     urlSegments = new URL(options.url);
-  } catch (e) {
+  } catch (_e) {
     consoleSandbox(() => {
       // eslint-disable-next-line no-console
       console.warn(

@@ -60,7 +60,7 @@ export function patchRequestToCaptureBody(
                     `Dropping request body chunk because maximum body length of ${maxBodySize}b is exceeded.`,
                   );
                 }
-              } catch (err) {
+              } catch (_err) {
                 DEBUG_BUILD && debug.error(integrationName, 'Encountered error while storing body chunk.');
               }
 

@@ -89,7 +89,7 @@ export class SentryAnthropicAiInstrumentation extends InstrumentationBase<Anthro
     // The Anthropic property might have only a getter, so use defineProperty
     try {
       exports.Anthropic = WrappedAnthropic;
-    } catch (error) {
+    } catch (_error) {
       // If direct assignment fails, override the property descriptor
       Object.defineProperty(exports, 'Anthropic', {
         value: WrappedAnthropic,
@@ -105,7 +105,7 @@ export class SentryAnthropicAiInstrumentation extends InstrumentationBase<Anthro
     if (exports.default === Original) {
       try {
         exports.default = WrappedAnthropic;
-      } catch (error) {
+      } catch (_error) {
         // If direct assignment fails, override the property descriptor
         Object.defineProperty(exports, 'default', {
           value: WrappedAnthropic,

@@ -103,7 +103,7 @@ export class SentryOpenAiInstrumentation extends InstrumentationBase<OpenAiInstr
     // The OpenAI property might have only a getter, so use defineProperty
     try {
       exports[exportKey] = WrappedOpenAI;
-    } catch (error) {
+    } catch (_error) {
       // If direct assignment fails, override the property descriptor
       Object.defineProperty(exports, exportKey, {
         value: WrappedOpenAI,
@@ -119,7 +119,7 @@ export class SentryOpenAiInstrumentation extends InstrumentationBase<OpenAiInstr
     if (exports.default === Original) {
       try {
         exports.default = WrappedOpenAI;
-      } catch (error) {
+      } catch (_error) {
         // If direct assignment fails, override the property descriptor
         Object.defineProperty(exports, 'default', {
           value: WrappedOpenAI,

@@ -262,7 +262,7 @@ function isCacheHit(key: string, value: unknown): boolean {
     }
 
     return validateCacheEntry(key, JSON.parse(String(value)) as CacheEntry);
-  } catch (error) {
+  } catch (_error) {
     // this is a best effort, so we return false if we can't validate the cache entry
     return false;
   }

@@ -157,7 +157,7 @@ export function hoistNonReactStatics<
           try {
             // Avoid failures from read-only properties
             defineProperty(targetComponent, key, descriptor);
-          } catch (e) {
+          } catch (_e) {
             // Silently ignore errors
           }
         }

@@ -92,7 +92,7 @@ function getManifest(): RouteManifest | null {
     cachedManifest = manifest;
     cachedManifestString = currentManifestString;
     return manifest;
-  } catch (error) {
+  } catch (_error) {
     DEBUG_BUILD && debug.warn('Could not extract route manifest');
     return null;
   }

@@ -314,7 +314,7 @@ describe('sentryHandle', () => {
     it('send errors to Sentry', async () => {
       try {
         await sentryHandle()({ event: mockEvent(), resolve: resolve(type, isError) });
-      } catch (e) {
+      } catch (_e) {
         expect(mockCaptureException).toBeCalledTimes(1);
         expect(mockCaptureException).toBeCalledWith(expect.any(Error), {
           mechanism: { handled: false, type: 'auto.function.sveltekit.handle' },
-Original file line number
+Diff line change
@@ Expand Up / @@ -102,7 +102,7 @@ async function run() { @@
               },
             ],
           });
-        } catch (error) {
+        } catch (_error) {
           // Expected error
         }
       });
@@ Expand Down @@