skillrepo 3.1.0 → 3.1.2

package/src/test/mergers/uninstall-settings.test.mjs

@@ -270,9 +270,20 @@ describe("removeSettingsSessionHook", () => {
     // refactor ever moves the fingerprint elsewhere or drops it,
     // this import breaks loudly and the architect tightening #3
     // (bidirectional registry/merger mapping) fails at import time.
+    //
+    // The fingerprint is deliberately platform-neutral — before
+    // v3.1.1 it contained `skillrepo`, but Windows `.cmd` shim
+    // paths broke the substring match. The current value is
+    // `update --session-hook` which appears in BOTH POSIX and
+    // Windows hook command shapes.
     assert.equal(typeof SESSION_HOOK_FINGERPRINT, "string");
     assert.ok(SESSION_HOOK_FINGERPRINT.length > 0);
-    assert.match(SESSION_HOOK_FINGERPRINT, /skillrepo/);
+    assert.match(
+      SESSION_HOOK_FINGERPRINT,
+      /--session-hook/,
+      "fingerprint must contain the --session-hook flag — that's what makes " +
+        "it specific to SkillRepo's installed hook and not a random user hook",
+    );
   });
 
   it("returns a structured error on unparseable JSON", () => {

package/src/test/mergers/uninstall-windsurf-mcp.test.mjs

@@ -23,15 +23,21 @@ import { join } from "node:path";
 import { tmpdir } from "node:os";
 
 import { removeWindsurfMcp } from "../../lib/removers/windsurf-mcp.mjs";
+import {
+  captureHome,
+  setSandboxHome,
+  restoreHome,
+} from "../helpers/sandbox-home.mjs";
 
 let sandbox;
-let originalHome;
+/** @type {import("../helpers/sandbox-home.mjs").HomeEnvSnapshot} */
+let originalHomeEnv;
 let mcpConfigPath;
 
 function setup() {
   sandbox = mkdtempSync(join(tmpdir(), "cli-unrm-windsurf-"));
-  originalHome = process.env.HOME;
-  process.env.HOME = sandbox;
+  originalHomeEnv = captureHome();
+  setSandboxHome(sandbox);
   // Sanity guard: asserting the override worked BEFORE any remover
   // runs is cheap insurance against forgetting to restore HOME in a
   // previous test's teardown or a parallel test cross-contamination.
@@ -44,7 +50,7 @@ function setup() {
 }
 
 function teardown() {
-  process.env.HOME = originalHome;
+  restoreHome(originalHomeEnv);
   if (sandbox) rmSync(sandbox, { recursive: true, force: true });
 }