fix(evlog): respect status/statusCode on Error instances (#218)

Co-authored-by: Hugo Richard <hugo.richard@vercel.com>

Author: benhid

.changeset/fix-parse-error-status.md

@@ -0,0 +1,7 @@
+---
+"evlog": patch
+---
+
+fix(parseError): respect `.status` / `.statusCode` on Error instances instead of hardcoding 500
+
+Frameworks like NestJS attach HTTP status directly on Error subclasses (e.g. `BadRequestException` has `.status = 400`). Previously, `parseError()` ignored these properties and always returned 500 for any `Error` instance without a `data` property. Now uses `extractErrorStatus()` to extract the correct status.

packages/evlog/src/runtime/utils/parseError.ts

@@ -1,5 +1,6 @@
 import type { FetchError } from 'ofetch'
 import type { ParsedError } from '../../types'
+import { extractErrorStatus } from '../../shared/errors'
 
 export type { ParsedError }
 
@@ -25,7 +26,7 @@ export function parseError(error: unknown): ParsedError {
   if (error instanceof Error) {
     return {
       message: error.message,
-      status: 500,
+      status: extractErrorStatus(error),
       raw: error,
     }
   }

packages/evlog/test/error.test.ts

@@ -317,4 +317,47 @@ describe('parseError', () => {
       expect(parsed.status).toBe(500)
     })
   })
+
+  describe('Error instances with status properties', () => {
+    it('respects .status on Error', () => {
+      const error = Object.assign(new Error('Bad Request'), { status: 400 })
+
+      const parsed = parseError(error)
+
+      expect(parsed.status).toBe(400)
+      expect(parsed.message).toBe('Bad Request')
+    })
+
+    it('respects .statusCode on Error', () => {
+      const error = Object.assign(new Error('Not Found'), { statusCode: 404 })
+
+      const parsed = parseError(error)
+
+      expect(parsed.status).toBe(404)
+    })
+
+    it('prefers .status over .statusCode', () => {
+      const error = Object.assign(new Error('Conflict'), { status: 409, statusCode: 500 })
+
+      const parsed = parseError(error)
+
+      expect(parsed.status).toBe(409)
+    })
+
+    it('defaults to 500 for plain Error without status', () => {
+      const error = new Error('Something broke')
+
+      const parsed = parseError(error)
+
+      expect(parsed.status).toBe(500)
+    })
+
+    it('defaults to 500 when status is not a valid number', () => {
+      const error = Object.assign(new Error('Weird'), { status: 'bad' })
+
+      const parsed = parseError(error)
+
+      expect(parsed.status).toBe(500)
+    })
+  })
 })