linkshell-cli 0.2.34 → 0.2.35

package/src/runtime/bridge-session.ts

@@ -15,7 +15,6 @@ import {
 import type { Envelope } from "@linkshell/protocol";
 import type { ProviderConfig } from "../providers.js";
 import { ScrollbackBuffer } from "./scrollback.js";
-import { AcpRelay } from "./acp-relay.js";
 import { ScreenFallback } from "./screen-fallback.js";
 import { ScreenShare } from "./screen-share.js";
 import { getLanIp } from "../utils/lan-ip.js";
@@ -42,9 +41,7 @@ const DEFAULT_TERMINAL_ID = "default";
 
 interface TerminalInstance {
   id: string;
-  pty: pty.IPty | null;
-  acpRelay: AcpRelay | null;
-  mode: "pty" | "acp";
+  pty: pty.IPty;
   cwd: string;
   projectName: string;
   provider: string;
@@ -284,29 +281,21 @@ export class BridgeSession {
       case "terminal.input": {
         const p = parseTypedPayload("terminal.input", envelope.payload);
         const term = this.terminals.get(tid);
-        if (term && term.status === "running" && term.pty) term.pty.write(p.data);
+        if (term && term.status === "running") term.pty.write(p.data);
         break;
       }
       case "terminal.resize": {
         const p = parseTypedPayload("terminal.resize", envelope.payload);
         const term = this.terminals.get(tid);
-        if (term && term.status === "running" && term.pty) term.pty.resize(p.cols, p.rows);
-        break;
-      }
-      case "acp.rpc": {
-        const term = this.terminals.get(tid);
-        if (term && term.status === "running" && term.acpRelay) {
-          term.acpRelay.write(envelope.payload);
-        }
+        if (term && term.status === "running") term.pty.resize(p.cols, p.rows);
         break;
       }
       case "terminal.spawn": {
         const p = parseTypedPayload("terminal.spawn", envelope.payload);
         const normalizedCwd = resolve(p.cwd);
-        const mode = p.mode ?? "pty";
-        // Dedup: if a running terminal already exists for this cwd and mode, return it
+        // Dedup: if a running terminal already exists for this cwd, return it
         const existing = [...this.terminals.values()].find(
-          (t) => t.status === "running" && resolve(t.cwd) === normalizedCwd && t.mode === mode,
+          (t) => t.status === "running" && resolve(t.cwd) === normalizedCwd,
         );
         if (existing) {
           this.send(createEnvelope({
@@ -318,7 +307,7 @@ export class BridgeSession {
         } else {
           const newId = `term-${Date.now()}-${Math.random().toString(36).slice(2, 6)}`;
           try {
-            await this.spawnTerminal(newId, normalizedCwd, p.provider, mode);
+            await this.spawnTerminal(newId, normalizedCwd, p.provider);
             this.send(createEnvelope({
               type: "terminal.spawned",
               sessionId: this.sessionId,
@@ -342,8 +331,7 @@ export class BridgeSession {
         const p = parseTypedPayload("terminal.kill", envelope.payload);
         const term = this.terminals.get(p.terminalId);
         if (term && term.status === "running") {
-          if (term.pty) term.pty.kill();
-          if (term.acpRelay) term.acpRelay.kill();
+          term.pty.kill();
         }
         break;
       }
@@ -425,7 +413,7 @@ export class BridgeSession {
         writeFileSync(tempPath, Buffer.from(p.data, "base64"));
         this.log(`image saved to ${tempPath}`);
         const term = this.terminals.get(tid);
-        if (term && term.status === "running" && term.pty) {
+        if (term && term.status === "running") {
           term.pty.write(`\x1b[200~${tempPath}\x1b[201~`);
         }
         break;
@@ -442,7 +430,6 @@ export class BridgeSession {
       projectName: t.projectName,
       provider: t.provider,
       status: t.status,
-      mode: t.mode,
     }));
     this.send(createEnvelope({
       type: "terminal.list",
@@ -473,7 +460,7 @@ export class BridgeSession {
     }
   }
 
-  private async spawnTerminal(terminalId: string, cwd: string, providerOverride?: string, mode: "pty" | "acp" = "pty"): Promise<void> {
+  private async spawnTerminal(terminalId: string, cwd: string, providerOverride?: string): Promise<void> {
     const cleanEnv: Record<string, string> = {};
     for (const [k, v] of Object.entries(this.options.providerConfig.env)) {
       if (v !== undefined) cleanEnv[k] = v;
@@ -482,58 +469,6 @@ export class BridgeSession {
     const provider = providerOverride ?? this.options.providerConfig.provider;
     const args = [...this.options.providerConfig.args];
 
-    // ACP mode: spawn as child process with JSON-RPC over stdio
-    if (mode === "acp") {
-      const term: TerminalInstance = {
-        id: terminalId,
-        pty: null,
-        acpRelay: new AcpRelay({
-          command: this.options.providerConfig.command,
-          args,
-          cwd,
-          env: cleanEnv,
-          sessionId: this.sessionId,
-          terminalId,
-          send: (envelope) => this.send(envelope),
-          log: (msg) => this.log(msg),
-        }),
-        mode: "acp",
-        cwd,
-        projectName: basename(cwd),
-        provider,
-        scrollback: new ScrollbackBuffer(1000),
-        outputSeq: 0,
-        statusSeq: 0,
-        status: "running",
-      };
-
-      term.acpRelay!.onExit((exitCode, signal) => {
-        term.status = "exited";
-        this.send(createEnvelope({
-          type: "terminal.exit",
-          sessionId: this.sessionId,
-          terminalId,
-          payload: { exitCode, signal: signal ? parseInt(signal, 10) || 0 : 0 },
-        }));
-        this.sendTerminalList();
-
-        const allExited = [...this.terminals.values()].every((t) => t.status === "exited");
-        if (allExited) {
-          this.exited = true;
-          setTimeout(() => {
-            this.stopHeartbeat();
-            this.socket?.close();
-          }, 500);
-          process.exitCode = exitCode ?? 0;
-        }
-      });
-
-      this.terminals.set(terminalId, term);
-      this.log(`spawned ACP terminal ${terminalId} in ${cwd}`);
-      return;
-    }
-
-    // PTY mode: existing behavior
     // Set up hook server for structured status (all supported providers)
     // For "custom" shell, set up hooks for all providers since user may launch any of them
     let hookServer: http.Server | undefined;
@@ -570,8 +505,6 @@ export class BridgeSession {
           env: cleanEnv,
         },
       ),
-      acpRelay: null,
-      mode: "pty",
       cwd,
       projectName: basename(cwd),
       provider,
@@ -584,7 +517,7 @@ export class BridgeSession {
       hookConfigPath,
     };
 
-    term.pty!.onData((data) => {
+    term.pty.onData((data) => {
       const seq = term.outputSeq++;
       const envelope = createEnvelope({
         type: "terminal.output",
@@ -603,7 +536,7 @@ export class BridgeSession {
       this.send(envelope);
     });
 
-    term.pty!.onExit(({ exitCode, signal }) => {
+    term.pty.onExit(({ exitCode, signal }) => {
       term.status = "exited";
       this.cleanupHookServer(term);
       this.send(createEnvelope({
@@ -1191,10 +1124,7 @@ export class BridgeSession {
     this.socket = undefined;
     for (const term of this.terminals.values()) {
       this.cleanupHookServer(term);
-      if (term.status === "running") {
-        if (term.pty) term.pty.kill();
-        if (term.acpRelay) term.acpRelay.kill();
-      }
+      if (term.status === "running") term.pty.kill();
     }
     this.terminals.clear();
     process.exitCode = exitCode;

package/src/runtime/acp-relay.ts

@@ -1,92 +0,0 @@
-import { spawn, type ChildProcess } from "node:child_process";
-import { createInterface } from "node:readline";
-import { createEnvelope, serializeEnvelope } from "@linkshell/protocol";
-import type { Envelope } from "@linkshell/protocol";
-
-export interface AcpRelayOptions {
-  command: string;
-  args: string[];
-  cwd: string;
-  env: Record<string, string>;
-  sessionId: string;
-  terminalId: string;
-  send: (envelope: Envelope) => void;
-  log: (msg: string) => void;
-}
-
-export class AcpRelay {
-  private child: ChildProcess;
-  private readonly options: AcpRelayOptions;
-  private exited = false;
-
-  constructor(options: AcpRelayOptions) {
-    this.options = options;
-
-    this.child = spawn(options.command, options.args, {
-      cwd: options.cwd,
-      env: options.env,
-      stdio: ["pipe", "pipe", "pipe"],
-    });
-
-    // Read newline-delimited JSON from stdout
-    const rl = createInterface({ input: this.child.stdout! });
-    rl.on("line", (line) => {
-      if (!line.trim()) return;
-      try {
-        const msg = JSON.parse(line);
-        // Wrap JSON-RPC message in acp.rpc envelope and send to gateway
-        this.options.send(createEnvelope({
-          type: "acp.rpc",
-          sessionId: this.options.sessionId,
-          terminalId: this.options.terminalId,
-          payload: msg,
-        }));
-      } catch (e) {
-        this.options.log(`acp-relay: failed to parse stdout line: ${e}`);
-      }
-    });
-
-    // Log stderr
-    this.child.stderr?.on("data", (chunk: Buffer) => {
-      this.options.log(`acp-relay stderr: ${chunk.toString().trimEnd()}`);
-    });
-
-    this.child.on("exit", (code, signal) => {
-      this.exited = true;
-      this.options.log(`acp-relay: process exited (code=${code}, signal=${signal})`);
-    });
-
-    this.child.on("error", (err) => {
-      this.options.log(`acp-relay: process error: ${err.message}`);
-    });
-  }
-
-  /** Forward an acp.rpc envelope payload to the child's stdin */
-  write(payload: unknown): void {
-    if (this.exited || !this.child.stdin?.writable) return;
-    try {
-      const line = JSON.stringify(payload) + "\n";
-      this.child.stdin.write(line);
-    } catch (e) {
-      this.options.log(`acp-relay: failed to write to stdin: ${e}`);
-    }
-  }
-
-  kill(): void {
-    if (!this.exited) {
-      this.child.kill("SIGTERM");
-    }
-  }
-
-  get isExited(): boolean {
-    return this.exited;
-  }
-
-  get pid(): number | undefined {
-    return this.child.pid;
-  }
-
-  onExit(cb: (code: number | null, signal: string | null) => void): void {
-    this.child.on("exit", cb);
-  }
-}