projectstore-claude 0.30.0 → 0.30.2

package/node_modules/projectstore/scripts/harness.mjs

@@ -302,9 +302,7 @@ export function hostPresence(manifest, { env = process.env, home = homedir() } =
         x.cli.bin,
     )?.[1] || null;
   const bin = s ? s.cli.bin : null;
-  const r = manifest?.runtime || {};
-  const fromEnv = r.home_env && env ? env[r.home_env] : null;
-  const homeDir = fromEnv || (home && r.home_default ? join(home, r.home_default) : null);
+  const homeDir = harnessHome(manifest, { env, home });
   let present = false;
   if (homeDir) {
     try {
@@ -314,6 +312,15 @@ export function hostPresence(manifest, { env = process.env, home = homedir() } =
   return { cli: bin ? whichOnPath(bin, env || {}) : null, home: present ? homeDir : null, bin, homeDir };
 }
 
+// A harness's own home directory, as the harness resolves it: its home
+// variable (runtime.home_env) when set, else home_default under the home
+// directory, else null. Whether it exists is the caller's question.
+export function harnessHome(manifest, { env = process.env, home = homedir() } = {}) {
+  const r = manifest?.runtime || {};
+  const fromEnv = r.home_env && env ? env[r.home_env] : null;
+  return fromEnv || (home && r.home_default ? join(home, r.home_default) : null);
+}
+
 // The agents block of a RUN (the story "One run plans the agents block once,
 // and a bare uninstall selects every harness the project uses", rules 1–2):
 // the files every manifest of the run names for it, in the FIRST manifest's
@@ -703,6 +710,9 @@ export function cachePaths({ env = process.env, home = homedir() } = {}) {
     fetch: root ? join(root, "fetch") : null,
     fetchRun: (id) => (root ? join(root, "fetch", id) : null),
     registry: root ? join(root, "registry.json") : null,
+    // A harness's model list, as its last live read answered (the ADR "Agent
+    // model lists are read from each harness's own catalog…", decision 4).
+    models: (id) => (root && id ? join(root, "models", `${id}.json`) : null),
     from: fromEnv ? variable : "the home directory",
     variable,
   };
@@ -779,6 +789,14 @@ export function hasBand(manifest) {
   return manifest?.surfaces?.band?.supported === true;
 }
 
+// Whether a harness has the status-line surface: the one predicate for "a
+// harness without a status line", read by configure (its spec, contract 2).
+// Install's own test is the same: a surface declared and not unsupported.
+export function hasStatusLine(manifest) {
+  const s = manifest?.surfaces?.statusline;
+  return Boolean(s) && s.supported !== false;
+}
+
 // The session id the host hands a tool process, by the manifest's name for
 // it (runtime.session_id_env), or null where the harness names none or the
 // process got none.

package/node_modules/projectstore/scripts/install-harness.mjs

@@ -1678,15 +1678,21 @@ function planKeepRecord(ctx) {
   });
 }
 
-// The status line's flag (the setup story, rule 9): setup's answer stored as
-// `statusline.enabled`, the key bind's interview writes; every other key of
-// a `statusline` object is kept. Planned for the run's harness that has the
-// status-line surface, when the binding the run leaves does not already say
-// the same; only when there is a binding to record it in.
+// The status line's flag (the setup story, rule 9; the configure spec,
+// contract 6): `statusline.enabled`, from setup's answer or from a run that
+// names the surface — install records on, uninstall off. This item is the
+// flag's one writer; every other key of a `statusline` object is kept.
+// Planned for the run's harness that has the status-line surface, when the
+// binding the run leaves does not already say the same; only when there is a
+// binding to record it in.
 const STATUSLINE_FLAG = "statusline.enabled";
