gscan 4.37.6 → 4.38.0

package/lib/checker.js

@@ -1,4 +1,3 @@
-const Promise = require('bluebird');
 const _ = require('lodash');
 const requireDir = require('require-dir');
 const debug = require('@tryghost/debug')('checker');
@@ -50,12 +49,12 @@ const check = function checkAll(themePath, options = {}) {
             // set the major version to check
             theme.checkedVersion = versions[version].major;
 
-            return Promise.reduce(_.values(checks), async function (themeToCheck, checkFunction) {
+            return Promise.all(_.values(checks).map(async (checkFunction) => {
                 const now = Date.now();
-                const result = await checkFunction(themeToCheck, options, themePath);
+                const result = await checkFunction(theme, options, themePath);
                 debug(checkFunction.name, 'took', Date.now() - now, 'ms');
                 return result;
-            }, theme);
+            })).then(() => theme);
         })
         .catch((error) => {
             throw new errors.ValidationError({

package/lib/read-theme.js

@@ -1,5 +1,4 @@
 const fs = require('fs-extra');
-const Promise = require('bluebird');
 const _ = require('lodash');
 const os = require('os');
 const path = require('path');
@@ -39,35 +38,43 @@ const readThemeStructure = function readThemeFiles(themePath, subPath, arr) {
     };
 
     return fs.readdir(themePath, {withFileTypes: true}).then(function (files) {
-        return Promise.reduce(files, function (result, dirent) {
-            const file = dirent.name;
-            var extMatch = file.match(/.*?(\.[0-9a-z]+$)/i),
-                subFilePath = path.join(subPath, file),
-                newPath = path.join(themePath, file);
-
-            /**
-             * don't process ignored paths, remove target file
-             *
-             * @TODO:
-             * - gscan extracts the target zip into a tmp directory
-             * - if you use gscan with `keepExtractedDir` the caller (Ghost) will use the tmp folder with the deleted ignore files
-             * - what we don't support right now is to delete the ignore files from the zip
-             */
-            if (ignore.indexOf(file) > -1) {
-                return inTmp
-                    ? fs.remove(newPath)
-                        .then(function () {
-                            return result;
-                        })
-                    : result;
-            }
+        let result = arr;
+
+        return files.reduce(function (promise, dirent) {
+            return promise.then(function () {
+                const file = dirent.name;
+                const extMatch = file.match(/.*?(\.[0-9a-z]+$)/i);
+                const subFilePath = path.join(subPath, file);
+                const newPath = path.join(themePath, file);
+
+                /**
+                 * don't process ignored paths, remove target file
+                 *
+                 * @TODO:
+                 * - gscan extracts the target zip into a tmp directory
+                 * - if you use gscan with `keepExtractedDir` the caller (Ghost) will use the tmp folder with the deleted ignore files
+                 * - what we don't support right now is to delete the ignore files from the zip
+                 */
+                if (ignore.indexOf(file) > -1) {
+                    return inTmp
+                        ? fs.remove(newPath)
+                            .then(function () {
+                                return result;
+                            })
+                        : Promise.resolve(result);
+                }
 
-            if (dirent.isDirectory()) {
-                return readThemeStructure(newPath, subFilePath, result);
-            } else {
-                return makeResult(result, subFilePath, extMatch !== null ? extMatch[1] : undefined, dirent.isSymbolicLink());
-            }
-        }, arr);
+                if (dirent.isDirectory()) {
+                    return readThemeStructure(newPath, subFilePath, result)
+                        .then((updatedResult) => {
+                            result = updatedResult;
+                        });
+                } else {
+                    result = makeResult(result, subFilePath, extMatch !== null ? extMatch[1] : undefined, dirent.isSymbolicLink());
+                    return Promise.resolve();
+                }
+            });
+        }, Promise.resolve()).then(() => result);
     });
 };
 
@@ -89,9 +96,8 @@ const readFiles = function readFiles(theme) {
     // Setup the helper object
     theme.helpers = {};
 
-    // CASE: we need the actual content of all css, hbs files, and package.json for
-    // our checks
-    return Promise.map(themeFilesContent, function (themeFile) {
+    // CASE: we need the actual content of all css, hbs files, and package.json for our checks
+    return Promise.all(themeFilesContent.map((themeFile) => {
         return fs.readFile(path.join(theme.path, themeFile.file), 'utf8').then(function (content) {
             themeFile.content = content;
 
@@ -119,13 +125,10 @@ const readFiles = function readFiles(theme) {
             const handlebarsMatch = themeFile.file.match(/\.hbs$/);
             if (handlebarsMatch) {
                 themeFile.parsed = ASTLinter.parse(themeFile.content, themeFile.file);
-
                 processHelpers(theme, themeFile);
             }
         });
-    }).then(function () {
-        return theme;
-    });
+    })).then(() => theme);
 };
 
 const processHelpers = function (theme, themeFile) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gscan",
-  "version": "4.37.6",
+  "version": "4.38.0",
   "description": "Scans Ghost themes looking for errors, deprecations, features and compatibility",
   "keywords": [
     "ghost",
@@ -40,15 +40,14 @@
     "gscan": "./bin/cli.js"
   },
   "dependencies": {
-    "@sentry/node": "7.64.0",
+    "@sentry/node": "7.66.0",
     "@tryghost/config": "0.2.18",
-    "@tryghost/debug": "0.1.25",
+    "@tryghost/debug": "0.1.26",
     "@tryghost/errors": "1.2.26",
     "@tryghost/logging": "2.4.6",
     "@tryghost/pretty-cli": "1.2.38",
     "@tryghost/server": "0.1.36",
     "@tryghost/zip": "1.1.37",
-    "bluebird": "3.7.2",
     "chalk": "4.1.2",
     "common-tags": "1.8.2",
     "express": "4.18.2",