@adaptivestone/framework 5.0.0-beta.4 → 5.0.0-beta.6

package/CHANGELOG.md

@@ -1,3 +1,12 @@
+### 5.0.0-beta.5
+
+[BREAKING] remove minimist CLI parsing and replace it by commandArguments parser
+[UPDATE] migrated from eslint-plugin-import to eslint-plugin-import-x
+
+### 5.0.0-beta.5
+
+[UPDATE] migrate to eslint 9 and away from aibnb styles (they are abonded)
+
 ### 5.0.0-beta.4
 
 [NEW] on shutdown event now after timeout we are forcing to shutdown
@@ -462,7 +471,7 @@ await doc.populate([
 [UPDATE] update deps
 [UPDATE] winston console transport now using timestapms
 [UPDATE] PrepareAppInfo middleware now a global one. Do not need to include it on every controller
-[NEW] Request anso works with req.query, but req.body have bigger priority
+[NEW] Request also works with req.query, but req.body have bigger priority
 
 ### 2.18.0
 

package/Cli.js

@@ -1,4 +1,3 @@
-import parseArgs from 'minimist';
 import mongoose from 'mongoose';
 import BaseCli from './modules/BaseCli.js';
 import Server from './server.js';
@@ -8,14 +7,14 @@ class Cli extends BaseCli {
     mongoose.set('autoIndex', false); // we do not need create indexes on CLI.
     const server = new Server(serverConfig);
     super(server);
-    this.args = parseArgs(process.argv.slice(3));
   }
 
   async run() {
     await this.server.init({ isSkipModelInit: true, isSkipModelLoading: true });
     const command = process.argv[2]?.toLowerCase();
-    await super.run(command, this.args);
+    await super.run(command);
     this.app.events.emit('shutdown');
+    return true;
   }
 }
 

package/cluster.js

@@ -4,7 +4,6 @@ import { cpus } from 'node:os';
 const numCPUs = cpus().length;
 
 if (cluster.isPrimary) {
-  // eslint-disable-next-line no-console
   console.log(`Master ${process.pid} is running`);
   // Fork workers.
   for (let i = 0; i < numCPUs; i += 1) {
@@ -12,7 +11,6 @@ if (cluster.isPrimary) {
   }
 
   cluster.on('exit', (worker, code, signal) => {
-    // eslint-disable-next-line no-console
     console.log(
       `Worker \x1B[45m ${
         worker.process.pid

package/commands/CreateUser.js

@@ -6,6 +6,37 @@ class CreateUser extends AbstractCommand {
     return 'Create user in a database';
   }
 
+  /**
+   * You able to add command arguments for parsing there.
+   * @returns {import("../types/ICommandArguments.js").ICommandArguments}
+   */
+  static get commandArguments() {
+    return {
+      id: {
+        type: 'string',
+        description: 'User id to find user',
+      },
+      email: {
+        type: 'string',
+        description: 'User id to find/create user',
+      },
+      password: {
+        type: 'string',
+        description: 'New password for user',
+      },
+      roles: {
+        type: 'string',
+        description:
+          'User roles comma separated string (--roles=user,admin,someOtherRoles)',
+      },
+      update: {
+        type: 'boolean',
+        default: false,
+        description: 'Update user if it exists',
+      },
+    };
+  }
+
   async run() {
     const User = this.app.getModel('User');
     const { id, email, password, roles, update } = this.args;

package/commands/DropIndex.js

@@ -5,12 +5,21 @@ class DropIndex extends AbstractCommand {
     return 'Drop indexes of model';
   }
 
-  async run() {
-    if (!this.args.model) {
-      this.logger.error('Please provide model name as "--model=BestUserModel"');
-      return false;
-    }
+  /**
+   * You able to add command arguments for parsing there.
+   * @returns {import("../types/ICommandArguments.js").ICommandArguments}
+   */
+  static get commandArguments() {
+    return {
+      model: {
+        type: 'string',
+        description: 'Model name',
+        required: true,
+      },
+    };
+  }
 
+  async run() {
     const Model = this.app.getModel(this.args.model);
 
     if (!Model) {

package/commands/GetOpenApiJson.js

@@ -8,6 +8,18 @@ class GetOpenApiJson extends AbstractCommand {
   static get description() {
     return 'Generate documentation (openApi) ';
   }
+  /**
+   * You able to add command arguments for parsing there.
+   * @returns {import("../types/ICommandArguments.js").ICommandArguments}
+   */
+  static get commandArguments() {
+    return {
+      output: {
+        type: 'string',
+        description: 'Output file path',
+      },
+    };
+  }
 
   async run() {
     const { myDomain } = this.app.getConfig('http');
@@ -18,7 +30,7 @@ class GetOpenApiJson extends AbstractCommand {
 
     try {
       jsonFile = JSON.parse(await fs.readFile(jsonFile, 'utf8'));
-    } catch (e) {
+    } catch {
       this.logger.error(
         'No npm package detected. Please start this command via NPM as it depends on package.json',
       );
@@ -123,7 +135,6 @@ class GetOpenApiJson extends AbstractCommand {
 
         let routeName = Object.keys(route)[0];
         if (routeName === '/') {
-          // eslint-disable-next-line no-continue
           continue;
         }
 

package/commands/SyncIndexes.js

@@ -29,6 +29,7 @@ class SyncIndexes extends AbstractCommand {
         this.logger.info(`Model - ${modelName} NO removed indexes`);
       }
     }
+    return true;
   }
 
   static get description() {

package/commands/migration/Create.js

@@ -7,28 +7,39 @@ class CreateMigration extends AbstractCommand {
     return 'Create new migration';
   }
 
+  /**
+   * You able to add command arguments for parsing there.
+   * @returns {import("../../types/ICommandArguments.js").ICommandArguments}
+   */
+  static get commandArguments() {
+    return {
+      name: {
+        type: 'string',
+        description: 'Migration name',
+        required: true,
+      },
+    };
+  }
+
   async run() {
-    if (!this.args.name) {
-      return this.logger.error(
-        'Please provide migration name with key "--name={someName}"',
-      );
-    }
     if (this.args.name.match(/^\d/)) {
-      return this.logger.error('Command cant start from nubmer');
+      this.logger.error('Command cant start from nubmer');
+      return false;
     }
-    const fileName = `${Date.now()}_${this.constructor.camelSentence(
+    const fileName = `${Date.now()}_${CreateMigration.camelSentence(
       this.args.name,
     )}.js`;
 
-    const fileContent = this.constructor.getTemplate(
-      this.constructor.camelSentence(this.args.name),
+    const fileContent = CreateMigration.getTemplate(
+      CreateMigration.camelSentence(this.args.name),
     );
 
     await fs.writeFile(
       path.join(this.app.foldersConfig.migrations, fileName),
       fileContent,
     );
-    return this.logger.info(`Migration created ${fileName}`);
+    this.logger.info(`Migration created ${fileName}`);
+    return true;
   }
 
   static camelSentence(str) {

package/commands/migration/Migrate.js

@@ -49,6 +49,7 @@ class Migrate extends AbstractCommand {
     this.logger.info(
       `=== Migration Finished. Migrated ${migrations.length} files ===`,
     );
+    return true;
   }
 }
 

package/controllers/Auth.js

@@ -1,4 +1,4 @@
-import yup from 'yup';
+import { object, string } from 'yup';
 import AbstractController from '../modules/AbstractController.js';
 import GetUserByToken from '../services/http/middleware/GetUserByToken.js';
 import RateLimiter from '../services/http/middleware/RateLimiter.js';
@@ -9,58 +9,52 @@ class Auth extends AbstractController {
       post: {
         '/login': {
           handler: this.postLogin,
-          request: yup.object().shape({
-            email: yup.string().email().required('auth.emailProvided'), // if not provided then error will be generated
-            password: yup.string().required('auth.passwordProvided'), // possible to provide values from translation
+          request: object().shape({
+            email: string().email().required('auth.emailProvided'), // if not provided then error will be generated
+            password: string().required('auth.passwordProvided'), // possible to provide values from translation
           }),
         },
         '/register': {
           handler: this.postRegister,
-          request: yup.object().shape({
-            email: yup
-              .string()
+          request: object().shape({
+            email: string()
               .email('auth.emailValid')
               .required('auth.emailProvided'),
-            password: yup
-              .string()
+            password: string()
               .matches(
                 /^[a-zA-Z0-9!@#$%ˆ^&*()_+\-{}[\]<>]+$/,
                 'auth.passwordValid',
               )
               .required('auth.passwordProvided'),
-            nickName: yup
-              .string()
-              .matches(/^[a-zA-Z0-9_\-.]+$/, 'auth.nickNameValid'),
-            firstName: yup.string(),
-            lastName: yup.string(),
+            nickName: string().matches(
+              /^[a-zA-Z0-9_\-.]+$/,
+              'auth.nickNameValid',
+            ),
+            firstName: string(),
+            lastName: string(),
           }),
         },
         '/logout': this.postLogout,
         '/verify': this.verifyUser,
         '/send-recovery-email': {
           handler: this.sendPasswordRecoveryEmail,
-          request: yup
-            .object()
-            .shape({ email: yup.string().email().required() }),
+          request: object().shape({ email: string().email().required() }),
         },
         '/recover-password': {
           handler: this.recoverPassword,
-          request: yup.object().shape({
-            password: yup
-              .string()
+          request: object().shape({
+            password: string()
               .matches(
                 /^[a-zA-Z0-9!@#$%ˆ^&*()_+\-{}[\]<>]+$/,
                 'auth.passwordValid',
               )
               .required(),
-            passwordRecoveryToken: yup.string().required(),
+            passwordRecoveryToken: string().required(),
           }),
         },
         '/send-verification': {
           handler: this.sendVerification,
-          request: yup
-            .object()
-            .shape({ email: yup.string().email().required() }),
+          request: object().shape({ email: string().email().required() }),
         },
       },
     };
@@ -136,7 +130,7 @@ class Auth extends AbstractController {
       user = await User.getUserByVerificationToken(
         req.query.verification_token,
       );
-    } catch (e) {
+    } catch {
       return res.status(400).json({
         message: req.i18n.t('email.alreadyVerifiedOrWrongToken'),
       });

package/eslint.config.js

@@ -0,0 +1,68 @@
+import globals from 'globals';
+import pluginJs from '@eslint/js';
+import importPlugin from 'eslint-plugin-import-x';
+import vitest from '@vitest/eslint-plugin';
+import eslintConfigPrettier from 'eslint-config-prettier';
+import prettierPlugin from 'eslint-plugin-prettier/recommended';
+
+/** @type {import('eslint').Linter.Config[]} */
+// @ts-ignore
+export default [
+  pluginJs.configs.recommended,
+  importPlugin.flatConfigs.recommended,
+  eslintConfigPrettier,
+  prettierPlugin,
+  {
+    languageOptions: {
+      sourceType: 'module',
+      ecmaVersion: 'latest',
+      globals: {
+        ...globals.es2023,
+        ...globals.node,
+      },
+    },
+  },
+  {
+    rules: {
+      'no-await-in-loop': 'error',
+      'no-param-reassign': 'error',
+      'class-methods-use-this': 'error',
+      'no-shadow': 'error',
+      'prefer-const': 'error',
+      'import-x/no-extraneous-dependencies': ['error'],
+      'import-x/first': ['error'],
+      camelcase: ['error', { properties: 'never', ignoreDestructuring: false }],
+      'prefer-destructuring': [
+        'error',
+        {
+          VariableDeclarator: {
+            array: false,
+            object: true,
+          },
+          AssignmentExpression: {
+            array: true,
+            object: false,
+          },
+        },
+        {
+          enforceForRenamedProperties: false,
+        },
+      ],
+      'no-plusplus': 'error',
+      'consistent-return': 'error',
+      'no-return-await': 'error',
+      'arrow-body-style': 'error',
+      'dot-notation': 'error',
+      curly: 'error',
+    },
+  },
+  {
+    files: ['**/*.test.js'],
+    plugins: {
+      vitest,
+    },
+    rules: {
+      ...vitest.configs.recommended.rules,
+    },
+  },
+];

package/helpers/files.js

@@ -73,7 +73,4 @@ const getFilesPathWithInheritance = async ({
   return filesToLoad;
 };
 
-export {
-  // eslint-disable-next-line import/prefer-default-export
-  getFilesPathWithInheritance,
-};
+export { getFilesPathWithInheritance };

package/helpers/logger.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 const levels = ['error', 'warn', 'info', 'http', 'verbose', 'debug', 'silly'];
 
 const consoleLogger = (level, message) => {

package/helpers/yup.js

@@ -13,12 +13,10 @@ class YupFile extends Schema {
   constructor() {
     super({
       type: 'file',
+      // @ts-ignore
       check: (value) => value.every((item) => item instanceof PersistentFile),
     });
   }
 }
 
-export {
-  // eslint-disable-next-line import/prefer-default-export
-  YupFile,
-};
+export { YupFile };

package/jsconfig.json

@@ -1,9 +1,9 @@
 {
   "compilerOptions": {
-    "module": "node16",
+    "module": "NodeNext",
     "target": "ES2022",
-    "moduleResolution": "node16",
+    "moduleResolution": "nodenext",
     "checkJs": true
   },
-  "exclude": ["node_modules"]
+  "exclude": ["node_modules", "coverage"]
 }

package/models/User.js

@@ -1,5 +1,3 @@
-/* eslint-disable no-param-reassign */
-
 import { scrypt } from 'node:crypto';
 
 import { promisify } from 'node:util';

package/modules/AbstractCommand.js

@@ -26,9 +26,18 @@ class AbstractCommand extends Base {
     return `CLI: ${commandName} ${JSON.stringify(args)}`;
   }
 
+  /**
+   * You able to add command arguments for parsing there.
+   * @see https://nodejs.org/api/util.html#utilparseargsconfig in config.options plus extended with description and required
+   * @returns {import("../types/ICommandArguments.js").ICommandArguments}
+   */
+  static get commandArguments() {
+    return {};
+  }
+
   /**
    * Entry point to every command. This method should be overridden
-   * @return {Promise<boolean>} resut
+   * @return {Promise<boolean>} result
    */
   async run() {
     this.logger.error('You should implement run method');

package/modules/AbstractController.js

@@ -1,5 +1,3 @@
-/* eslint-disable no-restricted-syntax */
-/* eslint-disable guard-for-in */
 import express from 'express';
 
 import Base from './Base.js';
@@ -88,7 +86,6 @@ class AbstractController extends Base {
         this.logger.error(
           `Method ${verb} not exist for router. Please check your codebase`,
         );
-        // eslint-disable-next-line no-continue
         continue;
       }
       for (const path in routes[verb]) {
@@ -112,7 +109,6 @@ class AbstractController extends Base {
                 routeObject.handler
               }' for controller '${this.getConstructorName()}'`,
             );
-            // eslint-disable-next-line no-continue
             continue;
           }
         }
@@ -300,14 +296,12 @@ class AbstractController extends Base {
         let realPath = path;
         if (typeof realPath !== 'string') {
           this.logger.error(`Path not a string ${realPath}. Please check it`);
-          // eslint-disable-next-line no-continue
           continue;
         }
         if (!realPath.startsWith('/')) {
           method = realPath.split('/')[0]?.toLowerCase();
           if (!method) {
             this.logger.error(`Method not found for ${realPath}`);
-            // eslint-disable-next-line no-continue
             continue;
           }
           realPath = realPath.substring(method.length);
@@ -316,7 +310,6 @@ class AbstractController extends Base {
           this.logger.error(
             `Method ${method} not exist for middleware. Please check your codebase`,
           );
-          // eslint-disable-next-line no-continue
           continue;
         }
         const fullPath = `/${httpPath}/${realPath.toUpperCase()}`

package/modules/Base.js

@@ -21,7 +21,7 @@ class Base {
     let l;
     try {
       l = this.#realLogger;
-    } catch (e) {
+    } catch {
       console.warn(
         `You try to accees logger not from class. that can be ok in case of models.`,
       );
@@ -29,8 +29,9 @@ class Base {
     }
 
     if (!l) {
+      const { loggerGroup } = /** @type {typeof Base} */ (this.constructor);
       this.#realLogger = this.getLogger(
-        this.constructor.loggerGroup + this.getConstructorName(),
+        loggerGroup + this.getConstructorName(),
       );
     }
     return this.#realLogger;

package/modules/BaseCli.js

@@ -1,6 +1,6 @@
-/* eslint-disable no-console */
 import path from 'node:path';
 import * as url from 'node:url';
+import { parseArgs } from 'node:util';
 import Base from './Base.js';
 
 class Cli extends Base {
@@ -35,7 +35,6 @@ class Cli extends Base {
     console.log('Available commands:');
     let commandsClasses = [];
     for (const c of commands) {
-      // eslint-disable-next-line no-await-in-loop
       commandsClasses.push(import(this.commands[c]));
       // console.log(
       //   ` \x1b[36m${c.padEnd(maxLength)}\x1b[0m - ${f.default.description}`,
@@ -43,14 +42,41 @@ class Cli extends Base {
     }
     commandsClasses = await Promise.all(commandsClasses);
     for (const [key, c] of Object.entries(commands)) {
-      // eslint-disable-next-line no-await-in-loop
       console.log(
         ` \x1b[36m${c.padEnd(maxLength)}\x1b[0m - ${commandsClasses[key].default.description}`,
       );
     }
+    console.log(
+      '\nUsage (use one of option): \n node cli.js <command> [options] \n npm run cli <command>  -- [options]',
+    );
+  }
+
+  static showHelp(Command, finalArguments) {
+    console.log(`\n\x1b[32m${Command.description}\x1b[0m`);
+    let output = '';
+
+    Object.entries(finalArguments).forEach(([key, opt]) => {
+      const outputLocal = [];
+      outputLocal.push(`\n\x1b[36m  --${key} \x1b[0m`);
+      if (opt.type !== 'boolean') {
+        outputLocal.push(`<${opt.type}>`);
+        // flag += `<${opt.type}>`;
+      }
+      if (opt.required) {
+        outputLocal.push('(required)');
+      }
+      outputLocal.push(`\n      \x1b[2m${opt.description}`);
+      if (opt.default !== undefined) {
+        outputLocal.push(` (default: ${opt.default})`);
+      }
+      outputLocal.push('\x1b[0m');
+      output += outputLocal.join(' ');
+    });
+
+    console.log(output);
   }
 
-  async run(command, args) {
+  async run(command) {
     await this.loadCommands();
 
     if (!command) {
@@ -66,20 +92,54 @@ class Cli extends Base {
     }
     const { default: Command } = await import(this.commands[command]);
 
+    const defaultArgs = {
+      help: {
+        type: 'boolean',
+        description: 'Show help',
+      },
+    };
+
+    const finalArguments = {
+      ...Command.commandArguments,
+      ...defaultArgs,
+    };
+
+    const parsedArgs = parseArgs({
+      args: process.argv.slice(3), // remove command name
+      options: finalArguments,
+      tokens: true,
+    });
+
+    // @ts-ignore
+    if (parsedArgs.values.help) {
+      Cli.showHelp(Command, finalArguments);
+      return true;
+    }
+
+    for (const [key, opt] of Object.entries(finalArguments)) {
+      if (opt.required && !parsedArgs.values[key]) {
+        console.log(
+          `\x1b[31mRequired field not proivded. Please provide "${key}" argument\x1b[0m`,
+        );
+        Cli.showHelp(Command, finalArguments);
+        return false;
+      }
+    }
+
     if (Command.isShouldInitModels) {
       this.logger.debug(
         `Command ${command} isShouldInitModels called. If you want to skip loading and init models, please set isShouldInitModels to false in tyou command`,
       );
       process.env.MONGO_APP_NAME = Command.getMongoConnectionName(
         command,
-        args,
+        parsedArgs.values,
       );
       await this.server.initAllModels();
     } else {
       this.logger.debug(`Command ${command} NOT need to isShouldInitModels`);
     }
 
-    const c = new Command(this.app, this.commands, args);
+    const c = new Command(this.app, this.commands, parsedArgs.values);
     let result = false;
 
     result = await c.run().catch((e) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adaptivestone/framework",
-  "version": "5.0.0-beta.4",
+  "version": "5.0.0-beta.6",
   "description": "Adaptive stone node js framework",
   "main": "index.js",
   "type": "module",
@@ -18,8 +18,8 @@
     "test": "vitest run",
     "t": "vitest --coverage=false --reporter=default",
     "prettier": "prettier --check '**/*.(js|jsx|ts|tsx|json|css|scss|md)'",
-    "lint": "eslint '**/*.js'",
-    "lint:fix": "eslint '**/*.js' --fix",
+    "lint": "eslint",
+    "lint:fix": "eslint --fix",
     "codestyle": "npm run prettier && npm run lint",
     "prepare": "husky",
     "cli": "node cliCommand",
@@ -37,7 +37,6 @@
     "i18next": "^24.0.0",
     "i18next-fs-backend": "^2.0.0",
     "juice": "^11.0.0",
-    "minimist": "^1.2.5",
     "mongoose": "^8.0.0",
     "nodemailer": "^6.6.3",
     "nodemailer-sendmail-transport": "^1.0.2",
@@ -50,12 +49,15 @@
     "yup": "^1.0.0"
   },
   "devDependencies": {
+    "@eslint/js": "^9.19.0",
+    "@types/node": "^22.13.1",
     "@vitest/coverage-v8": "^3.0.0",
-    "eslint": "^8.0.0",
-    "eslint-config-airbnb-base": "^15.0.0",
+    "@vitest/eslint-plugin": "^1.1.25",
+    "eslint": "^9.0.0",
     "eslint-config-prettier": "^10.0.0",
+    "eslint-plugin-import-x": "^4.6.1",
     "eslint-plugin-prettier": "^5.0.0",
-    "eslint-plugin-vitest": "^0.4.0",
+    "globals": "^15.14.0",
     "husky": "^9.0.0",
     "lint-staged": "^15.0.0",
     "mongodb-memory-server": "^10.0.0",

package/server.d.ts

@@ -10,6 +10,7 @@ import Cache from './services/cache/Cache';
 import winston from 'winston';
 
 import HttpServer from './services/http/HttpServer.js';
+import ControllerManager from './services/http/ControllerManager.js';
 
 type ServerConfig = {
   folders: ExpandDeep<TFolderConfig>;
@@ -27,7 +28,7 @@ declare class Server {
     get cache(): Server['cacheService'];
     get logger(): winston.Logger;
     httpServer: HttpServer | null;
-    controllerManager: null;
+    controllerManager: ControllerManager | null;
   };
   cacheService: Cache;
 
@@ -92,7 +93,7 @@ declare class Server {
   /**
    * Run cli command into framework (http, ws, etc)
    */
-  runCliCommand(commandName: string, args: {}): Promise<BaseCli['run']>;
+  runCliCommand(commandName: string, args?: {}): Promise<BaseCli['run']>;
 }
 
 export default Server;

package/server.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import EventEmitter from 'node:events';
 import { hrtime, loadEnvFile } from 'node:process';
 import * as url from 'node:url';
@@ -12,7 +11,7 @@ import Cache from './services/cache/Cache.js';
 
 try {
   loadEnvFile();
-} catch (e) {
+} catch {
   console.warn('No env file found. This is ok. But please check youself.');
 }
 
@@ -323,7 +322,7 @@ class Server {
     function IsConstructor(f) {
       try {
         Reflect.construct(String, [], f);
-      } catch (e) {
+      } catch {
         return false;
       }
       return true;
@@ -415,7 +414,7 @@ class Server {
    * @param {String} commandName name of command to load
    * @param {Object} args list of arguments to pass into command
    */
-  async runCliCommand(commandName, args) {
+  async runCliCommand(commandName, args = {}) {
     if (!this.cli) {
       const { default: BaseCli } = await import('./modules/BaseCli.js'); // Speed optimisation
       this.cli = new BaseCli(this);

package/services/cache/Cache.js

@@ -94,7 +94,7 @@ class Cache extends Base {
           }
           return value;
         });
-      } catch (e) {
+      } catch {
         this.logger.warn(
           'Not able to parse json from redis cache. That can be a normal in case you store string here',
         );

package/services/http/HttpServer.js

@@ -43,7 +43,6 @@ class HttpServer extends Base {
     // eslint-disable-next-line no-unused-vars
     this.express.use((err, req, res, next) => {
       // error handling
-      // eslint-disable-next-line no-console
       console.error(err.stack);
       // TODO
       res.status(500).json({ message: 'Something broke!' });
@@ -55,15 +54,15 @@ class HttpServer extends Base {
       httpConfig.port,
       httpConfig.hostname,
       () => {
-        this.logger.info(
-          `App started and listening on port ${listener.address().port}`,
-        );
-        if (+listener.address().port !== +httpConfig.port) {
+        const address = listener.address();
+        const port = typeof address === 'string' ? 0 : address.port;
+        this.logger.info(`App started and listening on port ${port}`);
+        if (+port !== +httpConfig.port) {
           // in case we using port 0
-          this.app.updateConfig('http', { port: listener.address().port });
+          this.app.updateConfig('http', { port });
           this.logger.info(
             `Updating http config to use new port ${
-              listener.address().port
+              port
             }. Old was ${httpConfig.port} `,
           );
         }

package/services/http/middleware/AbstractMiddleware.js

@@ -1,4 +1,4 @@
-import yup from 'yup';
+import { object } from 'yup';
 import Base from '../../../modules/Base.js';
 
 class AbstractMiddleware extends Base {
@@ -18,13 +18,13 @@ class AbstractMiddleware extends Base {
   // eslint-disable-next-line class-methods-use-this
   get relatedQueryParameters() {
     // For example  yup.object().shape({page: yup.number().required(),limit: yup.number()})
-    return yup.object().shape({});
+    return object().shape({});
   }
 
   // eslint-disable-next-line class-methods-use-this
   get relatedRequestParameters() {
     // For example  yup.object().shape({page: yup.number().required(),limit: yup.number()})
-    return yup.object().shape({});
+    return object().shape({});
   }
 
   get relatedReqParameters() {

package/services/http/middleware/I18n.js

@@ -27,6 +27,7 @@ class I18n extends AbstractMiddleware {
     if (I18NConfig.enabled) {
       this.logger.info('Enabling i18n support');
       this.i18n = i18next;
+      // eslint-disable-next-line import-x/no-named-as-default-member
       i18next.use(BackendFS).init({
         backend: {
           loadPath: `${this.app.foldersConfig.locales}/{{lng}}/{{ns}}.json`,
@@ -98,7 +99,6 @@ class I18n extends AbstractMiddleware {
     for (const detectorName of this.detectorOrder) {
       const lng = this.detectors[detectorName](req);
       if (!lng) {
-        // eslint-disable-next-line no-continue
         continue;
       }
       if (i18next.services.languageUtils.isSupportedCode(lng)) {

package/services/http/middleware/Pagination.js

@@ -1,4 +1,4 @@
-import yup from 'yup';
+import { object, number } from 'yup';
 import AbstractMiddleware from './AbstractMiddleware.js';
 /**
  * Middleware for reusing pagination
@@ -10,9 +10,9 @@ class Pagination extends AbstractMiddleware {
 
   // eslint-disable-next-line class-methods-use-this
   get relatedQueryParameters() {
-    return yup.object().shape({
-      page: yup.number(),
-      limit: yup.number(),
+    return object().shape({
+      page: number(),
+      limit: number(),
     });
   }
 

package/services/messaging/email/index.js

@@ -123,6 +123,7 @@ class Mail extends Base {
         this.#renderTemplateFile(templates.style),
       ]);
 
+    // @ts-ignore
     juice.tableElements = ['TABLE'];
 
     const juiceResourcesAsync = promisify(juice.juiceResources);
@@ -139,7 +140,6 @@ class Mail extends Base {
       inlinedHTML,
     };
   }
-
   /**
    * Send email
    * @param {string} to email send to
@@ -150,7 +150,7 @@ class Mail extends Base {
   async send(to, from = null, aditionalNodemailerOptions = {}) {
     const { subject, text, inlinedHTML } = await this.renderTemplate();
 
-    return this.constructor.sendRaw(
+    return Mail.sendRaw(
       this.app,
       to,
       subject,

package/services/validate/ValidateService.js

@@ -1,4 +1,4 @@
-import yup from 'yup';
+import { isSchema } from 'yup';
 import YupValidator from './drivers/YupValidator.js';
 import CustomValidator from './drivers/CustomValidator.js';
 import Base from '../../modules/Base.js';
@@ -7,7 +7,7 @@ class ValidateService extends Base {
   constructor(app, validator) {
     super(app);
     this.validator = validator
-      ? this.constructor.getDriverByValidatorBody(app, validator)
+      ? ValidateService.getDriverByValidatorBody(app, validator)
       : null;
   }
 
@@ -30,7 +30,7 @@ class ValidateService extends Base {
     if (this.isValidatorExists(body)) {
       return body;
     }
-    if (yup.isSchema(body)) {
+    if (isSchema(body)) {
       const yupValidator = new YupValidator(app, body);
       return yupValidator;
     }
@@ -104,7 +104,7 @@ class ValidateService extends Base {
     const result = [];
 
     for (const validator of validators) {
-      const formatedValidator = this.constructor.getDriverByValidatorBody(
+      const formatedValidator = ValidateService.getDriverByValidatorBody(
         this.app,
         validator,
       );

package/services/validate/drivers/AbstractValidator.js

@@ -18,8 +18,8 @@ class AbstractValidator extends Base {
     return {};
   }
 
-  // eslint-disable-next-line class-methods-use-this
-  async validateFields() {
+  // eslint-disable-next-line class-methods-use-this, no-unused-vars
+  async validateFields(data, { query, body, appInfo }) {
     // IMPLENT;
     return true;
   }

package/services/validate/drivers/CustomValidator.js

@@ -1,4 +1,4 @@
-import yup from 'yup';
+import { ValidationError } from 'yup';
 import AbstractValidator from './AbstractValidator.js';
 
 class CustomValidator extends AbstractValidator {
@@ -19,7 +19,7 @@ class CustomValidator extends AbstractValidator {
     } catch (e) {
       this.logger.warn(`CustomValidator validateFields ${e}`);
       if (e.path) {
-        throw new yup.ValidationError({
+        throw new ValidationError({
           [e.path]: e.message,
         });
       }

package/services/validate/drivers/YupValidator.js

@@ -1,9 +1,9 @@
-import yup from 'yup';
+import { ValidationError } from 'yup';
 import AbstractValidator from './AbstractValidator.js';
 
 class YupValidator extends AbstractValidator {
   get fieldsInJsonFormat() {
-    return this.constructor.convertFieldsToJson(this.body);
+    return YupValidator.convertFieldsToJson(this.body);
   }
 
   static convertFieldsToJson(fields) {
@@ -76,7 +76,7 @@ class YupValidator extends AbstractValidator {
         });
       }
 
-      throw new yup.ValidationError({
+      throw new ValidationError({
         ...errorAnswer,
       });
     }

package/tests/setup.js

@@ -12,7 +12,9 @@ mongoose.set('autoIndex', false);
 
 let mongoMemoryServerInstance;
 
+// @ts-ignore
 jest.setTimeout(1000000);
+// @ts-ignore
 beforeAll(async () => {
   mongoMemoryServerInstance = await MongoMemoryReplSet.create({
     // binary: { version: '4.4.6' },
@@ -64,9 +66,7 @@ beforeAll(async () => {
         nick: 'testUserNickName',
       },
     }).catch((e) => {
-      // eslint-disable-next-line no-console
       console.error(e);
-      // eslint-disable-next-line no-console
       console.info(
         'That error can happens in case you have custom user model. Please use global.testSetup.disableUserCreate flag to skip user creating',
       );
@@ -79,6 +79,7 @@ beforeAll(async () => {
   await global.server.startServer();
 });
 
+// @ts-ignore
 beforeEach(() => {
   if (global.server) {
     const key = `test-${Math.random().toString(36).substring(7)}`;
@@ -88,6 +89,7 @@ beforeEach(() => {
   }
 });
 
+// @ts-ignore
 afterEach(async () => {
   if (global.server) {
     const { url, namespace } = global.server.getConfig('redis');
@@ -97,12 +99,13 @@ afterEach(async () => {
       await redisClient.connect();
       await clearRedisNamespace(redisClient, namespace);
       await redisClient.disconnect();
-    } catch (err) {
+    } catch {
       // that ok. No redis connection
     }
   }
 });
 
+// @ts-ignore
 afterAll(async () => {
   if (global.server) {
     global.server.app.httpServer.shutdown();

package/tests/setupVitest.js

@@ -57,9 +57,7 @@ beforeAll(async () => {
         nick: 'testUserNickName',
       },
     }).catch((e) => {
-      // eslint-disable-next-line no-console
       console.error(e);
-      // eslint-disable-next-line no-console
       console.info(
         'That error can happens in case you have custom user model. Please use global.testSetup.disableUserCreate flag to skip user creating',
       );
@@ -90,7 +88,7 @@ afterEach(async () => {
       await redisClient.connect();
       await clearRedisNamespace(redisClient, namespace);
       await redisClient.disconnect();
-    } catch (err) {
+    } catch {
       // that ok. No redis connection
     }
   }

package/types/ICommandArguments.d.ts

@@ -0,0 +1,41 @@
+// this is from nodejs types
+interface ParseArgsOptionConfig {
+  /**
+   * Type of argument.
+   */
+  type: 'string' | 'boolean';
+  /**
+   * Whether this option can be provided multiple times.
+   * If `true`, all values will be collected in an array.
+   * If `false`, values for the option are last-wins.
+   * @default false.
+   */
+  multiple?: boolean | undefined;
+  /**
+   * A single character alias for the option.
+   */
+  short?: string | undefined;
+  /**
+   * The default option value when it is not set by args.
+   * It must be of the same type as the the `type` property.
+   * When `multiple` is `true`, it must be an array.
+   * @since v18.11.0
+   */
+  default?: string | boolean | string[] | boolean[] | undefined;
+}
+
+interface ParseArgsOptionsConfigExtended extends ParseArgsOptionConfig {
+  /**
+   * A description of the option.
+   */
+  description?: string;
+
+  /**
+   * Is it required?
+   */
+  required?: boolean;
+}
+
+export interface ICommandArguments {
+  [longOption: string]: ParseArgsOptionsConfigExtended;
+}

package/vitest.config.js

@@ -1,4 +1,4 @@
-// eslint-disable-next-line import/no-unresolved
+// eslint-disable-next-line import-x/no-unresolved
 import { defineConfig } from 'vitest/config';
 
 export default defineConfig({
@@ -9,6 +9,7 @@ export default defineConfig({
     outputFile: './coverage/rspec.xml',
     reporters: ['default', 'junit'],
     coverage: {
+      provider: 'v8',
       enabled: true,
       reporter: ['text', 'html', 'clover', 'json', 'cobertura'],
     },