agent-relay 2.0.18 → 2.0.20

package/packages/wrapper/dist/relay-pty-orchestrator.js

@@ -87,6 +87,8 @@ export class RelayPtyOrchestrator extends BaseWrapper {
     sessionStartTime = 0;
     // Track if agent is being gracefully stopped (vs crashed)
     isGracefulStop = false;
+    // Track early process exit for better error messages
+    earlyExitInfo;
     // Memory/CPU monitoring
     memoryMonitor;
     memoryAlertHandler = null;
@@ -383,7 +385,13 @@ export class RelayPtyOrchestrator extends BaseWrapper {
             throw new Error('relay-pty binary not found. Build with: cd relay-pty && cargo build --release');
         }
         this.log(` Using binary: ${binaryPath}`);
-        // Connect to relay daemon first
+        // Spawn relay-pty process FIRST (before connecting to daemon)
+        // This ensures the CLI is actually running before we register with the daemon
+        await this.spawnRelayPty(binaryPath);
+        // Wait for socket to become available and connect
+        await this.connectToSocket();
+        // Connect to relay daemon AFTER CLI is spawned
+        // This prevents the spawner from seeing us as "registered" before the CLI runs
         try {
             await this.client.connect();
             this.log(` Relay daemon connected`);
@@ -391,10 +399,6 @@ export class RelayPtyOrchestrator extends BaseWrapper {
         catch (err) {
             this.logError(` Relay connect failed: ${err.message}`);
         }
-        // Spawn relay-pty process
-        await this.spawnRelayPty(binaryPath);
-        // Wait for socket to become available and connect
-        await this.connectToSocket();
         this.running = true;
         // DON'T set readyForMessages yet - wait for CLI to be ready first
         // This prevents messages from being injected during CLI startup
@@ -540,6 +544,8 @@ export class RelayPtyOrchestrator extends BaseWrapper {
             ...(this.config.args ?? []),
         ];
         this.log(` Spawning: ${binaryPath} ${args.join(' ')}`);
+        // Reset early exit info from any previous spawn attempt
+        this.earlyExitInfo = undefined;
         // For interactive mode, let Rust directly inherit stdin/stdout from the terminal
         // This is more robust than manual forwarding through pipes
         // We still pipe stderr to capture JSON parsed commands
@@ -566,10 +572,14 @@ export class RelayPtyOrchestrator extends BaseWrapper {
                 this.handleOutput(text);
             });
         }
+        // Capture stderr for early exit diagnosis
+        let stderrBuffer = '';
         // Handle stderr (relay-pty logs and JSON output) - always needed
+        // Also captures to buffer for error diagnostics if process dies early
         if (proc.stderr) {
             proc.stderr.on('data', (data) => {
                 const text = data.toString();
+                stderrBuffer += text;
                 this.handleStderr(text);
             });
         }
@@ -577,6 +587,10 @@ export class RelayPtyOrchestrator extends BaseWrapper {
         proc.on('exit', (code, signal) => {
             const exitCode = code ?? (signal === 'SIGKILL' ? 137 : 1);
             this.log(` Process exited: code=${exitCode} signal=${signal}`);
+            // Capture early exit info for better error messages if socket not yet connected
+            if (!this.socketConnected) {
+                this.earlyExitInfo = { code, signal, stderr: stderrBuffer };
+            }
             this.running = false;
             // Get crash context before unregistering from memory monitor
             const crashContext = this.memoryMonitor.getCrashContext(this.config.name);
@@ -942,6 +956,22 @@ export class RelayPtyOrchestrator extends BaseWrapper {
     // =========================================================================
     // Socket communication
     // =========================================================================
+    /**
+     * Check if the relay-pty process is still alive
+     */
+    isProcessAlive() {
+        if (!this.relayPtyProcess || this.relayPtyProcess.exitCode !== null) {
+            return false;
+        }
+        try {
+            // Signal 0 checks if process exists without killing it
+            process.kill(this.relayPtyProcess.pid, 0);
+            return true;
+        }
+        catch {
+            return false;
+        }
+    }
     /**
      * Connect to the relay-pty socket
      */
@@ -949,6 +979,20 @@ export class RelayPtyOrchestrator extends BaseWrapper {
         const timeout = this.config.socketConnectTimeoutMs ?? 5000;
         const maxAttempts = this.config.socketReconnectAttempts ?? 3;
         for (let attempt = 1; attempt <= maxAttempts; attempt++) {
+            // Check if relay-pty process died before attempting connection
+            if (!this.isProcessAlive()) {
+                const exitInfo = this.earlyExitInfo;
+                if (exitInfo) {
+                    const exitReason = exitInfo.signal
+                        ? `signal ${exitInfo.signal}`
+                        : `code ${exitInfo.code ?? 'unknown'}`;
+                    const stderrHint = exitInfo.stderr
+                        ? `\n  stderr: ${exitInfo.stderr.trim().slice(0, 500)}`
+                        : '';
+                    throw new Error(`relay-pty process died early (${exitReason}).${stderrHint}`);
+                }
+                throw new Error('relay-pty process died before socket could be created');
+            }
             try {
                 await this.attemptSocketConnection(timeout);
                 this.log(` Socket connected`);
@@ -961,6 +1005,17 @@ export class RelayPtyOrchestrator extends BaseWrapper {
                 }
             }
         }
+        // Final check for process death after all attempts
+        if (!this.isProcessAlive() && this.earlyExitInfo) {
+            const exitInfo = this.earlyExitInfo;
+            const exitReason = exitInfo.signal
+                ? `signal ${exitInfo.signal}`
+                : `code ${exitInfo.code ?? 'unknown'}`;
+            const stderrHint = exitInfo.stderr
+                ? `\n  stderr: ${exitInfo.stderr.trim().slice(0, 500)}`
+                : '';
+            throw new Error(`relay-pty process died during socket connection (${exitReason}).${stderrHint}`);
+        }
         throw new Error(`Failed to connect to socket after ${maxAttempts} attempts`);
     }
     /**

package/packages/wrapper/dist/tmux-wrapper.js

@@ -805,6 +805,22 @@ export class TmuxWrapper extends BaseWrapper {
                 requires_ack: cmd.meta.ackRequired,
             };
         }
+        // Check if target is a channel (starts with #)
+        if (cmd.to.startsWith('#')) {
+            // Use CHANNEL_MESSAGE protocol for channel targets
+            this.logStderr(`→ [channel] ${cmd.to}: ${cmd.body.substring(0, Math.min(RELAY_LOG_TRUNCATE_LENGTH, cmd.body.length))}...`);
+            const success = this.client.sendChannelMessage(cmd.to, cmd.body, {
+                thread: cmd.thread,
+                data: cmd.data,
+            });
+            if (success) {
+                this.sentMessageHashes.add(msgHash);
+                this.queuedMessageHashes.delete(msgHash);
+                this.trajectory?.message('sent', this.config.name, cmd.to, cmd.body);
+            }
+            return;
+        }
+        // Use SEND protocol for direct messages and broadcasts
         if (cmd.sync?.blocking) {
             this.client.sendAndWait(cmd.to, cmd.body, {
                 timeoutMs: cmd.sync.timeoutMs,

package/packages/wrapper/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-relay/wrapper",
-  "version": "2.0.18",
+  "version": "2.0.20",
   "description": "CLI agent wrappers for Agent Relay - tmux, pty integration",
   "type": "module",
   "main": "dist/index.js",
@@ -30,11 +30,11 @@
     "clean": "rm -rf dist"
   },
   "dependencies": {
-    "@agent-relay/api-types": "2.0.18",
-    "@agent-relay/protocol": "2.0.18",
-    "@agent-relay/config": "2.0.18",
-    "@agent-relay/continuity": "2.0.18",
-    "@agent-relay/resiliency": "2.0.18"
+    "@agent-relay/api-types": "2.0.20",
+    "@agent-relay/protocol": "2.0.20",
+    "@agent-relay/config": "2.0.20",
+    "@agent-relay/continuity": "2.0.20",
+    "@agent-relay/resiliency": "2.0.20"
   },
   "devDependencies": {
     "typescript": "^5.9.3",
@@ -59,5 +59,5 @@
     "url": "git+https://github.com/AgentWorkforce/relay.git",
     "directory": "packages/wrapper"
   },
-  "license": "MIT"
+  "license": "Apache-2.0"
 }

package/scripts/hooks/install.sh

@@ -0,0 +1,16 @@
+#!/bin/sh
+# Install git hooks for agent-relay
+# Usage: ./scripts/hooks/install.sh
+
+SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
+REPO_ROOT="$(cd "$SCRIPT_DIR/../.." && pwd)"
+HOOKS_DIR="$REPO_ROOT/.git/hooks"
+
+echo "Installing git hooks..."
+
+# Install pre-commit hook
+cp "$SCRIPT_DIR/pre-commit" "$HOOKS_DIR/pre-commit"
+chmod +x "$HOOKS_DIR/pre-commit"
+echo "  Installed pre-commit hook"
+
+echo "Done! Git hooks installed."

package/scripts/hooks/pre-commit

@@ -0,0 +1,60 @@
+#!/bin/sh
+# Pre-commit hook for agent-relay
+# Handles Rust formatting and bd (beads) sync
+#
+# Install: cp scripts/hooks/pre-commit .git/hooks/pre-commit && chmod +x .git/hooks/pre-commit
+# Or run: npm run hooks:install
+
+# ============================================================
+# Rust formatting (auto-format staged .rs files)
+# ============================================================
+RUST_STAGED=$(git diff --cached --name-only --diff-filter=ACM | grep '\.rs$' || true)
+if [ -n "$RUST_STAGED" ]; then
+    if command -v rustfmt >/dev/null 2>&1; then
+        echo "Formatting Rust files..."
+        for file in $RUST_STAGED; do
+            if [ -f "$file" ]; then
+                rustfmt "$file" 2>/dev/null || true
+                git add "$file"
+            fi
+        done
+    else
+        echo "Warning: rustfmt not found, skipping Rust formatting" >&2
+        echo "Install with: rustup component add rustfmt" >&2
+    fi
+fi
+
+# ============================================================
+# bd (beads) sync - flush pending changes before commit
+# ============================================================
+# Check if bd is available
+if ! command -v bd >/dev/null 2>&1; then
+    exit 0
+fi
+
+# Check if we're in a bd workspace
+if [ ! -d .beads ]; then
+    exit 0
+fi
+
+# Check if sync-branch is configured (if so, .beads goes to separate branch)
+SYNC_BRANCH="${BEADS_SYNC_BRANCH:-}"
+if [ -z "$SYNC_BRANCH" ] && [ -f .beads/config.yaml ]; then
+    SYNC_BRANCH=$(grep -E '^sync-branch:' .beads/config.yaml 2>/dev/null | head -1 | sed 's/^sync-branch:[[:space:]]*//' | sed 's/^["'"'"']//' | sed 's/["'"'"']$//')
+fi
+if [ -n "$SYNC_BRANCH" ]; then
+    exit 0
+fi
+
+# Flush pending changes
+if ! bd sync --flush-only >/dev/null 2>&1; then
+    echo "Error: Failed to flush bd changes" >&2
+    exit 1
+fi
+
+# Stage JSONL files
+for f in .beads/beads.jsonl .beads/issues.jsonl .beads/deletions.jsonl; do
+    [ -f "$f" ] && git add "$f" 2>/dev/null || true
+done
+
+exit 0

package/scripts/postinstall.js

@@ -73,6 +73,36 @@ function getRelayPtyBinaryName() {
   return `relay-pty-${targetPlatform}-${targetArch}`;
 }
 
+/**
+ * Re-sign a binary with ad-hoc signature on macOS.
+ * This is required because macOS code signature validation can fail
+ * when binaries are copied/downloaded, causing SIGKILL on execution.
+ *
+ * The codesign tool is always available on macOS (part of the system).
+ * This is a common solution for npm packages distributing native binaries.
+ * Similar approach is used by esbuild, swc, and other Rust/Go tools.
+ *
+ * @param {string} binaryPath - Path to the binary to sign
+ * @returns {boolean} - Whether signing succeeded
+ */
+function resignBinaryForMacOS(binaryPath) {
+  if (os.platform() !== 'darwin') {
+    return true; // Only needed on macOS
+  }
+
+  try {
+    // codesign is always available on macOS as a system utility
+    execSync(`codesign --force --sign - "${binaryPath}"`, { stdio: 'pipe' });
+    return true;
+  } catch (err) {
+    // This shouldn't happen on a normal macOS system, but handle gracefully
+    warn(`Failed to re-sign binary: ${err.message}`);
+    warn('The binary may fail to execute due to code signature issues.');
+    warn('You can manually fix this by running: codesign --force --sign - ' + binaryPath);
+    return false;
+  }
+}
+
 /**
  * Install the relay-pty binary for the current platform
  */
@@ -103,6 +133,9 @@ function installRelayPtyBinary() {
       const sourceStats = fs.statSync(sourcePath);
       const targetStats = fs.statSync(targetPath);
       if (sourceStats.size === targetStats.size) {
+        // Re-sign even if already installed to ensure signature is valid
+        // This fixes issues where previous installs have invalid signatures
+        resignBinaryForMacOS(targetPath);
         info('relay-pty binary already installed');
         return true;
       }
@@ -115,7 +148,14 @@ function installRelayPtyBinary() {
   try {
     fs.copyFileSync(sourcePath, targetPath);
     fs.chmodSync(targetPath, 0o755);
-    success(`Installed relay-pty binary for ${os.platform()}-${os.arch()}`);
+
+    // Re-sign the binary on macOS to prevent code signature validation failures
+    // Without this, macOS may SIGKILL the process immediately on execution
+    if (resignBinaryForMacOS(targetPath)) {
+      success(`Installed relay-pty binary for ${os.platform()}-${os.arch()}`);
+    } else {
+      warn(`Installed relay-pty binary but signing failed - may not work on macOS`);
+    }
     return true;
   } catch (err) {
     warn(`Failed to install relay-pty binary: ${err.message}`);