tryharness 0.7.22 → 0.7.25

package/lib/cards.mjs

@@ -14,96 +14,217 @@ import { choiceOf, describePick } from "./models.mjs";
 
 // How a member hears that work is waiting for it: how it is set up to be woken (its setup from the daemon, { state, how }),
 // else its roster wake.
-const ANSWERS = { run: "starts as soon as it may", email: "woken by email when work is posted", webhook: "woken when work is posted", "local-webhook": "woken when work is posted",
-  schedule: "picks work up on its own schedule", none: "only when the person next talks to it" };
-export const answersFor = (entry) => entry.setup?.state === "not_set_up" ? "not set up yet: work posted for it waits until the person sets it up in Harness"
-  : entry.setup?.state === "not_answering" ? "not answering right now: work posted for it waits until it does"
-  : ANSWERS[entry.setup?.how ?? member(entry.id)?.wake] ?? "when it next looks";
+const ANSWERS = {
+  run: "starts as soon as it may",
+  email: "woken by email when work is posted",
+  webhook: "woken when work is posted",
+  "local-webhook": "woken when work is posted",
+  schedule: "picks work up on its own schedule",
+  none: "only when the person next talks to it",
+};
+export const answersFor = (entry) =>
+  entry.setup?.state === "not_set_up"
+    ? "not set up yet: work posted for it waits until the person sets it up in Harness"
+    : entry.setup?.state === "not_answering"
+      ? "not answering right now: work posted for it waits until it does"
+      : (ANSWERS[entry.setup?.how ?? member(entry.id)?.wake] ??
+        "when it next looks");
 
 const NOTE_LIMIT = 300;
 const noteKey = (id) => `agent-note:${id}`;
 export const noteOf = (store, id) => store.meta(noteKey(id)) || null;
 /** The person's own words about one agent ("use it for reviews"); empty clears it. */
 export function setNote(store, id, text) {
-  const note = String(text || "").replace(/\s+/g, " ").trim().slice(0, NOTE_LIMIT);
+  const note = String(text || "")
+    .replace(/\s+/g, " ")
+    .trim()
+    .slice(0, NOTE_LIMIT);
   store.meta(noteKey(id), note);
   return note || null;
 }
 
 // The model a coding agent's own settings name, as it runs here (the daemon passes no model of its own). Only that key
 // is read; null when the settings name none.
-const readText = (file) => { try { return fs.readFileSync(file, "utf8"); } catch { return null; } };
-const readJson = (file) => { const text = readText(file); if (text === null) return null; try { return JSON.parse(text.replace(/^\s*\/\/.*$/gm, "")); } catch { return null; } };
+const readText = (file) => {
+  try {
+    return fs.readFileSync(file, "utf8");
+  } catch {
+    return null;
+  }
+};
+const readJson = (file) => {
+  const text = readText(file);
+  if (text === null) return null;
+  try {
+    return JSON.parse(text.replace(/^\s*\/\/.*$/gm, ""));
+  } catch {
+    return null;
+  }
+};
 const MODELS = {
-  "claude-code": (home) => readJson(path.join(home, ".claude", "settings.json"))?.model,
+  "claude-code": (home) =>
+    readJson(path.join(home, ".claude", "settings.json"))?.model,
   // Top-level keys come before the first [section].
-  codex: (home) => readText(path.join(home, ".codex", "config.toml"))?.split(/^\s*\[/m)[0].match(/^\s*model\s*=\s*"([^"]+)"/m)?.[1],
-  pi: (home) => { const settings = readJson(path.join(home, ".pi", "agent", "settings.json")); return settings?.defaultModel ? [settings.defaultProvider, settings.defaultModel].filter(Boolean).join("/") : null; },
-  opencode: (home) => (readJson(path.join(home, ".config", "opencode", "opencode.json")) ?? readJson(path.join(home, ".config", "opencode", "opencode.jsonc")))?.model,
+  codex: (home) =>
+    readText(path.join(home, ".codex", "config.toml"))
+      ?.split(/^\s*\[/m)[0]
+      .match(/^\s*model\s*=\s*"([^"]+)"/m)?.[1],
+  pi: (home) => {
+    const settings = readJson(path.join(home, ".pi", "agent", "settings.json"));
+    return settings?.defaultModel
+      ? [settings.defaultProvider, settings.defaultModel]
+          .filter(Boolean)
+          .join("/")
+      : null;
+  },
+  opencode: (home) =>
+    (
+      readJson(path.join(home, ".config", "opencode", "opencode.json")) ??
+      readJson(path.join(home, ".config", "opencode", "opencode.jsonc"))
+    )?.model,
 };
 export function modelOf(id, home = os.homedir()) {
   const found = MODELS[id]?.(home);
-  return typeof found === "string" && found.trim() ? found.trim().slice(0, 100) : null;
+  return typeof found === "string" && found.trim()
+    ? found.trim().slice(0, 100)
+    : null;
 }
 
 // The person's choice in the relay (models.mjs), else what the agent's own settings name.
 const modelLine = (store, id, home) => {
   const choice = choiceOf(store, id);
-  return choice?.auto ? "picked for each run by Auto" : choice ? describePick(choice) : modelOf(id, home);
+  return choice?.auto
+    ? "picked for each run by Auto"
+    : choice
+      ? describePick(choice)
+      : modelOf(id, home);
 };
 
 const RECENT = 50;
