chore: update linter deps to eslint v9

Author: 3imed-jaberi

.eslintignore

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

.github/workflows/ci.yml

@@ -1,5 +1,5 @@
 module.exports = {
   '*.md': (filenames) => filenames.map((filename) => `remark ${filename} -qfo`),
   'package.json': 'fixpack',
-  '*.js': 'xo --fix'
+  '*.js': 'eslint . --fix'
 };

.lintstagedrc.js

@@ -3,7 +3,9 @@
 > Router middleware for [Koa](https://github.com/koajs/koa). Maintained by [Forward Email][forward-email] and [Lad][].
 
 [![build status](https://github.com/koajs/router/actions/workflows/ci.yml/badge.svg)](https://github.com/koajs/router/actions/workflows/ci.yml)
-[![code style](https://img.shields.io/badge/code_style-XO-5ed9c7.svg)](https://github.com/sindresorhus/xo)
+
+<!-- [![code style](https://img.shields.io/badge/code_style-XO-5ed9c7.svg)](https://github.com/sindresorhus/xo) -->
+
 [![styled with prettier](https://img.shields.io/badge/styled_with-prettier-ff69b4.svg)](https://github.com/prettier/prettier)
 [![made with lass](https://img.shields.io/badge/made_with-lass-95CC28.svg)](https://lass.js.org)
 [![license](https://img.shields.io/github/license/koajs/router.svg)](LICENSE)

.xo-config.js

@@ -0,0 +1,31 @@
+const js = require('@eslint/js');
+const eslintPluginUnicorn = require('eslint-plugin-unicorn');
+
+module.exports = [
+  js.configs.recommended,
+  {
+    languageOptions: {
+      ecmaVersion: 2022,
+      sourceType: 'commonjs',
+      globals: {
+        console: true,
+        setTimeout: true,
+        __dirname: true,
+        // Mocha globals
+        before: true,
+        after: true,
+        beforeEach: true,
+        describe: true,
+        it: true
+      }
+    },
+    plugins: {
+      unicorn: eslintPluginUnicorn,
+    },
+    rules: {
+      'promise/prefer-await-to-then': 0,
+      'logical-assignment-operators': 0,
+      'arrow-body-style': 0,
+    }
+  }
+];

README.md

@@ -51,7 +51,7 @@ class Router {
    * @constructor
    */
   constructor(opts = {}) {
-    if (!(this instanceof Router)) return new Router(opts); // eslint-disable-line no-constructor-return
+    if (!(this instanceof Router)) return new Router(opts);  
 
     this.opts = opts;
     this.methods = this.opts.methods || [
@@ -581,7 +581,6 @@ class Router {
 
       debug('test %s %s', layer.path, layer.regexp);
 
-      // eslint-disable-next-line unicorn/prefer-regexp-test
       if (layer.match(path)) {
         matched.path.push(layer);
 
@@ -830,7 +829,7 @@ for (const method of methods) {
 }
 
 // Alias for `router.delete()` because delete is a reserved word
-// eslint-disable-next-line dot-notation
+ 
 Router.prototype.del = Router.prototype['delete'];
 
 module.exports = Router;

eslint.config.js

@@ -31,8 +31,8 @@
     "@commitlint/config-conventional": "^17.7.0",
     "@ladjs/env": "^4.0.0",
     "chalk": "^5.4.1",
-    "eslint": "^8.39.0",
-    "eslint-config-xo-lass": "^2.0.1",
+    "eslint": "^9.32.0",
+    "eslint-plugin-unicorn": "^60.0.0",
     "fixpack": "^4.0.0",
     "husky": "^9.1.7",
     "jsdoc-to-markdown": "^8.0.0",
@@ -42,11 +42,10 @@
     "nyc": "^17.0.0",
     "remark-cli": "11",
     "remark-preset-github": "^4.0.4",
-    "supertest": "^7.0.0",
-    "xo": "0.53.1"
+    "supertest": "^7.0.0"
   },
   "engines": {
-    "node": ">= 18"
+    "node": ">= 20"
   },
   "files": [
     "lib"
@@ -69,7 +68,7 @@
     "benchmark": "node bench/run.js",
     "coverage": "nyc npm run test",
     "docs": "NODE_ENV=test jsdoc2md -t ./lib/API_tpl.hbs --src ./lib/*.js  >| API.md",
-    "lint": "xo --fix && remark . -qfo && fixpack",
+    "lint": "eslint . --fix && remark . -qfo && fixpack",
     "prepare": "husky install",
     "pretest": "npm run lint",
     "test": "mocha test/**/*.js",

lib/router.js

@@ -270,7 +270,7 @@ describe('Router', () => {
     router
       .get(
         'users_single',
-        new RegExp('/users/:id(.*)'), // eslint-disable-line prefer-regex-literals
+        new RegExp('/users/:id(.*)'),  
         (ctx, next) => {
           ctx.body = { single: true };
           next();