agent-relay-runner 0.65.0 → 0.65.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-relay-runner",
-  "version": "0.65.0",
+  "version": "0.65.2",
   "description": "Unified provider lifecycle runner for Agent Relay",
   "type": "module",
   "bin": {

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.65.0",
+  "version": "0.65.2",
   "agentRelayContracts": {
     "providerPluginProtocol": 1
   }

package/plugins/claude/hooks/relay-status.sh

@@ -31,6 +31,44 @@ relay_post_status_clearing_subagents() {
   relay_post_status "$1" "${2:-}" "${3:-}" "${4:-}" "${5:-}" "${6:-}" subagent
 }
 
+# #502: count the background shells still running at the moment this turn ends.
+# Claude's Stop payload carries a `background_tasks` array (CC v2.1.157+) listing
+# the shells launched with run_in_background that have not yet finished. It is a
+# full live snapshot, so reporting it each turn-end can never wedge: a completed
+# task wakes Claude, the next Stop re-reports the (now smaller/empty) set. Older
+# Claude builds omit the field → count 0 → graceful no-op (no false busy).
+relay_background_task_count() {
+  local input="${1:-}"
+  [ -z "$input" ] && { printf '0'; return 0; }
+  if command -v jq >/dev/null 2>&1; then
+    local n
+    n="$(printf '%s' "$input" | jq -r '(.background_tasks // []) | length' 2>/dev/null)"
+    case "$n" in ''|*[!0-9]*) printf '0' ;; *) printf '%s' "$n" ;; esac
+    return 0
+  fi
+  # jq-less fallback: a non-empty array literal means at least one is running.
+  if printf '%s' "$input" | tr -d '[:space:]' | grep -q '"background_tasks":\[{'; then
+    printf '1'
+  else
+    printf '0'
+  fi
+}
+
+# #502: end the provider turn (idle) clearing any stale subagent work — the
+# critical path (#199) — then report the live background-script snapshot so an
+# agent whose turn ended while a `run_in_background` shell keeps running stays
+# busy instead of falsely reading idle. A single full-snapshot post per turn-end:
+# >0 → busy(background-script); 0 → idle(background-script) which clears any prior.
+relay_post_turn_end_status() {
+  local bg_count="${1:-0}"
+  relay_post_status_clearing_subagents idle
+  case "$bg_count" in
+    ''|0|*[!0-9]*) relay_post_status idle background-script background-tasks ;;
+    1) relay_post_status busy background-script background-tasks "1 background task" ;;
+    *) relay_post_status busy background-script background-tasks "${bg_count} background tasks" ;;
+  esac
+}
+
 # Report a provider usage/rate-limit hold to the runner (#286). The runner turns
 # this into a `providerState: blocked` (reason rate_limit), posts a chat notice,
 # and auto-resumes the agent once the window resets. reset_time (unix seconds,

package/plugins/claude/hooks/stop.sh

@@ -8,10 +8,16 @@ relay_install_hook_guard stop
 # exception is the reply-obligation block path, which deliberately keeps the agent
 # busy to answer — it opts out via the flag before exiting.
 _relay_clear_idle_on_exit=1
-trap '[ "${_relay_clear_idle_on_exit:-0}" = "1" ] && relay_post_status_clearing_subagents idle' EXIT
+# bg_count is read from the payload below and consumed by the trap at exit time
+# (#502): the turn-end status carries the live background-script snapshot so an
+# agent whose turn ended while a run_in_background shell keeps running stays busy
+# instead of falsely reading idle.
+bg_count=0
+trap '[ "${_relay_clear_idle_on_exit:-0}" = "1" ] && relay_post_turn_end_status "${bg_count:-0}"' EXIT
 
 payload="$(cat || true)"
 stop_hook_active="$(relay_json_bool_field stop_hook_active "$payload")"
+bg_count="$(relay_background_task_count "$payload")"
 
 # Session-mirror capture is orthogonal to reply-obligation gating (#332): a recursive Stop
 # (stop_hook_active=true) must NOT suppress capturing this turn's response, or the live mirror

package/src/adapter.ts

@@ -4,7 +4,7 @@ import type { SessionEvent } from "./session-insights";
 import { messageBodyMaxCharsFromEnv } from "./config";
 
 export type SemanticStatus = "idle" | "busy" | "offline" | "error";
-type ProviderWorkKind = "provider-turn" | "subagent";
+type ProviderWorkKind = "provider-turn" | "subagent" | "background-script";
 
 export interface ProviderStatusEvent {
   status: SemanticStatus;

package/src/claim-tracker.ts

@@ -1,5 +1,5 @@
 type ClaimKind = "message" | "task" | "command";
-type RunnerWorkKind = "provider-turn" | "subagent";
+type RunnerWorkKind = "provider-turn" | "subagent" | "background-script";
 type RunnerBusyReason = RunnerWorkKind | ClaimKind;
 type RunnerSemanticStatus = "idle" | "busy" | "offline" | "error";
 

package/src/control-server.ts

@@ -439,12 +439,12 @@ async function handleStatus(req: Request, options: ControlServerOptions): Promis
     return Response.json({ error: "status must be idle, busy, offline, or error" }, { status: 400 });
   }
   const reason = body?.reason;
-  if (reason !== undefined && reason !== "provider-turn" && reason !== "subagent") {
-    return Response.json({ error: "reason must be provider-turn or subagent" }, { status: 400 });
+  if (reason !== undefined && reason !== "provider-turn" && reason !== "subagent" && reason !== "background-script") {
+    return Response.json({ error: "reason must be provider-turn, subagent, or background-script" }, { status: 400 });
   }
-  const clear = Array.isArray(body?.clear)
-    ? body.clear.filter((item): item is "provider-turn" | "subagent" => item === "provider-turn" || item === "subagent")
-    : undefined;
+  const isWorkKind = (item: unknown): item is "provider-turn" | "subagent" | "background-script" =>
+    item === "provider-turn" || item === "subagent" || item === "background-script";
+  const clear = Array.isArray(body?.clear) ? body.clear.filter(isWorkKind) : undefined;
   const timeline = statusTimelineEvent(body);
   const update: ProviderStatusEvent = {
     status,

package/src/runner-core.ts

@@ -919,6 +919,7 @@ export class AgentRunner {
       this.claims.clearTerminalStatus();
       this.claims.clearWorkKind("provider-turn");
       this.claims.clearWorkKind("subagent");
+      this.claims.clearWorkKind("background-script");
       if (this.stopped) return;
       this.process = await this.spawnProvider();
       this.processStartedAt = Date.now();
@@ -997,7 +998,7 @@ export class AgentRunner {
           status,
           reason: "provider-turn",
           id: `provider-exit-${this.providerSessionId}`,
-          clear: ["provider-turn", "subagent"],
+          clear: ["provider-turn", "subagent", "background-script"],
         });
         return;
       }