@sleds/cli 0.1.32 → 0.1.33

package/dist/sleds.mjs

@@ -5274,11 +5274,26 @@ var init_statusMap = __esm(() => {
     status: false,
     assignee: false,
     children: false,
-    comments: false
+    comments: false,
+    fields: false
   };
   WAY_SWITCHES = {
-    jira: { create: true, status: true, assignee: true, children: true, comments: true },
-    moving: { create: false, status: true, assignee: true, children: true, comments: true },
+    jira: {
+      create: true,
+      status: true,
+      assignee: true,
+      children: true,
+      comments: true,
+      fields: true
+    },
+    moving: {
+      create: false,
+      status: true,
+      assignee: true,
+      children: true,
+      comments: true,
+      fields: true
+    },
     readonly: { ...DEFAULT_WRITES }
   };
 });
@@ -6075,7 +6090,8 @@ function nodePrompt(o) {
     `- Work on branch \`${o.branch}\` from the latest \`origin/${base}\` (create it if this checkout is not on it). One branch, one PR.`,
     `- Open the PR against \`${base}\` with \`gh pr create\`${o.prTitle ? ` and title it exactly \`${o.prTitle}\` (the repo's CI checks it)` : ""}. Its body carries \`SLEDS-Node: ${claim.node}\` on the line under \`SLEDS-Source:\` when the pack names a source.`,
     "- Run the repo's own gate before you push. Never merge: a person (or the musher, when the attachment says so) does.",
-    "- Stay inside this node's scope. Work another node needs goes in the PR description, not in this branch."
+    "- Stay inside this node's scope. Work another node needs goes in the PR description, not in this branch.",
+    `- End every comment and reply you post on the PR with the line \`${ENGINE_MARK}\`, so the engine knows it is yours and not a reviewer's.`
   ];
   if (o.sledsTools && claim.runId)
     lines.push(`- If only a person can decide something, first search the record (the pack, the decision log in it, the specs and docs it names); if it is still open, call the SLEDS \`${toolName("run", o.engineMcp)}\` tool with \`{runId: "${claim.runId}", park: {question, options?, irreversible?}}\` — say in the question what you already read — and wait; do not guess. An answerer takes it first, then a person. Mark \`irreversible: true\` when acting on either answer could not be undone by reverting the PR (data deleted or migrated in place, a deploy, anything that leaves the org, spend, access, the brief). The engine leaves this session open for the answer.`, `- Before you finish, \`${toolName("land", o.engineMcp)}\` with \`{project: "${claim.item}", node: "${claim.node}", state: "pr", ref: <the PR url>, notes}\` — notes are what the next node should know${deviations}.`);
@@ -6332,9 +6348,15 @@ function eventPrompt(o) {
     return conflictPrompt({ ...o, event });
   return ciTwicePrompt({ ...o, event });
 }