-const median = (values) => { const sorted = [...values].sort((a, b) => a - b); return sorted.length ? sorted[Math.floor(sorted.length / 2)] : null; };
+const median = (values) => {
+  const sorted = [...values].sort((a, b) => a - b);
+  return sorted.length ? sorted[Math.floor(sorted.length / 2)] : null;
+};
 
 /** Cards for the given roster entries ({ id, setup? }), in their order. A card's setup is its state alone ("not_set_up"). */
 export function swarmCards(store, entries, { home } = {}) {
   const work = new Map();
   for (const row of store.agentWork()) {
-    const entry = work.get(row.agent) ?? { done: 0, failed: 0, minutes: [], working: 0, waiting: 0 };
+    const entry = work.get(row.agent) ?? {
+      done: 0,
+      failed: 0,
+      minutes: [],
+      working: 0,
+      waiting: 0,
+    };
     if (row.status === "done") {
       entry.done++;
       // Minutes from when it could start (the person's Start, else the post) to its report, over its latest work.
-      const minutes = (Date.parse(row.reported_at) - Date.parse(row.started_at)) / 60_000;
-      if (entry.minutes.length < RECENT && Number.isFinite(minutes) && minutes >= 0) entry.minutes.push(minutes);
+      const minutes =
+        (Date.parse(row.reported_at) - Date.parse(row.started_at)) / 60_000;
+      if (
+        entry.minutes.length < RECENT &&
+        Number.isFinite(minutes) &&
+        minutes >= 0
+      )
+        entry.minutes.push(minutes);
     } else if (row.status === "failed") entry.failed++;
-    else if (row.status === "claimed" || row.status === "running") entry.working++;
+    else if (row.status === "claimed" || row.status === "running")
+      entry.working++;
     else if (row.status === "open" && row.targeted) entry.waiting++;
     work.set(row.agent, entry);
   }
-  const seen = new Map(store.agentsSeen().map((row) => [row.agent, row.last_seen_at]));
+  const seen = new Map(
+    store.agentsSeen().map((row) => [row.agent, row.last_seen_at])
+  );
+  // What its own sessions showed after its changes, from the relay's receipts (a store from before receipts has none).
+  const checks = store.receiptCounts?.() ?? {};
   return entries.map((entry) => {
     const roster = member(entry.id) ?? {};
-    const done = work.get(entry.id) ?? { done: 0, failed: 0, minutes: [], working: 0, waiting: 0 };
+    const done = work.get(entry.id) ?? {
+      done: 0,
+      failed: 0,
+      minutes: [],
+      working: 0,
+      waiting: 0,
+    };
     const minutes = median(done.minutes);
     return {
-      id: entry.id, name: roster.name ?? entry.id, about: roster.about ?? "", limits: roster.limits ?? null, where: roster.where ?? null,
-      ...(roster.runs === "machine" ? { review: roster.review === true, model: modelLine(store, entry.id, home) } : {}),
+      id: entry.id,
+      name: roster.name ?? entry.id,
+      about: roster.about ?? "",
+      limits: roster.limits ?? null,
+      where: roster.where ?? null,
+      ...(roster.runs === "machine"
+        ? {
+            review: roster.review === true,
+            model: modelLine(store, entry.id, home),
+          }
+        : {}),
       answers: answersFor(entry),
-      record: { done: done.done, failed: done.failed, medianMinutes: minutes === null ? null : Math.round(minutes * 10) / 10 },
-      working: done.working, waiting: done.waiting, lastSeen: seen.get(entry.id) ?? null, note: noteOf(store, entry.id),
+      record: {
+        done: done.done,
+        failed: done.failed,
+        medianMinutes: minutes === null ? null : Math.round(minutes * 10) / 10,
+        ...(checks[entry.id] ? { checks: checks[entry.id] } : {}),
+      },
+      working: done.working,
+      waiting: done.waiting,
+      lastSeen: seen.get(entry.id) ?? null,
+      note: noteOf(store, entry.id),
       ...(entry.setup?.state ? { setup: String(entry.setup.state) } : {}),
     };
   });
 }
 
