@agentdeck/bridge 1.1.0 → 1.2.0

package/dist/cli.js

@@ -28,6 +28,25 @@ function warnIfDaemonHostIgnored(opts) {
         console.error('[agentdeck] --daemon-host / AGENTDECK_DAEMON_HOST is set but ignored: remote attach requires --remote-daemon (or AGENTDECK_REMOTE_DAEMON=1). Session stays local-only.');
     }
 }
+/**
+ * User-visible compatibility notice for the legacy per-session bridge. Keep
+ * this pure so CLI tests can pin the public migration contract without starting
+ * a PTY or importing its native dependencies.
+ */
+export function managedPtyCompatibilityNotice(command) {
+    const directLaunch = command === 'monitor'
+        ? 'run `claude`, `codex`, or `opencode` normally'
+        : `run \`${command}\` directly`;
+    return [
+        `[agentdeck] LEGACY COMPATIBILITY MODE: \`agentdeck ${command}\` uses the managed per-session bridge; daemon-first launch is the recommended default.`,
+        `[agentdeck] For ordinary local sessions, run \`agentdeck daemon install\`, then ${directLaunch}.`,
+        '[agentdeck] --remote-daemon, --weight, AGENTDECK_<AGENT>_ARGS, and terminal-only controls do not yet have daemon-first equivalents. No removal date is set. Design: https://github.com/puritysb/AgentDeck/discussions/278 Tracking: https://github.com/puritysb/AgentDeck/issues/273',
+    ];
+}
+function warnManagedPtyCompatibility(command) {
+    for (const line of managedPtyCompatibilityNotice(command))
+        console.error(line);
+}
 function log(msg) {
     process.stderr.write(msg + '\n');
 }
