mikser-io-layouts 2.4.0 → 2.4.1

package/index.js

@@ -110,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
@@ -410,12 +410,12 @@ export function layouts(userOptions = {}) {
                 if (!skipWrite) {
                     const destinationFile = path.join(writeBase, entity.destination)
                     // writeOutput skips the write when the bytes already on
-                    // disk are identical. Invalidation is deliberately
-                    // conservative — an entity that merely READ another
-                    // re-renders, because the engine cannot know which field
-                    // was read — so byte-identical output is routine, and
-                    // rewriting it moved mtime for live-reload, rsync and
-                    // `find -newer` alike. mkdir/unlink moved inside it.
+                    // 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, ''))

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.4.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",