@weborigami/async-tree 0.0.55 → 0.0.56

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@weborigami/async-tree",
-  "version": "0.0.55",
+  "version": "0.0.56",
   "description": "Asynchronous tree drivers based on standard JavaScript classes",
   "type": "module",
   "main": "./main.js",
@@ -11,7 +11,7 @@
     "typescript": "5.4.5"
   },
   "dependencies": {
-    "@weborigami/types": "0.0.55"
+    "@weborigami/types": "0.0.56"
   },
   "scripts": {
     "test": "node --test --test-reporter=spec",

package/src/FileTree.js

@@ -6,6 +6,7 @@ import {
   getRealmObjectPrototype,
   hiddenFileNames,
   isPacked,
+  isPlainObject,
   naturalOrder,
 } from "./utilities.js";
 
@@ -165,6 +166,9 @@ export default class FileTree {
       await fs.mkdir(this.dirname, { recursive: true });
       // Write out the value as the contents of a file.
       await fs.writeFile(destPath, value);
+    } else if (isPlainObject(value) && Object.keys(value).length === 0) {
+      // Special case: empty object means create an empty directory.
+      await fs.mkdir(destPath, { recursive: true });
     } else if (Tree.isTreelike(value)) {
       // Treat value as a tree and write it out as a subdirectory.
       const destTree = Reflect.construct(this.constructor, [destPath]);

package/src/operations/cache.js

@@ -1,8 +1,10 @@
 import { DeepObjectTree, Tree } from "../internal.js";
 
 /**
- * Caches values from a source tree in a second cache tree. If no second tree is
- * supplied, an in-memory cache is used.
+ * Caches values from a source tree in a second cache tree. Cache source tree
+ * keys in memory.
+ *
+ * If no second tree is supplied, an in-memory value cache is used.
  *
  * An optional third filter tree can be supplied. If a filter tree is supplied,
  * only values for keys that match the filter will be cached.
@@ -22,6 +24,7 @@ export default function treeCache(sourceTreelike, cacheTree, filterTreelike) {
 
   /** @type {AsyncMutableTree} */
   const cache = cacheTree ?? new DeepObjectTree({});
+  let keys;
   return {
     description: "cache",
 
@@ -65,14 +68,7 @@ export default function treeCache(sourceTreelike, cacheTree, filterTreelike) {
     },
 
     async keys() {
-      const keys = new Set(await source.keys());
-
-      // We also add the cache's keys in case the keys provided by the source
-      // tree have changed since the cache was updated.
-      for (const key of await cache.keys()) {
-        keys.add(key);
-      }
-
+      keys ??= source.keys();
       return keys;
     },
   };

package/test/FileTree.test.js

@@ -9,7 +9,7 @@ import { Tree } from "../src/internal.js";
 const dirname = path.dirname(fileURLToPath(import.meta.url));
 const tempDirectory = path.join(dirname, "fixtures/temp");
 
-describe("FileTree", async () => {
+describe.only("FileTree", async () => {
   test("can get the keys of the tree", async () => {
     const fixture = createFixture("fixtures/markdown");
     assert.deepEqual(Array.from(await fixture.keys()), [
@@ -61,6 +61,23 @@ describe("FileTree", async () => {
     await removeTempDirectory();
   });
 
+  test.only("can create empty subfolder via set()", async () => {
+    await createTempDirectory();
+
+    // Write out new, empty folder called "empty".
+    const tempFiles = new FileTree(tempDirectory);
+    await tempFiles.set("empty", {});
+
+    // Verify folder exists and has no contents.
+    const folderPath = path.join(tempDirectory, "empty");
+    const stats = await fs.stat(folderPath);
+    assert(stats.isDirectory());
+    const files = await fs.readdir(folderPath);
+    assert.deepEqual(files, []);
+
+    await removeTempDirectory();
+  });
+
   test("can write out subfolder via set()", async () => {
     await createTempDirectory();