ghosty-acp 0.0.2 → 0.0.4

package/README.md

@@ -52,6 +52,23 @@ Install the **ACP Client** extension, then in `settings.json`:
 
 The token is optional — omit it if the other end doesn't require one.
 
+## The working directory (`cwd`)
+
+Your editor runs on your machine; the agent lives in a remote box, and they share no disk.
+Every ACP client sends the path of the project **open on your machine** in `session/new` —
+a path that does not exist over there. The agent rejects it (`Invalid params: invalid
+directory path`) and editors show it as *"Failed to connect"*, which sends you off to check
+the URL and the token, the two things that were fine.
+
+So the bridge rewrites that one field: `cwd` becomes `/root`, or whatever you set in
+`GHOSTY_ACP_CWD`. Nothing else in the message is touched.
+
+```json
+"env": { "GHOSTY_ACP_TOKEN": "…", "GHOSTY_ACP_CWD": "/workspace" }
+```
+
+The agent edits **what is in the box**, not your working copy.
+
 ## ⚠️ Put the token in `env`, never in `args`
 
 Any process on your machine can read another process's arguments with `ps`. The token is
@@ -67,6 +84,7 @@ else there breaks the editor.
 | `rechazado (1006)` | Bad credential, or the server refused the connection |
 | `la máquina estaba en reposo…` | The agent was asleep; connecting wakes it |
 | `conexión cerrada (…)` | The other end closed; the bridge exits so the editor notices |
+| `cwd … → …` | The project path was remapped to one the agent can see |
 
 Requires Node 22+. MIT.
 
@@ -127,6 +145,23 @@ En `settings.json`:
 
 El token es opcional: si el otro extremo no pide credencial, omítelo.
 
+## El directorio de trabajo (`cwd`)
+
+Tu editor corre en tu máquina y el agente vive en una caja remota: no comparten disco.
+Todo cliente ACP manda en `session/new` la ruta del proyecto **abierto en tu máquina**, una
+ruta que allá no existe. El agente la rechaza (`Invalid params: invalid directory path`) y
+los editores lo pintan como *«Failed to connect»*, que manda a revisar la URL y el token
+—justo lo que sí estaba bien—.
+
+Por eso el puente reescribe ese único campo: `cwd` pasa a ser `/root`, o lo que pongas en
+`GHOSTY_ACP_CWD`. Del resto del mensaje no toca nada.
+
+```json
+"env": { "GHOSTY_ACP_TOKEN": "…", "GHOSTY_ACP_CWD": "/workspace" }
+```
+
+El agente edita **lo que hay en la caja**, no tu copia de trabajo.
+
 ## ⚠️ El token va en `env`, nunca en `args`
 
 Los argumentos de un proceso los puede leer cualquier otro proceso de la máquina con un
@@ -142,5 +177,6 @@ flujo JSON-RPC, porque cualquier otra cosa ahí rompe al editor.
 | `rechazado (1006)` | La credencial no vale, o el servidor no aceptó la conexión |
 | `la máquina estaba en reposo…` | El agente estaba dormido; la conexión lo despierta |
 | `conexión cerrada (…)` | El otro extremo cerró; el puente sale para que el editor se entere |
+| `cwd … → …` | Se remapeó la ruta del proyecto a una que el agente sí ve |
 
 Necesita Node 22 o superior. MIT.

package/bridge.mjs

@@ -17,6 +17,7 @@
 
 const url = process.argv[2];
 const token = process.env.GHOSTY_ACP_TOKEN || "";
