gscan 5.4.1 → 5.4.3

package/README.md

@@ -96,9 +96,12 @@ gscan.checkZip({
 
 ### Publish
 
-(Core team only)
+Ghost core team only
 
-- `yarn ship`
+1. run `yarn ship` - this bumps the version, commits, tags and pushes to `main`
+2. npm publishing is handled by GitHub Actions in `.github/workflows/publish.yml`
+
+Manual preview is available via workflow dispatch with `dry-run: true`
 
 ### Tools
 When developing new rules or testing gscan following tools are great to have in the toolbelt:

package/lib/ast-linter/rules/internal/scope.js

@@ -147,7 +147,11 @@ class Frame {
                 throw new Error('fileName must be passed as an option when constructing a template-level Frame');
             }
 
-            const {context, locals} = getTemplateContext(options.fileName);
+            const templateContext = getTemplateContext(options.fileName);
+            if (!templateContext) {
+                throw new Error(`No template context found for template: ${options.fileName}`);
+            }
+            const {context, locals} = templateContext;
             this.context = context;
             this.locals = locals; // eg. {post: {...postContext}}
             return;

package/lib/checker.js

@@ -121,8 +121,8 @@ const checkZip = async function checkZip(path, options) {
         throw error;
     } finally {
         if (!options.keepExtractedDir && zip.origPath) {
-            const fs = require('fs-extra');
-            await fs.remove(zip.origPath);
+            const fs = require('fs/promises');
+            await fs.rm(zip.origPath, {recursive: true, force: true});
         }
     }
 };

package/lib/read-theme.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra');
+const fs = require('fs/promises');
 const _ = require('lodash');
 const os = require('os');
 const path = require('path');
@@ -59,7 +59,7 @@ const readThemeStructure = function readThemeFiles(themePath, subPath, arr) {
                  */
                 if (ignore.indexOf(file) > -1) {
                     return inTmp
-                        ? fs.remove(newPath)
+                        ? fs.rm(newPath, {recursive: true, force: true})
                             .then(function () {
                                 return result;
                             })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gscan",
-  "version": "5.4.1",
+  "version": "5.4.3",
   "description": "Scans Ghost themes looking for errors, deprecations, features and compatibility",
   "keywords": [
     "ghost",
@@ -37,13 +37,13 @@
     "posttest": "yarn lint",
     "preship": "yarn test",
     "ship": "pro-ship",
-    "postship": "npm publish"
+    "ship:ci": "npm publish"
   },
   "bin": {
     "gscan": "./bin/cli.js"
   },
   "dependencies": {
-    "@sentry/node": "10.46.0",
+    "@sentry/node": "10.47.0",
     "@tryghost/config": "2.0.3",
     "@tryghost/debug": "2.0.3",
     "@tryghost/errors": "3.0.3",
@@ -55,9 +55,8 @@
     "chalk": "5.6.2",
     "express": "5.2.1",
     "express-handlebars": "7.1.3",
-    "fs-extra": "11.3.4",
     "glob": "13.0.6",
-    "lodash": "4.17.23",
+    "lodash": "4.18.1",
     "multer": "2.1.1",
     "semver": "7.7.4",
     "validator": "^13.0.0"
@@ -66,18 +65,16 @@
     "@eslint/compat": "2.0.3",
     "@eslint/eslintrc": "3.3.5",
     "@eslint/js": "10.0.1",
-    "@tryghost/pro-ship": "1.0.7",
+    "@tryghost/pro-ship": "1.0.8",
     "@vitest/coverage-v8": "4.1.2",
     "eslint": "10.1.0",
     "eslint-plugin-ghost": "3.5.0",
     "nodemon": "3.1.14",
-    "should": "13.2.3",
-    "sinon": "21.0.3",
     "vitest": "4.1.2"
   },
   "resolutions": {
     "node-loggly-bulk": "4.0.2",
-    "node-loggly-bulk/axios": "1.13.6"
+    "node-loggly-bulk/axios": "1.14.0"
   },
   "files": [
     "lib",