@epublishing/grunt-epublishing 1.2.6 → 1.2.8

package/lib/cli.js

@@ -60,10 +60,12 @@ function printOptions(options) {
 /**
  * Run npm install in jade and jade child gem directories.
  * Installs packages from each gem's package.json (excluding site).
+ * Skips if package-lock.json hasn't changed since the last install.
  * @param {Object} config - Build configuration with paths
  * @param {Object} options - Build options
  */
 async function runNpmInstall(config, options) {
+  const crypto = require('crypto');
   const Resolver = require('@epublishing/jade-resolver');
   const paths = config.paths || {};
   // Build paths object with ONLY jade and jade child gem directories (absolute paths)
@@ -89,10 +91,25 @@ async function runNpmInstall(config, options) {
 
   for (const dir of dirs) {
     const basename = path.basename(dir);
+    const lockFile = path.join(dir, 'package-lock.json');
+    const hasLock = fs.existsSync(lockFile);
+    const stampFile = path.join(dir, 'node_modules', '.package-lock.installed');
+
+    // Skip reinstall if package-lock.json hasn't changed since last npm ci
+    if (hasLock && fs.existsSync(stampFile)) {
+      const currentHash = crypto.createHash('sha1').update(fs.readFileSync(lockFile)).digest('hex');
+      const stampHash = fs.readFileSync(stampFile, 'utf8').trim();
+      if (currentHash === stampHash) {
+        if (options.verbose) {
+          console.log(chalk.gray(`  Skipping npm install in ${basename} (package-lock.json unchanged)`));
+        }
+        continue;
+      }
+    }
+
     const spinner = ora(`Installing npm modules in ${basename}...`).start();
 
     try {
-      const hasLock = fs.existsSync(path.join(dir, 'package-lock.json'));
       const cmd = hasLock ? 'ci' : 'install';
       await new Promise((resolve, reject) => {
         const proc = spawn('npm', [cmd], {
@@ -114,6 +131,16 @@ async function runNpmInstall(config, options) {
           reject(err);
         });
       });
+
+      // Write stamp so we can skip next time if package-lock.json is unchanged
+      if (hasLock) {
+        try {
+          const hash = crypto.createHash('sha1').update(fs.readFileSync(lockFile)).digest('hex');
+          fs.writeFileSync(stampFile, hash + '\n');
+        } catch {
+          // Stamp write failure is non-fatal — we'll just reinstall next time
+        }
+      }
     } catch (err) {
       throw err;
     }

package/lib/webpack.config.js

@@ -102,12 +102,30 @@ function createWebpackConfig(config, options = {}) {
     resolveAlias['jade-engine'] = path.resolve(paths.jade, paths.js_src || 'app/js', 'jade-engine.js');
   }
 
+  // Read jade gem version from lib/jade/version.rb (Ruby is the source of truth)
+  let jadeVersion = 'unknown';
+  if (paths.jade && typeof paths.jade === 'string') {
+    try {
+      const versionRb = fs.readFileSync(
+        path.join(paths.jade, 'lib/jade/version.rb'),
+        'utf8',
+      );
+      const major = versionRb.match(/MAJOR\s*=\s*['"]?(\d+)['"]?/)?.[1];
+      const minor = versionRb.match(/MINOR\s*=\s*['"]?([\w.]+)['"]?/)?.[1];
+      const build = versionRb.match(/BUILD\s*=\s*['"]?([\w.]+)['"]?/)?.[1];
+      if (major && minor && build) jadeVersion = `${major}.${minor}.${build}`;
+    } catch {
+      // Fall through to 'unknown' — never fail the build over a version banner
+    }
+  }
+
   // Build plugins array
   const plugins = [
     // Environment variables
     new webpack.DefinePlugin({
       'process.env.NODE_ENV': JSON.stringify(NODE_ENV),
       'process.env.DEBUG': JSON.stringify(process.env.DEBUG || false),
+      __JADE_VERSION__: JSON.stringify(jadeVersion),
     }),
 
     // Limit moment.js locales to en

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@epublishing/grunt-epublishing",
   "description": "Modern front-end build tools for ePublishing Jade and client sites.",
-  "version": "1.2.6",
+  "version": "1.2.8",
   "homepage": "https://www.epublishing.com",
   "contributors": [
     {