node-pluginsmanager 3.8.0 → 3.8.2

package/lib/cjs/PluginsManager.d.ts

@@ -54,6 +54,9 @@ export default class PluginsManager extends EventEmitter<{
     installViaGithub(user: string, repo: string, ...data: unknown[]): Promise<Orchestrator>;
     private _isExternalPlugin;
     private _updateWithoutGit;
+    private _rejectReleasedUpdate;
+    private _splicePlugin;
+    private _destroyReleasedPlugin;
     updateViaGithub(plugin: Orchestrator, ...data: unknown[]): Promise<Orchestrator>;
     uninstall(plugin: Orchestrator, ...data: unknown[]): Promise<string>;
 }

package/lib/cjs/PluginsManager.js

@@ -15,6 +15,7 @@ const check_node_engine_1 = __importDefault(require("check-node-engine"));
 const semver_1 = __importDefault(require("semver"));
 // locals
 const rmdirp_1 = __importDefault(require("./utils/rmdirp"));
+const clearRequireCache_1 = __importDefault(require("./utils/clearRequireCache"));
 const checkAbsoluteDirectory_1 = __importDefault(require("./checkers/checkAbsoluteDirectory"));
 const checkFunction_1 = __importDefault(require("./checkers/checkFunction"));
 const checkNonEmptyArray_1 = __importDefault(require("./checkers/checkNonEmptyArray"));
@@ -38,6 +39,18 @@ const npmUpdate_1 = __importDefault(require("./cmd/npm/npmUpdate"));
 // consts
 const DEFAULT_PLUGINS_DIRECTORY = (0, node_path_1.join)((0, node_os_1.homedir)(), "node-pluginsmanager-plugins");
 const DEFAULT_RESOURCES_DIRECTORY = (0, node_path_1.join)((0, node_os_1.homedir)(), "node-pluginsmanager-resources");
+// private
+// methods
+function _removeReleasedDirectory(directory) {
+    return (0, rmdirp_1.default)(directory).catch(() => {
+        return (0, promises_1.rm)(directory, {
+            "recursive": true,
+            "force": true
+        });
+    }).catch(() => {
+        return Promise.resolve();
+    });
+}
 // module
 class PluginsManager extends node_events_1.default {
     // attributes
@@ -491,6 +504,30 @@ class PluginsManager extends node_events_1.default {
             return (0, gitInstall_1.default)(directory, user, repo);
         });
     }
+    _rejectReleasedUpdate(pluginName, directory, err) {
+        if (this.getPluginsNames().includes(pluginName)) {
+            return Promise.reject(err);
+        }
+        (0, clearRequireCache_1.default)(directory);
+        return (0, rmdirp_1.default)((0, node_path_1.join)(this.externalResourcesDirectory, pluginName)).catch(() => {
+            return Promise.resolve();
+        }).then(() => {
+            return _removeReleasedDirectory(directory);
+        }).then(() => {
+            return Promise.reject(err);
+        });
+    }
+    _splicePlugin(key) {
+        if (-1 < key) {
+            this.plugins.splice(key, 1);
+        }
+    }
+    _destroyReleasedPlugin(plugin, key, ...data) {
+        return plugin.destroy(...data).catch((err) => {
+            this._splicePlugin(key);
+            return Promise.reject(err);
+        });
+    }
     // update a plugin via its github repo, using "data" in arguments for "release", "update" and "init" plugin's methods
     updateViaGithub(plugin, ...data) {
         let directory = "";
@@ -543,12 +580,10 @@ class PluginsManager extends node_events_1.default {
                 const key = this.plugins.indexOf(plugin);
                 return plugin.release(...data).then(() => {
                     this.emit("released", plugin, ...data);
-                    return plugin.destroy(...data);
+                    return this._destroyReleasedPlugin(plugin, key, ...data);
                 }).then(() => {
                     this.emit("destroyed", pluginName, ...data);
-                    if (-1 < key) {
-                        this.plugins.splice(key, 1);
-                    }
+                    this._splicePlugin(key);
                     return latestTag;
                 });
                 // update plugin
@@ -590,6 +625,8 @@ class PluginsManager extends node_events_1.default {
                     this.plugins.push(_plugin);
                     return _plugin;
                 });
+            }).catch((err) => {
+                return this._rejectReleasedUpdate(pluginName, directory, err);
             });
         });
     }

