agent-relay-server 0.119.1 → 0.119.2

package/docs/openapi.json

@@ -2,7 +2,7 @@
   "openapi": "3.1.0",
   "info": {
     "title": "Agent Relay API",
-    "version": "0.119.1",
+    "version": "0.119.2",
     "description": "Real-time message bus for inter-agent communication. Agent-first: this spec is designed for machine consumption — agents can self-discover the full API surface via GET /api/spec.",
     "license": {
       "name": "MIT",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-relay-server",
-  "version": "0.119.1",
+  "version": "0.119.2",
   "description": "Lightweight HTTP message relay for inter-agent communication across machines",
   "module": "src/index.ts",
   "type": "module",
@@ -37,9 +37,9 @@
     "CONTRIBUTING.md"
   ],
   "dependencies": {
-    "agent-relay-channels-host": "0.119.1",
+    "agent-relay-channels-host": "0.119.2",
     "agent-relay-providers": "0.104.2",
-    "agent-relay-sdk": "0.2.103",
+    "agent-relay-sdk": "0.2.104",
     "ajv": "^8.20.0"
   },
   "scripts": {

package/runner/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.119.1",
+  "version": "0.119.2",
   "agentRelayContracts": {
     "providerPluginProtocol": 1
   }

package/runner/plugins/claude/monitors/relay-monitor.provisioned.mjs

@@ -344,6 +344,10 @@ function relayTokenFromEnv() {
 }
 // sdk/src/bus-client.ts
 import { EventEmitter } from "events";
+function unrefTimer(timer) {
+  timer?.unref?.();
+}
+
 class MemoryCursorStore {
   value = null;
   async load() {
@@ -655,6 +659,7 @@ class RelayBusClient extends EventEmitter {
     this.heartbeat = setInterval(() => {
       this.sendHeartbeat();
     }, this.options.heartbeatIntervalMs ?? 30000);
+    unrefTimer(this.heartbeat);
   }
   stopHeartbeat() {
     if (this.heartbeat)
@@ -667,6 +672,7 @@ class RelayBusClient extends EventEmitter {
     this.stopHeartbeatWatchdog();
     const intervalMs = Math.max(100, Math.min(5000, this.heartbeatIntervalMs()));
     this.heartbeatWatchdog = setInterval(() => this.checkHeartbeatAck(), intervalMs);
+    unrefTimer(this.heartbeatWatchdog);
   }
   stopHeartbeatWatchdog() {
     if (this.heartbeatWatchdog)
@@ -753,6 +759,10 @@ class RelayBusClient extends EventEmitter {
   }
 }
 // sdk/src/provider-base.ts
+function unrefTimer2(timer) {
+  timer?.unref?.();
+}
+
 class ProviderBase {
   options;
   client;
@@ -814,6 +824,7 @@ class ProviderBase {
         this.client.status({ agentStatus: "online", ready: false });
       }
     }, 30000);
+    unrefTimer2(this.drain);
   }
 }
 // sdk/src/claim-tracker.ts

package/src/db/migrations.ts

@@ -276,6 +276,7 @@ export function applyMigrations(): void {
   getDb().run("CREATE INDEX IF NOT EXISTS idx_msg_thread ON messages(thread_id)");
   getDb().run("CREATE UNIQUE INDEX IF NOT EXISTS idx_msg_idempotency ON messages(from_agent, idempotency_key) WHERE idempotency_key IS NOT NULL");
   getDb().run("CREATE INDEX IF NOT EXISTS idx_msg_delivery_status ON messages(delivery_status)");
+  getDb().run("CREATE INDEX IF NOT EXISTS idx_msg_claimed_expires ON messages(claimed_by, claim_expires_at, id) WHERE claimed_by IS NOT NULL");
   // (#846) composite replaces the old single-column idx_msg_resolved_to_agent; created
   // here — AFTER the resolved_to_agent ALTER above — so existing DBs without the column
   // don't crash (same hazard as idx_msg_from_kind_id / #483/#559).
@@ -1076,6 +1077,7 @@ export function applyMigrations(): void {
   addTaskCol("owner_agent_label", "owner_agent_label TEXT");
   addTaskCol("dispatch", "dispatch TEXT");
   addTaskCol("reaped_at", "reaped_at INTEGER");
+  getDb().run("CREATE INDEX IF NOT EXISTS idx_tasks_message_status ON tasks(message_id, status) WHERE message_id IS NOT NULL");
   getDb().run("CREATE INDEX IF NOT EXISTS idx_tasks_kind ON tasks(kind, status, updated_at)");
   getDb().run("CREATE INDEX IF NOT EXISTS idx_tasks_branch ON tasks(branch) WHERE branch IS NOT NULL");
   getDb().run("CREATE INDEX IF NOT EXISTS idx_tasks_team ON tasks(team_id) WHERE team_id IS NOT NULL");

package/src/db/schema.ts

@@ -672,6 +672,9 @@ export function initDb(path: string = "agent-relay.db"): Database {
     CREATE INDEX IF NOT EXISTS idx_tasks_status ON tasks(status);
     CREATE INDEX IF NOT EXISTS idx_tasks_target ON tasks(target);
     CREATE INDEX IF NOT EXISTS idx_tasks_updated ON tasks(updated_at);
+    CREATE INDEX IF NOT EXISTS idx_tasks_message_status
+      ON tasks(message_id, status)
+      WHERE message_id IS NOT NULL;
     -- idx_tasks_kind / idx_tasks_branch / idx_tasks_team reference the #834 columns and are
     -- created in applyMigrations() AFTER the ALTERs — never inline here: an existing pre-#834
     -- tasks table makes CREATE TABLE IF NOT EXISTS a no-op, so an inline index over a column

package/src/issue-tracker/github.ts

@@ -3,8 +3,9 @@ import type { IssueTrackerAdapter, IssueTrackerCloseOptions, IssueTrackerIssueDe
 
 export const GITHUB_FIXED_LABEL = "fixed";
 const GITHUB_CREDENTIAL_REQUIREMENT = "GitHub issue tracker adapter requires gh authenticated with a token that has issues:write scope";
-// #838 — hard bound on the spawn-time issue read so hydration never stalls the spawn.
-const ISSUE_DETAIL_READ_TIMEOUT_MS = 10_000;
+// #935 — spawn-time issue hydration sits on the dashboard /agents/spawn -> 202 path.
+// Keep it sub-second and degrade to "spec unavailable" if GitHub is slow.
+const ISSUE_DETAIL_READ_TIMEOUT_MS = 750;
 
 type GitHubApiMethod = "GET" | "PATCH" | "POST";
 
@@ -83,19 +84,19 @@ async function ghApi(request: GitHubApiRequest): Promise<unknown> {
   // auth backed by a GitHub token with `issues:write` scope.
   const args = ["gh", "api", "--method", request.method, request.path, "--header", "Accept: application/vnd.github+json"];
   appendBodyArgs(args, request.body);
-  // `Bun.spawnSync` blocks the event loop synchronously, so a JS-level timeout race can't preempt
-  // a hung `gh`; the subprocess `timeout` (kills with SIGTERM on expiry) is the only effective bound.
-  const proc = Bun.spawnSync(args, {
+  const proc = Bun.spawn(args, {
     stdin: "ignore",
     stdout: "pipe",
     stderr: "pipe",
     env: process.env,
-    ...(request.timeoutMs ? { timeout: request.timeoutMs } : {}),
   });
-  const stdout = textFromBytes(proc.stdout).trim();
-  if (proc.exitCode !== 0) {
-    const stderr = textFromBytes(proc.stderr).trim();
-    const timedOut = request.timeoutMs && (proc.signalCode === "SIGTERM" || proc.exitCode === null);
+  const stdoutPromise = streamText(proc.stdout);
+  const stderrPromise = streamText(proc.stderr);
+  const timedOut = await waitForExitOrTimeout(proc, request.timeoutMs);
+  const [stdoutRaw, stderrRaw] = await Promise.all([stdoutPromise, stderrPromise]);
+  const stdout = stdoutRaw.trim();
+  if (timedOut || proc.exitCode !== 0) {
+    const stderr = stderrRaw.trim();
     throw new Error(`${GITHUB_CREDENTIAL_REQUIREMENT}; gh api ${timedOut ? `timed out after ${request.timeoutMs}ms` : "failed"}: ${stderr || stdout || `exit ${proc.exitCode}`}`);
   }
   if (!stdout) return {};
@@ -106,6 +107,40 @@ async function ghApi(request: GitHubApiRequest): Promise<unknown> {
   }
 }
 
+async function streamText(stream: ReadableStream<Uint8Array> | null): Promise<string> {
+  if (!stream) return "";
+  try {
+    return await new Response(stream).text();
+  } catch {
+    return "";
+  }
+}
+
+async function waitForExitOrTimeout(proc: Bun.Subprocess<"ignore", "pipe", "pipe">, timeoutMs: number | undefined): Promise<boolean> {
+  if (!timeoutMs || timeoutMs <= 0) {
+    await proc.exited;
+    return false;
+  }
+  let timedOut = false;
+  let kill: ReturnType<typeof setTimeout> | undefined;
+  const timeout = setTimeout(() => {
+    timedOut = true;
+    try { proc.kill("SIGTERM"); } catch {}
+    kill = setTimeout(() => {
+      try { proc.kill("SIGKILL"); } catch {}
+    }, 1_000);
+    kill.unref?.();
+  }, timeoutMs);
+  timeout.unref?.();
+  try {
+    await proc.exited;
+  } finally {
+    clearTimeout(timeout);
+    if (kill) clearTimeout(kill);
+  }
+  return timedOut;
+}
+
 function appendBodyArgs(args: string[], body: Record<string, unknown> | undefined): void {
   if (!body) return;
   for (const [key, value] of Object.entries(body)) {