agent-relay-runner 0.119.10 → 0.120.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-relay-runner",
-  "version": "0.119.10",
+  "version": "0.120.0",
   "description": "Unified provider lifecycle runner for Agent Relay",
   "type": "module",
   "bin": {
@@ -21,7 +21,7 @@
   },
   "dependencies": {
     "agent-relay-providers": "0.104.3",
-    "agent-relay-sdk": "0.2.106",
+    "agent-relay-sdk": "0.2.107",
     "callmux": "0.23.0"
   },
   "devDependencies": {

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

package/src/adapters/codex-agent-message-capture.ts

@@ -2,6 +2,10 @@ type EmitAgentMessage = (body: string) => void;
 type ResponseEntry = { itemId?: string; body: string };
 type CompleteItemOptions = { includeInResponse?: boolean; phase?: unknown };
 
+const MAX_CAPTURED_RESPONSE_CHARS = 16_000;
+const TRUNCATION_MARKER_PREFIX = "[truncated:";
+const TERSE_FINAL_MAX_CHARS = 280;
+
 export class CodexAgentMessageCapture {
   private segments: ResponseEntry[][] = [];
   private currentSegment: ResponseEntry[] = [];
@@ -44,9 +48,10 @@ export class CodexAgentMessageCapture {
 
   finish(emit: EmitAgentMessage): string | undefined {
     this.closeSegment(emit);
-    const finalSegment = this.segments.at(-1);
+    const selectedSegments = selectResponseSegments(this.segments);
     this.segments = [];
-    return finalSegment?.length ? finalSegment.map((entry) => entry.body).join("\n\n").trim() : undefined;
+    const text = selectedSegments.length ? selectedSegments.map(segmentBody).join("\n\n").trim() : undefined;
+    return truncateCapturedResponse(text);
   }
 
   private flushPending(emit: EmitAgentMessage): void {
@@ -90,3 +95,45 @@ export class CodexAgentMessageCapture {
       .filter((segment) => segment.length > 0);
   }
 }
+
+function selectResponseSegments(segments: ResponseEntry[][]): ResponseEntry[][] {
+  const nonEmptySegments = segments.filter((segment) => segment.length > 0);
+  const finalSegment = nonEmptySegments.at(-1);
+  if (!finalSegment) return [];
+  const finalBody = segmentBody(finalSegment);
+  if (!isTerseTerminalStatus(finalBody)) return [finalSegment];
+
+  // Codex often posts the real wrap-up across multiple assistant segments before a
+  // final workspace-ready/status line. Keep contiguous substantive segments after
+  // the last procedural assistant segment, then append the terminal status.
+  const selectedSegments: ResponseEntry[][] = [];
+  for (let index = nonEmptySegments.length - 2; index >= 0; index -= 1) {
+    const candidate = nonEmptySegments[index];
+    if (!candidate || !isSubstantiveResponseSegment(segmentBody(candidate))) break;
+    selectedSegments.unshift(candidate);
+  }
+  return [...selectedSegments, finalSegment];
+}
+
+function segmentBody(segment: ResponseEntry[]): string {
+  return segment.map((entry) => entry.body).join("\n\n").trim();
+}
+
+function isTerseTerminalStatus(body: string): boolean {
+  if (body.length > TERSE_FINAL_MAX_CHARS) return false;
+  if (/\n\s*(?:[-*]|\d+\.)\s+/.test(body)) return false;
+  return /\b(?:landed|ready for review|handed off|no further action|done|complete|completed|committed|merged)\b/i.test(body);
+}
+
+function isSubstantiveResponseSegment(body: string): boolean {
+  if (body.length >= 120) return true;
+  if (/\n\s*(?:[-*]|\d+\.)\s+/.test(body)) return true;
+  if (/\*\*(?:what changed|changes?|tests?|gates?|verification|risks?|residual risk|summary)\b/i.test(body)) return true;
+  if (/^#{1,3}\s+\S/m.test(body)) return true;
+  return body.split(/\n{2,}/).length >= 2;
+}
+
+function truncateCapturedResponse(text: string | undefined): string | undefined {
+  if (!text || text.length <= MAX_CAPTURED_RESPONSE_CHARS) return text;
+  return `${text.slice(0, MAX_CAPTURED_RESPONSE_CHARS).trimEnd()}\n\n${TRUNCATION_MARKER_PREFIX} showing first ${MAX_CAPTURED_RESPONSE_CHARS} of ${text.length} chars]`;
+}

package/src/adapters/codex.ts

@@ -533,9 +533,9 @@ export class CodexAdapter implements ProviderAdapter {
   }
 
   // Turn one completed Codex thread item into a session-mirror event. agentMessage text is
-  // mirrored as narration as it lands; only the final contiguous assistant segment is repeated
-  // as the turn-final response. The rest (user prompt echo, reasoning, tool steps) is surfaced
-  // as it lands.
+  // mirrored as narration as it lands; the captured turn-final response repeats the substantive
+  // assistant answer, including a substantive wrap-up segment before a terse terminal status.
+  // The rest (user prompt echo, reasoning, tool steps) is surfaced as it lands.
   private handleCodexItem(item: Record<string, unknown> | undefined): void {
     if (!item) return;
     const type = stringValue(item.type);

package/src/relay-mcp.ts

@@ -67,13 +67,16 @@ export function tomlString(value: string): string {
 // project-cwd `.mcp.json` is never discovered (its enable modal can't wedge a headless
 // agent). Whatever the assembler decides host mode should keep is injected here explicitly;
 // strict then preserves exactly that set and nothing else.
-import type { ProvisioningMcpServer } from "agent-relay-sdk";
+import { SHARED_CALLMUX_TOOL_CALL_TIMEOUT_MS, type ProvisioningMcpServer } from "agent-relay-sdk";
 
 // The shared-listener bridge as a provider-neutral stdio server descriptor: a `callmux bridge`
 // child fronting the host's shared listener, with per-agent `--cwd "$WORKTREE"` for session-cwd
 // isolation (#672). It flows through the existing claude/codex stdio injection unchanged.
 export function sharedMcpBridgeServer(url: string, cwd: string): ProvisioningMcpServer {
-  return { command: "callmux", args: ["bridge", "--url", url, "--cwd", cwd] };
+  return {
+    command: "callmux",
+    args: ["bridge", "--url", url, "--cwd", cwd, "--call-timeout", String(SHARED_CALLMUX_TOOL_CALL_TIMEOUT_MS)],
+  };
 }
 
 function claudeMcpServerEntry(server: ProvisioningMcpServer): Record<string, unknown> {