@@ -323,6 +342,14 @@ for (const b of ESP32_OTA_BOARDS) {
 export function resolveEsp32OtaDaemonTarget(target) {
     return ESP32_OTA_BY_TARGET[target]?.board ?? target;
 }
+export function inlineOtaPayload(target, firmwareB64, opts = {}) {
+    return {
+        target,
+        firmwareB64,
+        ...(opts.stage ? { stage: true } : {}),
+        ...(opts.identity ?? {}),
+    };
+}
 /** Resolve one exact OTA namespace from an integration manifest. Ambiguity is
  * an error: a release command must never guess which product or channel owns
  * an image. */
@@ -552,7 +579,7 @@ weatherCommand
 // ===== Agent session commands =====
 program
     .command('claude')
-    .description('Start Claude Code session (PTY + bridge)')
+    .description('[legacy compatibility] Start Claude Code in the managed PTY session bridge')
     .option('-p, --port <port>', 'Bridge server port', String(BRIDGE_WS_PORT))
     .option('-c, --command <cmd>', 'Command to spawn', 'claude')
     .option('-d, --debug', 'Enable debug logging')
@@ -568,6 +595,7 @@ program
     .option('--no-env-args', 'Ignore AGENTDECK_COMMANDER_ARGS and AGENTDECK_<AGENT>_ARGS for this invocation')
     .option('--weight <n>', 'Deck/tab sort order override (integer, lower first; default 0)', parseWeight)
     .action(async (opts) => {
+    warnManagedPtyCompatibility('claude');
     warnIfDaemonHostIgnored(opts);
     const { startSession } = await import('./index.js');
     await startSession({
@@ -596,7 +624,7 @@ program
 });
 program
     .command('codex')
-    .description('Start Codex CLI session (PTY + bridge)')
+    .description('[legacy compatibility] Start Codex in the managed PTY session bridge')
     .option('-p, --port <port>', 'Bridge server port', String(BRIDGE_WS_PORT))
     .option('-c, --command <cmd>', 'Command to spawn', 'codex')
     .option('-d, --debug', 'Enable debug logging')
@@ -610,6 +638,7 @@ program
     .option('--no-env-args', 'Ignore AGENTDECK_COMMANDER_ARGS and AGENTDECK_<AGENT>_ARGS for this invocation')
     .option('--weight <n>', 'Deck/tab sort order override (integer, lower first; default 0)', parseWeight)
     .action(async (opts) => {
+    warnManagedPtyCompatibility('codex');
     warnIfDaemonHostIgnored(opts);
     // Install Codex lifecycle hooks before starting the session so the
     // first prompt's UserPromptSubmit / Stop events reach the daemon.
@@ -657,7 +686,7 @@ program
 });
 program
     .command('opencode')
-    .description('Start OpenCode session (PTY + SSE bridge)')
+    .description('[legacy compatibility] Start OpenCode in the managed PTY/SSE session bridge')
     .option('-p, --port <port>', 'Bridge server port', String(BRIDGE_WS_PORT))
     .option('-c, --command <cmd>', 'Command to spawn', 'opencode')
     .option('-d, --debug', 'Enable debug logging')
@@ -671,6 +700,7 @@ program
     .option('--no-env-args', 'Ignore AGENTDECK_COMMANDER_ARGS and AGENTDECK_<AGENT>_ARGS for this invocation')
     .option('--weight <n>', 'Deck/tab sort order override (integer, lower first; default 0)', parseWeight)
     .action(async (opts) => {
+    warnManagedPtyCompatibility('opencode');
     warnIfDaemonHostIgnored(opts);
     // Install the OpenCode observer plugin so standalone `opencode` runs
     // (outside this managed session) also reach the daemon timeline. The
@@ -715,7 +745,7 @@ program
 });
 program
     .command('monitor')
-    .description('Start hook-only bridge (no PTY — run claude separately)')
+    .description('[legacy compatibility] Start the managed hook-only per-session bridge')
     .option('-p, --port <port>', 'Bridge server port', String(BRIDGE_WS_PORT))
     .option('-d, --debug', 'Enable debug logging')
     .option('--local', 'Disable all device modules')
@@ -726,6 +756,7 @@ program
     .option('--no-env-args', 'Ignore AGENTDECK_COMMANDER_ARGS for this invocation')
     .option('--weight <n>', 'Deck/tab sort order override (integer, lower first; default 0)', parseWeight)
     .action(async (opts) => {
+    warnManagedPtyCompatibility('monitor');
     warnIfDaemonHostIgnored(opts);
     const { startSession } = await import('./index.js');
     const port = parseInt(opts.port, 10);
@@ -1889,7 +1920,17 @@ program
         // Pull-OTA staging: for wake-sync-sleep boards (XTeink X3/X4) that never
         // hold a live WS. The daemon adverts the build on the board's feed pulls;
         // the board downloads + flashes itself on its next wake.
-        const { statusCode, body } = await postJsonWithTimeout(`http://127.0.0.1:${port}/esp32/ota`, { target: daemonTarget, firmwarePath, stage: true, ...(otaIdentity ?? {}) }, 30_000);
+        let { statusCode, body } = await postJsonWithTimeout(`http://127.0.0.1:${port}/esp32/ota`, { target: daemonTarget, firmwarePath, stage: true, ...(otaIdentity ?? {}) }, 30_000);
+        if (body.ok === false && String(body.error ?? '').startsWith('firmware_unreadable')) {
+            // The App Store daemon cannot reopen an arbitrary CLI path. Preserve
+            // the stage bit and exact product namespace when retrying inline — the
+            // live-OTA retry below is a different operation and must not consume a
+            // pull-stage request by accident.
+            log('Daemon cannot read the firmware path (sandbox) — resending stage inline...');
+            const { readFileSync } = await import('fs');
+            const firmwareB64 = readFileSync(firmwarePath).toString('base64');
+            ({ statusCode, body } = await postJsonWithTimeout(`http://127.0.0.1:${port}/esp32/ota`, inlineOtaPayload(daemonTarget, firmwareB64, { stage: true, identity: otaIdentity }), 30_000));
+        }
         if (statusCode < 200 || statusCode >= 300 || body.ok === false) {
             throw new Error(String(body.error ?? `HTTP ${statusCode}`));
         }
@@ -1919,7 +1960,7 @@ program
         log('Daemon cannot read the firmware path (sandbox) — resending inline...');
         const { readFileSync } = await import('fs');
         const firmwareB64 = readFileSync(firmwarePath).toString('base64');
-        ({ statusCode, body } = await postJsonWithTimeout(`http://127.0.0.1:${port}/esp32/ota`, { target: daemonTarget, firmwareB64 }, 15 * 60_000));
+        ({ statusCode, body } = await postJsonWithTimeout(`http://127.0.0.1:${port}/esp32/ota`, inlineOtaPayload(daemonTarget, firmwareB64), 15 * 60_000));
     }
     if (statusCode < 200 || statusCode >= 300 || body.ok === false) {
         throw new Error(String(body.error ?? `HTTP ${statusCode}`));
@@ -2344,8 +2385,8 @@ program
     .option('--json', 'Print target diagnostics as machine-readable JSON')
     .action(async (target, opts) => {
     if (target) {
-        if (target !== 'kiro') {
-            log(`Unknown diagnostic target: ${target}. Supported target: kiro`);
+        if (target !== 'kiro' && target !== 'agents') {
+            log(`Unknown diagnostic target: ${target}. Supported targets: agents, kiro`);
             process.exitCode = 1;
             return;
         }
@@ -2354,6 +2395,12 @@ program
             process.exitCode = 1;
             return;
         }
+        if (target === 'agents') {
+            const { collectAgentCliDiagnosticReport, formatAgentCliDiagnosticReport } = await import('./agent-cli-diagnostics.js');
+            const report = collectAgentCliDiagnosticReport();
+            process.stdout.write(`${opts.json ? JSON.stringify(report, null, 2) : formatAgentCliDiagnosticReport(report)}\n`);
+            return;
+        }
         const { collectKiroDiagnosticReport, formatKiroDiagnosticReport } = await import('./kiro-diagnostics.js');
         const report = await collectKiroDiagnosticReport();
         process.stdout.write(`${opts.json ? JSON.stringify(report, null, 2) : formatKiroDiagnosticReport(report)}\n`);