agent-relay-runner 0.57.1 → 0.58.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-relay-runner",
-  "version": "0.57.1",
+  "version": "0.58.0",
   "description": "Unified provider lifecycle runner for Agent Relay",
   "type": "module",
   "bin": {
@@ -20,7 +20,7 @@
     "directory": "runner"
   },
   "dependencies": {
-    "agent-relay-sdk": "0.2.33"
+    "agent-relay-sdk": "0.2.34"
   },
   "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.57.1",
+  "version": "0.58.0",
   "agentRelayContracts": {
     "providerPluginProtocol": 1
   }

package/src/control-server.ts

@@ -2,6 +2,7 @@ import type { Server, ServerWebSocket } from "bun";
 import type { Message, ReplyObligation } from "agent-relay-sdk";
 import { errMessage, isRecord } from "agent-relay-sdk";
 import type { ProviderPermissionDecisionInput, ProviderStatusEvent, SemanticStatus, TerminalAttachSpec } from "./adapter";
+import { obligationRequiresExplicitReply } from "./reply-obligation-cache";
 import { logger, parseLogLevel, LOG_LEVELS } from "./logger";
 import { buildRateLimitProviderState } from "./rate-limit";
 
@@ -199,7 +200,13 @@ function replyObligationSummary(obligations: ReplyObligation[]): Record<string,
 }
 
 function replyObligationStopDecision(obligations: ReplyObligation[]): Record<string, unknown> {
-  const summary = replyObligationSummary(obligations);
+  // #418 — never block the turn to nag for a `user` obligation: the operator already receives
+  // this turn via the session mirror, so an explicit /reply double-delivers. Only agent↔agent /
+  // spawner obligations (#413) gate the turn here; the mirror's replyTo-linking clears the user
+  // one. (The /reply-obligations summary endpoint is left unfiltered — the obligation IS pending
+  // until the mirror lands, so observability stays accurate.)
+  const enforceable = obligations.filter(obligationRequiresExplicitReply);
+  const summary = replyObligationSummary(enforceable);
   if (summary.pending !== true) return {};
   return {
     decision: "block",

package/src/reply-obligation-cache.ts

@@ -16,6 +16,17 @@ import { logger } from "./logger";
 // - A background interval keeps the snapshot warm; `markDirty()` requests an extra,
 //   debounced refresh when state likely just changed (a message arrived, a turn ended).
 
+// #418 — the human `user` sink is NOT delivered via an explicit relay reply: the agent's
+// turn is mirrored to the dashboard chat by the session-mirror lane (always, for managed
+// agents). So an explicit `/reply` to `user` double-delivers — the operator sees the mirrored
+// turn AND the explicit copy. Only agent↔agent / spawner obligations (#413) need an explicit
+// reply to clear and reach their recipient. This is the single home for that distinction —
+// both the Claude Stop-hook nag (control-server) and the non-claude re-injection (runner)
+// gate on it, so the rule must not drift between them.
+export function obligationRequiresExplicitReply(obligation: ReplyObligation): boolean {
+  return obligation.from !== "user";
+}
+
 type ReplyObligationFetch = () => Promise<ReplyObligation[]>;
 
 interface ReplyObligationCacheOptions {

package/src/runner.ts

@@ -11,7 +11,7 @@ import { messagesWithCachedAttachments } from "./attachment-cache";
 import { resolveProjectName } from "./config";
 import { ClaimTracker } from "./claim-tracker";
 import { startControlServer, type ControlServer } from "./control-server";
-import { ReplyObligationCache } from "./reply-obligation-cache";
+import { ReplyObligationCache, obligationRequiresExplicitReply } from "./reply-obligation-cache";
 import { Outbox, type OutboxRecord } from "./outbox";
 import { extractLastAssistantTurn, extractFinalAssistantMessage, extractHookAssistantMessage, extractLatestTurnSteps, stepDedupKeys, transcriptLooksComplete } from "./adapters/claude-transcript";
 import { computeContextRatio } from "./session-insights";
@@ -1215,9 +1215,11 @@ export class AgentRunner {
   // stop.sh hook; every other provider needs this in-process nudge.
   private async reInjectPendingObligation(): Promise<void> {
     if (!this.process) return;
-    // Filter to spawner obligations only (from ≠ "user") — user obligations go through
-    // the existing codex prompt-capture path (publishProviderSessionEvent).
-    const obligations = this.obligationCache.get().filter((o) => o.from !== "user");
+    // Filter to spawner obligations only — user obligations go through the existing codex
+    // prompt-capture path (publishProviderSessionEvent) and the session mirror (#418), never
+    // an explicit /reply. Shared predicate so this rule stays in lockstep with the Claude
+    // Stop-hook nag exemption (control-server.replyObligationStopDecision).
+    const obligations = this.obligationCache.get().filter(obligationRequiresExplicitReply);
     if (!obligations.length) return;
     const MAX_REINJECTIONS = 3;
     for (const obligation of obligations) {