-var DEFAULT_SEAT = "\x00default", MAX_SEAT_AT_ONCE = 16, DRAFT_NODE = "draft", BRANCH_FIELDS, PR_TITLE_FIELDS, ENGINE_MCP_SERVER = "sleds", RUN_TOKEN_ENV = "SLEDS_RUN_TOKEN", NO_RUN_TOKEN = "no run token from SLEDS: this session acts as the seat", toolName = (name, engineMcp) => engineMcp ? `mcp__${ENGINE_MCP_SERVER}__${name}` : name, APPROACH_SHAPE, AUTONOMY_SAYS, IRREVERSIBLE = "deleting or migrating data in place; deploying outside the playbook's window; anything that leaves the org (a message to a customer, a publish, a release, an email); spend past the band or the allowance; force-pushing or deleting a branch someone else holds; granting or removing access; changing the brief. When in doubt, it is irreversible.", oneLine = (s, max) => s.replace(/\s+/g, " ").trim().slice(0, max), VERDICT_SHAPE, codes = (xs) => xs.map((x) => `\`${x}\``).join(", "), RESOLVE_MAX_FILES = 8;
+var DEFAULT_SEAT = "\x00default", MAX_SEAT_AT_ONCE = 16, HOLD_DEFAULTS, DRAFT_NODE = "draft", BRANCH_FIELDS, PR_TITLE_FIELDS, ENGINE_MCP_SERVER = "sleds", RUN_TOKEN_ENV = "SLEDS_RUN_TOKEN", NO_RUN_TOKEN = "no run token from SLEDS: this session acts as the seat", toolName = (name, engineMcp) => engineMcp ? `mcp__${ENGINE_MCP_SERVER}__${name}` : name, ENGINE_MARK = "<!-- sleds:engine -->", APPROACH_SHAPE, AUTONOMY_SAYS, IRREVERSIBLE = "deleting or migrating data in place; deploying outside the playbook's window; anything that leaves the org (a message to a customer, a publish, a release, an email); spend past the band or the allowance; force-pushing or deleting a branch someone else holds; granting or removing access; changing the brief. When in doubt, it is irreversible.", oneLine = (s, max) => s.replace(/\s+/g, " ").trim().slice(0, max), VERDICT_SHAPE, codes = (xs) => xs.map((x) => `\`${x}\``).join(", "), RESOLVE_MAX_FILES = 8;
 var init_engine = __esm(() => {
   init_route();
+  HOLD_DEFAULTS = {
+    holdCpu: 85,
+    holdLoad: 1.5,
+    holdPressure: "warn",
+    holdUsage: 80
+  };
   BRANCH_FIELDS = ["key", "item", "node", "slug", "ticket"];
   PR_TITLE_FIELDS = ["key", "item", "node", "title", "ticket"];
   APPROACH_SHAPE = [
@@ -7369,9 +7391,12 @@ function parseRun(v) {
         url: v.pr.url,
         failing: Array.isArray(v.pr.failing) ? v.pr.failing.filter((x) => typeof x === "string") : [],
         feedback: typeof v.pr.feedback === "number" ? v.pr.feedback : 0,
+        ...typeof v.pr.authorFeedback === "number" ? { authorFeedback: v.pr.authorFeedback } : {},
+        ...typeof v.pr.selfNudges === "number" ? { selfNudges: v.pr.selfNudges } : {},
         ...v.pr.clear === true ? { clear: true } : {},
         ...typeof v.pr.ci === "string" ? { ci: v.pr.ci } : {},
         ...typeof v.pr.review === "string" ? { review: v.pr.review } : {},
+        ...typeof v.pr.posted === "string" ? { posted: v.pr.posted } : {},
         ...Array.isArray(v.pr.failedHeads) ? { failedHeads: v.pr.failedHeads.filter((x) => typeof x === "string") } : {},
         ...typeof v.pr.conflict === "string" ? { conflict: v.pr.conflict } : {},
         ...typeof v.pr.resolving === "string" ? { resolving: v.pr.resolving } : {},
@@ -7387,10 +7412,12 @@ function parseState(text) {
       return { runs: [] };
     const decisions = Array.isArray(raw.decisions) ? raw.decisions.map(parseDecision).filter((x) => x !== null) : [];
     const merged = Array.isArray(raw.merged) ? raw.merged.map(parseMerged).filter((x) => x !== null) : [];
+    const slowStarts = isObject8(raw.slowStarts) ? Object.fromEntries(Object.entries(raw.slowStarts).filter((e) => typeof e[1] === "number" && e[1] > 0)) : {};
     return {
       runs: raw.runs.map(parseRun).filter((r) => r !== null),
       ...decisions.length ? { decisions } : {},
-      ...merged.length ? { merged } : {}
+      ...merged.length ? { merged } : {},
+      ...Object.keys(slowStarts).length ? { slowStarts } : {}
     };
   } catch {
     return { runs: [] };
@@ -9258,6 +9285,8 @@ var defaultExec = (cmd, args, o) => {
   }
   return { status: r.status ?? 1, stdout: r.stdout ?? "", stderr: r.stderr ?? "" };
 };
+var isTimeout = (msg) => /\btime[ds]?[\s_-]?out\b|ETIMEDOUT/i.test(msg);
+var UNWATCHED = "the agent may be running in it unwatched";
 var shq = (s) => `'${s.replace(/'/g, `'\\''`)}'`;
 var agentShellCommand = (o) => {
   const prompt = `"$(cat ${shq(o.promptFile)})"`;
@@ -9350,6 +9379,18 @@ var cmuxIdForRef = (stdout, ref, name) => {
   }
   return null;
 };
+var cmuxIdForName = (stdout, name) => {
+  let found = null;
+  for (const words of namedLines(stdout, name)) {
+    const id = words[1]?.match(UUID)?.[0];
+    if (!id || found)
+      return null;
+    found = id.toLowerCase();
+  }
+  return found;
+};
+var namedLines = (stdout, name) => stdout.split(`
+`).map((line) => line.trim().replace(/^\*\s*/, "").split(/\s+/)).filter((words) => words.slice(2).includes(name));
 var cmuxWorkspaceListed = (status, stdout, id) => status !== 0 || stdout.toLowerCase().includes(id.toLowerCase());
 var cmuxGroupName = (g) => g.title.trim() ? `${g.key} · ${g.title.trim()}` : g.key;
 var str7 = (v) => typeof v === "string" && v ? v : undefined;
@@ -9410,6 +9451,28 @@ var cmuxProbeReason = (r) => {
   const why = line.replace(/^(cmux:\s*)?(error:\s*)?/i, "").trim();
   return `cmux: ${why || `exit ${r.status}`}`;
 };
+var cmuxAuthRefused = (r) => r.status !== 0 && /\b(auth_required|unauthorized|authentication required|invalid password)\b/i.test(`${r.stderr}
+${r.stdout}`);
+var CMUX_SETTLE_MS = 300;
+var CMUX_ENTER_RETRIES = 2;
+var PROMPT_LINE = /^[\s│|]*[>❯›]\s?(.*)$/;
+var PASTED = /\[Pasted text\b/i;
+var cmuxInputPending = (screen, text) => {
+  const squash = (s) => s.replace(/\s+/g, " ").trim();
+  const head = squash(text.split(`
+`).find((l) => l.trim()) ?? "").slice(0, 24);
+  let input = null;
+  for (const line of screen.split(`
+`)) {
+    const m = line.match(PROMPT_LINE);
+    if (m)
+      input = squash((m[1] ?? "").replace(/│\s*$/, ""));
+  }
+  if (!input || !head)
+    return false;
+  return PASTED.test(input) || input.startsWith(head) || input.length >= 8 && head.startsWith(input);
+};
+var sleepSync = (ms) => Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, ms);
 var CMUX_LANE = {
   working: "working",
   parked: "needs-attention",
@@ -9419,18 +9482,26 @@ var CMUX_LANE = {
 var launcherPath = (promptFile) => logPathFor(promptFile, "cmux.sh");
 var exitPath = (promptFile) => logPathFor(promptFile, "cmux.exit");
 function cmuxRunner(exec, deps = {}) {
+  const sleep = deps.sleep ?? sleepSync;
   const writeFile = deps.writeFile ?? ((p, t) => writeFileSync10(p, t, { mode: 384 }));
   const exists = deps.exists ?? existsSync11;
   const remove = deps.remove ?? ((p) => rmSync2(p, { force: true }));
   const bin = resolveCmuxBin(exists);
   const password = deps.password ?? (() => readCmuxPassword(exec));
-  let env = null;
+  let pw;
+  const run = (args) => pw ? exec(bin, args, { env: { CMUX_SOCKET_PASSWORD: pw } }) : exec(bin, args);
   const cmux = (args) => {
-    if (env === null) {
-      const pw = password();
-      env = pw ? { CMUX_SOCKET_PASSWORD: pw } : undefined;
-    }
-    return env ? exec(bin, args, { env }) : exec(bin, args);
+    const fresh = pw === undefined;
+    if (fresh)
+      pw = password();
+    const r = run(args);
+    if (fresh || !cmuxAuthRefused(r))
+      return r;
+    const now = password();
+    if (now === pw)
+      return r;
+    pw = now;
+    return pw ? run(args) : r;
   };
   const failed = (r) => r.stderr.trim() || r.stdout.trim() || `exit ${r.status}`;
   return {
@@ -9467,19 +9538,27 @@ function cmuxRunner(exec, deps = {}) {
         joined = undefined;
         r = cmux(args);
       }
-      if (r.status !== 0)
-        throw new Error(`cmux new-workspace ${o.name} failed: ${failed(r)}`);
+      if (r.status !== 0) {
+        const why = failed(r);
+        const l = cmux(["--id-format", "both", "list-workspaces"]);
+        const stray = l.status === 0 ? cmuxIdForName(l.stdout, o.name) : null;
+        const left = l.status !== 0 || (stray ? cmux(["close-workspace", "--workspace", stray]).status !== 0 : namedLines(l.stdout, o.name).length > 0);
+        throw new Error(`cmux new-workspace ${o.name} failed: ${why}${left ? `; ${UNWATCHED}` : ""}`);
+      }
       let id = parseCmuxWorkspace(r.stdout);
       const ref = id ? null : parseCmuxRef(r.stdout);
+      let closed = false;
       if (ref) {
         const l = cmux(["--id-format", "both", "list-workspaces"]);
-        if (l.status === 0)
+        if (l.status === 0) {
           id = cmuxIdForRef(l.stdout, ref, o.name);
-        if (!id)
-          cmux(["close-workspace", "--workspace", ref]);
+          const stray = id ? null : cmuxIdForName(l.stdout, o.name);
+          if (stray)
+            closed = cmux(["close-workspace", "--workspace", stray]).status === 0;
+        }
       }
       if (!id) {
-        throw new Error(ref ? `cmux new-workspace ${o.name} gave ${ref} but no workspace id for it; that workspace was closed` : `cmux new-workspace ${o.name} returned no workspace (${r.stdout.trim() || "nothing"}); the agent may be running in it unwatched`);
+        throw new Error(ref ? `cmux new-workspace ${o.name} gave ${ref} but no workspace id for it; ${closed ? "that workspace was closed" : UNWATCHED}` : `cmux new-workspace ${o.name} returned no workspace (${r.stdout.trim() || "nothing"}); ${UNWATCHED}`);
       }
       if (group && listed && !joined) {
         const c = cmux(["workspace-group", "create", "--name", group, "--from", id, "--json"]);
@@ -9505,9 +9584,26 @@ function cmuxRunner(exec, deps = {}) {
       const r = cmux(["send", "--workspace", h.id, "--", text]);
       if (r.status !== 0)
         throw new Error(`cmux send ${h.id} failed: ${failed(r)}`);
-      const k = cmux(["send-key", "--workspace", h.id, "enter"]);
-      if (k.status !== 0)
-        throw new Error(`cmux send-key ${h.id} failed: ${failed(k)}`);
+      const enter = () => {
+        const k = cmux(["send-key", "--workspace", h.id, "enter"]);
+        if (k.status !== 0)
+          throw new Error(`cmux send-key ${h.id} failed: ${failed(k)}`);
+      };
+      sleep(CMUX_SETTLE_MS);
+      enter();
+      const pending = () => {
+        sleep(CMUX_SETTLE_MS);
+        const s = cmux(["read-screen", "--workspace", h.id, "--lines", "20"]);
+        return s.status === 0 && cmuxInputPending(s.stdout, text);
+      };
+      let again = 0;
+      while (again < CMUX_ENTER_RETRIES && pending()) {
+        enter();
+        again++;
+      }
+      if (again === 0)
+        return;
+      return pending() ? `the Enter was swallowed; pressed it ${again} more times and the text is still in the input` : `the Enter was swallowed; pressed it again${again > 1 ? ` (${again}×)` : ""}`;
     },
     stop(h) {
       cmux(["close-workspace", "--workspace", h.id]);
@@ -9788,6 +9884,7 @@ function noteEvent(line) {
 }
 
 // src/engine/gh.ts
+init_src();
 var isGhError = (r) => r !== null && ("error" in r);
 function prForBranch(exec, repoPath, branch) {
   try {
@@ -9884,6 +9981,7 @@ function parsePrView(stdout) {
     pending,
     reviewDecision: typeof o.reviewDecision === "string" ? o.reviewDecision : "",
     feedback: 0,
+    authorFeedback: 0,
     ...typeof o.headRefOid === "string" && o.headRefOid ? { head: o.headRefOid } : {},
     ...typeof mergeCommit === "string" && mergeCommit ? { mergeCommit } : {}
   };
@@ -9892,22 +9990,39 @@ function prRef(url) {
   const m = /github\.com\/([^/]+\/[^/]+)\/pull\/(\d+)/.exec(url);
   return m?.[1] && m[2] ? { repo: m[1], number: Number(m[2]) } : null;
 }
+function countFeedback(stdout, author) {
+  const n = { all: 0, author: 0 };
+  for (const line of stdout.split(`
+`)) {
+    const [login = "", marked, hasBody] = line.trim().split("\t");
+    if (!login || marked === "true")
+      continue;
+    if (login === author && hasBody !== "true")
+      continue;
+    n.all++;
+    if (login === author)
+      n.author++;
+  }
+  return n;
+}
+var FEEDBACK_JQ = `.[] | select(.user.type != "Bot") | [.user.login, ((.body // "") | contains("${ENGINE_MARK}") | tostring), ((.body // "") | length > 0 | tostring)] | @tsv`;
 function prFeedback(exec, repoPath, url, author) {
   const ref = prRef(url);
   if (!ref)
     return null;
-  let total = 0;
+  const total = { all: 0, author: 0 };
   for (const path of [
     `repos/${ref.repo}/issues/${ref.number}/comments`,
     `repos/${ref.repo}/pulls/${ref.number}/reviews`,
     `repos/${ref.repo}/pulls/${ref.number}/comments`
   ]) {
     try {
-      const r = exec("gh", ["api", path, "--paginate", "--jq", '.[] | select(.user.type != "Bot") | .user.login'], { cwd: repoPath });
+      const r = exec("gh", ["api", path, "--paginate", "--jq", FEEDBACK_JQ], { cwd: repoPath });
       if (r.status !== 0)
         return null;
-      total += r.stdout.split(`
-`).filter((l) => l.trim() && l.trim() !== author).length;
+      const n = countFeedback(r.stdout, author);
+      total.all += n.all;
+      total.author += n.author;
     } catch {
       return null;
     }
@@ -9927,7 +10042,7 @@ function prView(exec, repoPath, url) {
     if (!view || view.state !== "OPEN")
       return view;
     const feedback = prFeedback(exec, repoPath, url, view.author);
-    return feedback === null ? view : { ...view, feedback };
+    return feedback === null ? view : { ...view, feedback: feedback.all, authorFeedback: feedback.author };
   } catch {
     return null;
   }
@@ -10400,6 +10515,17 @@ class SeatTokenError extends Error {
     this.name = "SeatTokenError";
   }
 }
+
+class ReportRefusedError extends Error {
+  status;
+  final;
+  constructor(status, message, final = false) {
+    super(message);
+    this.status = status;
+    this.final = final;
+    this.name = "ReportRefusedError";
+  }
+}
 async function readOrSay(client, path, what) {
   if (!client.getFull) {
     const r = await client.get(path);
@@ -10475,7 +10601,11 @@ function sledItems(r, armedOnly, engine) {
   }
   return out;
 }
-var HOLD_DEFAULTS = { holdCpu: 85, holdLoad: 1.5, holdPressure: "warn" };
+var HOLD_DEFAULTS2 = {
+  holdCpu: HOLD_DEFAULTS.holdCpu,
+  holdLoad: HOLD_DEFAULTS.holdLoad,
+  holdPressure: HOLD_DEFAULTS.holdPressure
+};
 var PROFILE_TEXT = [
   "runner",
   "launcher",
@@ -10498,7 +10628,7 @@ function profileOf(r) {
   const p = r.profile;
   const out = {
     pickup: p.pickup === "auto" ? "auto" : "approve",
-    ...HOLD_DEFAULTS,
+    ...HOLD_DEFAULTS2,
     holdOff: p.holdOff === true
   };
   for (const k of PROFILE_TEXT) {
@@ -10795,8 +10925,12 @@ function sledsSource(client, o) {
         throw new Error(`${UNREACHABLE2} — ${claim.item}/${claim.node} ${outcome.kind} not recorded`);
       if (r.status === 409 && path === "/proxy/run")
         return;
-      if (r.status >= 300)
-        throw new Error(`${claim.item}/${claim.node} ${outcome.kind}: ${serverMessage(r)}`);
+      if (r.status < 300)
+        return;
+      if (r.status === 401 || r.status === 403 || r.status === 404 && !r.json)
+        throw new SeatTokenError(r.status);
+      const final = r.json !== null && (r.status === 404 || r.status === 400 && r.json.badState === true);
+      throw new ReportRefusedError(r.status, `${claim.item}/${claim.node} ${outcome.kind}: ${serverMessage(r)}`, final);
     },
     async openDecision(item, node) {
       const route = await routeOf(item);
@@ -10879,6 +11013,14 @@ function sledsSource(client, o) {
         ] : [];
       });
     },
+    async prRead(item, node, read) {
+      try {
+        const r = await client.postFull("/proxy/prread", { project: item, node, ...read });
+        return !!r && r.status >= 200 && r.status < 300;
+      } catch {
+        return false;
+      }
+    },
     async answerDecision(a, engine, answer) {
       const r = await client.postFull("/proxy/run", { runId: a.runId, engine, answer });
       if (!r)
@@ -10997,6 +11139,13 @@ function removeBranch(exec, repoPath, branch, head) {
 }
 
 // src/engine/loop.ts
+function profileOnce(read) {
+  let p;
+  return () => {
+    p ??= read ? read().catch(() => null) : Promise.resolve(null);
+    return p;
+  };
+}
 function ciLine(v) {
   if (v.failing.length)
     return `failing: ${v.failing.join(", ")}`;
@@ -11090,6 +11239,7 @@ async function tick(deps) {
   } finally {
     if (d !== deps) {
       deps.runnerHold = d.runnerHold;
+      deps.startBackoff = d.startBackoff;
       deps.atOnce = d.atOnce;
     }
   }
@@ -11115,6 +11265,7 @@ async function tick(deps) {
   return report;
 }
 async function tickOnce(d) {
+  d.tickProfile = profileOnce(d.readProfile);
   const state = d.readState();
   const seat = seatNow(d);
   if (seat.changed) {
@@ -11130,6 +11281,8 @@ async function tickOnce(d) {
   const decided = [];
   const save = () => d.writeState(state);
   probeRunner(d);
+  if (d.startBackoff && --d.startBackoff.checks <= 0)
+    d.startBackoff = undefined;
   const keep = [];
   for (const run of state.runs) {
     const claim = claimOf(run);
@@ -11284,7 +11437,7 @@ async function tickOnce(d) {
   const answers = answersOf(d.source);
   if (answers) {
     for (const run of state.runs)
-      if (run.parked && !run.parked.answerer && !d.runnerHold) {
+      if (run.parked && !run.parked.answerer && !startsHeld(d)) {
         await startAnswerer(d, answers, run);
         save();
       }
@@ -11301,7 +11454,7 @@ async function tickOnce(d) {
   const items = await d.source.read();
   const takeNone = seat.gone;
   for (const it of items)
-    if (it.draft && !takeNone && !d.runnerHold)
+    if (it.draft && !takeNone && !startsHeld(d))
       await startDraft(d, it.draft.claim, it.draft.why, state, started, finished);
   const live = new Map(items.map((i) => [i.item, i]));
   const unheld = state.runs.filter((r) => r.parked && (live.get(r.item)?.landed ?? true));
@@ -11319,9 +11472,16 @@ async function tickOnce(d) {
     state.decisions = (state.decisions ?? []).filter((x) => alive(x) || x.answerer);
     save();
   }
+  const counted = Object.keys(state.slowStarts ?? {});
+  const gone = counted.filter((k) => !alive({ item: k.slice(0, k.lastIndexOf("/")) }));
+  if (gone.length) {
+    for (const k of gone)
+      forgetSlowStarts(state, k);
+    save();
+  }
   await rereadAtOnce(d);
   const cap = await machineCap(d);
-  const atOnce = d.runnerHold ? 0 : cap;
+  const atOnce = startsHeld(d) ? 0 : cap;
   const takes = planTakes({
     items,
     tracked: [
@@ -11349,7 +11509,7 @@ async function tickOnce(d) {
     return p;
   };
   for (const t of takeNone ? [] : takes) {
-    if (d.runnerHold)
+    if (startsHeld(d))
       break;
     const at = ref(t);
     let claim;
@@ -11455,10 +11615,26 @@ async function tickOnce(d) {
         await report({ kind: "release", why: `runner unavailable: ${down}` });
         continue;
       }
+      const slow = slowStart(d, message(e));
+      const inRow = (state.slowStarts?.[at] ?? 0) + 1;
+      if (slow && inRow <= SLOW_START_RELEASES) {
+        d.log(`${at}: released — ${slow}: ${message(e)}; backing off one check`);
+        d.startBackoff = { why: slow, checks: 2 };
+        state.slowStarts = { ...state.slowStarts, [at]: inRow };
+        save();
+        await report({ kind: "release", why: slow });
+        continue;
+      }
+      forgetSlowStarts(state, at);
+      save();
+      if (slow)
+        d.log(`${at}: ${slow} ${inRow} times in a row — failed, not released again`);
       d.log(`${at}: could not start: ${message(e)}`);
       await report({ kind: "fail", log: `The engine could not start the session: ${message(e)}` });
       continue;
     }
+    if (forgetSlowStarts(state, at))
+      save();
     const watch = safeWatch(d.runner, handle);
     const run = {
       item: claim.item,
@@ -11625,12 +11801,34 @@ function probeRunner(d) {
   }
   return why;
 }
+function profileOf2(d) {
+  return d.tickProfile ?? profileOnce(d.readProfile);
+}
+function forgetSlowStarts(state, at) {
+  if (!state.slowStarts || !(at in state.slowStarts))
+    return false;
+  const { [at]: _, ...rest } = state.slowStarts;
+  state.slowStarts = Object.keys(rest).length ? rest : undefined;
+  return true;
+}
+var startsHeld = (d) => Boolean(d.runnerHold || d.startBackoff);
+var SLOW_START_RELEASES = 3;
+function slowStart(d, msg) {
+  if (msg.includes(UNWATCHED))
+    return null;
+  if (isTimeout(msg))
+    return "the runner start timed out";
+  const hold = d.machine?.hold;
+  if (hold)
+    return `the runner start failed with the machine busy (${hold.signals.join(", ")})`;
+  return null;
+}
 async function rereadAtOnce(d) {
   if (!d.rereadAtOnce)
     return;
   let next = null;
   try {
-    next = await d.rereadAtOnce();
+    next = await d.rereadAtOnce(profileOf2(d));
   } catch {}
   if (next === null || next === d.atOnce)
     return;
@@ -11642,7 +11840,7 @@ async function rereadMachine(w, d) {
     return;
   let next = null;
   try {
-    next = await w.reread();
+    next = await w.reread(profileOf2(d));
   } catch {}
   if (!next)
     return;
@@ -11700,7 +11898,9 @@ async function resume(d, held, claim, state, started) {
       d.log(`${at}: answered, but the parked session is gone — starting a new one`);
       return false;
     }
-    runner.send(held.handle, claim.answer);
+    const note = runner.send(held.handle, claim.answer);
+    if (note)
+      d.log(`${at}: answered — ${note}`);
   } catch (e) {
     d.log(`${at}: answered, but the session could not be told (${message(e)}) — starting a new one`);
     stopOpen(d, held, runner);
@@ -11817,7 +12017,7 @@ async function roleLaunchFor(d, at, role) {
   };
   if (d.rereadRoles) {
     try {
-      const now = await d.rereadRoles();
+      const now = await d.rereadRoles(profileOf2(d));
       if (now !== null)
         roles = now;
     } catch {}
@@ -12175,8 +12375,9 @@ function endDraft(d, run, runner) {
 function ciNudge(url, failing) {
   return `CI failed on your PR (${failing.join(", ")}): ${url}. Read the failing logs (gh pr checks ${url}), fix the cause, run the repo's gate, and push to the same branch.`;
 }
+var SELF_NUDGES = 2;
 function reviewNudge(url, count) {
-  return `New review feedback on your PR (${count} ${count === 1 ? "comment" : "comments"} from people so far): ${url}. Read all of it (gh pr view ${url} --comments, and the inline threads), answer each comment on its own thread with the fix or a reasoned reply, push, and leave the threads for the reviewer to resolve.`;
+  return `New review feedback on your PR (${count} ${count === 1 ? "comment" : "comments"} from people so far): ${url}. Read all of it (gh pr view ${url} --comments, and the inline threads), answer each comment on its own thread with the fix or a reasoned reply, push, and leave the threads for the reviewer to resolve. End every comment and reply you post on the PR with the line ${ENGINE_MARK} so the engine knows it is yours and not the reviewer's.`;
 }
 async function follow(d, state, run, claim, runner, finished, attention, changes, decided) {
   const pr = run.pr;
@@ -12213,8 +12414,8 @@ async function follow(d, state, run, claim, runner, finished, attention, changes
     try {
       alive = runner.status(run.handle) === "running";
       if (alive) {
-        runner.send(run.handle, text);
-        d.log(`${ref(run)}: ${what} — told the session`);
+        const note = runner.send(run.handle, text);
+        d.log(`${ref(run)}: ${what} — told the session${note ? ` (${note})` : ""}`);
         return;
       }
     } catch {}
@@ -12231,16 +12432,38 @@ async function follow(d, state, run, claim, runner, finished, attention, changes
     pr.resolving = undefined;
     pr.conflicted = undefined;
   }
-  if (view.feedback > pr.feedback)
-    tell(reviewNudge(pr.url, view.feedback), `${view.feedback - pr.feedback} new review comment(s)`);
+  if (view.feedback > pr.feedback) {
+    const what = `${view.feedback - pr.feedback} new review comment(s)`;
+    const authorBefore = pr.authorFeedback ?? 0;
+    const othersGrew = view.feedback - view.authorFeedback > pr.feedback - authorBefore;
+    const selfNudges = othersGrew ? 0 : (pr.selfNudges ?? 0) + 1;
+    if (selfNudges > SELF_NUDGES) {
+      d.log(`${ref(run)}: ${what}, only the PR author's own after ${SELF_NUDGES} nudges in a row — needs a person (the session may be answering itself): ${pr.url}`);
+      attention.push({
+        ...at,
+        what: `${what} from the PR's author only, after ${SELF_NUDGES} nudges: is the session answering itself?`
+      });
+    } else {
+      tell(reviewNudge(pr.url, view.feedback), what);
+      pr.selfNudges = selfNudges || undefined;
+    }
+  }
   pr.failing = view.failing;
   pr.feedback = Math.max(pr.feedback, view.feedback);
+  pr.authorFeedback = Math.max(pr.authorFeedback ?? 0, view.authorFeedback) || undefined;
   const clear = view.failing.length === 0 && view.pending === 0 && view.reviewDecision === "APPROVED";
   if (clear && !pr.clear) {
     d.log(`${ref(run)}: ${pr.url} is green and approved — ready for your merge call`);
     attention.push({ ...at, what: "green and approved: ready for a merge" });
   }
   pr.clear = clear;
+  const prRead = d.source.prRead;
+  const posted = `${ci}|${review}|${clear}`;
+  if (prRead && posted !== pr.posted) {
+    const read = { ci, ...review ? { review } : {}, clear };
+    if (await prRead.call(d.source, run.item, run.node, read))
+      pr.posted = posted;
+  }
   return true;
 }
 function rippleRunnerWorktree(d, run, runner, head) {
@@ -12270,15 +12493,21 @@ async function ripple(d, state, run, claim, runner, url, finished, decided, head
     ...(state.merged ?? []).filter((m) => !(m.item === run.item && m.node === run.node)),
     { item: run.item, node: run.node, url, repoPath: run.repoPath }
   ];
+  let landed = true;
   try {
     await d.source.report(claim, outcome, d.engine);
   } catch (e) {
-    d.log(`${ref(run)}: merged (${url}), but the report failed — retrying next tick: ${message(e)}`);
-    return false;
+    if (!(e instanceof ReportRefusedError && e.final)) {
+      d.log(`${ref(run)}: merged (${url}), but the report failed — retrying next tick: ${message(e)}`);
+      return false;
+    }
+    d.log(`${ref(run)}: merged (${url}), but SLEDS refused the merged report (HTTP ${e.status}) — not retrying: ${message(e)}`);
+    landed = false;
   }
   finished.push({ item: run.item, node: run.node, outcome });
   setLane(d, runner, run.handle, "done");
-  await openLanded(d, state, run, url, mergeCommit, decided);
+  if (landed)
+    await openLanded(d, state, run, url, mergeCommit, decided);
   try {
     if (runner.status(run.handle) === "running")
       runner.stop(run.handle);
@@ -12333,7 +12562,7 @@ function eventAnswerNudge(e, answer) {
 
 ${answer}
 
-On your own branch, by a merge commit — never a rebase that needs a force-push — run the repo's gate, push, and comment on the PR how each file was resolved.`;
+On your own branch, by a merge commit — never a rebase that needs a force-push — run the repo's gate, push, and comment on the PR how each file was resolved, ending the comment with the line ${ENGINE_MARK}.`;
   if (e.kind === "conflict")
     return `The decider on your PR's conflict with ${e.sibling} (${e.pr}):
 
@@ -12563,9 +12792,13 @@ function deliver(d, state, dec, answer, attention, decided) {
   const url = dec.event.pr;
   if (kind === "conflict")
     conflictStep(run, "decidedAt", d.now());
+  let status;
   try {
-    if (run && runner && runner.status(run.handle) === "running") {
-      runner.send(run.handle, eventAnswerNudge(dec.event, answer));
+    status = run && runner ? runner.status(run.handle) : undefined;
+    if (run && runner && status === "running") {
+      const note = runner.send(run.handle, eventAnswerNudge(dec.event, answer));
+      if (note)
+        d.log(`${ref(run)}: ${KIND_SAYS[kind]} answered — ${note}`);
       if (run.pr && baseConflict(dec.event))
         run.pr.resolving = dec.event.baseSha;
       if (kind === "conflict")
@@ -12581,7 +12814,8 @@ function deliver(d, state, dec, answer, attention, decided) {
     }
   } catch {}
   const event = dec.event;
-  if (run && baseConflict(event) && startResolver(d, run, event, answer)) {
+  const gone = status !== undefined && status !== "running";
+  if (run && gone && baseConflict(event) && startResolver(d, run, event, answer)) {
     conflictStep(run, "fixingAt", d.now());
     decided.push({
       item: dec.item,
@@ -12670,6 +12904,9 @@ function startResolver(d, run, e, answer) {
   if (!runner || runner.makesWorktree || run.handle.cwd === run.repoPath)
     return false;
   const name = sessionName(run.item, `${run.node}-resolve`);
+  try {
+    runner.stop(run.handle);
+  } catch {}
   try {
     const launch = nodeLaunchFor(d, ref(run));
     run.handle = runner.start({
@@ -12680,7 +12917,9 @@ function startResolver(d, run, e, answer) {
       base: e.base,
       promptFile: d.writePrompt(name, eventAnswerNudge(e, answer)),
       permissionMode: d.permissionMode,
-      ...launchOptions(launch)
+      ...launchOptions(launch),
+      ...run.mcpConfig ? { mcpConfig: run.mcpConfig, ...d.strictMcp ? { strictMcp: true } : {} } : {},
+      ...run.group ? { group: run.group } : {}
     });
   } catch (err) {
     d.log(`${ref(run)}: could not start a resolver session: ${message(err)}`);
@@ -12834,7 +13073,7 @@ async function settleEvent(d, events, state, dec, attention, decided) {
   if (!dec.answerer && baseConflict(dec.event) && events.answerDecision)
     return answerBaseConflict(d, events, state, dec, attention, decided);
   if (!dec.answerer)
-    return d.runnerHold ? true : startEventAnswerer(d, events, state, dec, decided);
+    return startsHeld(d) ? true : startEventAnswerer(d, events, state, dec, decided);
   const a = dec.answerer;
   const runner = runnerFor(d, a.handle);
   if (!runner) {
@@ -13542,17 +13781,17 @@ function flagsFromProfile(flags, given, profile, home) {
   }
   if (profile.holdCpu !== undefined && !has("hold-cpu")) {
     out.holdCpu = profile.holdCpu;
-    if (profile.holdCpu !== HOLD_DEFAULTS.holdCpu)
+    if (profile.holdCpu !== HOLD_DEFAULTS2.holdCpu)
       used.push(`hold at cpu ${profile.holdCpu}%`);
   }
   if (profile.holdLoad !== undefined && !has("hold-load")) {
     out.holdLoad = profile.holdLoad;
-    if (profile.holdLoad !== HOLD_DEFAULTS.holdLoad)
+    if (profile.holdLoad !== HOLD_DEFAULTS2.holdLoad)
       used.push(`hold at load ${profile.holdLoad}× cores`);
   }
   if (profile.holdPressure && !has("hold-pressure")) {
     out.holdPressure = profile.holdPressure;
-    if (profile.holdPressure !== HOLD_DEFAULTS.holdPressure)
+    if (profile.holdPressure !== HOLD_DEFAULTS2.holdPressure)
       used.push(`hold at memory pressure ${profile.holdPressure}`);
   }
   if (profile.holdOff !== undefined && !has("hold")) {
@@ -13565,10 +13804,10 @@ function flagsFromProfile(flags, given, profile, home) {
 function machineSettings(flags) {
   return {
     thresholds: {
-      cpuPct: flags.holdCpu ?? HOLD_DEFAULTS.holdCpu,
-      loadPerCore: flags.holdLoad ?? HOLD_DEFAULTS.holdLoad
+      cpuPct: flags.holdCpu ?? HOLD_DEFAULTS2.holdCpu,
+      loadPerCore: flags.holdLoad ?? HOLD_DEFAULTS2.holdLoad
     },
-    pressure: flags.holdPressure ?? HOLD_DEFAULTS.holdPressure,
+    pressure: flags.holdPressure ?? HOLD_DEFAULTS2.holdPressure,
     off: flags.holdOff === true
   };
 }
@@ -15184,17 +15423,17 @@ function flagsFromProfile2(flags, given, profile, home) {
   }
   if (profile.holdCpu !== undefined && !has("hold-cpu")) {
     out.holdCpu = profile.holdCpu;
-    if (profile.holdCpu !== HOLD_DEFAULTS.holdCpu)
+    if (profile.holdCpu !== HOLD_DEFAULTS2.holdCpu)
       used.push(`hold at cpu ${profile.holdCpu}%`);
   }
   if (profile.holdLoad !== undefined && !has("hold-load")) {
     out.holdLoad = profile.holdLoad;
-    if (profile.holdLoad !== HOLD_DEFAULTS.holdLoad)
+    if (profile.holdLoad !== HOLD_DEFAULTS2.holdLoad)
       used.push(`hold at load ${profile.holdLoad}× cores`);
   }
   if (profile.holdPressure && !has("hold-pressure")) {
     out.holdPressure = profile.holdPressure;
-    if (profile.holdPressure !== HOLD_DEFAULTS.holdPressure)
+    if (profile.holdPressure !== HOLD_DEFAULTS2.holdPressure)
       used.push(`hold at memory pressure ${profile.holdPressure}`);
   }
   if (profile.holdOff !== undefined && !has("hold")) {
@@ -15207,15 +15446,15 @@ function flagsFromProfile2(flags, given, profile, home) {
 function machineSettings2(flags) {
   return {
     thresholds: {
-      cpuPct: flags.holdCpu ?? HOLD_DEFAULTS.holdCpu,
-      loadPerCore: flags.holdLoad ?? HOLD_DEFAULTS.holdLoad
+      cpuPct: flags.holdCpu ?? HOLD_DEFAULTS2.holdCpu,
+      loadPerCore: flags.holdLoad ?? HOLD_DEFAULTS2.holdLoad
     },
-    pressure: flags.holdPressure ?? HOLD_DEFAULTS.holdPressure,
+    pressure: flags.holdPressure ?? HOLD_DEFAULTS2.holdPressure,
     off: flags.holdOff === true
   };
 }
-async function rereadMachine2(client, flags, given) {
-  const profile = await readProfile(client);
+async function rereadMachine2(read, flags, given) {
+  const profile = await read();
   return profile ? machineSettings2(flagsFromProfile2(flags, given, profile).flags) : null;
 }
 var EFFORT_SAYS2 = `${EFFORTS.slice(0, -1).join(", ")} or ${EFFORTS.at(-1)}`;
@@ -15230,8 +15469,8 @@ var ROLE_KEYS = [
   ["drafter-model", "drafterModel"],
   ["drafter-effort", "drafterEffort"]
 ];
-async function rereadRoles(client, flags, given) {
-  const profile = ROLE_KEYS.every(([flag]) => given.includes(flag)) ? undefined : await readProfile(client);
+async function rereadRoles(read, flags, given) {
+  const profile = ROLE_KEYS.every(([flag]) => given.includes(flag)) ? undefined : await read();
   if (profile === null)
     return null;
   const out = {};
@@ -15243,10 +15482,10 @@ async function rereadRoles(client, flags, given) {
   }
   return out;
 }
-async function rereadAtOnce2(client, flags, given) {
+async function rereadAtOnce2(read, flags, given) {
   if (given.includes("at-once"))
     return flags.atOnce;
-  const profile = await readProfile(client);
+  const profile = await read();
   return profile ? profile.atOnce ?? 1 : null;
 }
 var PROFILE_RETRY_MS2 = [2000, 5000];
@@ -15729,16 +15968,18 @@ var engineCmd = async (o) => {
     });
   }
   const deps = engineDeps2(o, flags, source, runner, `engine:${flags.name}`);
-  if (client && deps.machine) {
+  if (client) {
     const c = client;
+    deps.readProfile = () => readProfile(c);
+  }
+  if (client && deps.machine) {
     const started = flags;
-    deps.machine.reread = () => rereadMachine2(c, started, parsed.given);
+    deps.machine.reread = (read) => rereadMachine2(read, started, parsed.given);
   }
   if (client) {
-    const c = client;
     const started = flags;
-    deps.rereadRoles = () => rereadRoles(c, started, parsed.given);
-    deps.rereadAtOnce = () => rereadAtOnce2(c, started, parsed.given);
+    deps.rereadRoles = (read) => rereadRoles(read, started, parsed.given);
+    deps.rereadAtOnce = (read) => rereadAtOnce2(read, started, parsed.given);
   }
   log(engineHeader2(terminalOf2(o).c, deps.engine, (source.name === "jira" ? `Jira epic ${flags.jira}` : pickup === "auto" ? "your sled, Pickup: Auto (it arms the next ready item)" : "the routes armed on your sled") + (flags.org ? ` · org ${flags.org}` : "") + (flags.seat ? ` · as ${flags.seat}` : ""), runner.name, flags));
   if (flags.strictMcp && flags.agent)
@@ -16878,8 +17119,8 @@ function withEstimated(rows, a) {
 function withActual(rows, a) {
   if (a.state === "started")
     return rows;
-  const key = sourceKey(a.source);
-  return rows.filter((r) => r.key !== key);
+  const key = a.source ? sourceKey(a.source) : a.key;
+  return key ? rows.filter((r) => r.key !== key) : rows;
 }
 function owedBySession(rows, sessionId) {
   return rows.filter((r) => r.sessionId === sessionId && r.askedAt === undefined);
@@ -16934,11 +17175,11 @@ async function usageTool2(deps) {
 }
 var isObj2 = (v) => typeof v === "object" && v !== null && !Array.isArray(v);
 function trackingClient(client, home, now) {
-  const after = (path, body, status) => {
-    if (status < 200 || status >= 300 || !isObj2(body.source))
+  const after = (path, body, status, json) => {
+    if (status < 200 || status >= 300)
       return;
     try {
-      if (path === "/proxy/estimate") {
+      if (path === "/proxy/estimate" && isObj2(body.source)) {
         const spec = typeof body.spec === "string" ? body.spec : "";
         writeOwed(home, withEstimated(readOwed(home, now()), {
           source: body.source,
@@ -16947,9 +17188,12 @@ function trackingClient(client, home, now) {
           now: now(),
           sessionId: readSession(home)?.sessionId
         }));
-      } else if (path === "/proxy/estimate/actual") {
+      } else if (path === "/proxy/estimate/actual" || path === "/proxy/land") {
+        const key = typeof json?.key === "string" ? json.key : undefined;
+        if (!isObj2(body.source) && !key)
+          return;
         writeOwed(home, withActual(readOwed(home, now()), {
-          source: body.source,
+          ...isObj2(body.source) ? { source: body.source } : { key },
           state: typeof body.state === "string" ? body.state : undefined
         }));
       }
@@ -16959,13 +17203,13 @@ function trackingClient(client, home, now) {
     async post(path, body) {
       const r = await client.post(path, body);
       if (r)
-        after(path, body, 200);
+        after(path, body, 200, r);
       return r;
     },
     async postFull(path, body) {
       const r = await client.postFull(path, body);
       if (r)
-        after(path, body, r.status);
+        after(path, body, r.status, r.json);
       return r;
     },
     get: (path) => client.get(path)
@@ -17402,23 +17646,53 @@ var hooksOrg2 = (home, env = process.env) => {
 
 // src/commands/init.ts
 var DEFAULT_PROXY_URL = "https://proxy.sleds.ai";
-var promptStdin = (question, input = process.stdin, output = process.stderr) => new Promise((resolve) => {
-  const rl = createInterface({ input, output });
+var lineReaders = new WeakMap;
+var lineReader = (input, output) => {
+  const cached = lineReaders.get(input);
+  if (cached)
+    return cached;
+  const rl = createInterface({ input, output, terminal: false });
   const events = rl;
-  let answer;
-  events.once("line", (line) => {
-    answer = line;
-    rl.close();
+  const queued = [];
+  const waiting = [];
+  let closed = false;
+  events.on("line", (line) => {
+    const take = waiting.shift();
+    if (take)
+      take(line);
+    else
+      queued.push(line);
+    if (waiting.length === 0 && !closed)
+      rl.pause();
   });
   events.once("close", () => {
-    if (answer === undefined)
-      output.write(`
-(stdin closed before an answer; taking it as no answer)
-`);
-    resolve((answer ?? "").trim());
+    closed = true;
+    for (const take of waiting.splice(0))
+      take(undefined);
   });
+  const reader = {
+    next: () => new Promise((resolve) => {
+      if (queued.length > 0)
+        return resolve(queued.shift());
+      if (closed)
+        return resolve(undefined);
+      waiting.push(resolve);
+      rl.resume();
+    })
+  };
+  lineReaders.set(input, reader);
+  return reader;
+};
+var promptStdin = async (question, input = process.stdin, output = process.stderr) => {
+  const reader = lineReader(input, output);
   output.write(question);
-});
+  const answer = await reader.next();
+  if (answer === undefined)
+    output.write(`
+(stdin closed before an answer; taking it as no answer)
+`);
+  return (answer ?? "").trim();
+};
 async function signupIndividual(proxyUrl, email, credential, fetchFn = fetch, code) {
   return signupRequest(proxyUrl, { email, apiKey: credential, ...code ? { code } : {} }, fetchFn);
 }
@@ -18320,6 +18594,7 @@ init_config();
 init_state();
 init_settings();
 var DEFAULT_PROXY_URL2 = "https://proxy.sleds.ai";
+var lineReaders2 = new WeakMap;
 
 // src/commands/on.ts
 var on = (o) => {
@@ -19012,7 +19287,8 @@ async function decisionCmd(o, argv) {
   return said(o, key, json, r, () => log(seen ? `${key}: marked seen` : overrule ? `${key}: overruled — your answer stands in the log` : answeredLine(key, r.json?.route ?? route, r.json?.held)));
 }
 var DECISIONS_USAGE = "usage: sleds route decisions [<key>] [--flagged] [--json]";
-function decisionsLines(j) {
+function decisionsLines(raw) {
+  const j = clean(raw);
   const rows = Array.isArray(j.decisions) ? j.decisions : [];
   if (!rows.length)
     return ["no decisions"];
@@ -20686,7 +20962,8 @@ function ago2(at, now) {
   const h = Math.floor(m / 60);
   return h < 24 ? `${h}h ago` : `${Math.floor(h / 24)}d ago`;
 }
-function webhookLines(j, now) {
+function webhookLines(raw, now) {
+  const j = clean(raw);
   const rows = Array.isArray(j.webhooks) ? j.webhooks : [];
   if (!rows.length)
     return ["no webhooks — add one in the app: Settings › Integrations › Webhooks"];
@@ -20703,7 +20980,8 @@ function webhookLines(j, now) {
     return `${off}  ${str10(h.name)}  ${host}  ${events.includes("*") ? "all events" : events.join(",")}  ${lastText}${why}  [${str10(h.id)}]`;
   });
 }
-function deliveryLines(j, now) {
+function deliveryLines(raw, now) {
+  const j = clean(raw);
   const rows = Array.isArray(j.deliveries) ? j.deliveries : [];
   const name = str10(j.webhook?.name);
   if (!rows.length)
@@ -21422,7 +21700,7 @@ ${renderHelpText(argv[1])}`);
     case "version":
     case "--version":
     case "-v":
-      console.log("sleds 0.1.32");
+      console.log("sleds 0.1.33");
       return 0;
     default:
       console.error(`unknown command: ${cmd}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sleds/cli",
-  "version": "0.1.32",
+  "version": "0.1.33",
   "description": "SLEDS CLI — the cost and limits of AI agent work, decided at the wall. Point Claude Code at a SLEDS proxy; estimate before you run; hold, switch model, or run overnight when you hit a limit.",
   "license": "UNLICENSED",
   "type": "module",