@saihu/common 1.1.93 → 1.1.95

package/dist/errors/unified-exception.filter.d.ts

@@ -6,6 +6,11 @@ export declare class UnifiedExceptionFilter implements ExceptionFilter {
     catch(exception: any, host: ArgumentsHost): void;
     private mapAppErrorCodeToHttpStatus;
     private logError;
+    /**
+     * Determine if an error is "expected" (business logic, validation)
+     * vs "unexpected" (infrastructure failures, bugs)
+     */
+    private isExpectedError;
     /**
      * Recursively sanitize sensitive information from details object
      * Handles nested objects like { headers: { authorization: "..." } }

package/dist/errors/unified-exception.filter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"unified-exception.filter.d.ts","sourceRoot":"","sources":["../../src/errors/unified-exception.filter.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,eAAe,EAEf,aAAa,EAGd,MAAM,gBAAgB,CAAC;AAOxB,qBACa,sBAAuB,YAAW,eAAe;IAC5D,OAAO,CAAC,QAAQ,CAAC,MAAM,CAA2C;IAElE,OAAO,CAAC,QAAQ,CAAC,cAAc,CAM7B;IACF,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAAQ;IAE3C,KAAK,CAAC,SAAS,EAAE,GAAG,EAAE,IAAI,EAAE,aAAa;IA+CzC,OAAO,CAAC,2BAA2B;IAUnC,OAAO,CAAC,QAAQ;IA2BhB;;;OAGG;IACH,OAAO,CAAC,eAAe;IAUvB;;OAEG;IACH,OAAO,CAAC,cAAc;IAmCtB;;;OAGG;IACH,OAAO,CAAC,cAAc;CAQvB"}
+{"version":3,"file":"unified-exception.filter.d.ts","sourceRoot":"","sources":["../../src/errors/unified-exception.filter.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,eAAe,EAEf,aAAa,EAGd,MAAM,gBAAgB,CAAC;AAQxB,qBACa,sBAAuB,YAAW,eAAe;IAC5D,OAAO,CAAC,QAAQ,CAAC,MAAM,CAA2C;IAElE,OAAO,CAAC,QAAQ,CAAC,cAAc,CAM7B;IACF,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAAQ;IAE3C,KAAK,CAAC,SAAS,EAAE,GAAG,EAAE,IAAI,EAAE,aAAa;IA+CzC,OAAO,CAAC,2BAA2B;IAUnC,OAAO,CAAC,QAAQ;IAiChB;;;OAGG;IACH,OAAO,CAAC,eAAe;IAoBvB;;;OAGG;IACH,OAAO,CAAC,eAAe;IAUvB;;OAEG;IACH,OAAO,CAAC,cAAc;IAmCtB;;;OAGG;IACH,OAAO,CAAC,cAAc;CAQvB"}

package/dist/errors/unified-exception.filter.js

@@ -13,6 +13,7 @@ const domain_error_1 = require("./domain.error");
 const application_error_1 = require("./application.error");
 const infrastructure_error_1 = require("./infrastructure.error");
 const dto_1 = require("../dto");
+const times_1 = require("../util/times");
 let UnifiedExceptionFilter = UnifiedExceptionFilter_1 = class UnifiedExceptionFilter {
     constructor() {
         this.logger = new common_1.Logger(UnifiedExceptionFilter_1.name);
@@ -66,7 +67,7 @@ let UnifiedExceptionFilter = UnifiedExceptionFilter_1 = class UnifiedExceptionFi
         errorResponse.errorCode = errorCode;
         if (details)
             errorResponse.details = this.sanitizeDetails(details);
-        errorResponse.timestamp = new Date().toISOString();
+        errorResponse.timestamp = (0, times_1.getCurrentChinaTime)().toISOString();
         response.status(statusCode).json(errorResponse);
     }
     mapAppErrorCodeToHttpStatus(code) {
@@ -91,9 +92,36 @@ let UnifiedExceptionFilter = UnifiedExceptionFilter_1 = class UnifiedExceptionFi
         ]
             .filter(Boolean)
             .join(' | ');
-        this.logger.error(logMessage, exception.stack);
-        if (cause === null || cause === void 0 ? void 0 : cause.stack)
-            this.logger.error(`Caused by: ${cause.stack}`);
+        // Expected errors (business/validation) → warn, no stack
+        // Unexpected errors (infra/unknown) → error with stack
+        if (this.isExpectedError(exception, code)) {
+            this.logger.warn(logMessage);
+        }
+        else {
+            this.logger.error(logMessage, exception.stack);
+            if (cause === null || cause === void 0 ? void 0 : cause.stack)
+                this.logger.error(`Caused by: ${cause.stack}`);
+        }
+    }
+    /**
+     * Determine if an error is "expected" (business logic, validation)
+     * vs "unexpected" (infrastructure failures, bugs)
+     */
+    isExpectedError(exception, code) {
+        // Domain errors are always expected (business rule violations)
+        if (exception instanceof domain_error_1.DomainError)
+            return true;
+        // Application errors with these codes are expected
+        const expectedCodes = [
+            application_error_1.ErrorCodes.RESOURCE_NOT_FOUND,
+            application_error_1.ErrorCodes.INVALID_INPUT,
+        ];
+        if (exception instanceof application_error_1.ApplicationError &&
+            expectedCodes.includes(code)) {
+            return true;
+        }
+        // Infrastructure errors and unknown errors are unexpected
+        return false;
     }
     /**
      * Recursively sanitize sensitive information from details object

package/package.json

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