@vltpkg/package-info 1.0.8 → 1.0.9

package/dist/index.js

@@ -1,5 +1,6 @@
 import { error } from '@vltpkg/error-cause';
 import { clone, resolve as gitResolve, revs } from '@vltpkg/git';
+import { logRequest } from '@vltpkg/output';
 import { PackageJson } from '@vltpkg/package-json';
 import { pickManifest } from '@vltpkg/pick-manifest';
 import { Spec } from '@vltpkg/spec';
@@ -10,8 +11,10 @@ import { XDG } from '@vltpkg/xdg';
 import { createHash, randomBytes } from 'node:crypto';
 import { mkdir, readFile, rm, stat, symlink, unlink, writeFile, } from 'node:fs/promises';
 import { basename, dirname, resolve as pathResolve, relative, } from 'node:path';
+import { debuglog } from 'node:util';
 import { create as tarC } from 'tar';
 import { rename } from "./rename.js";
+const debug = debuglog('vlt');
 const xdg = new XDG('vlt');
 export const delimiter = '~';
 // the maximum duration of a manifest cache file
@@ -129,6 +132,25 @@ export class PackageInfoClient {
                 // fallthrough if a remote tarball url present
             }
             case 'registry': {
+                // if the tarball is already on disk, unpack it straight from
+                // the cache file: it never has to be held in the client's
+                // in-memory cache. anything unexpected falls through to the
+                // fetch path, which throws its own error if the body is
+                // genuinely bad.
+                const cached = (await this.getRegistryClient()).cachedBody(r.resolved, { integrity: r.integrity });
+                if (cached) {
+                    try {
+                        await (await this.getTarPool()).unpack(cached.body, target);
+                        logRequest(r.resolved, 'cache');
+                        return r;
+                    }
+                    catch (er) {
+                        // a systematically failing fast path (every entry still
+                        // gzipped, EACCES, ENOSPC...) doubles the IO of the fetch
+                        // path, so leave a trace behind. NODE_DEBUG=vlt to see it.
+                        debug('cached tarball unpack failed: %s: %s', cached.path, er);
+                    }
+                }
                 const fetchTarball = async (useCache) => {
                     const trustIntegrity = this.#trustedIntegrities.get(r.resolved) === r.integrity;
                     const response = await (await this.getRegistryClient()).request(r.resolved, {
@@ -244,7 +266,7 @@ export class PackageInfoClient {
                 const st = await stat(path);
                 if (st.isFile()) {
                     try {
-                        await (await this.getTarPool()).unpack(await this.tarball(spec, options), target);
+                        await (await this.getTarPool()).unpackFile(path, target);
                     }
                     catch (er) {
                         throw this.#resolveError(spec, options, 'tar unpack failed', { cause: er });
@@ -586,7 +608,7 @@ export class PackageInfoClient {
                 const s = spec;
                 return await this.#tmpdir(async (dir) => {
                     try {
-                        await (await this.getTarPool()).unpack(await readFile(path), dir);
+                        await (await this.getTarPool()).unpackFile(path, dir);
                     }
                     catch (er) {
                         throw this.#resolveError(s, options, 'tar unpack failed', { cause: er });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@vltpkg/package-info",
   "description": "Resolve and fetch package metadata and tarballs",
-  "version": "1.0.8",
+  "version": "1.0.9",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/vltpkg/vltpkg.git",
@@ -13,16 +13,17 @@
     "url": "http://vlt.sh"
   },
   "dependencies": {
-    "@vltpkg/error-cause": "1.0.8",
-    "@vltpkg/git": "1.0.8",
-    "@vltpkg/package-json": "1.0.8",
-    "@vltpkg/pick-manifest": "1.0.8",
-    "@vltpkg/registry-client": "1.0.8",
-    "@vltpkg/spec": "1.0.8",
-    "@vltpkg/tar": "1.0.8",
-    "@vltpkg/types": "1.0.8",
-    "@vltpkg/workspaces": "1.0.8",
-    "@vltpkg/xdg": "1.0.8",
+    "@vltpkg/error-cause": "1.0.9",
+    "@vltpkg/git": "1.0.9",
+    "@vltpkg/output": "1.0.9",
+    "@vltpkg/package-json": "1.0.9",
+    "@vltpkg/pick-manifest": "1.0.9",
+    "@vltpkg/registry-client": "1.0.9",
+    "@vltpkg/spec": "1.0.9",
+    "@vltpkg/tar": "1.0.9",
+    "@vltpkg/types": "1.0.9",
+    "@vltpkg/workspaces": "1.0.9",
+    "@vltpkg/xdg": "1.0.9",
     "ssri": "^13.0.0",
     "tar": "^7.5.2"
   },
@@ -31,8 +32,8 @@
     "@types/node": "^22.19.2",
     "@types/pacote": "^11.1.8",
     "@vltpkg/benchmark": "0.0.0",
-    "@vltpkg/cache-unzip": "1.0.8",
-    "@vltpkg/vlt-json": "1.0.8",
+    "@vltpkg/cache-unzip": "1.0.9",
+    "@vltpkg/vlt-json": "1.0.9",
     "eslint": "^9.39.1",
     "pacote": "^21.0.4",
     "prettier": "^3.7.4",