elseware-nodejs 1.8.6 → 1.8.7

package/dist/index.d.cts

@@ -1809,10 +1809,10 @@ declare class AppError extends Error {
     status: string;
     isOperational: boolean;
     code?: string;
-    details?: unknown;
+    details?: Record<string, unknown>;
     constructor(message: string, statusCode?: number, options?: {
         code?: string;
-        details?: unknown;
+        details?: Record<string, unknown>;
     });
 }
 

package/dist/index.d.ts

@@ -1809,10 +1809,10 @@ declare class AppError extends Error {
     status: string;
     isOperational: boolean;
     code?: string;
-    details?: unknown;
+    details?: Record<string, unknown>;
     constructor(message: string, statusCode?: number, options?: {
         code?: string;
-        details?: unknown;
+        details?: Record<string, unknown>;
     });
 }
 

package/dist/index.js

@@ -5097,16 +5097,23 @@ var handleCastErrorDB = (err) => new AppError(`Invalid ${err.path}: ${err.value}
   code: "INVALID_ID"
 });
 var handleDuplicateFieldsDB = (err) => {
-  const value = err.keyValue ? JSON.stringify(err.keyValue) : "duplicate value";
-  return new AppError(`Duplicate field value: ${value}`, 400, {
+  const field = err.keyValue ? Object.keys(err.keyValue)[0] : "field";
+  return new AppError(`${field} already exists`, 400, {
     code: "DUPLICATE_FIELD"
   });
 };
 var handleValidationErrorDB = (err) => {
-  const errors = Object.values(err.errors).map((el) => el.message);
-  return new AppError("Invalid input data", 400, {
+  const errors = Object.entries(err.errors).reduce(
+    (acc, [key, value]) => {
+      acc[key] = value.message;
+      return acc;
+    },
+    {}
+  );
+  return new AppError("Validation failed", 400, {
     code: "VALIDATION_ERROR",
     details: errors
+    // structured field errors
   });
 };
 var handleJWTError = () => new AppError("Invalid token. Please log in again.", 401);
@@ -5144,21 +5151,27 @@ var GlobalErrorHandler = (isProd = false) => (err, _req, res, _next) => {
   } else {
     logger.danger("Operational error", { message: error.message });
   }
+  const baseResponse = {
+    success: false,
+    message: error.isOperational ? error.message : "Something went wrong",
+    ...error.details && {
+      meta: {
+        errors: error.details
+      }
+    }
+  };
   if (!isProd) {
     return APIResponse.send(res, {
-      success: false,
       statusCode: error.statusCode,
-      message: error.message,
+      ...baseResponse,
       data: {
-        stack: err?.stack,
-        details: error.details
+        stack: err?.stack
       }
     });
   }
   return APIResponse.send(res, {
-    success: false,
     statusCode: error.statusCode,
-    message: error.isOperational ? error.message : "Something went wrong"
+    ...baseResponse
   });
 };