package/lib/cjs/cmd/npm/npmInstall.js

@@ -5,12 +5,45 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = npmInstall;
+// natives
+const promises_1 = require("node:fs/promises");
+const node_path_1 = require("node:path");
 // locals
 const cmd_1 = __importDefault(require("../cmd"));
+// private
+// methods
+function _hasInstallScripts(packageData) {
+    if ("object" !== typeof packageData.scripts || null === packageData.scripts) {
+        return false;
+    }
+    const scripts = packageData.scripts;
+    const installDev = scripts["install-dev"];
+    const installProd = scripts["install-prod"];
+    if ("string" !== typeof installDev || "string" !== typeof installProd) {
+        return false;
+    }
+    return "" !== installDev.trim() && "" !== installProd.trim();
+}
 // module
 function npmInstall(directory, isBuildMode = false) {
-    return (0, cmd_1.default)(directory, "npm", [
-        "install",
-        ...(isBuildMode ? ["--no-optional"] : ["--omit=dev", "--no-optional"])
-    ]);
+    return (0, promises_1.readFile)((0, node_path_1.join)(directory, "package.json"), "utf-8").then((content) => {
+        const packageData = JSON.parse(content);
+        if (_hasInstallScripts(packageData)) {
+            return (0, cmd_1.default)(directory, "npm", [
+                "run",
+                isBuildMode ? "install-dev" : "install-prod"
+            ]);
+        }
+        const flags = isBuildMode ? ["--omit=optional"] : ["--omit=dev", "--omit=optional"];
+        return (0, cmd_1.default)(directory, "npm", [
+            "install",
+            ...flags
+        ]).then(() => {
+            return (0, cmd_1.default)(directory, "npm", [
+                "audit",
+                "fix",
+                ...flags
+            ]);
+        });
+    });
 }

package/lib/cjs/cmd/npm/npmUpdate.js

@@ -12,6 +12,6 @@ function npmUpdate(directory) {
     return (0, cmd_1.default)(directory, "npm", [
         "update",
         "--omit=dev",
-        "--no-optional"
+        "--omit=optional"
     ]);
 }

package/lib/cjs/createPluginByDirectory.js

@@ -15,6 +15,7 @@ const node_path_1 = require("node:path");
 const checkAbsoluteDirectory_1 = __importDefault(require("./checkers/checkAbsoluteDirectory"));
 const checkOrchestrator_1 = __importDefault(require("./checkers/checkOrchestrator"));
 const checkFunction_1 = __importDefault(require("./checkers/checkFunction"));
