agent-relay-orchestrator 0.27.2 → 0.28.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-relay-orchestrator",
-  "version": "0.27.2",
+  "version": "0.28.0",
   "description": "Agent Relay orchestrator — manages agent lifecycle across hosts",
   "type": "module",
   "bin": {
@@ -16,7 +16,7 @@
     "test": "bun test"
   },
   "dependencies": {
-    "agent-relay-sdk": "0.2.16"
+    "agent-relay-sdk": "0.2.17"
   },
   "devDependencies": {
     "@types/bun": "latest",

package/src/terminal-stream.ts

@@ -144,6 +144,19 @@ export function decodeControlOutput(data: string): Uint8Array {
   return Uint8Array.from(out);
 }
 
+// The control stream is read as latin1 so the `%output` octal path above sees faithful
+// bytes. But command-reply blocks (capture-pane grid rows) arrive as raw UTF-8, so their
+// lines reach us as the byte-faithful latin1 representation. Re-decode them to real UTF-8
+// here, or multi-byte glyphs (box-drawing, powerline) double-encode on the next repaint
+// and render as mojibake (#270). Safe to do per line: no UTF-8 continuation byte is
+// 0x0A/0x0D, so the newline split never lands mid-sequence.
+const REPLY_UTF8_DECODER = new TextDecoder("utf-8");
+export function latin1LineToUtf8(line: string): string {
+  const bytes = new Uint8Array(line.length);
+  for (let i = 0; i < line.length; i++) bytes[i] = line.charCodeAt(i) & 0xff;
+  return REPLY_UTF8_DECODER.decode(bytes);
+}
+
 export type ControlLine =
   | { type: "output"; pane: string; bytes: Uint8Array }
   | { type: "exit"; reason?: string }
@@ -428,7 +441,7 @@ class SessionStream {
     if (!entry) return; // unexpected extra block — drop it rather than mis-correlate
     if (entry.timer) clearTimeout(entry.timer);
     if (isError) entry.reject(new Error(block.lines.join(" ").trim() || "tmux command error"));
-    else entry.resolve(block.lines);
+    else entry.resolve(block.lines.map(latin1LineToUtf8));
   }
 
   private enqueue(bytes: Uint8Array): void {

package/src/workspace-probe.ts

@@ -1086,7 +1086,9 @@ function workspaceId(input: WorkspaceResolutionInput): string {
 
 function branchName(input: WorkspaceResolutionInput, id: string): string {
   const owner = input.policyName || input.label || input.automationId || "manual";
-  return `agent/${safeSegment(owner, 48)}/${safeSegment(id.replace(/^sp[_-]?/, ""), 24)}`;
+  // 40 fits a full UUID (36) plus the `sp_`-stripped slack. A tighter cap (was 24)
+  // sliced the session UUID mid-string, leaving an unaddressable, dangling ref (#282).
+  return `agent/${safeSegment(owner, 48)}/${safeSegment(id.replace(/^sp[_-]?/, ""), 40)}`;
 }
 
 /** Next free `<branch>-N` cycle name for a recycled worktree (#206). Strips any