hologit 0.48.1 → 0.49.1

package/lib/BlobObject.js

@@ -3,11 +3,9 @@ class BlobObject {
 
     static async write (repo, content) {
         const git = await repo.getGit();
-        const hashObject = await git.hashObject({ w: true, stdin: true, $spawn: true });
+        const hash = await git.$putBlob(content);
 
-        return new BlobObject(repo, {
-            hash: await hashObject.captureOutputTrimmed(content)
-        });
+        return new BlobObject(repo, { hash });
     }
 
     static async writeFromFile (repo, filePath) {
@@ -38,7 +36,7 @@ class BlobObject {
 
     async read () {
         const git = await this.repo.getGit();
-        return git.catFile({ p: true }, this.hash);
+        return git.$getBlob(this.hash);
     }
 }
 

package/lib/Lens.js

@@ -261,7 +261,7 @@ class Lens extends Configurable {
 
 
         // determine spec type from content
-        const specToml = await git.catFile({ p: true }, specHash);
+        const specToml = await git.$getBlob(specHash);
         const { holospec: { lens: spec } } = TOML.parse(specToml);
         const specType = spec.container ? 'container' : spec.package ? 'habitat' : null;
 
@@ -319,7 +319,7 @@ class Lens extends Configurable {
         const git = await repo.getGit();
 
         // read and parse spec file
-        const specToml = await git.catFile({ p: true }, specHash);
+        const specToml = await git.$getBlob(specHash);
         const {
             holospec: {
                 lens: spec
@@ -474,7 +474,7 @@ class Lens extends Configurable {
             }
         } else {
             // load spec
-            const specToml = await git.catFile({ p: true }, specHash);
+            const specToml = await git.$getBlob(specHash);
             const {
                 holospec: {
                     lens: spec

package/lib/Repo.js

@@ -157,7 +157,7 @@ class Repo {
      */
     async hasCommit (commit) {
         const git = await this.getGit();
-        return 'commit' == await git.catFile({ t: true }, commit, { $nullOnError: true });
+        return 'commit' == await git.$objectExists(commit);
     }
 
     async hashWorkTree () {

package/lib/TreeObject.js

@@ -10,12 +10,20 @@ const EMPTY_TREE_HASH = '4b825dc642cb6eb9a060e54bf8d69288fbee4904';
 
 const cache = {};
 
+const stats = { treesWritten: 0, treesSkippedClean: 0, treesSkippedCached: 0, cacheHits: 0, cacheMisses: 0 };
+
 function cacheRead (hash) {
     if (hash == EMPTY_TREE_HASH) {
         return {};
     }
 
-    return cache[hash] || null;
+    const result = cache[hash] || null;
+    if (result) {
+        stats.cacheHits++;
+    } else {
+        stats.cacheMisses++;
+    }
+    return result;
 }
 
 function cacheWrite (hash, children) {
@@ -120,21 +128,20 @@ class TreeObject {
         if (!cachedHashChildren) {
             cachedHashChildren = {};
 
-            const treeLines = (await git.lsTree(preloadChildren ? { r: true, t: true, z: true, 'full-tree': true } : { z: true, 'full-tree': true }, this.hash)).split('\0');
-            const preloadedTrees = {};
-
-            for (const treeLine of treeLines) {
-                if (!treeLine) {
-                    continue;
-                }
+            if (preloadChildren) {
+                // recursive ls-tree: one subprocess call to load the entire tree hierarchy
+                const treeLines = (await git.lsTree({ r: true, t: true, z: true, 'full-tree': true }, this.hash)).split('\0');
+                const preloadedTrees = {};
 
-                const [, mode, type, hash, childPath] = treeLineRe.exec(treeLine);
+                for (const treeLine of treeLines) {
+                    if (!treeLine) {
+                        continue;
+                    }
 
-                if (preloadChildren) {
+                    const [, mode, type, hash, childPath] = treeLineRe.exec(treeLine);
                     const parentTreePathLength = childPath.lastIndexOf('/');
 
                     if (type == 'tree') {
-                        // any tree listed will have children, begin cache entry
                         preloadedTrees[childPath] = {
                             hash,
                             children: {}
@@ -142,25 +149,32 @@ class TreeObject {
                     }
 
                     if (parentTreePathLength == -1) {
-                        // direct child, add to current result
                         cachedHashChildren[childPath] = { type, hash, mode };
                     } else {
                         preloadedTrees[childPath.substr(0, parentTreePathLength)]
                             .children[childPath.substr(parentTreePathLength+1)] = { type, hash, mode };
                     }
-                } else {
-                    cachedHashChildren[childPath] = { type, hash, mode };
                 }
-            }
 
-            cacheWrite(this.hash, cachedHashChildren);
+                // populate git-client's known-objects cache
+                for (const name in cachedHashChildren) {
+                    git.$knowObject(cachedHashChildren[name].hash);
+                }
 
-            if (preloadChildren) {
                 for (const treePath in preloadedTrees) {
                     const tree = preloadedTrees[treePath];
                     cacheWrite(tree.hash, tree.children);
+                    git.$knowObject(tree.hash);
+                }
+            } else {
+                // non-recursive: read single tree level via cat-file --batch
+                const entries = await git.$getTree(this.hash);
+                for (const { mode, type, hash, name } of entries) {
+                    cachedHashChildren[name] = { type, hash, mode };
                 }
             }
+
+            cacheWrite(this.hash, cachedHashChildren);
         }
 
 
@@ -368,7 +382,8 @@ class TreeObject {
             this.hash = EMPTY_TREE_HASH;
         } else {
             const git = await this.repo.getGit();
-            this.hash = await git.mktreeBatch(lines);
+            this.hash = await git.$putTree(lines);
+            stats.treesWritten++;
         }
 
 
@@ -546,6 +561,8 @@ class TreeObject {
 }
 
 TreeObject.treeLineRe = treeLineRe;
+TreeObject.stats = stats;
+TreeObject.resetStats = () => Object.keys(stats).forEach(k => stats[k] = 0);
 
 TreeObject.prototype.isTree = true;
 TreeObject.prototype.type = 'tree';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hologit",
-  "version": "0.48.1",
+  "version": "0.49.1",
   "description": "Hologit automates the projection of layered composite file trees based on flat, declarative plans",
   "repository": "https://github.com/JarvusInnovations/hologit",
   "main": "lib/index.js",
@@ -11,18 +11,18 @@
   "dependencies": {
     "@iarna/toml": "^2.2.5",
     "async-exit-hook": "^2.0.1",
-    "axios": "^1.13.5",
+    "axios": "^1.13.6",
     "chokidar": "^5.0.0",
     "debounce": "^3.0.0",
     "fb-watchman": "^2.0.2",
-    "git-client": "^1.9.4",
+    "git-client": "^1.11.0",
     "hab-client": "^1.1.3",
     "handlebars": "^4.7.8",
-    "minimatch": "^10.2.2",
+    "minimatch": "^10.2.4",
     "mz": "^2.7.0",
     "mz-modules": "^2.1.0",
     "object-squish": "^1.1.0",
-    "parse-url": "^10.0.3",
+    "parse-url": "^11.1.0",
     "shell-quote-word": "^1.0.1",
     "sort-keys": "^6.0.0",
     "toposort": "^2.0.2",
@@ -49,9 +49,10 @@
   ],
   "devDependencies": {
     "@types/jest": "^30.0.0",
-    "jest": "^30.2.0"
+    "jest": "^30.3.0"
   },
   "scripts": {
-    "test": "jest"
+    "test": "jest",
+    "bench": "node test/benchmark/projection.bench.js"
   }
 }