Add support for generic errors with stacktrace and knexjs errors

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tree-house-errors",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "NodeJS default error definitions with an error parser utility function",
   "main": "build/index.js",
   "types": "build/index.d.ts",

src/lib/errorParser.ts

@@ -4,8 +4,18 @@ import { errorConfig as errors } from './errorConfig';
 import { errorDefaults } from './constants';
 
 export function parseErrors(error: any) {
+  const metaData: any = {};
   let parsedError = new ApiError(errorDefaults.DEFAULT_HTTP_CODE, errorDefaults.DEFAULT_ERROR); // Default error
 
+  // Other errors
+  if (error instanceof Error) {
+    Object.assign(metaData, { stack: JSON.stringify(error.stack) });
+    if (error.hasOwnProperty('schema') && error.hasOwnProperty('detail')) { // knex.js specific errors
+      const errorData = <any>error;
+      Object.assign(metaData, errorData);
+    }
+  }
+
   // Express middleware validation errors
   if (error instanceof ev.ValidationError) {
     parsedError = new ValidationError(errors.INVALID_INPUT, {
@@ -25,6 +35,6 @@ export function parseErrors(error: any) {
     code: parsedError.code,
     title: parsedError.message,
     detail: parsedError.detail || parsedError.message,
-    meta: error instanceof Error ? { stack: parsedError.stack } : undefined, // At least add stacktrace for unknown errors
+    meta: Object.keys(metaData).length !== 0 ? metaData : undefined,
   };
 }

src/lib/errors.ts

@@ -16,7 +16,7 @@ export class ApiError extends Error {
     this.code = error.code;
     this.status = status;
     this.detail = detail;
-    if (stack) this.stack = JSON.stringify(stack);
+    if (stack) this.stack = stack;
   }
 }
 

tests/errorParser.test.ts

@@ -22,6 +22,27 @@ describe('errorParser', () => {
     // TODO: Other custom errors extending from Error
   });
 
+
+  describe('Knex.js errors', () => {
+    const error = new Error('Some error...');
+    Object.assign(error, { stack: 'myStack', detail: 'some details', schema: 'public' });
+
+    const parsedError = parseErrors(error);
+    expect(parsedError).toMatchObject({
+      id: expect.any(String),
+      status: defaultError.status,
+      code: defaultError.code,
+      title: defaultError.message,
+      detail: defaultError.message,
+      meta: {
+        stack: JSON.stringify('myStack'),
+        detail: 'some details',
+        schema: 'public',
+      },
+    });
+
+  });
+
   describe('Express Validation errors', () => {
     const options = {
       flatten: false,