feat: update node version 18

BREAKING CHANGE:
- node version >= 18 required
- APIGatewayProxyHandler body mandatory response field

Author: sleistner

.husky/pre-commit

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+. "$(dirname -- "$0")/_/husky.sh"
+
+npm run lint && npm run test -- -b

.node-version

@@ -0,0 +1 @@
+v18

jest.config.js

@@ -0,0 +1,16 @@
+module.exports = {
+    "verbose": true,
+    "preset": "ts-jest",
+    "testEnvironment": "node",
+    "coverageDirectory": "test-reports",
+    "roots": [
+        "src"
+    ],
+    "reporters": [
+        "default",
+        ["jest-junit", {
+            "outputDirectory": "test-reports/jest",
+            "outputName": "report.xml"
+        }]
+    ]
+};

package-lock.json

@@ -4,17 +4,21 @@
     "description": "An opinionated Typescript package that facilitates specifying AWS Lambda handlers including input validation, error handling and response formatting.",
     "main": "index.js",
     "types": "index.d.ts",
+    "engines": {
+        "node": ">=18"
+    },
     "scripts": {
         "lint": "eslint './src/**/*.ts'",
-        "test": "NODE_ENV=testing jest -b --reporters jest-dot-reporter",
+        "test": "NODE_ENV=testing jest --detectOpenHandles --no-cache -b --reporters jest-progress-bar-reporter",
         "test:verbose": "npm run test -- --reporters default",
         "test:coverage": "npm run test -- --coverage",
         "test:coverage:verbose": "npm run test:coverage -- --reporters default",
         "test:ci": "npm run test:coverage:verbose -- --ci",
         "build": "npm run clean && npm run build:compile && npm run build:copy",
         "build:compile": "tsc -p tsconfig.build.json",
         "build:copy": "cp LICENSE README.md package.json dist/",
-        "clean": "rimraf ./dist"
+        "clean": "rimraf ./dist",
+        "prepare": "[ ! -x ./node_modules/.bin/husky ] && exit 0; husky install"
     },
     "repository": {
         "type": "git",
@@ -27,53 +31,35 @@
     ],
     "author": "Steffen Leistner",
     "contributors": [
-        "Martin Pirkl <[email protected]>"
+        "Martin Pirkl <[email protected]>",
+        "Steve Waldowski <[email protected]>"
     ],
     "license": "Apache-2.0",
     "bugs": {
         "url": "https://github.com/enter-at/node-aws-lambda-handlers/issues"
     },
     "homepage": "https://github.com/enter-at/node-aws-lambda-handlers#readme",
-    "dependencies": {},
     "devDependencies": {
-        "@enter-at/eslint-config-typescript-prettier": "1.5.2",
-        "@types/aws-lambda": "8.10.76",
+        "@enter-at/eslint-config-typescript-prettier": "1.6.0",
+        "@faker-js/faker": "7.6.0",
+        "@tsconfig/node18": "1.0.1",
+        "@types/aws-lambda": "8.10.109",
         "@types/aws-sdk": "2.7.0",
-        "@types/faker": "4.1.12",
-        "@types/jest": "26.0.24",
+        "@types/jest": "29.2.4",
         "aws-lambda": "1.0.7",
-        "eslint": "7.32.0",
-        "aws-sdk": "2.1087.0",
-        "faker": "4.1.0",
-        "husky": "4.3.8",
-        "jest": "26.6.3",
-        "jest-dot-reporter": "1.0.14",
-        "jest-junit": "11.1.0",
-        "prettier": "2.2.1",
+        "aws-sdk": "2.1277.0",
+        "eslint": "8.30.0",
+        "husky": "8.0.2",
+        "jest": "29.3.1",
+        "jest-junit": "15.0.0",
+        "jest-progress-bar-reporter": "1.0.25",
+        "prettier": "2.8.1",
         "rimraf": "3.0.2",
-        "ts-jest": "26.5.6",
-        "typescript": "3.9.10"
-    },
-    "jest": {
-        "verbose": true,
-        "preset": "ts-jest",
-        "testEnvironment": "node",
-        "coverageDirectory": "test-reports",
-        "roots": [
-            "src"
-        ],
-        "reporters": [
-            "default",
-            "jest-junit"
-        ]
+        "ts-jest": "29.0.3",
+        "typescript": "4.9.4"
     },
     "jest-junit": {
         "outputDirectory": "test-reports/jest",
         "outputName": "report.xml"
-    },
-    "husky": {
-        "hooks": {
-            "pre-commit": "npm run lint && npm run test -- -b"
-        }
     }
 }

package.json

@@ -4,5 +4,5 @@ type Handler = (_target: unknown, _propertyName: string, propertyDescriptor: Pro
 
 export function APIGatewayProxyHandler(args?: handlers.APIGatewayProxyHandlerArguments): Handler {
     const handler = new handlers.APIGatewayProxyHandler(args);
-    return (handler.decorator.bind(handler) as unknown) as Handler;
+    return handler.decorator.bind(handler) as unknown as Handler;
 }

src/decorator/handler/index.ts

@@ -13,4 +13,4 @@ export class InputFormat {
     }
 }
 
-export const json: Format = (InputFormat.json as unknown) as Format;
+export const json: Format = InputFormat.json as unknown as Format;

src/format/InputFormat.ts

@@ -8,4 +8,4 @@ export class OutputFormat {
     }
 }
 
-export const json: Format = (OutputFormat.json as unknown) as Format;
+export const json: Format = OutputFormat.json as unknown as Format;

src/format/OutputFormat.ts

@@ -96,7 +96,7 @@ export class APIGatewayProxyHandler extends BaseHandler {
     protected formatOutput(result: APIGatewayResponse): APIGatewayProxyResult {
         const { body, ...properties } = result;
         return {
-            ...(body && { body: this.outputFormat.apply(body) }),
+            body: body ? this.outputFormat.apply(body) : "",
             ...properties,
         };
     }

src/handler/APIGatewayProxyHandler.ts

@@ -115,7 +115,7 @@ export abstract class BaseHandler {
                 const instance = this;
                 return handler.after(await method.apply(instance, handler.before(event, context)));
             } catch (err) {
-                return handler.onException(err);
+                return handler.onException(err as Error);
             }
         };
     }