Use eslint v9.

.github/workflows/main.yml

@@ -24,7 +24,7 @@ jobs:
     timeout-minutes: 10
     strategy:
       matrix:
-        node-version: [20.x]
+        node-version: [24.x]
     steps:
     - uses: actions/checkout@v4
     - name: Use Node.js ${{ matrix.node-version }}
@@ -40,7 +40,7 @@ jobs:
     timeout-minutes: 10
     strategy:
       matrix:
-        node-version: [18.x, 20.x]
+        node-version: [18.x, 20.x, 22.x, 24.x]
     steps:
     - uses: actions/checkout@v4
     - name: Use Node.js ${{ matrix.node-version }}
@@ -61,7 +61,7 @@ jobs:
     timeout-minutes: 10
     strategy:
       matrix:
-        node-version: [20.x]
+        node-version: [24.x]
     steps:
     - uses: actions/checkout@v4
     - name: Use Node.js ${{ matrix.node-version }}

CHANGELOG.md

@@ -1,5 +1,10 @@
 # `@bedrock/core` ChangeLog
 
+## 6.3.1 - 2025-08-xx
+
+### Changed
+- Update to support `eslint@9`.
+
 ## 6.3.0 - 2024-11-19
 
 ### Added

.eslintrc.cjs → eslint.config.js

@@ -15,19 +15,8 @@
  *
  * SPDX-License-Identifier: Apache-2.0
  */
-module.exports = {
-  root: true,
-  env: {
-    node: true
-  },
-  extends: [
-    'digitalbazaar',
-    'digitalbazaar/jsdoc',
-    'digitalbazaar/module'
-  ],
-  ignorePatterns: ['node_modules/'],
-  rules: {
-    'unicorn/prefer-node-protocol': 'error'
-  }
-};
+import config from '@digitalbazaar/eslint-config/node-recommended';
 
+export default [
+  ...config
+];

lib/helpers.js

@@ -16,7 +16,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-// eslint-disable-next-line max-len
+// eslint-disable-next-line @stylistic/max-len
 const PROPERTY_REGEX = /[^.[\]]+|\[(?:(-?\d+(?:\.\d+)?)|(["'])((?:(?!\2)[^\\]|\\.)*?)\2)\]|(?=(?:\.|\[\])(?:\.|\[\]|$))/g;
 const ESCAPE_REGEX = /\\(\\)?/g;
 

lib/index.js

@@ -552,7 +552,7 @@ async function _runWorker(startTime) {
   if(config.ensureConfigOverride.enable) {
     configOverrideSnapshot = _snapshotOverrideFields({
       config,
-      fields: config.ensureConfigOverride.fields,
+      fields: config.ensureConfigOverride.fields
     });
   }
 
@@ -797,5 +797,7 @@ async function _logExit(code) {
       logger.end();
     });
     await p;
-  } finally {}
+  } finally {
+    // continue regardless of error
+  }
 }

lib/loggers/fileLogger.js

@@ -36,15 +36,15 @@ cc({
 export async function init({transports}) {
   transports.app = new winston.transports.File({
     ...config.loggers.app,
-    format: formatters.fromConfig(config.loggers.app.bedrock),
+    format: formatters.fromConfig(config.loggers.app.bedrock)
   });
   transports.access = new winston.transports.File({
     ...config.loggers.access,
-    format: formatters.fromConfig(config.loggers.access.bedrock),
+    format: formatters.fromConfig(config.loggers.access.bedrock)
   });
   transports.error = new winston.transports.File({
     ...config.loggers.error,
-    format: formatters.fromConfig(config.loggers.error.bedrock),
+    format: formatters.fromConfig(config.loggers.error.bedrock)
   });
 
   // ensure all files are created and have ownership set to the

lib/loggers/formatters.js

@@ -31,7 +31,7 @@ export const bedrock = format(info => {
     splat: undefined,
     timestamp: undefined,
     workerId: undefined,
-    workerPid: undefined,
+    workerPid: undefined
   }, null, 2);
   const workerInfo = `workerPid=${workerPid}, workerId=${workerId}`;
   if(stringifiedRest !== '{}') {
@@ -89,7 +89,7 @@ export function fromConfig(config) {
       }
       fmts.push(
         format.timestamp(),
-        modulePrefix(),
+        modulePrefix()
       );
       if(colorize) {
         fmts.push(format.colorize());
@@ -99,12 +99,12 @@ export function fromConfig(config) {
     } else if(formatter === 'json') {
       fmt = format.combine(
         format.timestamp(),
-        format.json(),
+        format.json()
       );
     } else if(formatter === 'logstash') {
       fmt = format.combine(
         format.timestamp(),
-        format.logstash(),
+        format.logstash()
       );
     }
   } else {

lib/loggers/index.js

@@ -87,7 +87,7 @@ container.add = function(id) {
       const child = createChild.apply(wrapper, [childMeta]);
       // if child `defaultMeta` is not set, add it to ensure that the parent
       // meta is included
-      if(!Object.hasOwnProperty(child, 'defaultMeta')) {
+      if(!Object.prototype.hasOwnProperty.call(child, 'defaultMeta')) {
         child.defaultMeta = {...wrapper.defaultMeta, ...childMeta};
       }
       return child;
@@ -104,7 +104,7 @@ if(cluster.isPrimary) {
     access: null,
     app: null,
     console: null,
-    error: null,
+    error: null
   };
 }
 
@@ -119,7 +119,7 @@ container.init = async () => {
     const transports = container.transports;
     transports.console = new winston.transports.Console({
       ...config.loggers.console,
-      format: formatters.fromConfig(config.loggers.console.bedrock),
+      format: formatters.fromConfig(config.loggers.console.bedrock)
     });
 
     if(config.loggers.enableFileTransport) {

package.json

@@ -14,7 +14,7 @@
   ],
   "scripts": {
     "start": "node index.js",
-    "lint": "eslint ."
+    "lint": "eslint"
   },
   "repository": {
     "type": "git",
@@ -36,12 +36,11 @@
     "winston-transport": "^4.5.0"
   },
   "devDependencies": {
-    "eslint": "^8.57.0",
-    "eslint-config-digitalbazaar": "^5.0.1",
-    "eslint-plugin-jsdoc": "^50.4.1",
-    "eslint-plugin-unicorn": "^56.0.0",
-    "jsdoc": "^4.0.2",
-    "jsdoc-to-markdown": "^9.0.2"
+    "@digitalbazaar/eslint-config": "^6.2.0",
+    "eslint": "^9.36.0",
+    "eslint-plugin-jsdoc": "^60.1.1",
+    "eslint-plugin-unicorn": "^61.0.2",
+    "jsdoc": "^4.0.4"
   },
   "engines": {
     "node": ">=18"