improve error handling

Author: omermecitoglu

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@omer-x/next-openapi-route-handler",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "a Next.js plugin to generate OpenAPI documentation from route handlers",
   "keywords": [
     "next.js",

package.json

@@ -2,7 +2,7 @@ import type { HttpMethod } from "~/types/http";
 import type { FixedRequest } from "~/types/request";
 import { resolveContent } from "./content";
 import type { RequestBodyObject } from "@omer-x/openapi-types/request-body";
-import type { ZodType } from "zod";
+import type { ZodError, ZodType } from "zod";
 
 export function resolveRequestBody(source?: ZodType<unknown> | string, isFormData: boolean = false) {
   if (!source) return undefined;
@@ -26,7 +26,23 @@ export async function parseRequestBody<B>(
     const body = Array.from(formData.keys()).reduce((collection, key) => ({
       ...collection, [key]: formData.get(key),
     }), {});
-    return schema.parse(body);
+    try {
+      return schema.parse(body);
+    } catch (error) {
+      if (process.env.NODE_ENV !== "production") {
+        // eslint-disable-next-line no-console
+        console.log((error as ZodError).issues);
+      }
+      throw new Error("PARSE_FORM_DATA");
+    }
+  }
+  try {
+    return schema.parse(await request.json());
+  } catch (error) {
+    if (process.env.NODE_ENV !== "production") {
+      // eslint-disable-next-line no-console
+      console.log((error as ZodError).issues);
+    }
+    throw new Error("PARSE_REQUEST_BODY");
   }
-  return schema.parse(await request.json());
 }

src/core/body.ts

@@ -1,7 +1,15 @@
 import { safeParse } from "./zod-error-handler";
-import type { ZodType } from "zod";
+import type { ZodError, ZodType } from "zod";
 
 export default function parsePathParams<T>(source?: T, schema?: ZodType<T>) {
   if (!schema || !source) return null;
-  return safeParse(schema, source as Record<string, unknown>);
+  try {
+    return safeParse(schema, source as Record<string, unknown>);
+  } catch (error) {
+    if (process.env.NODE_ENV !== "production") {
+      // eslint-disable-next-line no-console
+      console.log((error as ZodError).issues);
+    }
+    throw new Error("PARSE_PATH_PARAMS");
+  }
 }

src/core/path-params.ts

@@ -1,5 +1,5 @@
 import { safeParse } from "./zod-error-handler";
-import type { ZodType } from "zod";
+import type { ZodError, ZodType } from "zod";
 
 function serializeArray(value: string[]) {
   return value.join(",");
@@ -19,5 +19,13 @@ export default function parseSearchParams<T>(source: URLSearchParams, schema?: Z
       [key]: serializeArray(values),
     };
   }, {});
-  return safeParse(schema, params);
+  try {
+    return safeParse(schema, params);
+  } catch (error) {
+    if (process.env.NODE_ENV !== "production") {
+      // eslint-disable-next-line no-console
+      console.log((error as ZodError).issues);
+    }
+    throw new Error("PARSE_SEARCH_PARAMS");
+  }
 }

src/core/search-params.ts

@@ -47,8 +47,15 @@ export default function createRoute<M extends HttpMethod, PP, QP, RB>(input: Rou
       const body = await parseRequestBody(request, input.method, input.requestBody ?? undefined, input.hasFormData) as RB;
       return await input.action({ pathParams, queryParams, body });
     } catch (error) {
-      if (error instanceof Error && error.constructor.name === "ZodError") {
-        return new Response(null, { status: 400 });
+      if (error instanceof Error) {
+        switch (error.message) {
+          case "PARSE_FORM_DATA":
+          case "PARSE_REQUEST_BODY":
+          case "PARSE_SEARCH_PARAMS":
+            return new Response(null, { status: 400 });
+          case "PARSE_PATH_PARAMS":
+            return new Response(null, { status: 404 });
+        }
       }
       return new Response(null, { status: 500 });
     }