javi-forge 1.34.0 → 1.34.1

package/dist/lib/__fixtures__/fake-secure-fs.d.ts

@@ -27,6 +27,11 @@ export interface FakeFaults {
     writeRefuse?: (name: string, callIndex: number) => boolean;
     /** Refuse renameInDir when the destination base name matches. */
     renameRefuse?: (to: string) => boolean;
+    /**
+     * Refuse applyExactMode for a target (chmod fault). Lets a test drive the
+     * rollback's prior-mode restore into failure without touching the bytes.
+     */
+    applyModeRefuse?: (target: string) => boolean;
     /**
      * Refuse proveManagedContainer for a path on its Nth call (a foreign
      * add/delete-child ACE on a managed container we own — the win32 CREATE_PARENT_DIR

package/dist/lib/__fixtures__/fake-secure-fs.js

@@ -152,6 +152,8 @@ export function makeFakeSecureFs() {
             return ok();
         },
         async applyExactMode(target, mode) {
+            if (fake.faults.applyModeRefuse?.(target))
+                return unsafe(`applyMode ${target}`);
             const file = files.get(target);
             if (!file)
                 return unsafe(`applyMode enoent ${target}`);

package/dist/lib/claude-hook-manager.js

@@ -191,10 +191,12 @@ function settingsSignals(value, classification, currentAssetSha) {
     };
 }
 const REMEDIATION = {
-    absent: "install the managed $ (Slice 3)",
-    "released-outdated": "upgrade the managed $ (Slice 3)",
-    "exact-legacy": "migrate the legacy $ (Slice 3)",
-    "edited-managed": "repair the managed $ with --force (Slice 3)",
+    // User-facing remediation: name the exact command to run, never an internal
+    // SDD slice number (which means nothing outside this repo's planning docs).
+    absent: "install the managed $ with: javi-forge hooks install claude",
+    "released-outdated": "upgrade the managed $ with: javi-forge hooks install claude",
+    "exact-legacy": "migrate the legacy $ with: javi-forge hooks install claude",
+    "edited-managed": "repair the managed $ with: javi-forge hooks repair claude --force",
     foreign: "manually review the $",
     symlink: "manually review the $",
     "non-regular": "manually review the $",

package/dist/lib/secure-fs-transaction.js

@@ -286,7 +286,16 @@ export async function runTransaction(input) {
                     errors.push(`STOP: cannot stage rollback for ${entry.path}`);
                     return;
                 }
-                await secureFs.applyExactMode(path.join(entry.dir.path, rName), entry.prior.mode);
+                // Prior-mode restore is BEST-EFFORT (JD-B-003): a chmod miss fails
+                // toward MORE restrictive perms (the 0600 staging mode), so it is not
+                // a security regression and must not halt the rollback — restoring the
+                // prior BYTES matters more than the prior mode. Record an
+                // INFORMATIONAL note (never `STOP:`, which signals a halted rollback)
+                // and continue to the rename.
+                const remoded = await secureFs.applyExactMode(path.join(entry.dir.path, rName), entry.prior.mode);
+                if (!remoded.ok) {
+                    errors.push(`note: restored ${entry.path}; prior-mode restore failed, verify permissions (${remoded.detail ?? remoded.refusal ?? "unknown"})`);
+                }
                 const restored = await secureFs.renameInDir(entry.dir, rName, base);
                 if (!restored.ok) {
                     errors.push(`STOP: cannot restore ${entry.path}; prior payload staged at ${path.join(entry.dir.path, rName)} for manual recovery`);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "javi-forge",
-	"version": "1.34.0",
+	"version": "1.34.1",
 	"description": "Project scaffolding and AI-ready CI bootstrap",
 	"type": "module",
 	"bin": {