node-pluginsmanager 3.3.0 → 3.4.1

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,53 @@ 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) => {
+                    if (!isPluginADirectory) {
+                        throw new Error("\"" + repo + "\" plugin directory is not created");
+                    }
+                });
+                // work around package.json
+            }).then(() => {
+                const packageFile = (0, node_path_1.join)(directory, "package.json");
+                // check if plugin has a valid package.json
+                return (0, isFile_1.default)(packageFile).then((isPluginAPackageFile) => {
+                    if (!isPluginAPackageFile) {
+                        throw new Error("\"" + repo + "\" plugin has no valid package.json");
+                    }
+                    // read package.json and parse it
+                }).then(() => {
+                    return (0, promises_1.readFile)(packageFile, "utf-8").then((content) => {
+                        return JSON.parse(content);
+                    });
+                }).then((packageData) => {
+                    const entryPoint = (0, node_path_1.join)(directory, packageData.main);
+                    // check if the plugin has a valid entry point
+                    return (0, isFile_1.default)(entryPoint).then((hasPluginEntryPoint) => {
+                        if (!hasPluginEntryPoint) {
+                            throw new Error("\"" + repo + "\" plugin has no valid entry point");
+                        }
+                        // check if the plugin is builded
+                        // @TODO : "build installed plugin" feature to be implemented
+                    }).then(() => {
+                        return (0, isFile_1.default)(entryPoint).then((isEntryPointAFile) => {
+                            if (!isEntryPointAFile) {
+                                throw new Error("\"" + repo + "\" plugin entry point is not builded");
+                            }
+                        });
+                    }).then(() => {
+                        if ("object" !== typeof packageData.dependencies || null === packageData.dependencies || 0 >= Object.keys(packageData.dependencies).length) {
+                            return Promise.resolve();
+                        }
+                        return (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 +336,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.1",
   "description": "A plugins manager.",
 
   "type": "commonjs",
@@ -47,25 +47,25 @@
   },
 
   "dependencies": {
-    "check-node-engine": "1.0.0",
+    "check-node-engine": "1.1.0",
     "check-version-modules": "2.5.0"
   },
   "devDependencies": {
     "@types/express": "5.0.6",
-    "@types/node": "25.9.1",
+    "@types/node": "25.9.2",
     "@types/socket.io": "3.0.2",
     "@types/ws": "8.18.1",
     "eslint-plugin-personnallinter": "git+ssh://git@github.com/Psychopoulet/eslint-plugin-personnallinter",
     "express": "5.2.1",
     "husky": "9.1.7",
     "mocha": "11.7.6",
-    "node-pluginsmanager-plugin": "7.2.0",
+    "node-pluginsmanager-plugin": "7.3.2",
     "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"
   },