fix linting, improve comments

Author: s1gr1d

packages/core/src/integrations/express/index.ts

@@ -223,8 +223,8 @@ export function expressErrorHandler(options?: ExpressHandlerOptions): ExpressErr
   ): void {
     // When an error happens, the `expressRequestHandler` middleware does not run, so we set it here too
     setSDKProcessingMetadata(request);
-    // oxlint-disable-next-line typescript/no-deprecated
     const shouldHandleError =
+      // oxlint-disable-next-line typescript/no-deprecated
       options?.shouldHandleError ?? getIntegrationShouldHandleError() ?? defaultShouldHandleError;
 
     if (shouldHandleError === false) {
@@ -249,8 +249,7 @@ export function expressErrorHandler(options?: ExpressHandlerOptions): ExpressErr
  * The error handler must be before any other middleware and after all controllers.
  *
  * @param app The Express instances
- * @param options {ExpressHandlerOptions} Configuration options for the handler. Deprecated: set
- * `shouldHandleError` on `expressIntegration()` instead. This parameter will be removed in v11.
+ * @param options {ExpressHandlerOptions} Configuration options for the handler
  *
  * @example
  * ```javascript

packages/core/src/integrations/express/types.ts

@@ -228,7 +228,22 @@ export interface ExpressHandlerOptions {
    *
    * @param error Captured middleware error
    *
-   * @deprecated Set `shouldHandleError` on `expressIntegration()` instead. This option will be removed in v11.
+   * @deprecated Configure `shouldHandleError` on `expressIntegration()` rather than here. Keep calling
+   * `setupExpressErrorHandler(app)` as that is what captures the errors. This option will be removed in v11.
+   *
+   * @example
+   *
+   * ```javascript
+   * Sentry.init({
+   *   integrations: [
+   *     Sentry.expressIntegration({
+   *       shouldHandleError(error) {
+   *         return (error.statusCode ?? 500) >= 500;
+   *       },
+   *     }),
+   *   ],
+   * });
+   * ```
    */
   shouldHandleError?(this: void, error: MiddlewareError): boolean;
 }

packages/node/src/integrations/tracing/fastify/index.ts

@@ -77,8 +77,10 @@ interface FastifyHandlerOptions {
    * @param request Fastify request (or any object containing at least method, routeOptions.url, and routerPath)
    * @param reply Fastify reply (or any object containing at least statusCode)
    *
-   * @deprecated Set `shouldHandleError` on `fastifyIntegration()` instead. It applies to every supported
-   * Fastify version. This option will be removed in v11.
+   * @deprecated Configure `shouldHandleError` on `fastifyIntegration()` rather than here, where it
+   * applies to every supported Fastify version. This option will be removed in v11.
+   *
+   * @example
    *
    * ```javascript
    * Sentry.init({
@@ -145,8 +147,7 @@ export const fastifyIntegration = defineIntegration((options: Partial<FastifyInt
  * Add an Fastify error handler to capture errors to Sentry.
  *
  * @param fastify The Fastify instance to which to add the error handler
- * @param options Configuration options for the handler. Deprecated: set `shouldHandleError` on
- * `fastifyIntegration()` instead. This parameter will be removed in v11.
+ * @param options Configuration options for the handler
  *
  * @example
  * ```javascript