cruo-agent 0.1.1 → 0.1.2

package/README.md

@@ -60,6 +60,28 @@ Run it from the repository the agent should work on. If that repository needs
 installing before its tests will run, add `--prepare 'npm ci'` or whatever your
 equivalent is — a fresh checkout has your source and none of your dependencies.
 
+Checkouts go under `~/.cruo-work/<agent>`, one directory per agent, so **running
+several agents at once needs no extra flags**. `--worktree-root` overrides it if
+you want them somewhere else; pointing two agents at one root is safe too, since
+the sweep that clears abandoned checkouts at startup only removes ones too old
+for any run to still be inside.
+
+## When it stops doing anything
+
+A supervisor sitting idle costs nothing — it is a database query on a timer. If
+the *harness* refuses to work, though, the supervisor says so and slows down:
+
+```
+! the harness is refusing to work: the account's spend limit is reached
+  No card is charged for this, and polling backs off until it changes.
+```
+
+That distinction matters more than it looks. A refusal is a fact about your
+account, not about the card, so those runs are **not** counted against the
+card's `--max-attempts` — otherwise one billing problem would quietly set aside
+every issue the agent was holding, and you would come back to a board that had
+given up on work nobody abandoned.
+
 ## Common options
 
 | flag | |
@@ -74,7 +96,7 @@ equivalent is — a fresh checkout has your source and none of your dependencies
 | `--harness-timeout <seconds>` | kill a run that wedges (default 600) |
 | `--max-attempts <n>` | give up on a card after n unproductive runs (default 3) |
 
-The full list, and what each is for: <https://cruo.space/agents>
+The full list, and what each is for: <https://cruo.space/docs#agents>
 
 ## Running it somewhere that is not your laptop
 

package/dist/VERSION

@@ -1 +1 @@
-0.1.1
+0.1.2

package/dist/cli.js

@@ -46294,17 +46294,25 @@ async function pollMentions(ctx, identity) {
   );
 }
 async function claim(ctx, hit) {
-  const expiresAt = new Date(Date.now() + options.harnessTimeoutMs + 3e4);
-  const { error: error51 } = await ctx.client.schema("pm").from("issue_claims").upsert(
-    {
-      issue_id: hit.issue.id,
-      claimed_by: ctx.userId,
-      claimed_at: (/* @__PURE__ */ new Date()).toISOString(),
-      expires_at: expiresAt.toISOString()
-    },
-    { onConflict: "issue_id" }
-  );
-  if (error51) log(`  could not claim ${hit.ref}: ${error51.message}`);
+  const ttlMs = options.harnessTimeoutMs + 3e4;
+  const { data, error: error51 } = await ctx.client.schema("public").rpc("claim_issue", {
+    issue: hit.issue.id,
+    ttl_ms: ttlMs
+  });
+  if (error51) {
+    log(`  could not claim ${hit.ref}: ${error51.message} \u2014 leaving it`);
+    return false;
+  }
+  return data === true;
+}
+async function claimedElsewhere(ctx, hits) {
+  if (hits.length === 0) return /* @__PURE__ */ new Set();
+  const { data, error: error51 } = await ctx.client.schema("pm").from("issue_claims").select("issue_id, claimed_by").in("issue_id", hits.map((h) => h.issue.id)).gt("expires_at", (/* @__PURE__ */ new Date()).toISOString()).neq("claimed_by", ctx.userId);
+  if (error51) {
+    log(`  could not read claims: ${error51.message} \u2014 relying on the claim itself`);
+    return /* @__PURE__ */ new Set();
+  }
+  return new Set((data ?? []).map((r) => r.issue_id));
 }
 async function release(ctx, hit) {
   const { error: error51 } = await ctx.client.schema("pm").from("issue_claims").delete().eq("issue_id", hit.issue.id).eq("claimed_by", ctx.userId);
@@ -46634,11 +46642,17 @@ async function tick(ctx, identity, deadTicks) {
     seen.add(h.ref);
     return true;
   });
-  const attemptsByHit = await loadAttempts(ctx, hits);
+  const heldByOthers = await claimedElsewhere(ctx, hits);
+  const free = hits.filter((h) => !heldByOthers.has(h.issue.id));
+  if (heldByOthers.size > 0) {
+    log(`${heldByOthers.size} issue(s) already being worked by someone else`);
+  }
+  const attemptsByHit = await loadAttempts(ctx, free);
   const attemptsFor = (h) => attemptsByHit.get(`${h.reason}:${h.issue.id}`) ?? 0;
-  const actionable = hits.filter((h) => attemptsFor(h) < options.maxAttempts);
+  const actionable = free.filter((h) => attemptsFor(h) < options.maxAttempts);
   if (actionable.length === 0) {
-    if (hits.length > 0) log(`${hits.length} issue(s) waiting, all set aside`);
+    if (free.length > 0) log(`${free.length} issue(s) waiting, all set aside`);
+    else if (heldByOthers.size > 0) log(`nothing free \u2014 everything waiting is claimed`);
     await beat(ctx, { holding: 0, deadTicks, intervalMs: options.intervalMs });
     return { invoked: 0, failedToRun: 0 };
   }
@@ -46667,6 +46681,10 @@ async function tick(ctx, identity, deadTicks) {
     invoked += 1;
     const before = { status: hit.issue.status_id, assignee: hit.issue.assignee_id };
     const startedAt = (/* @__PURE__ */ new Date()).toISOString();
+    if (!await claim(ctx, hit)) {
+      log(`  ${hit.ref} is being worked by another agent \u2014 skipping`);
+      continue;
+    }
     let worktree = null;
     if (worktreeConfig) {
       try {
@@ -46674,12 +46692,12 @@ async function tick(ctx, identity, deadTicks) {
         log(`  ${hit.ref} worktree ${worktree.path} on ${worktree.branch}`);
       } catch (error51) {
         failedToRun += 1;
+        await release(ctx, hit);
         log(`  ${hit.ref} could not prepare a worktree \u2014 not counted against this issue`);
         log(`    ${error51 instanceof Error ? error51.message.split("\n")[0] : String(error51)}`);
         continue;
       }
     }
-    await claim(ctx, hit);
     let run;
     let committed = 0;
     try {
@@ -47006,7 +47024,7 @@ Common options
   --push                 publish the branch after a run that commits
   --interval <seconds>   how often to poll (default 20)
 
-  Full list and what each one is for: https://cruo.space/agents
+  Full list and what each one is for: https://cruo.space/docs#agents
 
 The token comes from CRUO_TOKEN, then --token, then whatever \`login\` stored.
 `;

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "cruo-agent",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Run a Cruo agent: it watches your board, picks up the cards you assign it, and works them.",
   "license": "MIT",
-  "homepage": "https://cruo.space/agents",
+  "homepage": "https://cruo.space/docs#agents",
   "keywords": [
     "cruo",
     "agent",
@@ -36,6 +36,6 @@
     "@cruo/mcp": "workspace:*"
   },
   "bugs": {
-    "url": "https://cruo.space/agents"
+    "url": "https://cruo.space/docs#agents"
   }
 }