chore: upgrade to eslint v9

eslint.config.mjs

@@ -0,0 +1,47 @@
+// @ts-check
+import eslint from '@eslint/js';
+import eslintPluginPrettierRecommended from 'eslint-plugin-prettier/recommended';
+import globals from 'globals';
+import tseslint from 'typescript-eslint';
+
+export default tseslint.config(
+  {
+    ignores: ['tests/**'],
+  },
+  eslint.configs.recommended,
+  ...tseslint.configs.recommendedTypeChecked,
+  eslintPluginPrettierRecommended,
+  {
+    languageOptions: {
+      globals: {
+        ...globals.node,
+        ...globals.jest,
+      },
+      ecmaVersion: 5,
+      sourceType: 'module',
+      parserOptions: {
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
+      },
+    },
+  },
+  {
+    rules: {
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/no-unsafe-assignment': 'off',
+      '@typescript-eslint/no-unsafe-call': 'off',
+      '@typescript-eslint/no-unsafe-member-access': 'off',
+      '@typescript-eslint/no-unsafe-function-type': 'off',
+      '@typescript-eslint/no-unsafe-argument': 'off',
+      '@typescript-eslint/no-unsafe-return': 'off',
+      '@typescript-eslint/require-await': 'warn',
+      '@typescript-eslint/no-unused-vars': 'off',
+      '@typescript-eslint/unbound-method': 'off',
+      '@typescript-eslint/no-unsafe-enum-comparison': 'off',
+      '@typescript-eslint/no-redundant-type-constituents': 'warn',
+      '@typescript-eslint/only-throw-error': 'warn',
+      'no-useless-escape': 'off',
+      '@typescript-eslint/no-require-imports': 'off'
+    },
+  },
+);

lib/decorators/api-callbacks.decorator.ts

@@ -1,6 +1,6 @@
 import { DECORATORS } from '../constants';
 import { createMixedDecorator } from './helpers';
