chore: update eslint (#286)

Author: D-Sketon

.eslintrc.json

@@ -0,0 +1,44 @@
+const config = require('eslint-config-hexo/ts');
+const testConfig = require('eslint-config-hexo/test');
+
+module.exports = [
+  // Configurations applied globally
+  ...config,
+  {
+    'rules': {
+      '@typescript-eslint/no-explicit-any': 0,
+      '@typescript-eslint/no-var-requires': 0,
+      '@typescript-eslint/no-require-imports': 0,
+      '@typescript-eslint/ban-ts-comment': 0,
+      'no-use-before-define': 0,
+      'n/no-missing-require': 0,
+      'n/no-missing-import': 0,
+      '@typescript-eslint/no-unused-vars': [
+        'error', {
+          'argsIgnorePattern': '^_'
+        }
+      ]
+    }
+  },
+  // Configurations applied only to test files
+  {
+    files: [
+      'test/**/*.ts'
+    ],
+    languageOptions: {
+      ...testConfig.languageOptions
+    },
+    rules: {
+      ...testConfig.rules,
+      '@typescript-eslint/ban-ts-comment': 0,
+      '@typescript-eslint/no-unused-expressions': 0,
+      '@typescript-eslint/no-unused-vars': [
+        'error', {
+          'varsIgnorePattern': '^_',
+          'argsIgnorePattern': '^_',
+          'caughtErrorsIgnorePattern': '^_'
+        }
+      ]
+    }
+  }
+];

eslint.config.js

@@ -49,8 +49,8 @@
     "c8": "^10.1.3",
     "chai": "^4.3.6",
     "chai-as-promised": "^7.1.1",
-    "eslint": "^8.26.0",
-    "eslint-config-hexo": "^5.0.0",
+    "eslint": "^9.17.0",
+    "eslint-config-hexo": "^6.0.0",
     "lodash": "^4.17.21",
     "mocha": "11.0.2",
     "sinon": "^19.0.2",

package.json

@@ -54,7 +54,7 @@ async function exportAsync(database: Database, path: string): Promise<void> {
       throw e;
     }
   } finally {
-    await new Promise<void>((resolve, reject) => {
+    await new Promise<void>((resolve, _reject) => {
       writeStream.end(resolve);
     });
   }
@@ -84,9 +84,9 @@ class Database {
   constructor(options?: { path: string } & Partial<DatabaseOptions>) {
     this.options = {
       version: 0,
-      // eslint-disable-next-line @typescript-eslint/no-empty-function
+
       onUpgrade() {},
-      // eslint-disable-next-line @typescript-eslint/no-empty-function
+
       onDowngrade() {},
       ...options
     };

src/database.ts

@@ -108,7 +108,7 @@ export function parse(path: string | any[], map = null) {
         const nextKey = path[i];
         if (!nextKey) return;
 
-        // eslint-disable-next-line no-constant-condition
+
         while (true) {
           c = j === this.stack.length ? this : this.stack[j];
           if (!c) return;

src/lib/jsonstream/index.ts

@@ -152,7 +152,7 @@ class Model<T> extends EventEmitter {
   _acquireWriteLock(): BluebirdPromise.Disposer<void> {
     const mutex = this._mutex;
 
-    return new BluebirdPromise((resolve, reject) => {
+    return new BluebirdPromise((resolve, _reject) => {
       mutex.lock(resolve);
     }).disposer(() => {
       mutex.unlock();
@@ -170,7 +170,7 @@ class Model<T> extends EventEmitter {
     const schema = this.schema;
 
     // Apply getters
-    const data = (data_ instanceof this.Document ? data_ : this.new(data_));
+    const data = data_ instanceof this.Document ? data_ : this.new(data_);
     const id = data._id;
 
     // Check ID
@@ -333,7 +333,7 @@ class Model<T> extends EventEmitter {
     (data_ as any)._id = id;
 
     // Apply getters
-    const data = (data_ instanceof this.Document ? data_ : this.new(data_));
+    const data = data_ instanceof this.Document ? data_ : this.new(data_);
 
     // Apply setters
     const result = data.toObject();
@@ -867,7 +867,7 @@ class Model<T> extends EventEmitter {
    * @return {Function}
    * @private
    */
-  _populateGetter(data: PropertyKey, model: Model<T>, options: unknown): () => Document<T> {
+  _populateGetter(data: PropertyKey, model: Model<T>, _options: unknown): () => Document<T> {
     let hasCache = false;
     let cache: Document<T>;
 

src/model.ts

@@ -75,7 +75,7 @@ class UpdateParser {
     };
   }
 
-  // eslint-disable-next-line no-useless-constructor
+
   constructor(private paths: Record<string, SchemaType<any>>) { }
 
   /**
@@ -132,7 +132,7 @@ class UpdateParser {
  * @private
  */
 class QueryParser {
-  // eslint-disable-next-line no-useless-constructor
+
   constructor(private paths: Record<string, SchemaType<any>>) { }
 
   /**
@@ -765,7 +765,7 @@ class Schema<T = any> {
 
       if (!item.model) {
         const path = paths[key];
-        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+
         // @ts-ignore
         const ref = path.child ? path.child.options.ref : path.options.ref;
 

src/schema.ts

@@ -78,7 +78,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {*}
    */
-  cast(value?: unknown, data?: unknown): unknown {
+  cast(value?: unknown, _data?: unknown): unknown {
     if (value == null) {
       return this.default();
     }
@@ -93,7 +93,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {*|Error}
    */
-  validate(value: unknown, data?: unknown): unknown {
+  validate(value: unknown, _data?: unknown): unknown {
     if (this.options.required && value == null) {
       throw new ValidationError(`\`${this.name}\` is required!`);
     }
@@ -136,7 +136,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {*}
    */
-  value(value: unknown, data?: unknown): any {
+  value(value: unknown, _data?: unknown): any {
     return value;
   }
 
@@ -148,7 +148,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {Boolean}
    */
-  match(value: T, query: unknown, data?: unknown): boolean {
+  match(value: T, query: unknown, _data?: unknown): boolean {
     return value === query;
   }
 
@@ -160,7 +160,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$exist(value: unknown, query: unknown, data?: unknown): boolean {
+  q$exist(value: unknown, query: unknown, _data?: unknown): boolean {
     return (value != null) === query;
   }
 
@@ -184,7 +184,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$lt(value: unknown, query: unknown, data?: unknown): boolean {
+  q$lt(value: unknown, query: unknown, _data?: unknown): boolean {
     return value < query;
   }
 
@@ -196,7 +196,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$lte(value: unknown, query: unknown, data?: unknown): boolean {
+  q$lte(value: unknown, query: unknown, _data?: unknown): boolean {
     return value <= query;
   }
 
@@ -208,7 +208,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$gt(value: unknown, query: unknown, data?: unknown): boolean {
+  q$gt(value: unknown, query: unknown, _data?: unknown): boolean {
     return value > query;
   }
 
@@ -220,7 +220,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$gte(value: unknown, query: unknown, data?: unknown): boolean {
+  q$gte(value: unknown, query: unknown, _data?: unknown): boolean {
     return value >= query;
   }
 
@@ -232,7 +232,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$in(value: unknown, query: unknown[], data?: unknown): boolean {
+  q$in(value: unknown, query: unknown[], _data?: unknown): boolean {
     return query.includes(value);
   }
 
@@ -244,7 +244,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$nin(value: unknown, query: unknown[], data?: unknown): boolean {
+  q$nin(value: unknown, query: unknown[], _data?: unknown): boolean {
     return !query.includes(value);
   }
 
@@ -256,7 +256,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {*}
    */
-  u$set<T>(value: unknown, update: T, data?: unknown): T {
+  u$set<T>(value: unknown, update: T, _data?: unknown): T {
     return update;
   }
 
@@ -268,7 +268,7 @@ class SchemaType<T> {
    * @param {Object} data
    * @return {*}
    */
-  u$unset<T>(value: T, update: boolean, data?: unknown): T | undefined { return update ? undefined : value; }
+  u$unset<T>(value: T, update: boolean, _data?: unknown): T | undefined { return update ? undefined : value; }
 
   /**
    * Renames a field.

src/schematype.ts

@@ -187,7 +187,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$size(value?: unknown[], query?: unknown, data?: unknown): boolean {
+  q$size(value?: unknown[], query?: unknown, _data?: unknown): boolean {
     return (value ? value.length : 0) === query;
   }
 
@@ -199,7 +199,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$in(value?: unknown[], query?: unknown[], data?: unknown): boolean {
+  q$in(value?: unknown[], query?: unknown[], _data?: unknown): boolean {
     if (!value) return false;
 
     for (let i = 0, len = query.length; i < len; i++) {
@@ -217,7 +217,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$nin<T>(value?: T[], query?: T[], data?: unknown): boolean {
+  q$nin<T>(value?: T[], query?: T[], _data?: unknown): boolean {
     if (!value) return true;
 
     for (let i = 0, len = query.length; i < len; i++) {
@@ -235,7 +235,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Boolean}
    */
-  q$all<T>(value?: T[], query?: T[], data?: unknown): boolean {
+  q$all<T>(value?: T[], query?: T[], _data?: unknown): boolean {
     if (!value) return false;
 
     for (let i = 0, len = query.length; i < len; i++) {
@@ -253,7 +253,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Array}
    */
-  u$push<T>(value?: T[], update?: T | T[], data?: unknown): T[] {
+  u$push<T>(value?: T[], update?: T | T[], _data?: unknown): T[] {
     if (isArray(update)) {
       return value ? value.concat(update) : update;
     }
@@ -274,7 +274,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Array}
    */
-  u$unshift<T>(value?: T[], update?: T | T[], data?: unknown): T[] {
+  u$unshift<T>(value?: T[], update?: T | T[], _data?: unknown): T[] {
     if (isArray(update)) {
       return value ? update.concat(value) : update;
     }
@@ -295,7 +295,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Array}
    */
-  u$pull<T>(value?: T[], update?: T | T[], data?: unknown): T[] {
+  u$pull<T>(value?: T[], update?: T | T[], _data?: unknown): T[] {
     if (!value) return value;
 
     if (isArray(update)) {
@@ -313,7 +313,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Array}
    */
-  u$shift<T>(value?: T[], update?: number | boolean, data?: unknown): T[] {
+  u$shift<T>(value?: T[], update?: number | boolean, _data?: unknown): T[] {
     if (!value || !update) return value;
 
     if (update === true) {
@@ -333,7 +333,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Array}
    */
-  u$pop<T>(value?: T[], update?: number | boolean, data?: unknown): T[] {
+  u$pop<T>(value?: T[], update?: number | boolean, _data?: unknown): T[] {
     if (!value || !update) return value;
 
     const length = value.length;
@@ -355,7 +355,7 @@ class SchemaTypeArray<I, T extends SchemaType<I>> extends SchemaType<I[]> {
    * @param {Object} data
    * @return {Array}
    */
-  u$addToSet<T>(value?: T[], update?: T | T[], data?: unknown): T[] {
+  u$addToSet<T>(value?: T[], update?: T | T[], _data?: unknown): T[] {
     if (isArray(update)) {
       if (!value) return update;
 

src/types/array.ts

@@ -1,6 +1,6 @@
 import SchemaType from '../schematype';
 import ValidationError from '../error/validation';
-import type { WithImplicitCoercion } from 'buffer';
+type WithImplicitCoercion<T> = T | { valueOf(): T };
 
 /**
  * Boolean schema type.