ref(nestjs): clarify exception predicate name

Author: psh4607

packages/nestjs/src/helpers.ts

@@ -25,21 +25,20 @@ export function isExpectedError(exception: unknown): boolean {
     return true;
   }
 
-  // RpcException / WsException (same duck-type shape)
-  if (isWsException(exception)) {
+  if (isWsOrRpcException(exception)) {
     return true;
   }
 
   return false;
 }
 
 /**
- * Determines if the exception is a WsException (or RpcException, which has the same shape).
+ * Determines if the exception is a WsException or RpcException, which have the same shape.
  * Both have `getError()` and `initMessage()` methods.
  *
  * We use duck-typing to avoid importing from `@nestjs/websockets` or `@nestjs/microservices`.
  */
-export function isWsException(exception: unknown): boolean {
+export function isWsOrRpcException(exception: unknown): boolean {
   if (typeof exception !== 'object' || exception === null) {
     return false;
   }

packages/nestjs/src/setup.ts

@@ -10,7 +10,7 @@ import { Catch, Global, HttpException, Injectable, Logger, Module } from '@nestj
 import { APP_INTERCEPTOR, BaseExceptionFilter } from '@nestjs/core';
 import { captureException, debug, getDefaultIsolationScope, getIsolationScope } from '@sentry/core';
 import type { Observable } from 'rxjs';
-import { isExpectedError, isWsException } from './helpers';
+import { isExpectedError, isWsOrRpcException } from './helpers';
 
 // Partial extract of FastifyRequest interface
 // https://github.com/fastify/fastify/blob/87f9f20687c938828f1138f91682d568d2a31e53/types/request.d.ts#L41
@@ -164,7 +164,7 @@ class SentryGlobalFilter extends BaseExceptionFilter {
 
       const client = host.switchToWs().getClient<{ emit?: (event: string, data: unknown) => void }>();
 
-      if (isWsException(exception)) {
+      if (isWsOrRpcException(exception)) {
         const result = (exception as { getError: () => unknown }).getError();
         const response = typeof result === 'object' && result !== null ? result : { status: 'error', message: result };
         client.emit?.('exception', response);

packages/nestjs/test/sentry-global-filter.test.ts

@@ -8,7 +8,7 @@ import { SentryGlobalFilter } from '../src/setup';
 
 vi.mock('../src/helpers', () => ({
   isExpectedError: vi.fn(),
-  isWsException: vi.fn(),
+  isWsOrRpcException: vi.fn(),
 }));
 
 vi.mock('@sentry/core', () => ({
@@ -28,7 +28,7 @@ describe('SentryGlobalFilter', () => {
   let mockLoggerError: any;
   let mockLoggerWarn: any;
   let isExpectedErrorMock: any;
-  let isWsExceptionMock: any;
+  let isWsOrRpcExceptionMock: any;
 
   beforeEach(() => {
     vi.clearAllMocks();
@@ -59,7 +59,7 @@ describe('SentryGlobalFilter', () => {
     mockCaptureException = vi.spyOn(SentryCore, 'captureException').mockReturnValue('mock-event-id');
 
     isExpectedErrorMock = vi.mocked(Helpers.isExpectedError).mockImplementation(() => false);
-    isWsExceptionMock = vi.mocked(Helpers.isWsException).mockImplementation(() => false);
+    isWsOrRpcExceptionMock = vi.mocked(Helpers.isWsOrRpcException).mockImplementation(() => false);
   });
 
   describe('HTTP context', () => {
@@ -274,7 +274,7 @@ describe('SentryGlobalFilter', () => {
 
     it('does not capture expected WebSocket exceptions and emits their response', () => {
       isExpectedErrorMock.mockReturnValueOnce(true);
-      isWsExceptionMock.mockReturnValueOnce(true);
+      isWsOrRpcExceptionMock.mockReturnValueOnce(true);
       const exception = {
         getError: () => 'Expected WebSocket exception',
         initMessage: vi.fn(),