@sleds/cli 0.1.37 → 0.1.38

package/dist/sleds.mjs

@@ -7952,7 +7952,8 @@ function parseRun(v) {
         ...typeof v.pr.conflict === "string" ? { conflict: v.pr.conflict } : {},
         ...typeof v.pr.resolving === "string" ? { resolving: v.pr.resolving } : {},
         ...conflicted ? { conflicted } : {},
-        ...typeof v.pr.reviewed === "string" ? { reviewed: v.pr.reviewed } : {}
+        ...typeof v.pr.reviewed === "string" ? { reviewed: v.pr.reviewed } : {},
+        ...typeof v.pr.look === "string" ? { look: v.pr.look } : {}
       }
     } : {}
   };
@@ -8023,6 +8024,7 @@ function parseLock(text) {
       runner: typeof v.runner === "string" ? v.runner : "",
       atOnce: n(v.atOnce, 1),
       interval: n(v.interval, 0),
+      ...typeof v.cliVersion === "string" && v.cliVersion ? { cliVersion: v.cliVersion } : {},
       ...typeof v.checkedAt === "number" ? { checkedAt: v.checkedAt } : {},
       ...typeof v.nextCheckAt === "number" ? { nextCheckAt: v.nextCheckAt } : {},
       ...machine ? { machine } : {},
@@ -10345,7 +10347,7 @@ function beatBody(state, flags, host = {}, cap = {}) {
     name: flags.name,
     os: host.os ?? process.platform,
     runner: flags.runner,
-    version: host.version ?? "0.1.37" ?? "dev",
+    version: host.version ?? "0.1.38" ?? "dev",
     running: live.length,
     items: live.slice(0, BEAT_ITEMS).map((r) => `${r.item}/${r.node}`),
     ...cap.atOnce !== undefined ? { atOnce: cap.atOnce } : {},
@@ -10657,6 +10659,15 @@ function prForBranch(exec, repoPath, branch) {
     return { error: e instanceof Error ? e.message : String(e) };
   }
 }
+var URL_RE = /https:\/\/[^\s)>\]"']+/;
+var ARTIFACT_RE = /https:\/\/claude\.ai\/[^\s)>\]"']*artifact\/[^\s)>\]"']+/;
+function mockOf(body) {
+  const header = /<!--\s*sleds:header\s*-->([\s\S]*?)<!--\s*\/sleds:header\s*-->/.exec(body)?.[1];
+  const line = header?.split(`
+`).find((l) => /\*\*Mocks\*\*\s*:/i.test(l));
+  const fromHeader = line ? URL_RE.exec(line)?.[0] : undefined;
+  return fromHeader ?? ARTIFACT_RE.exec(body)?.[0];
+}
 var FAILED = new Set([
   "FAILURE",
   "ERROR",
@@ -10702,6 +10713,7 @@ function parsePrView(stdout) {
       pending++;
   }
   const mergeCommit = typeof o.mergeCommit === "object" && o.mergeCommit !== null ? o.mergeCommit.oid : undefined;
+  const mock = typeof o.body === "string" ? mockOf(o.body) : undefined;
   return {
     state: o.state,
     author: loginOf(o.author),
@@ -10712,6 +10724,8 @@ function parsePrView(stdout) {
     reviewDecision: typeof o.reviewDecision === "string" ? o.reviewDecision : "",
     feedback: 0,
     authorFeedback: 0,
+    draft: o.isDraft === true,
+    ...mock ? { mock } : {},
     ...typeof o.headRefOid === "string" && o.headRefOid ? { head: o.headRefOid } : {},
     ...typeof mergeCommit === "string" && mergeCommit ? { mergeCommit } : {}
   };
@@ -10766,7 +10780,7 @@ function prView(exec, repoPath, url) {
       "view",
       url,
       "--json",
-      "state,author,statusCheckRollup,reviewDecision,comments,reviews,headRefOid,mergeable,baseRefName,mergeCommit"
+      "state,author,statusCheckRollup,reviewDecision,comments,reviews,headRefOid,mergeable,baseRefName,mergeCommit,isDraft,body"
     ], { cwd: repoPath });
     const view = r.status === 0 ? parsePrView(r.stdout) : null;
     if (!view || view.state !== "OPEN")
@@ -11232,6 +11246,10 @@ init_machine();
 
 // src/engine/sledsSource.ts
 init_src();
+function looksOf(s) {
+  const l = s;
+  return l && typeof l.openLook === "function" && typeof l.clearLook === "function" ? s : null;
+}
 var FAIL_LOG_MAX = 2000;
 var UNREACHABLE2 = "SLEDS unreachable";
 function keysIn(states, state) {
@@ -11373,20 +11391,34 @@ var PROFILE_TEXT = [
   "reposDir",
   "engine"
 ];
+var SHELL_TEXT = ["agent", "launcher"];
 function profileOf(r) {
   if (!r || r.ok === false || !isObject8(r.profile))
     return null;
   const p = r.profile;
+  const member = isObject8(r.member) ? r.member : {};
   const out = {
     pickup: p.pickup === "auto" ? "auto" : "approve",
     ...HOLD_DEFAULTS2,
     holdOff: p.holdOff === true
   };
   for (const k of PROFILE_TEXT) {
+    if (SHELL_TEXT.includes(k))
+      continue;
     const v = str6(p[k])?.trim();
     if (v)
       out[k] = v;
   }
+  const notYours = [];
+  for (const k of SHELL_TEXT) {
+    const mine = str6(member[k])?.trim();
+    if (mine)
+      out[k] = mine;
+    else if (str6(p[k])?.trim())
+      notYours.push(k);
+  }
+  if (notYours.length)
+    out.notYours = notYours;
   const atOnce = num5(p.atOnce);
   if (atOnce !== undefined && atOnce >= 1)
     out.atOnce = Math.round(atOnce);
@@ -11746,6 +11778,36 @@ function sledsSource(client, o) {
         throw new Error(serverMessage(r));
       return eventOpenOf(r.json);
     },
+    async openLook(runId, look, engine) {
+      const r = await client.postFull("/proxy/run", {
+        runId,
+        engine,
+        event: {
+          kind: "look",
+          md: look.md,
+          about: { prs: [look.pr], ...look.mock ? { mock: look.mock } : {} }
+        }
+      });
+      if (!r)
+        throw new Error(`${UNREACHABLE2} — the look not raised`);
+      if (r.status >= 300 || !r.json || r.json.ok === false)
+        throw new Error(serverMessage(r));
+      const decisionId = str6(r.json.decisionId);
+      const state = str6(r.json.state);
+      return {
+        opened: r.json.opened === true,
+        ...decisionId ? { decisionId } : {},
+        ...state ? { state } : {}
+      };
+    },
+    async clearLook(runId, clear, engine) {
+      const r = await client.postFull("/proxy/run", { runId, engine, clearLook: clear });
+      if (!r)
+        throw new Error(`${UNREACHABLE2} — the look not cleared`);
+      if (r.status >= 300 || !r.json || r.json.ok === false)
+        throw new Error(serverMessage(r));
+      return r.json.cleared === true;
+    },
     async decisionOf(item, decisionId) {
       const route = await routeOf(item);
       const decisions = Array.isArray(route?.decisions) ? route.decisions.filter(isObject8) : [];
@@ -12207,9 +12269,13 @@ async function tick(deps) {
       deps.providerHold = d.providerHold;
       deps.atOnce = d.atOnce;
     }
+    removeEndedMcpConfigs(deps, made);
   }
+  return report;
+}
+function removeEndedMcpConfigs(deps, made) {
   if (!deps.removeMcpConfig || made.size === 0)
-    return report;
+    return;
   const state = deps.readState();
   const live = liveMcpConfigs(state);
   for (const p of made) {
@@ -12227,7 +12293,6 @@ async function tick(deps) {
       r.mcpConfig = undefined;
     deps.writeState(state);
   }
-  return report;
 }
 async function tickOnce(d) {
   d.tickProfile = profileOnce(d.readProfile);
@@ -13513,11 +13578,47 @@ 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. 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.`;
 }
+function lookQuestion(url, mock) {
+  return `A draft PR waits for your look: ${url}.${mock ? ` The mock: ${mock}.` : " Its body names no mock; the PR is the place to look."} Looks good marks it ready for review.`;
+}
+async function judgeLook(d, run, view) {
+  const looks = looksOf(d.source);
+  const pr = run.pr;
+  if (!looks || !run.runId)
+    return;
+  const draft = view.state === "OPEN" && view.draft;
+  if (draft && !pr.look) {
+    try {
+      const r = await looks.openLook(run.runId, {
+        md: lookQuestion(pr.url, view.mock),
+        pr: pr.url,
+        ...view.mock ? { mock: view.mock } : {}
+      }, d.engine);
+      if (!r.decisionId)
+        return;
+      pr.look = r.decisionId;
+      if (r.opened)
+        d.log(`${ref(run)}: ${pr.url} is a draft — a look waits for a person`);
+    } catch (e) {
+      d.log(`${ref(run)}: could not raise the look on ${pr.url}: ${message(e)}`);
+    }
+  } else if (!draft && pr.look) {
+    const why = view.state === "MERGED" ? "the PR merged" : view.state === "CLOSED" ? "the PR closed without a merge" : "the PR left draft";
+    try {
+      if (await looks.clearLook(run.runId, { pr: pr.url, why }, d.engine))
+        d.log(`${ref(run)}: the look on ${pr.url} is cleared — ${why}`);
+      pr.look = undefined;
+    } catch (e) {
+      d.log(`${ref(run)}: could not clear the look on ${pr.url}: ${message(e)}`);
+    }
+  }
+}
 async function follow(d, state, run, claim, runner, finished, attention, changes, decided) {
   const pr = run.pr;
   const view = d.prView(d.exec, run.repoPath, pr.url);
   if (!view)
     return true;
+  await judgeLook(d, run, view);
   if (view.state === "MERGED") {
     const landed = await ripple(d, state, run, claim, runner, pr.url, finished, decided, view.head, view.mergeCommit);
     return !landed;
@@ -15429,6 +15530,8 @@ function flagsFromProfile(flags, given, profile, home) {
   const out = { ...flags };
   const used = [];
   const skipped = [];
+  for (const k of profile.notYours ?? [])
+    skipped.push(`${k} (the workspace default — the engine runs only one you set yourself)`);
   if (profile.atOnce !== undefined && !has("at-once")) {
     out.atOnce = profile.atOnce;
     used.push(`at-once ${profile.atOnce}`);
@@ -15819,7 +15922,8 @@ async function driveEngine(o, flags, deps, until = {}) {
     startedAt: (o.now ?? Date.now)(),
     runner: deps.runner.name,
     atOnce: flags.atOnce,
-    interval: flags.interval
+    interval: flags.interval,
+    cliVersion: "0.1.38"
   };
   const beat = (checkedAt) => {
     const m = deps.machine;
@@ -17421,6 +17525,8 @@ function flagsFromProfile2(flags, given, profile, home) {
   const out = { ...flags };
   const used = [];
   const skipped = [];
+  for (const k of profile.notYours ?? [])
+    skipped.push(`${k} (the workspace default — the engine runs only one you set yourself)`);
   if (profile.atOnce !== undefined && !has("at-once")) {
     out.atOnce = profile.atOnce;
     used.push(`at-once ${profile.atOnce}`);
@@ -17849,7 +17955,8 @@ async function driveEngine2(o, flags, deps, until = {}) {
     startedAt: (o.now ?? Date.now)(),
     runner: deps.runner.name,
     atOnce: flags.atOnce,
-    interval: flags.interval
+    interval: flags.interval,
+    cliVersion: "0.1.38"
   };
   const beat = (checkedAt) => {
     const m = deps.machine;
@@ -18827,6 +18934,7 @@ function engineStatus(i) {
       ...opt("runner", lock.runner || undefined),
       atOnce: lock.atOnce,
       interval: lock.interval,
+      cliVersion: lock.cliVersion ?? "",
       ...opt("checkedAt", iso2(lock.checkedAt)),
       ...opt("nextCheckAt", alive ? iso2(lock.nextCheckAt) : undefined),
       ...opt("machine", lock.machine ? { ...lock.machine, at: new Date(lock.machine.at).toISOString() } : undefined),
@@ -20764,8 +20872,9 @@ var mcp = async (o) => {
     (o.warn ?? ((line) => process.stderr.write(`${line}
 `)))("sleds mcp: on this run's token (SLEDS_RUN_TOKEN), not the seat's");
   let buffer = "";
+  const decoder = new TextDecoder;
   for await (const chunk of stdin) {
-    buffer += typeof chunk === "string" ? chunk : new TextDecoder().decode(chunk);
+    buffer += typeof chunk === "string" ? chunk : decoder.decode(chunk, { stream: true });
     let nl = buffer.indexOf(`
 `);
     while (nl !== -1) {
@@ -20778,6 +20887,7 @@ var mcp = async (o) => {
 `);
     }
   }
+  buffer += decoder.decode();
   if (buffer.trim().length > 0) {
     const out = await handleLine2(buffer, { home: o.home, env });
     if (out !== null)
@@ -21381,13 +21491,13 @@ init_src();
 import { readFileSync as readFileSync24 } from "node:fs";
 import { hostname as hostname12 } from "node:os";
 var ARM_USAGE = "usage: sleds route arm <key> [--off] [--json] [--name <engine>] [--at-once <n>] [--runner tmux|headless|command|orca|cmux] [--runner-cmd '<launcher>'] [--agent '<command>'] [--mode <permission-mode>] [--model <id>] [--effort <e>] [--branch '<template>'] [--pr-title '<template>'] [--repos-dir <dir>]";
-var USAGE = "usage: sleds route <key> [--long | --dot | --json]   |   sleds route arm <key> [--off] [--json]   |   sleds route answer <key>/<node> --stdin   |   sleds route propose <key> --file <nodes.json>   |   sleds route approve <key>   |   sleds route redraft <key>   |   sleds route single <key>   |   sleds route decisions [<key>] [--flagged]   |   sleds route decision <key> <id> --seen | --answer --stdin [--route accept|keep|redraft] | --keep-route | --apply [--answer --stdin] | --overrule --stdin   |   sleds route run <key> [--at-once <n>] [--runner tmux|headless|command|orca|cmux] [--runner-cmd '<launcher>'] [--agent '<command>'] [--interval <s>] [--lease <min>] [--idle <min>] [--repo <path>] [--repos-dir <dir>] [--mode <permission-mode>] [--model <id>] [--branch '<template>'] [--pr-title '<template>'] [--once]";
+var USAGE = "usage: sleds route <key> [--long | --dot | --json]   |   sleds route arm <key> [--off] [--json]   |   sleds route answer <key>/<node> --stdin   |   sleds route propose <key> --file <nodes.json>   |   sleds route approve <key>   |   sleds route redraft <key>   |   sleds route single <key>   |   sleds route decisions [<key>] [--flagged]   |   sleds route decision <key> <id> --seen | --answer --stdin [--route accept|keep|redraft] | --keep-route | --apply [--answer --stdin] | --overrule --stdin | --ready   |   sleds route run <key> [--at-once <n>] [--runner tmux|headless|command|orca|cmux] [--runner-cmd '<launcher>'] [--agent '<command>'] [--interval <s>] [--lease <min>] [--idle <min>] [--repo <path>] [--repos-dir <dir>] [--mode <permission-mode>] [--model <id>] [--branch '<template>'] [--pr-title '<template>'] [--once]";
 var ANSWER_USAGE = "usage: sleds route answer <key>/<node> <answer…>   |   sleds route answer <key>/<node> --stdin   [--json]";
 var PROPOSE_USAGE = "usage: sleds route propose <key> (--file <nodes.json> | --stdin) [--name <engine>] [--json]";
 var APPROVE_USAGE = "usage: sleds route approve <key> [--file <nodes.json>] [--json]";
 var REDRAFT_USAGE = "usage: sleds route redraft <key> [--json]";
 var SINGLE_USAGE = "usage: sleds route single <key> [--json]";
-var DECISION_USAGE = "usage: sleds route decision <key> <id> --seen   |   sleds route decision <key> <id> (--answer | --overrule) (<answer…> | --stdin)   |   sleds route decision <key> <id> [--answer (<answer…> | --stdin)] --route accept|keep|redraft   |   sleds route decision <key> <id> --keep-route   |   sleds route decision <key> <id> --apply [--answer (<answer…> | --stdin)]   [--json]";
+var DECISION_USAGE = "usage: sleds route decision <key> <id> --seen   |   sleds route decision <key> <id> (--answer | --overrule) (<answer…> | --stdin)   |   sleds route decision <key> <id> [--answer (<answer…> | --stdin)] --route accept|keep|redraft   |   sleds route decision <key> <id> --keep-route   |   sleds route decision <key> <id> --apply [--answer (<answer…> | --stdin)]   |   sleds route decision <key> <id> --ready   [--json]";
 var ROUTE_VERDICTS = ["accept", "keep", "redraft"];
 function nodesFromJson(text) {
   let parsed;
@@ -21716,6 +21826,8 @@ function appliedLine(key, applied) {
   return a.changed === false ? `${key}: ${what} was already set — answered` : `${key}: set ${what} — answered`;
 }
 async function decisionCmd(o, argv) {
+  if (argv.includes("--ready"))
+    return readyCmd(o, argv);
   const log = o.log ?? console.log;
   const error = o.error ?? console.error;
   const json = argv.includes("--json");
@@ -21769,6 +21881,68 @@ 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` : apply ? appliedLine(key, r.json?.applied) : answeredLine(key, r.json?.route ?? route, r.json?.held)));
 }
+var LOOK_READY_ANSWER = "Looks good — marked ready for review.";
+function lookToReady(route, id) {
+  const rows = Array.isArray(route?.decisions) ? route.decisions : [];
+  const d = rows.find((r) => r.id === id);
+  if (!d)
+    return { error: `no decision ${id} on this item` };
+  if (d.kind !== "look")
+    return { error: `decision ${id} is not a look — --ready marks a look` };
+  if (d.state !== "escalated")
+    return { error: `decision ${id} is ${String(d.state)} — not a look you owe` };
+  const about = d.about && typeof d.about === "object" ? d.about : {};
+  const pr = Array.isArray(about.prs) ? about.prs.find((p) => typeof p === "string" && p) : undefined;
+  return typeof pr === "string" ? { pr } : { error: `decision ${id} names no PR` };
+}
+async function readyCmd(o, argv) {
+  const log = o.log ?? console.log;
+  const error = o.error ?? console.error;
+  const json = argv.includes("--json");
+  const words = argv.filter((a) => a !== "--ready" && a !== "--json");
+  const [key, id] = words;
+  if (!key || !id || words.length !== 2 || words.some((w) => w.startsWith("--"))) {
+    error(DECISION_USAGE);
+    return 2;
+  }
+  const client = engineClient(o);
+  if ("error" in client) {
+    error(client.error);
+    return 1;
+  }
+  const reader = routeReader(o);
+  if ("error" in reader) {
+    error(reader.error);
+    return 1;
+  }
+  const route = await reader.read(key).catch((e) => {
+    error(e instanceof Error ? e.message : String(e));
+    return;
+  });
+  if (route === undefined)
+    return 1;
+  const look = lookToReady(route, id);
+  if ("error" in look) {
+    error(`${key}: ${look.error}`);
+    return 1;
+  }
+  const gh = (o.exec ?? defaultExec)("gh", ["pr", "ready", look.pr]);
+  if (gh.status !== 0) {
+    error(gh.stderr.trim() || `gh pr ready ${look.pr} exited ${gh.status}`);
+    error(`${key}: the look is not answered`);
+    return 1;
+  }
+  const r = await client.postFull("/proxy/route/decision", {
+    project: key,
+    id,
+    answer: LOOK_READY_ANSWER
+  });
+  if (!r) {
+    error(`${key}: ${look.pr} is ready for review, but SLEDS is unreachable — the look is not answered`);
+    return 1;
+  }
+  return said(o, key, json, r, () => log(`${key}: ${look.pr} marked ready for review — the look is answered`));
+}
 var DECISIONS_USAGE = "usage: sleds route decisions [<key>] [--flagged] [--json]";
 function decisionsLines(raw) {
   const j = clean(raw);
@@ -24215,7 +24389,7 @@ ${renderHelpText(argv[1])}`);
     case "version":
     case "--version":
     case "-v":
-      console.log("sleds 0.1.37");
+      console.log("sleds 0.1.38");
       return 0;
     default:
       console.error(`unknown command: ${cmd}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sleds/cli",
-  "version": "0.1.37",
+  "version": "0.1.38",
   "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",