@rallycry/conveyor-mcp 5.0.2 → 5.0.4

package/README.md

@@ -76,6 +76,12 @@ CONVEYOR_PROJECT_ID = "<project-id>"
 | `CONVEYOR_PROJECT_ID`    | no       | Default project for unqualified tools. Omit it for multi-project mode.                |
 | `CONVEYOR_SUBPROJECT_ID` | no       | Default board. Unqualified `create_task`/`list_tasks`/`search_tasks` target it.       |
 
+## How the server exits
+
+The server's lifetime is its stdin pipe. It exits when stdin reaches EOF, on
+`SIGINT` or `SIGTERM`, and when its parent process changes — so a client that
+force-quits, crashes, or is torn down never leaves a server behind.
+
 ## Why `npx -y …@latest`
 
 Every form above launches the server through

package/dist/chunk-5KU3QS6A.js

@@ -0,0 +1,45 @@
+// src/lifecycle.ts
+function watchForClientExit(onExit, deps = {}) {
+  const stdin = deps.stdin ?? process.stdin;
+  const signals = deps.signals ?? process;
+  const getPpid = deps.getPpid ?? (() => process.ppid);
+  const intervalMs = deps.intervalMs ?? 3e4;
+  const watchStdin = deps.watchStdin ?? true;
+  let fired = false;
+  const fire = () => {
+    if (fired) return;
+    fired = true;
+    onExit();
+  };
+  signals.on("SIGINT", fire);
+  signals.on("SIGTERM", fire);
+  signals.on("SIGHUP", fire);
+  if (watchStdin && stdin.isTTY !== true) {
+    if (deps.resumeStdin) stdin.resume?.();
+    stdin.on("end", fire);
+    stdin.on("close", fire);
+  }
+  const initialPpid = getPpid();
+  const orphanCheck = setInterval(() => {
+    if (getPpid() !== initialPpid) fire();
+  }, intervalMs);
+  orphanCheck.unref?.();
+  return fire;
+}
+function installLifecycle(conn, deps = {}) {
+  const exit = deps.exit ?? ((code) => process.exit(code));
+  let stopping = false;
+  function stop(code = 0) {
+    if (stopping) return;
+    stopping = true;
+    conn.disconnect();
+    exit(code);
+  }
+  watchForClientExit(() => stop(), deps);
+  return stop;
+}
+
+export {
+  watchForClientExit,
+  installLifecycle
+};

package/dist/{chunk-XLDG5QEX.js → chunk-EUCQ26F7.js}

@@ -49,10 +49,14 @@ function enrichToolError(error, fallback) {
 var ConveyorConnection = class {
   socket = null;
   config;
+  socketFactory;
   /** project slug → id, for resolving `<project>/<card>` card paths. */
   projectSlugIds = /* @__PURE__ */ new Map();
-  constructor(config) {
+  /** Session rooms this connection has joined, replayed after every reconnect. */
+  subscribedSessionIds = /* @__PURE__ */ new Set();
+  constructor(config, socketFactory = io) {
     this.config = config;
+    this.socketFactory = socketFactory;
   }
   get projectId() {
     return this.resolveProjectId();
@@ -86,12 +90,32 @@ var ConveyorConnection = class {
     if (Array.isArray(record.data)) return record.data;
     return [];
   }
+  /**
+   * Re-join every room this connection needs.
+   *
+   * Runs on the FIRST connect and on every reconnect. A socket.io reconnect
+   * starts a fresh server-side socket that belongs to no rooms, and Cloud Run
+   * cuts each websocket at its 60-minute request limit, so a subscribe sent
+   * only once left long-lived MCP connections silently outside the project room
+   * for the rest of the day.
+   */
+  rejoinRooms() {
+    const socket = this.socket;
+    if (!socket) return;
+    socket.emit("projectService:subscribe", { id: this.config.projectId });
+    for (const sessionId of this.subscribedSessionIds) {
+      socket.emit("agentSessionService:subscribe", {
+        entryId: sessionId,
+        requiredLevel: "Read"
+      });
+    }
+  }
   connect() {
     return new Promise((resolve, reject) => {
       let settled = false;
       let attempts = 0;
       const maxAttempts = 15;
-      this.socket = io(
+      this.socket = this.socketFactory(
         this.config.apiUrl,
         buildConveyorSocketOptions({
           projectToken: this.config.projectToken,
@@ -99,12 +123,15 @@ var ConveyorConnection = class {
         })
       );
       this.socket.on("connect", () => {
+        this.rejoinRooms();
         if (!settled) {
           settled = true;
-          this.socket?.emit("projectService:subscribe", { id: this.config.projectId });
           resolve();
         }
       });
+      this.socket.on("disconnect", (reason) => {
+        if (reason === "io server disconnect") this.socket?.connect();
+      });
       this.socket.on("connect_error", (err) => {
         const message = err?.message ?? "unknown error";
         if (!settled && AUTH_ERROR_RE.test(message)) {
@@ -943,6 +970,7 @@ var ConveyorConnection = class {
   subscribeToSession(sessionId) {
     const socket = this.socket;
     if (!socket) throw new Error("Not connected");
+    this.subscribedSessionIds.add(sessionId);
     socket.emit("agentSessionService:subscribe", {
       entryId: sessionId,
       requiredLevel: "Read"
@@ -1013,6 +1041,7 @@ var ConveyorConnection = class {
   disconnect() {
     this.socket?.disconnect();
     this.socket = null;
+    this.subscribedSessionIds.clear();
   }
 };