@augurworks/augur 0.15.10 → 0.15.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@augurworks/augur",
-  "version": "0.15.10",
+  "version": "0.15.11",
   "files": [
     "scripts/",
     "build.js",

package/scripts/hook.mjs

@@ -5,13 +5,15 @@
 // JSON on stdin (docs/drafts-that-land.md §7). Both answer with an exit code: 0 lets the
 // edit through, 2 refuses it and puts one sentence on stderr for the agent to read in its
 // next tool result. NOTHING ELSE EXITS NON-ZERO: an unreadable payload, a missing
-// registry, an exception — every accident exits 0 and says nothing, because the hook is
-// installed machine-wide and a bug here would take every editor down with it.
+// registry, an exception — every accident exits 0 and says nothing, because the hook runs
+// around every edit in the folder it is installed for and a bug here would take that
+// editor down with it.
 //
 //   pre    refuse a write into a read-only copy, or into a shared checkout's prototype
 //   post   save the draft the edited file belongs to; refused saves fail with the reason
 //
-// `install|remove|status` manage the adapters — `augur open` installs them once.
+// `install|remove|status` manage the adapters for the folder they run in (the agent tool's
+// project-local settings) — `augur open` installs them there the first time.
 import fs from "node:fs";
 import { ADAPTERS, denyDecision, saveDecision, installAdapters, removeAdapters } from "./lib/adapters.mjs";
 import { registryList, readState, unitClient, doSave } from "./lib/draft.mjs";
@@ -24,7 +26,7 @@ if (event === "install" || event === "remove" || event === "status") {
   const rows = event === "install" ? installAdapters() : event === "remove" ? removeAdapters() : installAdapters({ dryRun: true });
   for (const r of rows) {
     const verdict = event === "status" ? (r.result === "unchanged" ? "installed" : r.result === "absent" ? "absent" : "not installed") : r.result;
-    console.log(`${r.name}: ${verdict}${r.path ? ` (${r.path})` : ""}`);
+    console.log(`${r.name}: ${verdict}${r.path ? ` (${r.path})` : ""}${r.movedFrom ? ` — an older machine-wide entry in ${r.movedFrom} was removed` : ""}`);
   }
   process.exit(0);
 }

package/scripts/lib/adapters.mjs

@@ -84,7 +84,14 @@ export const ADAPTERS = Object.freeze([
   Object.freeze({
     id: "claude-code",
     name: "Claude Code",
-    settingsPath: (home) => path.join(home, ".claude", "settings.json"),
+    // The hooks live in the PROJECT's local settings — the folder `augur open` runs in, which
+    // is where the draft folder is made — never in the account-wide file. Three cold agents
+    // in a row flagged a write to ~/.claude/settings.json and their person asked for it to be
+    // undone; a hook whose command carries this machine's absolute path belongs in the
+    // per-machine local file anyway. `legacyPath` is where older engines wrote it; an entry
+    // found there is removed on the next install so the two never fire together.
+    settingsPath: (cwd) => path.join(cwd, ".claude", "settings.local.json"),
+    legacyPath: (home) => path.join(home, ".claude", "settings.json"),
     detect: (home) => fs.existsSync(path.join(home, ".claude")),
     matcher: "Write|Edit|MultiEdit|NotebookEdit",
     payload: (j) => {
@@ -154,20 +161,28 @@ function writeJson(p, v) {
   fs.renameSync(p + ".tmp", p);
 }
 
-function applyToAdapters({ home = os.homedir(), dryRun = false } = {}, decide) {
+function applyToAdapters({ home = os.homedir(), cwd = process.cwd(), dryRun = false } = {}, decide) {
   const out = [];
   for (const a of ADAPTERS) {
     if (!a.detect(home)) { out.push({ id: a.id, name: a.name, result: "absent" }); continue; }
-    const p = a.settingsPath(home);
+    const p = a.settingsPath(cwd);
     const cur = readJson(p);
     const r = decide(a, cur);
     if (r.changed && !dryRun) writeJson(p, r.settings);
-    out.push({ id: a.id, name: a.name, result: r.changed ? r.did : "unchanged", path: p });
+    // An entry an older engine left in the account-wide file: taken out, and said.
+    let movedFrom = null;
+    const lp = a.legacyPath ? a.legacyPath(home) : null;
+    const legacy = lp ? readJson(lp) : null;
+    if (legacy) {
+      const s = stripHooks(legacy);
+      if (s.changed) { movedFrom = lp; if (!dryRun) writeJson(lp, s.settings); }
+    }
+    out.push({ id: a.id, name: a.name, result: r.changed ? r.did : "unchanged", path: p, ...(movedFrom ? { movedFrom } : {}) });
   }
   return out;
 }
 
-/** Install the two hooks for every tool present on this machine. Idempotent; keeps every other hook. */
+/** Install the two hooks, for the folder `cwd` (the agent tool's project-local settings), for every tool present on this machine. Idempotent; keeps every other hook. */
 export function installAdapters(opts = {}) {
   return applyToAdapters(opts, (a, cur) => {
     const onPath = typeof opts.onPath === "boolean" ? opts.onPath : augurOnPath();

package/scripts/open.mjs

@@ -47,12 +47,14 @@ if (!r.ok) {
   die(`could not open: ${r.error || r.status}${r.reason ? ` (${r.reason})` : ""}${r.message ? ` — ${r.message}` : ""}`);
 }
 log(r.isNew ? `draft ${r.draftId} on ${unit} — a NEW prototype; ${dir} is empty, write its index.html there` : `draft ${r.draftId} on ${unit} — ${r.files} file(s) in ${dir}`);
-// The agent tool's hooks, installed for this machine the first time a draft is opened
-// here (idempotent; `AUGUR_NO_ADAPTERS=1` skips it — the suite and CI set it).
+// The agent tool's hooks, installed for THIS folder (its project-local settings) the first
+// time a draft is opened from here — never account-wide (idempotent; `AUGUR_NO_ADAPTERS=1`
+// skips it — the suite and CI set it).
 if (!process.env.AUGUR_NO_ADAPTERS) {
   for (const a of installAdapters()) {
-    if (a.result === "installed") log(`${a.name}: a save hook is now in ${a.path} — after each edit inside a draft folder it runs \`augur hook post\`, which saves that draft to ${origin}; before an edit it refuses writes into a shared checkout's prototypes. It does nothing outside draft folders and talks to nothing else; \`augur hook remove\` takes it out.`);
+    if (a.result === "installed") log(`${a.name}: a save hook is now in ${a.path} — for this folder only, nothing account-wide. After each edit inside a draft folder here it runs \`augur hook post\`, which saves that draft to ${origin}; before an edit it refuses writes into a shared checkout's prototypes. It does nothing else and talks to nothing else; \`augur hook remove\` run here takes it out.`);
     else if (a.result === "updated") log(`${a.name}: editor hooks updated (${a.path}).`);
+    if (a.movedFrom) log(`${a.name}: an older machine-wide entry in ${a.movedFrom} was removed; the hook lives with the folder now.`);
   }
 }
 if (r.others.length) {

package/src/_worker.js

@@ -9307,9 +9307,10 @@ function doorText(f) {
         ? `You need no source tree here: \`augur open <opportunity>/<prototype>\` fetches that one\n`
           + `prototype into a folder of its own (see below). Do not clone the workspace first.\n\n`
           + `Until \`npm i -g @augurworks/augur\`, every verb runs as \`npx @augurworks/augur <verb>\`.\n`
-          + `The first \`open\` on a machine also installs a save hook in the editor tool's\n`
-          + `settings (it says which file): after each edit inside a draft folder it saves that\n`
-          + `draft; it does nothing elsewhere, and \`augur hook remove\` takes it out.\n\n`
+          + `The first \`open\` in a folder also installs a save hook in the editor tool's settings\n`
+          + `for that folder only (.claude/settings.local.json there; it says so), never account-wide:\n`
+          + `after each edit inside a draft folder it saves that draft; it does nothing elsewhere,\n`
+          + `and \`augur hook remove\` in that folder takes it out.\n\n`
         : `With no source tree yet, \`npx @augurworks/augur clone --space ${f.workspace}\` then fetches\n`
           + `one (it reads the origin from the pairing).\n\n`)
     : `Device pairing is switched off on this workspace. Ask an admin for an invite;\n`