node-pluginsmanager 3.3.0 → 3.4.0

package/lib/cjs/PluginsManager.js

@@ -19,6 +19,8 @@ const checkFunction_1 = __importDefault(require("./checkers/checkFunction"));
 const checkNonEmptyArray_1 = __importDefault(require("./checkers/checkNonEmptyArray"));
 const checkNonEmptyString_1 = __importDefault(require("./checkers/checkNonEmptyString"));
 const checkOrchestrator_1 = __importDefault(require("./checkers/checkOrchestrator"));
+const isDirectory_1 = __importDefault(require("./utils/isDirectory"));
+const isFile_1 = __importDefault(require("./utils/isFile"));
 const createPluginByDirectory_1 = __importDefault(require("./createPluginByDirectory"));
 const loadSortedPlugins_1 = __importDefault(require("./loadSortedPlugins"));
 const initSortedPlugins_1 = __importDefault(require("./initSortedPlugins"));
@@ -280,8 +282,30 @@ class PluginsManager extends node_events_1.default {
                 });
             });
         }).then((directory) => {
-            return (0, gitInstall_1.default)(directory, user, repo).then(() => {
-                // install dependencies & execute install script
+            // download plugin
+            return Promise.resolve().then(() => {
+                return (0, gitInstall_1.default)(directory, user, repo);
+                // check if plugin directory is created
+            }).then(() => {
+                return (0, isDirectory_1.default)(directory).then((isPluginADirectory) => {
+                    return !isPluginADirectory ? Promise.reject(new Error("\"" + repo + "\" plugin directory is not created")) : Promise.resolve();
+                });
+                // check if plugin has a valid package.json
+            }).then(() => {
+                return (0, isFile_1.default)((0, node_path_1.join)(directory, "package.json")).then((isPluginAPackageFile) => {
+                    return !isPluginAPackageFile ? Promise.reject(new Error("\"" + repo + "\" plugin has no valid package.json")) : Promise.resolve();
+                });
+                // install dependencies if needed
+            }).then(() => {
+                return (0, promises_1.readFile)((0, node_path_1.join)(directory, "package.json"), "utf-8").then((content) => {
+                    return JSON.parse(content);
+                }).then(({ dependencies }) => {
+                    return "object" !== typeof dependencies || null === dependencies || 0 >= Object.keys(dependencies).length
+                        ? Promise.resolve()
+                        : (0, npmInstall_1.default)(directory);
+                });
+                // create plugin
+            }).then(() => {
                 return (0, createPluginByDirectory_1.default)(directory, this.externalResourcesDirectory, this._logger, ...data);
                 // check plugin modules versions
             }).then((plugin) => {
@@ -289,17 +313,19 @@ class PluginsManager extends node_events_1.default {
                     return Promise.resolve(plugin);
                 });
             }).then((plugin) => {
+                // execute plugin install script
                 return Promise.resolve().then(() => {
-                    return !plugin.dependencies ? Promise.resolve() : (0, npmInstall_1.default)(directory);
-                }).then(() => {
-                    return plugin.install(...data);
-                    // execute init script
+                    return plugin.install(...data).then(() => {
+                        this.emit("installed", plugin, ...data);
+                    });
+                    // execute init plugin script
                 }).then(() => {
-                    this.emit("installed", plugin, ...data);
-                    return plugin.init(...data);
+                    return plugin.init(...data).then(() => {
+                        this.emit("initialized", plugin, ...data);
+                        this.plugins.push(plugin);
+                    });
+                    // return installed plugin
                 }).then(() => {
-                    this.emit("initialized", plugin, ...data);
-                    this.plugins.push(plugin);
                     return Promise.resolve(plugin);
                 }).catch((err) => {
                     return this.uninstall(plugin, ...data).then(() => {

package/lib/cjs/checkers/checkDirectory.js

@@ -5,18 +5,13 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = checkDirectory;
-// natives
-const node_fs_1 = require("node:fs");
 // locals
 const checkNonEmptyString_1 = __importDefault(require("./checkNonEmptyString"));
+const isDirectory_1 = __importDefault(require("../utils/isDirectory"));
 // module
 function checkDirectory(dataName, directory) {
     return (0, checkNonEmptyString_1.default)(dataName, directory).then(() => {
-        return new Promise((resolve) => {
-            (0, node_fs_1.lstat)(directory, (err, stats) => {
-                return resolve(Boolean(!err && stats.isDirectory()));
-            });
-        });
+        return (0, isDirectory_1.default)(directory);
     }).then((exists) => {
         return exists ? Promise.resolve() : Promise.reject(new Error("\"" + dataName + "\" (" + directory + ") is not a valid directory"));
     });

package/lib/cjs/utils/isDirectory.js

@@ -14,7 +14,7 @@ function isDirectory(directory) {
             reject(new TypeError("\"directory\" argument is not a string"));
         }
         else if ("" === directory.trim()) {
-            reject(new Error("\"directory\" argument is empty"));
+            reject(new RangeError("\"directory\" argument is empty"));
         }
         else {
             (0, node_fs_1.lstat)(directory, (err, stats) => {

package/lib/cjs/utils/isFile.d.ts

@@ -0,0 +1 @@
+export default function isFile(directory: string): Promise<boolean>;

package/lib/cjs/utils/isFile.js

@@ -0,0 +1,25 @@
+"use strict";
+// deps
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = isFile;
+// natives
+const node_fs_1 = require("node:fs");
+// module
+function isFile(directory) {
+    return new Promise((resolve, reject) => {
+        if ("undefined" === typeof directory) {
+            reject(new ReferenceError("missing \"directory\" argument"));
+        }
+        else if ("string" !== typeof directory) {
+            reject(new TypeError("\"directory\" argument is not a string"));
+        }
+        else if ("" === directory.trim()) {
+            reject(new RangeError("\"directory\" argument is empty"));
+        }
+        else {
+            (0, node_fs_1.lstat)(directory, (err, stats) => {
+                return resolve(Boolean(!err && stats.isFile()));
+            });
+        }
+    });
+}

package/package.json

@@ -1,7 +1,7 @@
 {
 
   "name": "node-pluginsmanager",
-  "version": "3.3.0",
+  "version": "3.4.0",
   "description": "A plugins manager.",
 
   "type": "commonjs",
@@ -47,7 +47,7 @@
   },
 
   "dependencies": {
-    "check-node-engine": "1.0.0",
+    "check-node-engine": "1.1.0",
     "check-version-modules": "2.5.0"
   },
   "devDependencies": {
@@ -59,13 +59,13 @@
     "express": "5.2.1",
     "husky": "9.1.7",
     "mocha": "11.7.6",
-    "node-pluginsmanager-plugin": "7.2.0",
+    "node-pluginsmanager-plugin": "7.3.0",
     "nyc": "18.0.0",
     "proxyquire": "2.1.3",
     "rimraf": "6.1.3",
     "socket.io": "4.8.3",
     "typescript": "5.9.3",
-    "used-deps-analyzer": "0.3.0",
+    "used-deps-analyzer": "0.4.0",
     "ws": "8.21.0"
   },