feat(createError): inherit fields from cause (#795)

Co-authored-by: Pooya Parsa <pooya@pi0.io>

src/error.ts

@@ -93,8 +93,11 @@ export function createError<DataT = unknown>(
     return input;
   }
 
+  // Inherit H3Error properties from cause as fallback
+  const cause: unknown = input.cause;
+
   const err = new H3Error<DataT>(input.message ?? input.statusMessage ?? "", {
-    cause: input.cause || input,
+    cause: cause || input,
   });
 
   if (hasProp(input, "stack")) {
@@ -117,25 +120,32 @@ export function createError<DataT = unknown>(
     err.data = input.data;
   }
 
-  if (input.statusCode) {
-    err.statusCode = sanitizeStatusCode(input.statusCode, err.statusCode);
-  } else if (input.status) {
-    err.statusCode = sanitizeStatusCode(input.status, err.statusCode);
-  }
-  if (input.statusMessage) {
-    err.statusMessage = input.statusMessage;
-  } else if (input.statusText) {
-    err.statusMessage = input.statusText as string;
+  const statusCode =
+    input.statusCode ??
+    input.status ??
+    (cause as H3Error)?.statusCode ??
+    (cause as { status?: number })?.status;
+  if (typeof statusCode === "number") {
+    err.statusCode = sanitizeStatusCode(statusCode);
   }
-  if (err.statusMessage) {
-    err.statusMessage = sanitizeStatusMessage(err.statusMessage);
+
+  const statusMessage =
+    input.statusMessage ??
+    input.statusText ??
+    (cause as H3Error)?.statusMessage ??
+    (cause as { statusText?: string })?.statusText;
+  if (statusMessage) {
+    err.statusMessage = sanitizeStatusMessage(statusMessage);
   }
 
-  if (input.fatal !== undefined) {
-    err.fatal = input.fatal;
+  const fatal = input.fatal ?? (cause as H3Error)?.fatal;
+  if (fatal !== undefined) {
+    err.fatal = fatal;
   }
-  if (input.unhandled !== undefined) {
-    err.unhandled = input.unhandled;
+
+  const unhandled = input.unhandled ?? (cause as H3Error)?.unhandled;
+  if (unhandled !== undefined) {
+    err.unhandled = unhandled;
   }
 
   return err;

test/error.test.ts

@@ -103,4 +103,24 @@ describe("error", () => {
 
     expect(ctx.errors[0].cause).toBeInstanceOf(CustomError);
   });
+
+  it("can inherit from cause", async () => {
+    class CustomError extends Error {
+      cause = createError({
+        statusCode: 400,
+        statusMessage: "Bad Request",
+        unhandled: true,
+        fatal: true,
+      });
+    }
+
+    ctx.app.use("/", () => {
+      throw createError(new CustomError());
+    });
+
+    const res = await ctx.request.get("/");
+    expect(res.status).toBe(400);
+    expect(ctx.errors[0].unhandled).toBe(true);
+    expect(ctx.errors[0].fatal).toBe(true);
+  });
 });