fixed errors

Author: sdangol

packages/event-handler/src/http/errors.ts

@@ -1,5 +1,5 @@
 import type { JSONValue } from '@aws-lambda-powertools/commons/types';
-import { isDevMode } from '@aws-lambda-powertools/commons/utils/env';
+import type { StandardSchemaV1 } from '@standard-schema/spec';
 import type { HandlerResponse, HttpStatusCode } from '../types/http.js';
 import { HttpStatusCodes } from './constants.js';
 
@@ -174,36 +174,21 @@ class ServiceUnavailableError extends HttpError {
   }
 }
 
-class InvalidEventError extends Error {
-  constructor(message?: string) {
-    super(message);
-    this.name = 'InvalidEventError';
-  }
-}
-
-class InvalidHttpMethodError extends Error {
-  constructor(method: string) {
-    super(`HTTP method ${method} is not supported.`);
-    this.name = 'InvalidEventError';
-  }
-}
-
 class RequestValidationError extends HttpError {
   readonly statusCode = HttpStatusCodes.UNPROCESSABLE_ENTITY;
   readonly errorType = 'RequestValidationError';
 
   constructor(
-    message: string,
-    public readonly component: 'body' | 'headers' | 'path' | 'query',
-    public readonly originalError?: Error,
-    details?: Record<string, unknown>
+    message?: string,
+    issues?: StandardSchemaV1.FailureResult['issues'],
+    options?: ErrorOptions
   ) {
-    const errorDetails =
-      isDevMode() && originalError
-        ? { ...details, validationError: originalError.message }
-        : details;
-
-    super(message, { cause: originalError }, errorDetails);
+    super(message, options, {
+      issues: issues?.map((issue) => ({
+        message: issue.message,
+        path: issue.path,
+      })),
+    });
     this.name = 'RequestValidationError';
   }
 }
@@ -213,21 +198,34 @@ class ResponseValidationError extends HttpError {
   readonly errorType = 'ResponseValidationError';
 
   constructor(
-    message: string,
-    public readonly component: 'body' | 'headers',
-    public readonly originalError?: Error,
-    details?: Record<string, unknown>
+    message?: string,
+    issues?: StandardSchemaV1.FailureResult['issues'],
+    options?: ErrorOptions
   ) {
-    const errorDetails =
-      isDevMode() && originalError
-        ? { ...details, validationError: originalError.message }
-        : details;
-
-    super(message, { cause: originalError }, errorDetails);
+    super(message, options, {
+      issues: issues?.map((issue) => ({
+        message: issue.message,
+        path: issue.path,
+      })),
+    });
     this.name = 'ResponseValidationError';
   }
 }
 
+class InvalidEventError extends Error {
+  constructor(message?: string) {
+    super(message);
+    this.name = 'InvalidEventError';
+  }
+}
+
+class InvalidHttpMethodError extends Error {
+  constructor(method: string) {
+    super(`HTTP method ${method} is not supported.`);
+    this.name = 'InvalidEventError';
+  }
+}
+
 export {
   BadRequestError,
   ForbiddenError,

packages/event-handler/src/http/middleware/validation.ts

@@ -121,14 +121,18 @@ async function extractBody(source: Request | Response): Promise<unknown> {
       if (source instanceof Request) {
         throw new RequestValidationError(
           'Validation failed for request body',
-          'body',
-          new Error('Invalid JSON')
+          [],
+          {
+            cause: new Error('Invalid JSON body'),
+          }
         );
       }
       throw new ResponseValidationError(
         'Validation failed for response body',
-        'body',
-        new Error('Invalid JSON')
+        [],
+        {
+          cause: new Error('Invalid JSON body'),
+        }
       );
     }
   }
@@ -155,8 +159,7 @@ async function validateRequest<T>(
 
   if ('issues' in result) {
     const message = `Validation failed for request ${component}`;
-    const error = new Error('Validation failed');
-    throw new RequestValidationError(message, component, error);
+    throw new RequestValidationError(message, result.issues);
   }
 
   return result.value as T | Record<string, string>;
@@ -181,8 +184,7 @@ async function validateResponse<T>(
 
   if ('issues' in result) {
     const message = `Validation failed for response ${component}`;
-    const error = new Error('Validation failed');
-    throw new ResponseValidationError(message, component, error);
+    throw new ResponseValidationError(message, result.issues);
   }
 
   return result.value as T | Record<string, string>;