@saihu/common 1.1.61 → 1.1.63

package/dist/filters/base-error-rpc.filter.d.ts

@@ -1,7 +1,7 @@
 import { ArgumentsHost } from '@nestjs/common';
 import { BaseRpcExceptionFilter } from '@nestjs/microservices';
-import { BaseError } from '../dto/base-res.dto';
 export declare class BaseErrorRpcFilter extends BaseRpcExceptionFilter {
-    catch(exception: BaseError, host: ArgumentsHost): import("rxjs").Observable<any>;
+    private readonly logger;
+    catch(exception: any, host: ArgumentsHost): import("rxjs").Observable<any>;
 }
 //# sourceMappingURL=base-error-rpc.filter.d.ts.map

package/dist/filters/base-error-rpc.filter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"base-error-rpc.filter.d.ts","sourceRoot":"","sources":["../../src/filters/base-error-rpc.filter.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,aAAa,EAAS,MAAM,gBAAgB,CAAC;AACtD,OAAO,EAAE,sBAAsB,EAAgB,MAAM,uBAAuB,CAAC;AAC7E,OAAO,EAAE,SAAS,EAAE,MAAM,qBAAqB,CAAC;AAEhD,qBACa,kBAAmB,SAAQ,sBAAsB;IAC5D,KAAK,CAAC,SAAS,EAAE,SAAS,EAAE,IAAI,EAAE,aAAa;CAWhD"}
+{"version":3,"file":"base-error-rpc.filter.d.ts","sourceRoot":"","sources":["../../src/filters/base-error-rpc.filter.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,aAAa,EAAiB,MAAM,gBAAgB,CAAC;AAC9D,OAAO,EAAE,sBAAsB,EAAgB,MAAM,uBAAuB,CAAC;AAG7E,qBACa,kBAAmB,SAAQ,sBAAsB;IAC5D,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAuC;IAE9D,KAAK,CAAC,SAAS,EAAE,GAAG,EAAE,IAAI,EAAE,aAAa;CAwC1C"}

package/dist/filters/base-error-rpc.filter.js

@@ -5,26 +5,56 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
     else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
     return c > 3 && r && Object.defineProperty(target, key, r), r;
 };
+var BaseErrorRpcFilter_1;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.BaseErrorRpcFilter = void 0;
 // @saihu/common/src/filters/base-error-rpc.filter.ts
 const common_1 = require("@nestjs/common");
 const microservices_1 = require("@nestjs/microservices");
 const base_res_dto_1 = require("../dto/base-res.dto");
-let BaseErrorRpcFilter = class BaseErrorRpcFilter extends microservices_1.BaseRpcExceptionFilter {
+let BaseErrorRpcFilter = BaseErrorRpcFilter_1 = class BaseErrorRpcFilter extends microservices_1.BaseRpcExceptionFilter {
+    constructor() {
+        super(...arguments);
+        this.logger = new common_1.Logger(BaseErrorRpcFilter_1.name);
+    }
     catch(exception, host) {
         var _a;
-        // Wrap your BaseError into RpcException payload
+        // Skip if already an RpcException
+        if (exception instanceof microservices_1.RpcException) {
+            return super.catch(exception, host);
+        }
+        // Check if it's a BaseError using duck-typing
+        // This is more reliable than instanceof for serialized errors
+        const isBaseErrorLike = exception &&
+            typeof exception === 'object' &&
+            exception.name === 'BaseError' &&
+            typeof exception.code === 'number' &&
+            typeof exception.message === 'string';
+        if (isBaseErrorLike || exception instanceof base_res_dto_1.BaseError) {
+            const payload = {
+                code: exception.code,
+                msg: exception.message,
+                data: (_a = exception.data) !== null && _a !== void 0 ? _a : null,
+            };
+            this.logger.log(`BaseError caught: [${payload.code}] ${payload.msg}`);
+            return super.catch(new microservices_1.RpcException(payload), host);
+        }
+        // Handle all other errors
+        this.logger.error('Unhandled RPC error:', {
+            name: exception === null || exception === void 0 ? void 0 : exception.name,
+            message: exception === null || exception === void 0 ? void 0 : exception.message,
+            code: exception === null || exception === void 0 ? void 0 : exception.code,
+            stack: exception === null || exception === void 0 ? void 0 : exception.stack,
+        });
         const payload = {
-            code: exception.code,
-            msg: exception.message,
-            data: (_a = exception.data) !== null && _a !== void 0 ? _a : null,
+            code: 500,
+            msg: (exception === null || exception === void 0 ? void 0 : exception.message) || 'Internal server error',
+            data: null,
         };
-        // Delegate to the base filter with a proper RpcException
         return super.catch(new microservices_1.RpcException(payload), host);
     }
 };
 exports.BaseErrorRpcFilter = BaseErrorRpcFilter;
-exports.BaseErrorRpcFilter = BaseErrorRpcFilter = __decorate([
-    (0, common_1.Catch)(base_res_dto_1.BaseError)
+exports.BaseErrorRpcFilter = BaseErrorRpcFilter = BaseErrorRpcFilter_1 = __decorate([
+    (0, common_1.Catch)() // Catch everything
 ], BaseErrorRpcFilter);

package/dist/filters/index.d.ts

@@ -1,3 +1,4 @@
 export * from './base-error.filter';
+export * from './base-error-rpc.filter';
 export * from './validation-exception.filter';
 //# sourceMappingURL=index.d.ts.map

package/dist/filters/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/filters/index.ts"],"names":[],"mappings":"AAAA,cAAc,qBAAqB,CAAC;AACpC,cAAc,+BAA+B,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/filters/index.ts"],"names":[],"mappings":"AAAA,cAAc,qBAAqB,CAAC;AACpC,cAAc,yBAAyB,CAAC;AACxC,cAAc,+BAA+B,CAAC"}

package/dist/filters/index.js

@@ -15,4 +15,5 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 __exportStar(require("./base-error.filter"), exports);
+__exportStar(require("./base-error-rpc.filter"), exports);
 __exportStar(require("./validation-exception.filter"), exports);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saihu/common",
-  "version": "1.1.61",
+  "version": "1.1.63",
   "description": "Common utilities for NestJS applications",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",