-const span = (minutes) => minutes < 1 ? "under a minute" : minutes < 60 ? `${Math.round(minutes)} min`
-  : minutes < 48 * 60 ? `${Math.floor(minutes / 60)} h${Math.round(minutes % 60) ? ` ${Math.round(minutes % 60)} min` : ""}` : `${Math.round(minutes / 1_440)} days`;
-const sentence = (text) => `${text.charAt(0).toUpperCase()}${text.slice(1)}${/[.!?]$/.test(text) ? "" : "."}`;
+const span = (minutes) =>
+  minutes < 1
+    ? "under a minute"
+    : minutes < 60
+      ? `${Math.round(minutes)} min`
+      : minutes < 48 * 60
+        ? `${Math.floor(minutes / 60)} h${Math.round(minutes % 60) ? ` ${Math.round(minutes % 60)} min` : ""}`
+        : `${Math.round(minutes / 1_440)} days`;
+const sentence = (text) =>
+  `${text.charAt(0).toUpperCase()}${text.slice(1)}${/[.!?]$/.test(text) ? "" : "."}`;
 
 /** One card as one line of plain text. */
 export function cardLine(card, now = Date.now()) {
   const parts = [`@${card.id} (${card.name}): ${sentence(card.about)}`];
   if (card.limits) parts.push(card.limits);
-  if (card.review !== undefined) parts.push(card.review ? "Can review without changing files." : "Has no review-only mode: its runs may change files.");
-  if (card.where) parts.push(`Runs ${card.where}${card.model ? `, model ${card.model}` : ""}.`);
+  if (card.review !== undefined)
+    parts.push(
+      card.review
+        ? "Can review without changing files."
+        : "Has no review-only mode: its runs may change files."
+    );
+  if (card.where)
+    parts.push(
+      `Runs ${card.where}${card.model ? `, model ${card.model}` : ""}.`
+    );
   parts.push(sentence(card.answers));
   const { done, failed, medianMinutes } = card.record;
   // Work for an agent whose tasks live on Harness's server is not on this board, so none here says nothing about it.
-  if (done || failed) parts.push(`On this board: ${done} done, ${failed} failed${medianMinutes === null ? "" : `; a median of ${span(medianMinutes)} from start to report`}.`);
-  else if (member(card.id)?.home === "machine") parts.push("Has finished no work on this board yet.");
-  if (card.working || card.waiting) parts.push(`Now: ${card.working} running, ${card.waiting} waiting for it.`);
+  if (done || failed)
+    parts.push(
+      `On this board: ${done} done, ${failed} failed${medianMinutes === null ? "" : `; a median of ${span(medianMinutes)} from start to report`}.`
+    );
+  const checks = card.record.checks;
+  if (checks)
+    parts.push(
+      `Its own sessions show its changes tested after the last edit ${checks.tested} of ${checks.tested + checks.untested + checks.failing} times${checks.failing ? ` (${checks.failing} with tests failing)` : ""}.`
+    );
+  else if (member(card.id)?.home === "machine")
+    parts.push("Has finished no work on this board yet.");
+  if (card.working || card.waiting)
+    parts.push(`Now: ${card.working} running, ${card.waiting} waiting for it.`);
   // An agent that picks work up itself: when it last looked says whether it is around.
-  if (member(card.id)?.wake !== "run") parts.push(card.lastSeen ? `Last checked the board ${span((now - Date.parse(card.lastSeen)) / 60_000)} ago.` : "Has not checked the board yet.");
+  if (member(card.id)?.wake !== "run")
+    parts.push(
+      card.lastSeen
+        ? `Last checked the board ${span((now - Date.parse(card.lastSeen)) / 60_000)} ago.`
+        : "Has not checked the board yet."
+    );
   if (card.note) parts.push(`The person says: ${sentence(card.note)}`);
   return `- ${parts.join(" ")}`;
 }
