@adaptivestone/framework 4.6.0 → 4.8.0

package/CHANGELOG.md

@@ -1,10 +1,23 @@
+### 4.8.0
+
+[BREAKING] Minimum node js version id 18.17.0 now.
+[BREAKING] Removed getFileWithExtendingInhirence. This was internal method and not suppose to use externally
+[UPDATE] update Base getFilesPathWithInheritance to use fs.read dir resursive option
+[UPDATE] update cache (refactor+tets)
+[UPDATE] update config and model inits
+
+### 4.7.0
+
+[UPDATE] update logger init (refactor)
+[UPDATE] updated deps
+
 ### 4.6.0
 
 [NEW] migrated from JEST to vitest
 
 ### 4.5.0
 
-[NEW] Now getSuper() available as a method on monsgoose models
+[NEW] Now getSuper() available as a method on mongoose models
 [UPDATE] Update rate-limiter-flexible to v3
 [UPDATE] Update test runner to suport ESM. In case problem with test please copy babel.config.js from framework to your project directory
 

package/Cli.js

@@ -10,6 +10,7 @@ class Cli extends BaseCli {
   }
 
   async run() {
+    await this.server.init();
     const command = process.argv[2]?.toLowerCase();
     await super.run(command, this.args);
     this.app.events.emit('shutdown');

package/cluster.js

@@ -21,6 +21,5 @@ if (cluster.isMaster) {
     cluster.fork();
   });
 } else {
-  // eslint-disable-next-line global-require
-  require('./index');
+  import('./server.js');
 }

package/commands/GetOpenApiJson.js

@@ -13,8 +13,7 @@ class GetOpenApiJson extends AbstractCommand {
     }
 
     try {
-      // eslint-disable-next-line import/no-dynamic-require, global-require
-      jsonFile = require(jsonFile);
+      jsonFile = JSON.parse(await fs.readFile(jsonFile, 'utf8'));
     } catch (e) {
       this.logger.error(
         'No npm package detected. Please start this command via NPM as it depends on package.json',

package/commands/SyncIndexes.js

@@ -18,7 +18,7 @@ class SyncIndexes extends AbstractCommand {
     for (const modelName of models) {
       const Model = this.app.getModel(modelName);
       // eslint-disable-next-line no-await-in-loop
-      const removedIndexes = await Model.syncIndexes(); // that not a bug. Lets do one by one
+      const removedIndexes = await Model.syncIndexes(); // await in loop not a bug. Lets do one by one
       if (removedIndexes.length) {
         this.logger.info(
           `Model - ${modelName} removed indexes: ${removedIndexes}`,

package/commands/migration/Migrate.js

@@ -31,8 +31,8 @@ class Migrate extends AbstractCommand {
 
     for (const migration of migrations) {
       this.logger.info(`=== Start migration ${migration.file} ===`);
-      // eslint-disable-next-line import/no-dynamic-require, global-require
-      const MigrationCommand = require(migration.path);
+      // eslint-disable-next-line no-await-in-loop
+      const MigrationCommand = await import(migration.path);
       const migrationCommand = new MigrationCommand(this.app);
       // eslint-disable-next-line no-await-in-loop
       await migrationCommand.up();

package/controllers/Auth.js

@@ -114,7 +114,7 @@ class Auth extends AbstractController {
     const { isAuthWithVefificationFlow } = this.app.getConfig('auth');
     if (isAuthWithVefificationFlow) {
       const answer = await user.sendVerificationEmail(req.i18n).catch((e) => {
-        this.logger.error(e.message);
+        this.logger.error(e);
       });
       if (!answer) {
         return res.status(500).json();
@@ -165,7 +165,7 @@ class Auth extends AbstractController {
       await user.sendPasswordRecoveryEmail(req.i18n);
       return res.status(200).json();
     } catch (e) {
-      this.logger.error(e.message);
+      this.logger.error(e);
       return res.status(400).json({ message: req.i18n.t('auth.errorUExist') });
     }
   }
@@ -175,9 +175,7 @@ class Auth extends AbstractController {
     const user = await User.getUserByPasswordRecoveryToken(
       req.appInfo.request.passwordRecoveryToken,
     ).catch((e) => {
-      this.logger.error(e.message);
-      // eslint-disable-next-line no-console
-      console.log(e);
+      this.logger.error(e);
     });
 
     if (!user) {

package/controllers/index.js

@@ -30,11 +30,8 @@ class ControllerManager extends Base {
     });
     const controllers = [];
     for (const controller of controllersToLoad) {
-      // TODO wait until https://github.com/nodejs/node/issues/35889
       controllers.push(
         import(controller.path).then(({ default: ControllerModule }) => {
-          // eslint-disable-next-line import/no-dynamic-require, global-require
-          // const ControllerModule = require(controller.path);
           const contollerName = ControllerModule.name.toLowerCase();
           let prefix = path.dirname(controller.file);
           if (prefix === '.') {