+const cwdRemoto = process.env.GHOSTY_ACP_CWD || "/root";
 
 if (!url) {
   console.error("uso: node bridge.mjs <wss://…/acp>   (token en GHOSTY_ACP_TOKEN)");
@@ -47,8 +48,11 @@ const ws = new WebSocket(target);
 // sueltan en orden al abrir.
 let cola = [];
 
+let abierto = false;
+
 ws.onopen = () => {
   clearTimeout(avisoLento);
+  abierto = true;
   diag("listo");
   for (const linea of cola) ws.send(linea + "\n");
   cola = [];
@@ -65,7 +69,9 @@ ws.onclose = (e) => {
   clearTimeout(avisoLento);
   // 1006 sin motivo es lo que se ve cuando el servidor rechaza el upgrade: casi siempre
   // el token. Decirlo aquí evita el rato de mirar el editor pensando que es cosa suya.
-  if (e.code === 1006 && !e.reason) {
+  // Sólo es un rechazo si nunca llegó a abrir: un 1006 tras «listo» es la caja o la red
+  // cortando una conexión que sí autenticó, y mandar a revisar el token ahí despista.
+  if (e.code === 1006 && !e.reason && !abierto) {
     diag(token ? "rechazado (1006): revisa el token de este agente" : "rechazado (1006): falta GHOSTY_ACP_TOKEN");
   } else {
     diag(`conexión cerrada (${e.code}${e.reason ? `: ${e.reason}` : ""})`);
@@ -91,6 +97,35 @@ ws.onmessage = (e) => {
   }
 };
 
+// ── `cwd`: LO ÚNICO que este puente traduce ──────────────────────────────────────
+//
+// Rompe a propósito el "no traduce nada" de arriba, y vale la pena decir por qué. El
+// editor corre en tu máquina y el agente vive en una microVM: NO comparten disco. Todo
+// cliente ACP manda en `session/new` el directorio del proyecto ABIERTO EN TU MÁQUINA
+// (`/Users/tu/proyecto`), una ruta que dentro de la caja sencillamente no existe. goose
+// la rechaza con `-32602 Invalid params: invalid directory path`, y los editores pintan
+// eso como «Failed to connect» — un mensaje que manda a revisar la URL y el token, que
+// son justamente lo que sí estaba bien. Se pierde la tarde.
+//
+// Se reescribe SÓLO el campo `cwd` y SÓLO en los dos métodos que lo llevan. El resto del
+// mensaje pasa intacto; si no es JSON, o no es uno de esos métodos, no se toca nada.
+const METODOS_CON_CWD = new Set(["session/new", "session/load"]);
+
+const remapearCwd = (linea) => {
+  // Un mensaje que no parsea no es asunto nuestro: se mueve tal cual, como todo lo demás.
+  let msg;
+  try {
+    msg = JSON.parse(linea);
+  } catch {
+    return linea;
+  }
+  if (!METODOS_CON_CWD.has(msg?.method) || typeof msg?.params?.cwd !== "string") return linea;
+  if (msg.params.cwd === cwdRemoto) return linea;
+  diag(`cwd ${msg.params.cwd} → ${cwdRemoto} (el agente no ve tu disco)`);
+  msg.params.cwd = cwdRemoto;
+  return JSON.stringify(msg);
+};
+
 // ── editor → caja ────────────────────────────────────────────────────────────────
 //
 // Aquí SÍ hace falta acumular: stdin es un flujo y un mensaje puede llegar partido en
@@ -101,9 +136,12 @@ process.stdin.on("data", (chunk) => {
   buffer += chunk;
   let nl;
   while ((nl = buffer.indexOf("\n")) !== -1) {
-    const linea = buffer.slice(0, nl).trim();
+    const cruda = buffer.slice(0, nl).trim();
     buffer = buffer.slice(nl + 1);
-    if (!linea) continue;
+    if (!cruda) continue;
+    // Antes de la cola y antes del envío: así la línea que se guarda dormida ya va
+    // corregida y no hay dos caminos que mantener.
+    const linea = remapearCwd(cruda);
     // Si el socket aún no está abierto la línea se descarta y se dice: mandarla al vacío
     // en silencio dejaría al editor esperando para siempre una respuesta que nadie oyó.
     if (ws.readyState === WebSocket.CONNECTING) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ghosty-acp",
-  "version": "0.0.2",
+  "version": "0.0.4",
   "description": "Conecta tu editor a un agente ACP remoto. Puente entre entrada/salida estándar y WebSocket, sin dependencias.",
   "type": "module",
   "bin": { "ghosty-acp": "bridge.mjs" },