agent-relay-orchestrator 0.27.2 → 0.29.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-relay-orchestrator",
-  "version": "0.27.2",
+  "version": "0.29.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.18"
   },
   "devDependencies": {
     "@types/bun": "latest",

package/src/control.ts

@@ -192,7 +192,7 @@ function shutdownTimeoutMs(ctrl: Record<string, any>): number | undefined {
 //     input and stays defensive against junk).
 //   - cwd: `source.cwd || baseDir` — an empty-string cwd now falls back to
 //     baseDir on the restart path too (was a latent bug; empty cwd is invalid).
-export function spawnOptionsFromRecord(source: Record<string, any>, config: OrchestratorConfig): SpawnOptions {
+function spawnOptionsFromRecord(source: Record<string, any>, config: OrchestratorConfig): SpawnOptions {
   return {
     provider: source.provider === "codex" ? "codex" : "claude",
     cwd: source.cwd || config.baseDir,

package/src/relay.ts

@@ -18,7 +18,7 @@ export interface RelayClient {
   connected: boolean;
 }
 
-export interface RunnerTokenRemint {
+interface RunnerTokenRemint {
   token: string;
   record: { jti: string; profileId?: string; expiresAt?: number };
 }

package/src/self-supervision.ts

@@ -32,7 +32,7 @@ export function detectSelfSupervision(moduleUrl: string = import.meta.url): Self
 }
 
 /** Reset the cache. Test-only. */
-export function resetSelfSupervisionCache(): void {
+function resetSelfSupervisionCache(): void {
   cached = undefined;
 }
 

package/src/self-upgrade.ts

@@ -16,7 +16,7 @@ const CACHE_RACE_RE = /ETARGET|No matching version found|notarget/i;
 const DEFAULT_INSTALL_RETRIES = 4;
 const DEFAULT_INSTALL_RETRY_BASE_MS = 2000;
 
-export interface SelfUpgradeOptions {
+interface SelfUpgradeOptions {
   /** Sleep between install retries (injectable for tests). */
   sleep?: (ms: number) => Promise<void>;
   /** Max extra install attempts after the first on a cache-race error. */
@@ -55,7 +55,7 @@ const defaultRunner: SelfUpgradeRunner = {
   },
 };
 
-export interface SelfUpgradePlan {
+interface SelfUpgradePlan {
   targetVersion: string;
   providers: string[];
   unit: string;

package/src/spawn.ts

@@ -51,7 +51,7 @@ interface SessionInfo {
   logFile: string;
 }
 
-export interface TerminalGuestSession {
+interface TerminalGuestSession {
   session: string;
   mode: "guest";
   provider: string;

package/src/terminal-stream.ts

@@ -144,7 +144,20 @@ export function decodeControlOutput(data: string): Uint8Array {
   return Uint8Array.from(out);
 }
 
-export type ControlLine =
+// 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);
+}
+
+type ControlLine =
   | { type: "output"; pane: string; bytes: Uint8Array }
   | { type: "exit"; reason?: string }
   // Command-reply block framing. Every command written to the control client's stdin is
@@ -191,7 +204,7 @@ export function parseControlLine(line: string): ControlLine {
 // scroll-region TUIs like Claude Code emit constantly). So we track whether the outbound
 // stream is at a sequence boundary (ground) and only inject there. This is a deliberately
 // minimal VT state machine: enough to know "are we mid-sequence?", not a full parser.
-export type AnsiState = "ground" | "esc" | "esc-charset" | "csi" | "string" | "string-esc";
+type AnsiState = "ground" | "esc" | "esc-charset" | "csi" | "string" | "string-esc";
 
 export function advanceAnsiState(state: AnsiState, byte: number): AnsiState {
   // CAN (0x18) / SUB (0x1a) abort any in-progress sequence from any state → ground.
@@ -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

@@ -441,7 +441,7 @@ export function refreshWorkspaceDeps(repoRoot: string, worktreePath: string, opt
  * install (full isolation, no shared cache), or =none to skip entirely.
  * Never throws — provisioning failure must not block the spawn.
  */
-export function provisionWorkspaceDeps(repoRoot: string, worktreePath: string): WorkspaceDepsProvision {
+function provisionWorkspaceDeps(repoRoot: string, worktreePath: string): WorkspaceDepsProvision {
   const requested = (process.env.AGENT_RELAY_WORKSPACE_DEPS || "symlink").toLowerCase();
   if (requested === "none") return { mode: "none" };
 
@@ -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