-function planStatuslineFlag(ctx, value) {
+function planStatuslineFlag(ctx, value, mode = "install") {
   const b = ctx.binding;
   if (!b || !b.bound || typeof value !== "boolean" || b.statusline === value) return null;
+  const off =
+    mode === "uninstall"
+      ? "the status line is removed in this run, and the SessionStart hook keeps it off"
+      : "the status line stays off, and setup does not ask again";
   return bindingItem({
     harness: ctx.harness.id,
     surface: "statusline_flag",
@@ -1697,7 +1703,7 @@ function planStatuslineFlag(ctx, value) {
     why: {
       [STATUSLINE_FLAG]: value
         ? "the status line is wired in this run, and the SessionStart hook keeps it current"
-        : "the status line stays off, and setup does not ask again",
+        : off,
     },
     said: {
       unbound: "the project is not bound; the status line's flag is recorded once it is",
@@ -2342,12 +2348,24 @@ export function plan(
       const s = loadHarness(id).surfaces?.statusline;
       return Boolean(s) && s.supported !== false;
     }) || null;
-  const recordFor = (ctx) => {
-    if (mode === "uninstall") return;
+  // The flag's value: setup's answer when it gave one; else, when the run
+  // names the status-line surface, what the run does to it — on for install,
+  // off for uninstall. Never on for a line this run neither wires nor finds
+  // ours: a foreign slot, the keep's branch, a root that is never wired.
+  const flagValue = (ctx, keep) => {
+    if (typeof statusline === "boolean") return statusline;
+    if (!(surfaces || []).includes("statusline")) return null;
+    if (mode === "uninstall") return false;
+    if (keep || ctx.slotForeign.has("statusline")) return null;
+    const row = out.items.find((i) => i.harness === ctx.harness.id && i.surface === "statusline");
+    return row && (row.action !== "skip" || row.state === "ours-current") ? true : null;
+  };
+  const recordFor = (ctx, { keep = false } = {}) => {
     if (ctx.harness.id === flagHarness) {
-      const flag = planStatuslineFlag(ctx, statusline);
+      const flag = planStatuslineFlag(ctx, flagValue(ctx, keep), mode);
       if (flag) marks.push(flag);
     }
+    if (mode === "uninstall") return;
     if (recordKeep && keeping && ctx.harness.id === keeping.harness.id) {
       const kept = planKeepRecord(ctx);
       if (kept) marks.push(kept);
@@ -2464,7 +2482,7 @@ export function plan(
       out.refusals.push(...ctx.refusals);
       if (ctx.incomplete) out.incomplete = true;
       stateFor(id, harness);
-      recordFor(ctx);
+      recordFor(ctx, { keep: true });
       continue;
     }
     const hostRows = [];

package/node_modules/projectstore/scripts/lib.mjs

@@ -90,10 +90,15 @@ export function sharedRoleForm(name) {
   return invocation(sourceHarness(), name, { kind: "agents" });
 }
 // What a welcome says about updates, and the product name a message uses.
-export function updateHint({ env = process.env } = {}) {
+export function updateHint({ env = process.env, root = pluginRoot(), home = homedir() } = {}) {
   const m = speakingHarness(env) || {};
   // A harness that declares no hint gets none — never another harness's advice.
-  const hint = m.update_hint || { line: "", welcome: [] };
+  // A copy loaded from another channel than the hint's own says that
+  // channel's way to update (installChannel: string tests, no file read, so
+  // the SessionStart budget holds): a git-marketplace copy keeps the host's
+  // auto-update toggle, the npm registration and a checkout get the npm form.
+  const channel = m.update_hint?.by_channel ? installChannel(root, { home, harness: m }) : null;
+  const hint = (channel && m.update_hint.by_channel[channel]) || m.update_hint || { line: "", welcome: [] };
   // `{upgrade}` is the core's upgrade for this harness, pinned to latest so it
   // runs as printed, with no --project: the hint says where to run it (the
   // story "User-facing text says where to run…", rule 2). Any hint that holds
@@ -1723,8 +1728,9 @@ export function listOf(fm, key) {
 // the policy survives clones and second machines), Obsidian hides it, and it
 // is intentionally NOT inside <vault>/.projectstore/, whose .gitignore ("*")
 // would defeat the whole point. Keys: spec_policy ("required"|"optional"),
-// lifecycle_gates ("on"|"off"), spec_policy_since (ISO-8601, stamped when
-// spec_policy first becomes "required").
+// lifecycle_gates ("on"|"off"), spec_policy_since (ISO-8601, stamped whenever
+// spec_policy becomes "required" from anything else; kept otherwise — the
+// configure spec, contract 7).
 export function vaultConfigPath(vault) {
   return join(vault, LAYOUT.vaultConfig);
 }
@@ -1740,8 +1746,38 @@ export function readVaultConfig(vault) {
   }
 }
 
+// The strict read a writer needs: a file that exists and is not a JSON
+// object is `corrupt`, never an empty config a write would then replace.
+export function readVaultConfigStrict(vault) {
+  const path = vaultConfigPath(vault);
+  if (!existsSync(path)) return { path, present: false, corrupt: false, cfg: {} };
+  try {
+    const v = JSON.parse(readFileSync(path, "utf8"));
+    if (v && typeof v === "object" && !Array.isArray(v)) return { path, present: true, corrupt: false, cfg: v };
+  } catch {}
+  return { path, present: true, corrupt: true, cfg: null };
+}
+
+// The config a change leaves, pure. A return to "required" re-stamps
+// spec_policy_since: a story done before the policy (re)took effect is exempt
+// (ADR-007 decision 6), and since every story the old stamp exempted closed
+// before the new one, no exemption is lost.
+export function nextVaultConfig(before, patch, now = new Date().toISOString()) {
+  const next = { ...before, ...patch };
+  if (patch.spec_policy === "required" && before.spec_policy !== "required") {
+    // Never earlier than the stamp it replaces: one written ahead of this
+    // clock (a hand edit, a machine whose clock ran fast) would otherwise
+    // shrink the exemption the old stamp gave.
+    const prev = before.spec_policy_since;
+    next.spec_policy_since = prev && String(prev) > now ? prev : now;
+  }
+  return next;
+}
+
+// The one writer of the vault config: atomic, and every key it was not asked
+// to change is kept as it was.
 export function writeVaultConfig(vault, cfg) {
-  writeFileSync(vaultConfigPath(vault), JSON.stringify(cfg, null, 2) + "\n", "utf8");
+  writeFileAtomic(vaultConfigPath(vault), JSON.stringify(cfg, null, 2) + "\n");
 }
 
 // Legacy exemption (ADR-007 Decision 6): a story is exempt from spec-first