wkappbot-code 0.3.25 → 0.3.26

package/lib/doctor-path.js

@@ -224,13 +224,19 @@ export function ensureLinks({ home, platform = process.platform, binDir, env = p
   // Windows runs the PATH logic even when both shims already existed: the doctor row reads the persistent PATH,
   // so a missing entry is a FAIL whether or not this run wrote a shim. POSIX keeps its old rule (a link was written).
   const pathWanted = needsPath && (created.length > 0 || platform === "win32");
+  // POSIX: note whether ~/.profile existed BEFORE addToPath, so uninstall deletes only a profile install created.
+  const profilePath = platform === "win32" ? "" : join(home, ".profile");
+  const profileExisted = !!profilePath && existsSync(profilePath);
   const pathAdded = pathWanted ? !!addToPath({ dir, home, platform, readRegPath }) : false;
-  return { created, dir, pathAdded };
+  const profileCreated = pathAdded && !profileExisted && !!profilePath && existsSync(profilePath) ? profilePath : "";
+  return { created, dir, pathAdded, profileCreated };
 }
 
 // removeCreatedLinks(created, opts) -> the entries actually removed. Deletes only the listed entries, nothing else.
 // A "PATH:<folder>" entry is the PATH entry install added: it is removed by removePath (exactly that folder),
-// never by unlink. opts: platform, home, and an injectable removePath({dir, home, platform}) -> boolean.
+// never by unlink. A "PROFILE:<path>" entry is the ~/.profile install created: it is deleted only when its text is
+// blank again (trim() empty), so a profile the user wrote into is never removed. opts: platform, home, and an
+// injectable removePath({dir, home, platform}) -> boolean.
 export function removeCreatedLinks(created = [], { platform = process.platform, home = process.env.HOME || process.env.USERPROFILE || "", removePath = defaultRemovePath } = {}) {
   const removed = [];
   for (const p of created) {
@@ -238,6 +244,13 @@ export function removeCreatedLinks(created = [], { platform = process.platform,
       try { if (removePath({ dir: p.slice(5), home, platform })) removed.push(p); } catch { /* not removed */ }
       continue;
     }
+    if (p.startsWith("PROFILE:")) {
+      try {
+        const f = p.slice(8);
+        if (existsSync(f) && readFileSync(f, "utf8").trim() === "") { unlinkSync(f); removed.push(p); }
+      } catch { /* not removed */ }
+      continue;
+    }
     try { unlinkSync(p); removed.push(p); } catch { /* already gone */ }
   }
   return removed;
@@ -259,6 +272,7 @@ export function installBothLinks({ dest, home, platform = process.platform, binD
   const r = ensureLinks({ home, platform, binDir: dir, env, targets, addToPath, readRegPath });
   const created = BINS.map(pathOf).filter((p) => r.created.includes(p) || ours.some((bin) => pathOf(bin) === p));
   if (r.pathAdded) created.push("PATH:" + dir);  // recorded only when this install wrote the PATH entry
+  if (r.profileCreated) created.push("PROFILE:" + r.profileCreated);  // right after PATH: the profile install made
   return { created, pathAdded: r.pathAdded, kept, dir };
 }
 

package/lib/hook-wire.js

@@ -61,11 +61,19 @@ function route(cmd, lite) {
   return null;
 }
 
+// A dev seat: a harness checkout beside this lite (its .claude/hooks/wkharness-run.sh exists) that is not a web user install
+// (no .wk-user-install marker in the harness folder). Python --wire uses the same two tests (tools/wkharness-lite.sh).
+export function isDevSeat(lite) {
+  const kih = dirname(dirname(lite));
+  return existsSync(join(kih, ".claude", "hooks", "wkharness-run.sh")) && !existsSync(join(kih, ".wk-user-install"));
+}
+
 // The node port of --wire, plus the create step of the web installer. Throws on a settings file that is not JSON.
 export function wireSettings(path, lite) {
   const r = readSettings(path);
   if (!r.ok) throw new Error(path + " is not plain JSON (" + r.err + ")");
   const d = r.json || {};
+  const dev = isDevSeat(lite) && harnessHooks(d).some((h) => /wkharness-run\.sh/.test(h.command));
   const out = {};
   for (const [ev, groups] of Object.entries(d.hooks && typeof d.hooks === "object" ? d.hooks : {})) {
     const keep = [];
@@ -75,6 +83,8 @@ export function wireSettings(path, lite) {
       const seen = new Set(), hs = [];
       for (const h of asHooks(g)) {
         const cmd = String((h && h.command) || "");
+        // a dev seat runs its full harness and never also the lite one (owner 2026-11-12): lite entries go, the rest stay verbatim
+        if (dev) { if (!/wkharness-lite\.sh/.test(cmd)) hs.push(h); continue; }
         const c = HARNESS_HOOK.test(cmd) ? route(cmd, lite) : null;  // only a harness hook is ever rewritten
         if (c && !seen.has(c)) { seen.add(c); hs.push({ type: "command", command: c, timeout: 10 }); }
         else if (!c && !HARNESS_HOOK.test(cmd)) hs.push(h);  // a user's own hook stays verbatim
@@ -83,7 +93,8 @@ export function wireSettings(path, lite) {
     }
     if (keep.length) out[ev] = keep;
   }
-  for (const [ev, arm, matcher] of ARMS) {
+  // a dev seat gets no lite arm added: its full harness is the only chain
+  if (!dev) for (const [ev, arm, matcher] of ARMS) {
     const c = '"' + lite + '" ' + arm;
     if (!(out[ev] || []).some((g) => (g.hooks || []).some((h) => h.command === c)))
       (out[ev] = out[ev] || []).push({ ...(matcher ? { matcher } : {}), hooks: [{ type: "command", command: c, timeout: 10 }] });
@@ -124,6 +135,16 @@ function wireClaudeHooksInner(o) {
     log("hooks: WARN " + o.settings + " is not plain JSON (" + before.err + "); left untouched -- please fix it, then run: wkappbot-code install");
     return res;
   }
+  // A dev seat with both its full hook and a lite entry (owner 2026-11-12: two Stop chains fired): drop the lite entries
+  // and keep the full hook, whatever state the settings reports as.
+  const liteOnDisk = join(o.kih, "tools", "wkharness-lite.sh");
+  if (before.hooks.some((h) => /wkharness-run\.sh/.test(h.command)) && before.hooks.some((h) => /wkharness-lite\.sh/.test(h.command)) && isDevSeat(liteOnDisk)) {
+    wireSettings(o.settings, liteOnDisk);
+    const dev = analyzeHooks(o.settings, o.kih);
+    res.after = dev.state; res.changed = true;
+    log("hooks: " + o.settings + " runs the full harness on this dev seat; its lite hook entries were removed");
+    return res;
+  }
   if (before.state === "ours") { log("hooks: " + o.settings + " already runs every harness hook from " + o.kih); return res; }
   // A FULL harness hook (wkharness-run.sh, the pwsh hub of a dev seat) is never rewritten into the lite one where pwsh
   // runs: the rewrite would downgrade the owner's own seats. Leave it and say what to do.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wkappbot-code",
-  "version": "0.3.25",
+  "version": "0.3.26",
   "description": "wkappbot-code -- Wisdom Kernel: the pinned OpenCode bridge plus the reflex harness plugin, the wkappbot core, the web harness and the latest public skills in one install",
   "type": "module",
   "scripts": {