Refactor to use native Node.js modules, support Node.js v20+ and modernize the code (#180)

* refactor: upgrade deps + refactor code to be with oop class style + support node.js 20+ ...etc

* fix: add fix for #179 issue

Author: 3imed-jaberi

Diff for: .github/workflows/ci.yml

@@ -10,9 +10,8 @@ jobs:
         os:
           - ubuntu-latest
         node_version:
-          - 14
-          - 16
-          - 18
+          - 20
+          - 22
     name: Node ${{ matrix.node_version }} on ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v3

Diff for: .xo-config.js

@@ -32,6 +32,8 @@ module.exports = {
     'unicorn/explicit-length-check': 'warn',
     'unicorn/no-array-reduce': 'warn',
     'unicorn/prefer-spread': 'warn',
-    'unicorn/prefer-node-protocol': 'off'
+    'unicorn/prefer-node-protocol': 'off',
+    'unicorn/expiring-todo-comments': 'off',
+    'max-nested-callbacks': 'off'
   }
 };

Diff for: README.md

@@ -69,6 +69,7 @@ See [API Reference](./API.md) for more documentation.
 | ---------------- |
 | **Alex Mingoia** |
 | **@koajs**       |
+| **Imed Jaberi**  |
 
 
 ## License

Diff for: bench/server.js

@@ -1,15 +1,20 @@
-const Koa = require('koa');
-const Router = require('../');
+const process = require('node:process');
 const env = require('@ladjs/env')({
   path: '../.env',
   includeProcessEnv: true,
   assignToProcessEnv: true
 });
+const Koa = require('koa');
+
+const Router = require('../');
 
 const app = new Koa();
 const router = new Router();
 
-const ok = (ctx) => (ctx.status = 200);
+const ok = (ctx) => {
+  ctx.status = 200;
+};
+
 const n = Number.parseInt(env.FACTOR || '10', 10);
 const useMiddleware = env.USE_MIDDLEWARE === 'true';
 

Diff for: lib/layer.js

@@ -8,41 +8,44 @@
     "email": "niftylettuce@gmail.com"
   },
   "contributors": [
-    "Alex Mingoia <talk@alexmingoia.com>",
-    "@koajs"
+    {
+      "name": "Alex Mingoia",
+      "email": "talk@alexmingoia.com"
+    },
+    {
+      "name": "@koajs"
+    },
+    {
+      "name": "Imed Jaberi",
+      "email": "imed-jaberi@outlook.com"
+    }
   ],
   "dependencies": {
-    "debug": "^4.3.4",
+    "debug": "^4.3.6",
     "http-errors": "^2.0.0",
     "koa-compose": "^4.1.0",
-    "methods": "^1.1.2",
-    "path-to-regexp": "^6.2.1"
+    "path-to-regexp": "^6.2.2"
   },
   "devDependencies": {
     "@commitlint/cli": "^17.7.2",
     "@commitlint/config-conventional": "^17.7.0",
     "@ladjs/env": "^4.0.0",
-    "ava": "^5.3.1",
-    "cross-env": "^7.0.3",
-    "eslint": "8.39.0",
+    "eslint": "^8.39.0",
     "eslint-config-xo-lass": "^2.0.1",
-    "expect.js": "^0.3.1",
     "fixpack": "^4.0.0",
     "husky": "^8.0.3",
     "jsdoc-to-markdown": "^8.0.0",
-    "koa": "^2.14.2",
+    "koa": "^2.15.3",
     "lint-staged": "^14.0.1",
-    "mocha": "^10.2.0",
-    "nyc": "^15.1.0",
+    "mocha": "^10.7.3",
+    "nyc": "^17.0.0",
     "remark-cli": "11",
     "remark-preset-github": "^4.0.4",
-    "should": "^13.2.3",
-    "supertest": "^6.3.3",
-    "wrk": "^1.2.1",
+    "supertest": "^7.0.0",
     "xo": "0.53.1"
   },
   "engines": {
-    "node": ">= 12"
+    "node": ">= 20"
   },
   "files": [
     "lib"

Diff for: lib/router.js

@@ -1,14 +1,12 @@
 /**
  * Module tests
  */
+const assert = require('node:assert');
 
-const should = require('should');
-
-describe('module', function () {
-  it('should expose Router', function (done) {
+describe('module', () => {
+  it('should expose Router', () => {
     const Router = require('..');
-    should.exist(Router);
-    Router.should.be.type('function');
-    done();
+    assert.strictEqual(Boolean(Router), true);
+    assert.strictEqual(typeof Router, 'function');
   });
 });