@saltcorn/plugins-loader 1.1.0-beta.12 → 1.1.0-beta.13

package/download_utils.js

@@ -69,13 +69,15 @@ const loadTarball = (rootFolder, url, name) => {
           } else
             reject(
               new Error(
-                `Error downloading tarball: http code ${redirect.statusCode}`
+                `Error downloading tarball from ${url}: http code ${redirect.statusCode}`
               )
             );
         });
       } else if (res.statusCode !== 200)
         reject(
-          new Error(`Error downloading tarball: http code ${res.statusCode}`)
+          new Error(
+            `Error downloading tarball from ${url}: http code ${res.statusCode}`
+          )
         );
       else {
         const filePath = await writeTarball(res);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saltcorn/plugins-loader",
-  "version": "1.1.0-beta.12",
+  "version": "1.1.0-beta.13",
   "description": "Saltcorn plugin loader",
   "homepage": "https://saltcorn.com",
   "scripts": {
@@ -11,7 +11,7 @@
   "dependencies": {
     "env-paths": "^2.2.0",
     "npm-registry-fetch": "17.1.0",
-    "@saltcorn/data": "1.1.0-beta.12"
+    "@saltcorn/data": "1.1.0-beta.13"
   },
   "author": "Christian Hugo",
   "license": "MIT",

package/plugin_installer.js

@@ -111,10 +111,10 @@ class PluginInstaller {
         await this.movePlugin(wasInstalled);
         if (await tarballExists(this.rootFolder, this.plugin))
           await removeTarball(this.rootFolder, this.plugin);
+        pckJSON = await readPackageJson(this.pckJsonPath);
+        const msg = this.checkEngineWarning(pckJSON);
+        if (msg && !msgs.includes(msg)) msgs.push(msg);
       }
-      pckJSON = await readPackageJson(this.pckJsonPath);
-      const msg = this.checkEngineWarning(pckJSON);
-      if (msg && !msgs.includes(msg)) msgs.push(msg);
     };
     await installer();
     let module = null;
@@ -293,7 +293,13 @@ class PluginInstaller {
           });
         }
         child.on("exit", (exitCode, signal) => {
-          resolve({ success: exitCode === 0 });
+          if (exitCode !== 0) {
+            reject(
+              new Error(
+                `NPM install failed for ${pckJSON.name} with exit code ${exitCode}`
+              )
+            );
+          } else resolve();
         });
         child.on("error", (msg) => {
           reject(msg);