vouchington-tooling 0.12.0 → 0.12.2

package/README.md

@@ -68,13 +68,15 @@ vouchington post-review
 vouchington stage-review-payload optional|required <source> <destination>
 ```
 
-For persistent `pnpm-install`, v4 metadata tracks structural inputs separately from the
+For persistent `pnpm-install`, v5 metadata tracks structural inputs separately from the
 `--install-scripts` policy. A warm scripts-enabled tree can therefore toggle
 `true → false → true` without forced reconciliation; a tree first installed with scripts disabled
 uses one script-suppressed verification install followed by `pnpm rebuild --pending --recursive`.
 When only newly pending dependency package IDs remain, it instead rebuilds those exact IDs without
 rerunning first-party workspace hooks.
-An isolated native-binary mismatch uses one strict forced install only when structural provenance
+If that generic rebuild leaves only pnpm's root importer marker, one root-only pending rebuild runs
+the root lifecycle hook; every other residual ID remains a hard failure. An isolated native-binary
+mismatch uses one strict forced install only when structural provenance
 matches, workspace links are valid, and pnpm records empty `ignoredBuilds` and `pendingBuilds` ledgers;
 otherwise it retains the script-free then strict reconciliation. Native and workspace-link health
 are verified before its metadata stamp is refreshed.

package/dist/pnpm-install/install-operations.mjs

@@ -1,7 +1,7 @@
 import { scheduler } from 'node:timers/promises';
 import { runPnpm } from './exec.mjs';
 import { nativeBinariesMatchRuntime, repairedNativeBinariesMatchRuntime } from './native-health.mjs';
-import { buildLedgersAllowNativeRepair, pendingBuilds } from './pending-builds.mjs';
+import { buildLedgersAllowNativeRepair, deduplicatePendingBuilds, pendingBuilds, } from './pending-builds.mjs';
 import { INSTALL_TERMINATION_FAILED } from './process.mjs';
 import { formatReleaseAgeFailure, isReleaseAgeViolation } from './release-age.mjs';
 import { findWorkspaceLinkMismatches, forcedInstallArgs, logWorkspaceLinkMismatches, } from './support.mjs';
@@ -51,6 +51,9 @@ export async function finalizePendingBuilds(options, runCapture, phase) {
     if (options.installScripts && before.kind === 'unknown')
         fail(`${phase} completed without a clear pending build ledger`);
     if (options.installScripts && before.kind === 'pending') {
+        const deduplicated = await deduplicatePendingBuilds();
+        if (deduplicated.kind !== 'pending')
+            fail(`${phase} completed without a clear pending build ledger`);
         await install(['rebuild', '--pending', '--recursive'], options, 'pending scripts rebuild');
         const after = await pendingBuilds();
         if (after.kind !== 'clear')

package/dist/pnpm-install/pending-builds.d.mts

@@ -7,4 +7,5 @@ export type PendingBuildState = {
     kind: 'unknown';
 };
 export declare function pendingBuilds(): Promise<PendingBuildState>;
+export declare function deduplicatePendingBuilds(): Promise<PendingBuildState>;
 export declare function buildLedgersAllowNativeRepair(): Promise<boolean>;

package/dist/pnpm-install/pending-builds.mjs

@@ -1,9 +1,10 @@
-import { readFile } from 'node:fs/promises';
+import { readFile, writeFile } from 'node:fs/promises';
 import path from 'node:path';
-import { parse } from 'yaml';
+import { parse, stringify } from 'yaml';
+const modulesPath = () => path.join(process.cwd(), 'node_modules', '.modules.yaml');
 async function buildLedgers() {
     try {
-        const value = parse(await readFile(path.join(process.cwd(), 'node_modules', '.modules.yaml'), 'utf8'));
+        const value = parse(await readFile(modulesPath(), 'utf8'));
         if (typeof value !== 'object' || value === null || Array.isArray(value))
             return undefined;
         // oxlint-disable-next-line no-mistakes/ts-no-const-aliases -- validate the parsed YAML object before reading its fields
@@ -18,6 +19,7 @@ async function buildLedgers() {
         return {
             ignoredBuilds: ignored,
             pendingBuilds: pending.length === 0 ? { kind: 'clear' } : { ids: pending.toSorted(), kind: 'pending' },
+            record,
         };
     }
     catch {
@@ -27,6 +29,22 @@ async function buildLedgers() {
 export async function pendingBuilds() {
     return (await buildLedgers())?.pendingBuilds ?? { kind: 'unknown' };
 }
+export async function deduplicatePendingBuilds() {
+    const ledgers = await buildLedgers();
+    if (ledgers === undefined || ledgers.pendingBuilds.kind !== 'pending')
+        return ledgers?.pendingBuilds ?? { kind: 'unknown' };
+    // `buildLedgers` has already verified every pending entry is a string.
+    const unique = [...new Set(ledgers.record.pendingBuilds)];
+    if (unique.length === ledgers.pendingBuilds.ids.length)
+        return ledgers.pendingBuilds;
+    try {
+        await writeFile(modulesPath(), stringify({ ...ledgers.record, pendingBuilds: unique }));
+    }
+    catch {
+        return { kind: 'unknown' };
+    }
+    return pendingBuilds();
+}
 export async function buildLedgersAllowNativeRepair() {
     const ledgers = await buildLedgers();
     return (ledgers !== undefined &&

package/dist/pnpm-install/pnpm-install-fake-pnpm.test-helpers.mjs

@@ -29,10 +29,10 @@ if [ "\${PNPM_FAIL_RELEASE_AGE_CALL:-0}" = "$calls" ]; then
   exit 1
 fi
 if [ -n "\${PNPM_SLEEP_SECONDS:-}" ]; then sleep "$PNPM_SLEEP_SECONDS"; fi
-if [ "\${PNPM_INVALID_PENDING_BUILDS:-0}" = 1 ]; then
+if [ "\${1:-}" != rebuild ] && [ "\${PNPM_INVALID_PENDING_BUILDS:-0}" = 1 ]; then
   mkdir -p "$PNPM_NODE_MODULES"
   printf 'pendingBuilds: invalid\n' > "$PNPM_NODE_MODULES/.modules.yaml"
-elif [ -n "\${PNPM_PENDING_BUILDS:-}" ]; then
+elif [ "\${1:-}" != rebuild ] && [ -n "\${PNPM_PENDING_BUILDS:-}" ]; then
   mkdir -p "$PNPM_NODE_MODULES"
   printf 'pendingBuilds: [%s]\\n' "$PNPM_PENDING_BUILDS" > "$PNPM_NODE_MODULES/.modules.yaml"
 elif [ ! -f "$PNPM_NODE_MODULES/.modules.yaml" ]; then
@@ -40,12 +40,19 @@ elif [ ! -f "$PNPM_NODE_MODULES/.modules.yaml" ]; then
   printf 'pendingBuilds: []\\n' > "$PNPM_NODE_MODULES/.modules.yaml"
 fi
 case " $* " in
+  *' rebuild --pending --workspace-root '*)
+    printf 'pendingBuilds: [%s]\n' "\${PNPM_WORKSPACE_ROOT_REBUILD_PENDING_BUILDS:-}" > "$PNPM_NODE_MODULES/.modules.yaml"
+    ;;
   *' rebuild '*)
     if [ "\${PNPM_REPAIR_NATIVE_ON_REBUILD:-0}" = 1 ]; then
       cp "$PNPM_NATIVE_REPLACEMENT" "$PNPM_NATIVE_ADDON"
     fi
     if [ "\${PNPM_REBUILD_INVALID_LEDGER:-0}" = 1 ]; then
       printf 'pendingBuilds: invalid\n' > "$PNPM_NODE_MODULES/.modules.yaml"
+    elif [ "\${PNPM_REBUILD_REQUIRES_DEDUPED:-0}" = 1 ] && sed -n 's/.*\\[\\(.*\\)\\].*/\\1/p' "$PNPM_NODE_MODULES/.modules.yaml" | tr ',' '\\n' | sed 's/^ *//; s/ *$//' | sort | uniq -d | grep -q .; then
+      cp "$PNPM_NODE_MODULES/.modules.yaml" "$PNPM_NODE_MODULES/.modules.yaml.rebuild-left-pending"
+    elif [ -n "\${PNPM_REBUILD_PENDING_BUILDS:-}" ]; then
+      printf 'pendingBuilds: [%s]\n' "$PNPM_REBUILD_PENDING_BUILDS" > "$PNPM_NODE_MODULES/.modules.yaml"
     else
       printf 'pendingBuilds: []\n' > "$PNPM_NODE_MODULES/.modules.yaml"
     fi

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vouchington-tooling",
-  "version": "0.12.0",
+  "version": "0.12.2",
   "description": "Vouchington CLI and extractable tooling libraries.",
   "homepage": "https://github.com/vouchington/vouchington-tooling/tree/main/packages/vouchington-tooling#readme",
   "bugs": {