+const clearRequireCache_1 = __importDefault(require("./utils/clearRequireCache"));
 // module
 function createPluginByDirectory(directory, externalResourcesDirectory, logger, ...data) {
     return (0, checkAbsoluteDirectory_1.default)("createPluginByDirectory/directory", directory).then(() => {
@@ -22,6 +23,7 @@ function createPluginByDirectory(directory, externalResourcesDirectory, logger,
     }).then(() => {
         return new Promise((resolve, reject) => {
             try {
+                (0, clearRequireCache_1.default)(directory);
                 resolve(require(directory));
             }
             catch (e) {

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

@@ -0,0 +1 @@
+export default function clearRequireCache(directory: string): void;

package/lib/cjs/utils/clearRequireCache.js

@@ -0,0 +1,23 @@
+"use strict";
+// deps
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = clearRequireCache;
+// natives
+const node_path_1 = require("node:path");
+// private
+// methods
+function _isUnderDirectory(directory, file) {
+    const fromDirectory = (0, node_path_1.relative)(directory, file);
+    if ("" === fromDirectory) {
+        return true;
+    }
+    return !fromDirectory.startsWith("..") && !(0, node_path_1.isAbsolute)(fromDirectory);
+}
+// module
+function clearRequireCache(directory) {
+    for (const file of Object.keys(require.cache)) {
+        if (_isUnderDirectory(directory, file)) {
+            Reflect.deleteProperty(require.cache, file);
+        }
+    }
+}

package/package.json

@@ -1,7 +1,7 @@
 {
 
   "name": "node-pluginsmanager",
-  "version": "3.8.0",
+  "version": "3.8.2",
   "description": "A plugins manager.",
 
   "type": "commonjs",
@@ -19,6 +19,9 @@
 
   "scripts": {
 
+    "install-prod": "npm install --omit=dev --omit=optional && npm audit fix --omit=dev --omit=optional",
+    "install-dev": "npm install --allow-git=root && npm audit fix --allow-git=root && npx husky",
+
     "lint-back": "npx eslint --config .eslintrc-back.js --ext .ts,.cts,.mts ./lib/src/**/*",
     "lint-tests": "npx eslint --config .eslintrc-tests.js --ext .js,.cjs,.mjs ./test/**/*.js",
     "lint": "npm run lint-back && npm run lint-tests",
@@ -32,10 +35,10 @@
     "check-node-engine": "npx check-node-engine",
     "check-requires": "npx used-deps-analyzer ./package.json ./lib/src --no-dev",
     "check-updates": "npx check-version-modules --no-fail-at-major --fail-at-minor --fail-at-patch",
-    "unit-tests": "npx mocha",
-    "unit-tests-local": "npx nyc --reporter=html --reporter=text mocha",
+    "unit-tests": "npx c8 --reporter=text-summary --all --include \"lib/src/**/*.ts\" --include \"lib/src/**/*.cts\" --extension .ts --extension .cts --check-coverage --statements 95 --branches 95 --functions 95 --lines 95 mocha --require ./test/register.js",
+    "unit-tests-local": "npx c8 --reporter=html --reporter=text --all --include \"lib/src/**/*.ts\" --include \"lib/src/**/*.cts\" --extension .ts --extension .cts mocha --require ./test/register.js",
 
-    "tests": "npm run lint && npm run check-node-engine && npm run check-requires && npm run check-updates && npm run build && npm run unit-tests-local"
+    "tests": "npm run lint && npm run check-node-engine && npm run check-requires && npm run check-updates && npm run build && npm run unit-tests"
 
   },
 
@@ -43,32 +46,36 @@
     "/lib/cjs"
   ],
   "engines": {
-    "node": ">=22.23.2"
+    "node": ">=24.22.2"
   },
 
   "dependencies": {
-    "check-node-engine": "1.2.0",
-    "check-version-modules": "2.5.0",
-    "semver": "7.8.5"
+    "check-node-engine": "1.2.1",
+    "check-version-modules": "2.5.2",
+    "semver": "7.8.6"
   },
   "devDependencies": {
     "@types/express": "5.0.6",
-    "@types/node": "26.4.1",
+    "@types/node": "26.6.6",
     "@types/semver": "7.8.0",
     "@types/socket.io": "3.0.2",
-    "@types/ws": "8.18.1",
+    "@types/ws": "8.18.2",
+    "c8": "12.0.0",
     "eslint-plugin-personnallinter": "git+ssh://git@github.com/Psychopoulet/eslint-plugin-personnallinter",
-    "express": "5.2.1",
+    "express": "5.3.0",
     "husky": "9.1.7",
-    "mocha": "12.0.0",
-    "node-pluginsmanager-plugin": "7.4.4",
-    "nyc": "18.0.0",
+    "mocha": "12.0.3",
+    "node-pluginsmanager-plugin": "7.4.5",
     "proxyquire": "2.1.3",
     "rimraf": "6.1.3",
-    "socket.io": "4.8.3",
-    "typescript": "5.9.3",
-    "used-deps-analyzer": "0.4.0",
-    "ws": "8.21.3"
+    "socket.io": "4.8.4",
+    "typescript": "6.0.3",
+    "used-deps-analyzer": "0.4.1",
+    "ws": "8.22.1"
+  },
+  "overrides": {
+    "glob": "14.0.3",
+    "minimatch": "11.0.1"
   },
 
   "keywords": [