@@ -111,11 +232,28 @@ export function cardLine(card, now = Date.now()) {
 /** The block an agent is told when its session starts: every other agent's card. */
 export function swarmBlock(cards, now = Date.now()) {
   if (!cards.length) return "";
-  return ["Your swarm, the person's other agents. Hand one work with post_task (target_agent) or ask one in a thread with post_to_room (ask).",
-    ...cards.map((card) => cardLine(card, now))].join("\n");
+  return [
+    "Your swarm, the person's other agents. Hand one work with post_task (target_agent) or ask one in a thread with post_to_room (ask).",
+    ...cards.map((card) => cardLine(card, now)),
+  ].join("\n");
 }
 
 /** What the block says that stays put (who is in the swarm, their facts, the person's notes), so it is told again only when that changes. */
-export const swarmDigest = (cards) => createHash("sha256")
-  .update(JSON.stringify(cards.map((card) => [card.id, card.about, card.limits, card.where, card.review ?? null, card.model ?? null, card.note, card.setup ?? null])))
-  .digest("hex").slice(0, 16);
+export const swarmDigest = (cards) =>
+  createHash("sha256")
+    .update(
+      JSON.stringify(
+        cards.map((card) => [
+          card.id,
+          card.about,
+          card.limits,
+          card.where,
+          card.review ?? null,
+          card.model ?? null,
+          card.note,
+          card.setup ?? null,
+        ])
+      )
+    )
+    .digest("hex")
+    .slice(0, 16);

package/lib/claudemem.mjs

@@ -12,13 +12,22 @@ import { spawnSync } from "node:child_process";
 
 function readSettings(env) {
   const dir = env.CLAUDE_MEM_DATA_DIR || path.join(os.homedir(), ".claude-mem");
-  try { return JSON.parse(fs.readFileSync(path.join(dir, "settings.json"), "utf8")); } catch { return null; }
+  try {
+    return JSON.parse(fs.readFileSync(path.join(dir, "settings.json"), "utf8"));
+  } catch {
+    return null;
+  }
 }
 
-const setting = (env, settings, key, fallback) => env[key] || settings[key] || fallback;
+const setting = (env, settings, key, fallback) =>
+  env[key] || settings[key] || fallback;
 
 function git(cwd, args) {
-  const result = spawnSync("git", args, { cwd, encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
+  const result = spawnSync("git", args, {
+    cwd,
+    encoding: "utf8",
+    stdio: ["ignore", "pipe", "ignore"],
+  });
   return result.status === 0 ? result.stdout.trim() || null : null;
 }
 
@@ -26,15 +35,30 @@ function git(cwd, args) {
 function remoteSlug(url) {
   if (!url || /^(file:|[/.~\\]|[a-z]:[\\/])/i.test(url)) return null;
   const cleaned = url.replace(/\/+$/, "").replace(/\.git$/i, "");
-  const tail = cleaned.match(/^[a-z][a-z0-9+.-]*:\/\/[^/]+\/(.+)$/i)?.[1] ?? cleaned.match(/^(?:[^/@\s]+@)?[^:/\s]+:(?!\/\/)(.+)$/)?.[1];
-  const segments = (tail || "").split("/").filter((segment) => segment && segment !== "_git");
+  const tail =
+    cleaned.match(/^[a-z][a-z0-9+.-]*:\/\/[^/]+\/(.+)$/i)?.[1] ??
+    cleaned.match(/^(?:[^/@\s]+@)?[^:/\s]+:(?!\/\/)(.+)$/)?.[1];
+  const segments = (tail || "")
+    .split("/")
+    .filter((segment) => segment && segment !== "_git");
   return segments.length ? segments.slice(-2).join("/") : null;
 }
 
 function markerRoot(dir) {
-  const stops = new Set([os.homedir(), os.tmpdir(), "/"].map((stop) => path.resolve(stop)));
-  for (let current = path.resolve(dir); !stops.has(current); current = path.dirname(current)) {
-    if ([".claude-mem-project", ".claude-mem.json"].some((marker) => fs.existsSync(path.join(current, marker)))) return current;
+  const stops = new Set(
+    [os.homedir(), os.tmpdir(), "/"].map((stop) => path.resolve(stop))
+  );
+  for (
+    let current = path.resolve(dir);
+    !stops.has(current);
+    current = path.dirname(current)
+  ) {
+    if (
+      [".claude-mem-project", ".claude-mem.json"].some((marker) =>
+        fs.existsSync(path.join(current, marker))
+      )
+    )
+      return current;
     if (path.dirname(current) === current) break;
   }
   return null;
@@ -45,33 +69,65 @@ export function projectNames(cwd, env, settings) {
   const root = git(cwd, ["rev-parse", "--show-toplevel"]);
   if (!root) return [path.basename(markerRoot(cwd) || cwd)];
   const names = [];
-  if (String(setting(env, settings, "CLAUDE_MEM_PROJECT_NAME_SOURCE", "path")).toLowerCase() === "git-remote") names.push(remoteSlug(git(root, ["remote", "get-url", "origin"])));
+  if (
+    String(
+      setting(env, settings, "CLAUDE_MEM_PROJECT_NAME_SOURCE", "path")
+    ).toLowerCase() === "git-remote"
+  )
+    names.push(remoteSlug(git(root, ["remote", "get-url", "origin"])));
   names.push(path.basename(root));
   // A worktree's memory can sit under the main checkout's name.
-  const common = git(cwd, ["rev-parse", "--path-format=absolute", "--git-common-dir"]);
-  if (common && path.basename(common) === ".git") names.push(path.basename(path.dirname(common)));
+  const common = git(cwd, [
+    "rev-parse",
+    "--path-format=absolute",
+    "--git-common-dir",
+  ]);
+  if (common && path.basename(common) === ".git")
+    names.push(path.basename(path.dirname(common)));
   return [...new Set(names.filter(Boolean))];
 }
 
 function render(observation) {
   const heading = `- ${String(observation.created_at).slice(0, 10)} · ${observation.type} · ${observation.title || "untitled"}`;
-  return [heading, observation.subtitle, observation.narrative].filter(Boolean).join("\n  ");
+  return [heading, observation.subtitle, observation.narrative]
+    .filter(Boolean)
+    .join("\n  ");
 }
 
 /** claude-mem's newest notes on the project `cwd` belongs to, or null when it has none or is not running. */
-export async function claudeMemNotes(cwd, { env = process.env, timeoutMs, maxChars = 20_000 } = {}) {
+export async function claudeMemNotes(
+  cwd,
+  { env = process.env, timeoutMs, maxChars = 20_000 } = {}
+) {
   const settings = cwd && readSettings(env);
   if (!settings) return null;
   const host = setting(env, settings, "CLAUDE_MEM_WORKER_HOST", "127.0.0.1");
-  const port = setting(env, settings, "CLAUDE_MEM_WORKER_PORT", String(37700 + ((process.getuid?.() ?? 77) % 100)));
+  const port = setting(
+    env,
+    settings,
+    "CLAUDE_MEM_WORKER_PORT",
+    String(37700 + ((process.getuid?.() ?? 77) % 100))
+  );
   const base = `http://${host.includes(":") ? `[${host}]` : host}:${port}`;
   const signal = AbortSignal.timeout(timeoutMs);
   try {
-    const known = new Set(((await (await fetch(`${base}/api/projects`, { signal })).json()).projects || []).map((name) => String(name).toLowerCase()));
-    const project = projectNames(cwd, env, settings).find((name) => known.has(name.toLowerCase()));
+    const known = new Set(
+      (
+        (await (await fetch(`${base}/api/projects`, { signal })).json())
+          .projects || []
+      ).map((name) => String(name).toLowerCase())
+    );
+    const project = projectNames(cwd, env, settings).find((name) =>
+      known.has(name.toLowerCase())
+    );
     if (!project) return null;
-    const limit = Number(setting(env, settings, "CLAUDE_MEM_CONTEXT_OBSERVATIONS", "50")) || 50;
-    const response = await fetch(`${base}/api/search?type=observations&format=json&project=${encodeURIComponent(project)}&limit=${limit}`, { signal });
+    const limit =
+      Number(setting(env, settings, "CLAUDE_MEM_CONTEXT_OBSERVATIONS", "50")) ||
+      50;
+    const response = await fetch(
+      `${base}/api/search?type=observations&format=json&project=${encodeURIComponent(project)}&limit=${limit}`,
+      { signal }
+    );
     if (!response.ok) return null;
     // Whole notes, newest first, as many as fit the task context's entry size; a note is never cut.
     const header = `Project "${project}", newest first:`;

package/lib/commands.mjs

@@ -4,14 +4,22 @@ import { spawnSync } from "node:child_process";
 
 export function resolveCommand(command, env = process.env) {
   const pathValue = env.PATH || "";
-  const extensions = process.platform === "win32"
-    ? (env.PATHEXT || ".EXE;.CMD;.BAT;.COM").split(";")
-    : [""];
+  const extensions =
+    process.platform === "win32"
+      ? (env.PATHEXT || ".EXE;.CMD;.BAT;.COM").split(";")
+      : [""];
   for (const dir of pathValue.split(path.delimiter).filter(Boolean)) {
     for (const extension of extensions) {
-      const candidate = path.join(dir, process.platform === "win32" ? command + extension.toLowerCase() : command);
+      const candidate = path.join(
+        dir,
+        process.platform === "win32"
+          ? command + extension.toLowerCase()
+          : command
+      );
       const alternate = path.join(dir, command + extension);
-      for (const file of candidate === alternate ? [candidate] : [candidate, alternate]) {
+      for (const file of candidate === alternate
+        ? [candidate]
+        : [candidate, alternate]) {
         try {
           fs.accessSync(file, fs.constants.X_OK);
           return file;