poops 1.5.2 → 1.5.3

package/lib/markups.js

@@ -551,6 +551,20 @@ export default class Markups {
     this.lastPageEntries = pageEntries
   }
 
+  // Watch hook for .json/.yaml changes. Data extensions are ambiguous: a
+  // .json may be a data file feeding the globals, or engine-owned markup
+  // (Shopify templates/*.json, section-group JSON). Engines that can tell
+  // claim theirs via isMarkupSource() and get the incremental path;
+  // everything else reloads data globals and full-compiles, as before.
+  async compileDataChange(file) {
+    await this.init()
+    if (this.engine && typeof this.engine.isMarkupSource === 'function' &&
+      this.engine.isMarkupSource(path.resolve(process.cwd(), file))) {
+      return this.compileIncremental(file)
+    }
+    return this.reloadDataFiles().then(() => this.compile())
+  }
+
   // Watch-mode entry point: rebuild only the pages whose last render touched
   // the changed file. Falls back to a full compile() whenever the result
   // isn't provably identical to a full build — unsupported engine, no prior
@@ -593,6 +607,14 @@ export default class Markups {
       if (this.isCollectionIndexOverride(rel.split(path.sep), collectionData)) return this.compile()
     }
 
+    // A page's own dep set always contains itself, so an edited page is in
+    // `affected` (exact match, no glob needed). If it isn't, but the file IS a
+    // page source, this is a new page whose basename collides with a recorded
+    // dep — rendering only the collided pages would leave it unbuilt.
+    if (!affected.includes(abs) && this.getMarkupFiles(markupIn).some((f) => path.resolve(f) === abs)) {
+      return this.compile()
+    }
+
     const markupStart = performance.now()
     const newEntries = this.lastPageEntries ? [] : null
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "poops",
   "description": "Straightforward, no-bullshit bundler for the web.",
-  "version": "1.5.2",
+  "version": "1.5.3",
   "license": "MIT",
   "type": "module",
   "main": "poops.js",

package/poops.js

@@ -156,7 +156,9 @@ function setupWatchers(config, modules) {
     }
 
     if (/(\.json|\.ya?ml)$/i.test(file)) {
-      modules.markups.reloadDataFiles().then(() => modules.markups.compile()).then(() => reload()).catch(err => console.error(err))
+      // Engine-owned markup with a data extension (Shopify templates/*.json)
+      // goes incremental; real data files reload globals + full compile.
+      modules.markups.compileDataChange(file).then(() => reload()).catch(err => console.error(err))
     }
   }