Fixes #30 /claim #30

src/middleware/with-response-object-check.ts

@@ -1,17 +1,21 @@
-import { ResponseValidationError } from "./http-exceptions.js"
+import { BadRequestError } from "./http-exceptions.js"
 import { Middleware } from "./types.js"
 import { RouteSpec } from "src/types/route-spec.js"
 
+export class RawJsonResponseError extends BadRequestError {
+  constructor() {
+    super("Use ctx.json({...}) instead of returning an object directly.")
+  }
+}
+
 export const withResponseObjectCheck: Middleware<
   { routeSpec: RouteSpec<any> },
   {}
 > = async (req, ctx, next) => {
   const rawResponse = await next(req, ctx)
 
   if (typeof rawResponse === "object" && !(rawResponse instanceof Response)) {
-    throw new Error(
-      "Use ctx.json({...}) instead of returning an object directly."
-    )
+    throw new RawJsonResponseError()
   }
 
   return rawResponse

src/middleware/with-unhandled-exception-handling.ts

@@ -13,12 +13,20 @@ export const withUnhandledExceptionHandling: Middleware<{
       logger.warn(
         "Caught unhandled HTTP exception thrown by WinterSpec provided middleware. Consider adding createWithDefaultExceptionHandling middleware to your global or route spec."
       )
-    } else {
-      logger.warn(
-        "Caught unknown unhandled exception; consider adding a exception handling middleware to your global or route spec."
+      logger.error(e)
+      return Response.json(
+        {
+          message: e.message,
+        },
+        {
+          status: e.status ?? 500,
+        }
       )
     }
 
+    logger.warn(
+      "Caught unknown unhandled exception; consider adding a exception handling middleware to your global or route spec."
+    )
     logger.error(e)
 
     return new Response(null, {