agent-relay-runner 0.11.3 → 0.11.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-relay-runner",
-  "version": "0.11.3",
+  "version": "0.11.4",
   "description": "Unified provider lifecycle runner for Agent Relay",
   "type": "module",
   "bin": {
@@ -20,7 +20,7 @@
     "directory": "runner"
   },
   "dependencies": {
-    "agent-relay-sdk": "0.2.2"
+    "agent-relay-sdk": "0.2.3"
   },
   "devDependencies": {
     "@types/bun": "latest",

package/plugins/claude/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "agent-relay-runner",
   "description": "Thin Agent Relay runner bridge for Claude Code",
-  "version": "0.11.3",
+  "version": "0.11.4",
   "agentRelayContracts": {
     "providerPluginProtocol": 1
   }

package/src/runner.ts

@@ -361,6 +361,7 @@ export class AgentRunner {
     const deliverable: Message[] = [];
     const providerAlreadyBusy = this.claims.reasons().includes("provider-turn");
     for (const message of messages) {
+      let toDeliver = message;
       if (message.claimable) {
         const claimed = await this.http.claimMessageResult(message.id, this.agentId).catch(() => ({ ok: false, claimExpiresAt: undefined }));
         if (!claimed.ok) continue;
@@ -374,9 +375,12 @@ export class AgentRunner {
             agentId: this.agentId,
             metadata: { messageId: message.id, completedBy: "runner" },
           }).catch((error) => console.error(`[runner] task ${taskId} in_progress update failed: ${error}`));
+          // Runner owns claim + status here; drop the server's self-claim instruction
+          // so the agent doesn't improvise a stray claim send (see stripRunnerClaimedGuidance).
+          toDeliver = { ...message, body: stripRunnerClaimedGuidance(message.body) };
         }
       }
-      deliverable.push(message);
+      deliverable.push(toDeliver);
     }
     if (deliverable.length === 0) {
       this.delivering = false;
@@ -1075,6 +1079,19 @@ export function taskIdFromMessage(message: Pick<Message, "payload">): number | u
   return Number.isSafeInteger(taskId) ? taskId as number : undefined;
 }
 
+// The server appends a self-claim/status instruction to task message bodies
+// (taskMessageBody in src/db.ts: "Claim this task before working it, then update
+// task status when finished."). For runner-managed agents the runner already owns
+// claim + status, so that line is not only redundant but actively harmful: the
+// agent tries to follow it, has no real claim command, and improvises a stray
+// `agent-relay /send-claimable claim <id>` that leaves an orphan claimable message.
+// Strip the trailing guidance from messages the runner has auto-claimed.
+const RUNNER_CLAIMED_GUIDANCE_RE = /\s*Claim this task before working it, then update task status when finished\.\s*$/;
+
+export function stripRunnerClaimedGuidance(body: string): string {
+  return body.replace(RUNNER_CLAIMED_GUIDANCE_RE, "");
+}
+
 function csvTags(raw: string | undefined): string[] {
   return (raw || "").split(",").map((tag) => tag.trim()).filter(Boolean);
 }