mikser-io-layouts 2.3.0 → 2.4.1

package/index.js

@@ -21,6 +21,7 @@ import {
     checksumsByCollection,
     checksum as fileChecksum, checksumOf,
     useDatabase,
+    writeOutput, reportUnchanged,
 } from 'mikser-io'
 
 import { createInspect } from './lib/inspect.js'
@@ -109,9 +110,9 @@ export function layouts(userOptions = {}) {
             const { relativePath } = context
 
             // A .js file under the layouts folder is a SIDECAR (or something
-            // a sidecar imports), never a layout. It used to have its `.js`
-            // stripped and be created as an entity, which produced a phantom
-            // `/layouts/page` — holding JS source as its content — beside the
+            // a sidecar imports), never a layout. Treating one as a layout —
+            // stripping its `.js` and creating an entity — yields a phantom
+            // `/layouts/page` holding JS source as its content, beside the
             // real `/layouts/page.liquid`.
             //
             // Sidecars are inputs to a layout's checksum instead (see
@@ -408,12 +409,20 @@ export function layouts(userOptions = {}) {
 
                 if (!skipWrite) {
                     const destinationFile = path.join(writeBase, entity.destination)
-                    await mkdir(path.dirname(destinationFile), { recursive: true })
-                    try {
-                        await unlink(destinationFile)
-                    } catch { }
-                    await writeFile(destinationFile, output.result)
-                    logger.debug('Layout render finished: %s', entity.destination.replace(runtime.options.workingFolder, ''))
+                    // writeOutput skips the write when the bytes already on
+                    // disk are identical, and owns the mkdir/unlink.
+                    // Invalidation is deliberately conservative — an entity
+                    // that merely READS another re-renders, since the engine
+                    // cannot know which field was read — so byte-identical
+                    // output is routine, and rewriting it moves mtime for
+                    // live-reload, rsync and `find -newer` alike.
+                    const wrote = await writeOutput(destinationFile, output.result)
+                    if (wrote) {
+                        logger.debug('Layout render finished: %s', entity.destination.replace(runtime.options.workingFolder, ''))
+                    } else {
+                        reportUnchanged(entity)
+                        logger.debug('Layout render finished, output unchanged: %s', entity.destination.replace(runtime.options.workingFolder, ''))
+                    }
                 } else {
                     logger.debug('Layout render finished (save:false, bytes only): %s', entity.id)
                 }

package/lib/matching.js

@@ -221,14 +221,14 @@ export function createOnProcessed({
                     // pattern-match / auto-layout paths.
                     // Persist the primary destination.
                     //
-                    // entity.destination used to exist ONLY as a field the
-                    // render-task builder set per task, and was never written
-                    // back — so the catalog had none, and runtime.href()
-                    // (which needs the TARGET's destination to make a
-                    // relative URL) fell through and returned the whole
-                    // entity instead of { url }. The catalog is only written
+                    // The primary destination has to be persisted, not just
+                    // set per render task. runtime.href() needs the TARGET's
+                    // destination to build a relative URL, and it reads the
+                    // catalog: a destination that only ever exists on a
+                    // task makes href() fall through and return the whole
+                    // entity instead of { url }. The catalog is written only
                     // through the journal, and this hook is where that
-                    // happens, so it has to be derived here.
+                    // happens, so it is derived here.
                     //
                     // "Primary" means page 1 / the sole output of the FIRST
                     // matched layout — the same choice entity.layout already

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mikser-io-layouts",
-  "version": "2.3.0",
+  "version": "2.4.1",
   "description": "SSG-flavor task-production policy for mikser-io. Multi-match layout assignment, per-layout destinations, postprocess chain attachment, layout inspection.",
   "main": "index.js",
   "type": "module",
@@ -18,7 +18,7 @@
   },
   "homepage": "https://github.com/almero-digital-marketing/mikser-io-layouts#readme",
   "peerDependencies": {
-    "mikser-io": "^9.14.0"
+    "mikser-io": "^9.18.0"
   },
   "dependencies": {
     "globby": "^14.0.0",