-import { CallBackObject } from '../interfaces/callback-object.interface'
+import { CallBackObject } from '../interfaces/callback-object.interface';
 
 export function ApiCallbacks(...callbackObject: Array<CallBackObject<any>>) {
   return createMixedDecorator(DECORATORS.API_CALLBACKS, callbackObject);

lib/decorators/api-hide-property.decorator.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-empty-function */
 export function ApiHideProperty(): PropertyDecorator {
   return (target: Record<string, any>, propertyKey: string | symbol) => {};
 }

lib/explorers/api-callbacks.explorer.ts

@@ -1,7 +1,7 @@
 import { Type } from '@nestjs/common';
 import { DECORATORS } from '../constants';
 import { getSchemaPath } from '../utils';
-import { CallBackObject } from '../interfaces/callback-object.interface'
+import { CallBackObject } from '../interfaces/callback-object.interface';
 
 export const exploreApiCallbacksMetadata = (
   instance: object,
@@ -11,39 +11,42 @@ export const exploreApiCallbacksMetadata = (
   const callbacksData = Reflect.getMetadata(DECORATORS.API_CALLBACKS, method);
   if (!callbacksData) return callbacksData;
 
-  return callbacksData.reduce((acc, callbackData: CallBackObject<string | Function>) => {
-    const {
-      name: eventName,
-      callbackUrl,
-      method: callbackMethod,
-      requestBody,
-      expectedResponse
-    } = callbackData;
-    return {
-      ...acc,
-      [eventName]: {
-        [callbackUrl]: {
-          [callbackMethod]: {
-            requestBody: {
-              required: true,
-              content: {
-                'application/json': {
-                  schema: {
-                    $ref: getSchemaPath(requestBody.type)
+  return callbacksData.reduce(
+    (acc, callbackData: CallBackObject<string | Function>) => {
+      const {
+        name: eventName,
+        callbackUrl,
+        method: callbackMethod,
+        requestBody,
+        expectedResponse
+      } = callbackData;
+      return {
+        ...acc,
+        [eventName]: {
+          [callbackUrl]: {
+            [callbackMethod]: {
+              requestBody: {
+                required: true,
+                content: {
+                  'application/json': {
+                    schema: {
+                      $ref: getSchemaPath(requestBody.type)
+                    }
                   }
                 }
-              }
-            },
-            responses: {
-              [expectedResponse.status]: {
-                description:
-                  expectedResponse.description ||
-                  'Your server returns this code if it accepts the callback'
+              },
+              responses: {
+                [expectedResponse.status]: {
+                  description:
+                    expectedResponse.description ||
+                    'Your server returns this code if it accepts the callback'
+                }
               }
             }
           }
         }
-      }
-    };
-  }, {});
+      };
+    },
+    {}
+  );
 };

lib/extra/swagger-shim.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-empty-function */
 export function ApiProperty() {
   return () => {};
 }

lib/interfaces/callback-object.interface.ts

@@ -1,12 +1,12 @@
-export interface CallBackObject<T>  {
-  name: string,
-  callbackUrl: string,
-  method: string,
+export interface CallBackObject<T> {
+  name: string;
+  callbackUrl: string;
+  method: string;
   requestBody: {
-    type: T
-  },
+    type: T;
+  };
   expectedResponse: {
-    status: number
-    description?: string
-  },
+    status: number;
+    description?: string;
+  };
 }

lib/plugin/utils/ast-utils.ts

@@ -15,7 +15,6 @@ import {
   ObjectFlags,
   ObjectType,
   PropertyAccessExpression,
-  SourceFile,
   SyntaxKind,
   Type,
   TypeChecker,
@@ -141,10 +140,8 @@ export function getDocComment(node: Node): DocComment {
   );
   return parserContext.docComment;
 }
-export function getMainCommentOfNode(
-  node: Node,
-  sourceFile: SourceFile
-): string {
+
+export function getMainCommentOfNode(node: Node): string {
   const docComment = getDocComment(node);
   return renderDocNode(docComment.summarySection).trim();
 }
@@ -155,7 +152,9 @@ export function parseCommentDocValue(docValue: string, type: ts.Type) {
   if (!type || !isString(type)) {
     try {
       value = JSON.parse(value);
-    } catch {}
+    } catch {
+      // Do nothing
+    }
   } else if (isString(type)) {
     if (value.split(' ').length !== 1 && !value.startsWith('"')) {
       value = null;
@@ -252,7 +251,9 @@ export function getTsDocErrorsOfNode(node: Node) {
           status: match[1],
           description: `"${match[2]}"`
         });
-      } catch (err) {}
+      } catch {
+        // Do nothing
+      }
     });
   };
   introspectTsDocTags(docComment);

lib/plugin/utils/plugin-utils.ts

@@ -115,7 +115,10 @@ export function getTypeReferenceAsString(
     if (type.aliasSymbol) {
       return { typeName: 'Object', arrayDepth };
     }
-    if (typeChecker.getApparentType(type).getSymbol().getEscapedName() === 'String') {
+    if (
+      typeChecker.getApparentType(type).getSymbol().getEscapedName() ===
+      'String'
+    ) {
       return { typeName: String.name, arrayDepth };
     }
     return { typeName: undefined };
@@ -162,7 +165,7 @@ export function replaceImportPath(
       typeReference,
       importPath: null
     };
-  } catch (_error) {
+  } catch {
     const from = options?.readonly
       ? convertPath(options.pathToSource)
       : posix.dirname(convertPath(fileName));
@@ -250,7 +253,8 @@ export function isAutoGeneratedEnumUnion(
       return undefined;
     }
     const undefinedTypeIndex = type.types.findIndex(
-      (type: any) => type.intrinsicName === 'undefined' || type.intrinsicName === 'null'
+      (type: any) =>
+        type.intrinsicName === 'undefined' || type.intrinsicName === 'null'
     );
     if (undefinedTypeIndex < 0) {
       return undefined;

lib/plugin/visitors/abstract.visitor.ts

@@ -1,7 +1,7 @@
 import * as ts from 'typescript';
 import { OPENAPI_NAMESPACE, OPENAPI_PACKAGE_NAME } from '../plugin-constants';
 
-const [major, minor] = ts.versionMajorMinor?.split('.').map((x) => +x);
+const [major, minor] = ts.versionMajorMinor.split('.').map((x) => +x);
 
 export class AbstractFileVisitor {
   updateImports(

lib/plugin/visitors/controller-class.visitor.ts

@@ -229,7 +229,7 @@ export class ControllerClassVisitor extends AbstractFileVisitor {
       }
     }
 
-    const extractedComments = getMainCommentOfNode(node, sourceFile);
+    const extractedComments = getMainCommentOfNode(node);
     if (!extractedComments) {
       return [];
     }

lib/plugin/visitors/model-class.visitor.ts

@@ -3,6 +3,10 @@ import { posix } from 'path';
 import * as ts from 'typescript';
 import { factory, PropertyAssignment } from 'typescript';
 import { ApiHideProperty, ApiProperty } from '../../decorators';
+import {
+  decoratorsProperties,
+  decoratorsPropertiesMappingType
+} from '../../services/decorators-properties';
 import { PluginOptions } from '../merge-options';
 import { METADATA_FACTORY_NAME } from '../plugin-constants';
 import { pluginDebugLogger } from '../plugin-debug-logger';
@@ -28,10 +32,6 @@ import {
 } from '../utils/plugin-utils';
 import { typeReferenceToIdentifier } from '../utils/type-reference-to-identifier.util';
 import { AbstractFileVisitor } from './abstract.visitor';
-import {
-  decoratorsProperties,
-  decoratorsPropertiesMappingType
-} from '../../services/decorators-properties';
 
 type ClassMetadata = Record<string, ts.ObjectLiteralExpression>;
 
@@ -160,7 +160,8 @@ export class ModelClassVisitor extends AbstractFileVisitor {
     metadata: ClassMetadata
   ) {
     const isPropertyStatic = (node.modifiers || []).some(
-      (modifier: ts.ModifierLike) => modifier.kind === ts.SyntaxKind.StaticKeyword
+      (modifier: ts.ModifierLike) =>
+        modifier.kind === ts.SyntaxKind.StaticKeyword
     );
     if (isPropertyStatic) {
       return node;
@@ -850,7 +851,7 @@ export class ModelClassVisitor extends AbstractFileVisitor {
       return [];
     }
     const propertyAssignments = [];
-    const comments = getMainCommentOfNode(node, sourceFile);
+    const comments = getMainCommentOfNode(node);
     const tags = getTsDocTagsOfNode(node, typeChecker);
 
     const keyOfComment = options.dtoKeyOfComment;

lib/services/parameter-metadata-accessor.ts

@@ -56,11 +56,12 @@ export class ParameterMetadataAccessor {
 
     const parametersWithType: ParamsWithType = mapValues(
       reverseObjectKeys(routeArgsMetadata),
-      (param: ParamMetadata) => ({
-        type: types[param.index],
-        name: param.data,
-        required: true
-      }) as unknown as ParamsWithType
+      (param: ParamMetadata) =>
+        ({
+          type: types[param.index],
+          name: param.data,
+          required: true
+        }) as unknown as ParamsWithType
     );
     const excludePredicate = (val: ParamWithTypeMetadata) =>
       val.in === PARAM_TOKEN_PLACEHOLDER || (val.name && val.in === 'body');

lib/services/response-object-factory.ts

@@ -41,7 +41,7 @@ export class ResponseObjectFactory {
     schemas: Record<string, SchemaObject>,
     factories: FactoriesNeededByResponseFactory
   ) {
-    const { type, isArray } = response as ApiResponseMetadata;
+    const { type, isArray } = response;
     response = omit(response, ['isArray']);
 
     if (!type) {

lib/services/schema-object-factory.ts

@@ -179,9 +179,7 @@ export class SchemaObjectFactory {
     if (!prototype) {
       return;
     }
-    const extraModels = exploreGlobalApiExtraModelsMetadata(
-      type as Type<unknown>
-    );
+    const extraModels = exploreGlobalApiExtraModelsMetadata(type);
     extraModels.forEach((item) =>
       this.exploreModelSchema(item, schemas, pendingSchemasRefs)
     );
@@ -341,12 +339,12 @@ export class SchemaObjectFactory {
       const _enum = param.enum
         ? param.enum
         : param.schema
-        ? param.schema['items']
-          ? param.schema['items']['enum']
-          : param.schema['enum']
-        : param.isArray && param.items
-        ? param.items.enum
-        : undefined;
+          ? param.schema['items']
+            ? param.schema['items']['enum']
+            : param.schema['enum']
+          : param.isArray && param.items
+            ? param.items.enum
+            : undefined;
 
       schemas[enumName] = {
         type: