@vzn/vx-lockfile 0.0.649 → 0.0.650

package/README.md

@@ -80,7 +80,7 @@ These edges are what was installed, so they differ from a guess over manifest ra
 
 ## Cost
 
-Owning the file, the per-project key, the memo and the `--affected` diff are core's `lockfileClaim`; this package is the parsers, and they are internal: it exports `pnpm`, `bun`, `npm`, `yarn` and `LockfileOptions`. A lockfile is parsed **once per content**: the digests and each importer's edges are memoised under the cache dir (`lockfile-claims/<file>.json`) by the file's xxh3 and the plugin, its parsers' source and what core's digest step answers on a fixed probe graph included (an upgrade that changes how a digest is built never replays an old digest), so a warm run pays one read, one hash and one small JSON read — never a parse — and the read happens once per run, not per task. `--affected` reads which files the lockfile names from the same memo. Under `scope: 'workspace'` the file is parsed for its edges alone, once per content, so an unreadable lockfile is refused there too. The settings files are read once per run. The digest is one hash per strongly connected component of the dependency graph (lockfiles carry cycles), children first, so a 1000-importer / 3000-package lockfile digests in ~20 ms when it does change.
+Owning the file, the per-project key, the memo and the `--affected` diff are core's `lockfileClaim`; this package is the parsers, and they are internal: it exports `pnpm`, `bun`, `npm`, `yarn` and `LockfileOptions`. A lockfile is parsed **once per content**: the digests, each importer's edges and the dependency names it records are memoised in core's cache.db (`lockfile_memos`, one row per file and claimant) by the file's xxh3 and the plugin, its parsers' source and what core's digest step answers on a fixed probe graph included (an upgrade that changes how a digest is built never replays an old digest), so a warm run pays one read, one hash and one row read — never a parse — and the read happens once per run, not per task. `--affected` reads which files the lockfile names from the same memo. Under `scope: 'workspace'` the file is parsed for its edges alone, once per content, so an unreadable lockfile is refused there too. The settings files are read once per run. The digest is one hash per strongly connected component of the dependency graph (lockfiles carry cycles), children first, so a 1000-importer / 3000-package lockfile digests in ~20 ms when it does change.
 
 ## `--affected`
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx-lockfile",
-  "version": "0.0.649",
+  "version": "0.0.650",
   "license": "MIT",
   "description": "Lockfile plugins for vx — pnpm(), bun(), npm(), yarn(): each keys every task on its project's own resolved dependency closure from the package manager's lockfile, so a lockfile change re-keys only the projects it reaches; --affected follows the same answer.",
   "keywords": [
@@ -35,7 +35,7 @@
     "access": "public"
   },
   "peerDependencies": {
-    "@vzn/vx": "^0.0.649"
+    "@vzn/vx": "^0.0.650"
   },
   "repository": {
     "type": "git",

package/src/index.ts

@@ -18,7 +18,6 @@
 //
 // Imports core only through the public `@vzn/vx` specifier.
 import { readdirSync, readFileSync } from 'node:fs'
-import { mkdir, rename, rm } from 'node:fs/promises'
 import { createRequire } from 'node:module'
 import path from 'node:path'
 import {
@@ -326,6 +325,10 @@ function plugin(manager: Manager, options: LockfileOptions): VxPlugin {
       version: digestVersion(),
       scope,
       importers: guarded((text: string) => manager.importers(text)),
+      recorded: guarded(
+        (text: string) =>
+          new Map([...manager.recorded(text)].map(([i, deps]) => [i, [...deps.keys()]])),
+      ),
       // Read before `digest` on the key path: a malformed bun.lock refused
       // there as a bare "JSONC Parse error", no file named (fuzzed, L-16).
       ...(extraFiles !== undefined
@@ -476,7 +479,9 @@ function plugin(manager: Manager, options: LockfileOptions): VxPlugin {
         const missing = await staleness(
           ctx.workspaceRoot,
           importers,
-          await recordedOnce(ctx.cacheDir, file, text, manager.recorded),
+          (await claimsAt(ctx.workspaceRoot)
+            .find((c) => c.file === file)!
+            .claim.recorded(ctx)) ?? new Map(),
           await dirs,
           manifest,
           manager.member,
@@ -539,46 +544,6 @@ function plugin(manager: Manager, options: LockfileOptions): VxPlugin {
   })
 }
 
-/**
- * The dependency names each importer records, kept per lockfile content:
- * parsing them (pnpm's importers block, ~2 ms on solid, and the parser's
- * load) was on every run, and only the lockfile text decides them.
- */
-async function recordedOnce(
-  cacheDir: string,
-  file: string,
-  text: string,
-  recorded: (text: string) => ReadonlyMap<string, ReadonlyMap<string, unknown>>,
-): Promise<ReadonlyMap<string, ReadonlySet<string>>> {
-  const hash = `${RECORDED_VERSION}:${Bun.hash.xxHash3(text).toString(16)}`
-  const memo = path.join(cacheDir, 'lockfile-claims', `${file}.recorded.json`)
-  try {
-    const kept = (await Bun.file(memo).json()) as { hash: string; names: Record<string, string[]> }
-    if (kept.hash === hash) {
-      return new Map(Object.entries(kept.names).map(([i, names]) => [i, new Set(names)]))
-    }
-  } catch {
-    // None kept, or unreadable: parsed below.
-  }
-  const out = new Map([...recorded(text)].map(([i, deps]) => [i, new Set(deps.keys())]))
-  const names = Object.fromEntries([...out].map(([i, set]) => [i, [...set]]))
-  // Write-then-rename, so a concurrent run reads a whole memo or none. The
-  // memo is a speed-up: a failed write costs the next run a parse.
-  const tmp = `${memo}.tmp-${process.pid}-${Date.now()}`
-  try {
-    await mkdir(path.dirname(memo), { recursive: true })
-    await Bun.write(tmp, JSON.stringify({ hash, names }))
-    await rename(tmp, memo)
-  } catch {
-    // Left behind, one more temp file piled up in the cache every run.
-    await rm(tmp, { force: true }).catch(() => {})
-  }
-  return out
-}
-
-/** Bumped when what `recorded` returns for the same text changes. */
-const RECORDED_VERSION = 1
-
 /** Up to five names, then a count of the rest. */
 function listed(names: readonly string[]): string {
   return names.length <= 5