allow to run site.build() multiple times in the same process

Author: oscarotero

CHANGELOG.md

@@ -14,6 +14,7 @@ Go to the `v1` branch to see the changelog of Lume 1.
 - `site.initDebugBar()` to initialize manually the `debugBar`.
 
 ### Fixed
+- Issues running `site.build()` multiple times in the same process.
 - Updated dependencies: `bar`, `unocss`, `tailwindcss`, `esbuild`, `std`, `deno/loader` and some plugins.
 
 ## [3.0.6] - 2025-08-07

core/fs.ts

@@ -53,15 +53,16 @@ export default class FS {
   options: Options;
   entries = new Map<string, Entry>();
   remoteFiles = new Map<string, string>();
-  tree: Entry;
+  tree?: Entry;
 
   constructor(options: Options) {
     this.options = options;
-    this.tree = new Entry("", "/", "directory", options.root);
-    this.entries.set("/", this.tree);
   }
 
   init() {
+    this.tree = new Entry("", "/", "directory", this.options.root);
+    this.entries.clear();
+    this.entries.set("/", this.tree);
     this.#walkFs(this.tree);
     this.#walkRemote();
   }
@@ -196,7 +197,10 @@ export default class FS {
 
   addEntry(data: { path: string; type?: EntryType; src?: string }): Entry {
     const pieces = data.path.split("/").filter((p) => p);
-    let parent = this.tree;
+    let parent = this.tree as Entry;
+    if (!parent) {
+      throw new Error("FS not initialized");
+    }
 
     if (!data.src) {
       data.src = posix.join(this.options.root, data.path);

tests/core/scopes.test.ts

@@ -1,6 +1,6 @@
 import { assertStrictEquals as equals } from "../../deps/assert.ts";
 import Scopes from "../../core/scopes.ts";
-import FS from "../../core/fs.ts";
+import FS, { Entry } from "../../core/fs.ts";
 
 Deno.test("Scripts", async (t) => {
   const scopes = new Scopes();
@@ -11,6 +11,7 @@ Deno.test("Scripts", async (t) => {
     root: "/",
   });
 
+  fs.tree = new Entry("", "/", "directory", "/");
   fs.addEntry({ path: "/file1.foo", type: "file" });
   fs.addEntry({ path: "/file2.bar", type: "file" });
   fs.addEntry({ path: "/file3.css", type: "file" });