lume -s improvements

CHANGELOG.md

@@ -11,6 +11,10 @@ project adheres to [Semantic Versioning](http://semver.org/).
 - The `permalink` value can be a relative path (must starts with `./` or `../`), that will be resolved to the directory name.
 - The `url` filter allows urls starting with `~` to reference to source files that will be automatically resolved to the final url.
 
+### Fixed
+- On reload files in `lume --serve`, some ignored files were not correctly filtered.
+- Improved `lume --serve`: only rebuild the site if it's needed.
+
 ## [0.16.3] - 2021-03-21
 ### Fixed
 - SVGO dependency fails SVG plugin [#43]

site.js

@@ -231,19 +231,27 @@ export default class Site {
   async update(files) {
     await this.dispatchEvent({ type: "beforeUpdate", files });
 
+    let rebuildIsNeeded = false;
+
     for (const file of files) {
-      // file inside a _data file or folder
+      // It's an ignored file
+      if (this.source.isIgnored(file)) {
+        continue;
+      }
+
+      // It's inside a _data file or folder
       if (file.includes("/_data/") || file.match(/\/_data.\w+$/)) {
         await this.source.loadFile(file);
+        rebuildIsNeeded = true;
         continue;
       }
 
-      // file path contains /_ or /.
+      // The path contains /_ or /.
       if (file.includes("/_") || file.includes("/.")) {
         continue;
       }
 
-      //Static file
+      // It's a static file
       const entry = this.source.isStatic(file);
 
       if (entry) {
@@ -253,11 +261,15 @@ export default class Site {
         continue;
       }
 
-      //Default
+      // Default
       await this.source.loadFile(file);
+      rebuildIsNeeded = true;
+    }
+
+    if (rebuildIsNeeded) {
+      await this.#buildPages();
     }
 
-    await this.#buildPages();
     await this.dispatchEvent({ type: "afterUpdate", files });
   }
 

source.js

@@ -83,6 +83,19 @@ export default class Source {
     return false;
   }
 
+  /**
+   * Check whether a path is ignored or not
+   */
+  isIgnored(path) {
+    for (const pattern of this.ignored) {
+      if (path.startsWith(pattern)) {
+        return true;
+      }
+    }
+
+    return false;
+  }
+
   /**
    * Load a directory recursively
    */