pluriply 0.5.0 → 0.5.2

package/README.md

@@ -32,7 +32,7 @@ Pluriply MCP connector with each of them (idempotent — run it again any time).
 - `npx pluriply setup --only claude-code,codex` — limit to specific tools.
 - `npx pluriply setup --remove` — unregister Pluriply from every tool, disable headless workers and stop the hub. Your channels and task history under `~/.pluriply` stay; add `--purge` to delete them too.
 - `npx pluriply setup --remove --hooks-only` — take out only the Stop hooks; MCP registration, headless workers and the hub stay as they are.
-- Codex and Antigravity get a 600 s MCP tool timeout written into their config at registration (their default is 60 s, too short for `ask_agent`/`request_review` waits). If you registered with an earlier version, run `setup --remove` then `setup` again to pick it up.
+- Codex and Antigravity get a 600 s MCP tool timeout written into their config at registration (their default is 60 s, too short for `ask_agent`/`request_review` waits). Codex also gets a 30 s MCP startup timeout (its default is 10 s; the connector may have to wait for the hub to start). If you registered with an earlier version, run `setup` again to pick up anything missing — values you already set are kept (the config file is rewritten once, with a `.bak` copy of the original).
 - `setup` also installs a Stop hook for Claude Code, Codex and Antigravity CLI so a live session notices new tasks and finished results at the end of its turn (see _Warm reception_). `--no-hooks` skips it; `setup --remove` takes it out again.
 - After upgrading or cleaning the npx cache, run `npx pluriply@latest setup --hooks-only` — the hook command embeds the installed path, and this refreshes it without rewriting your MCP configuration.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pluriply",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Connect your AI coding tools into one collaboration channel",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE.md",

package/src/connector/tools.js

@@ -54,7 +54,7 @@ function staleHubMessage(stale) {
 /**
  * 허브에 정체성을 알리고 인스턴스 ID를 받는다. 재접속 때는 알고 있는 ID를 실어 그대로 인정받는다.
  * @param {import('./hub-client.js').HubClient} hub
- * @param {{agent: string, worker?: boolean, instanceId?: string, duringReconnect?: boolean}} opts
+ * @param {{agent: string, worker?: boolean, instanceId?: string|null, duringReconnect?: boolean}} opts
  * @returns {Promise<string>}
  */
 export async function hello(
@@ -63,7 +63,14 @@ export async function hello(
 ) {
   const r = await hub.request(
     "agent.hello",
-    { tool: agent, cwd: process.cwd(), worker, instanceId },
+    // null 을 그대로 보내면 허브가 "invalid instanceId: null" 로 거절한다(undefined 만 "새로 발급"이다).
+    // 구버전 허브로 시작해 정체성이 없던 커넥터가 재접속 때 새 id 를 받을 수 있게 비운다.
+    {
+      tool: agent,
+      cwd: process.cwd(),
+      worker,
+      instanceId: instanceId ?? undefined,
+    },
     { duringReconnect },
   );
   return r.instanceId;
@@ -73,10 +80,11 @@ export async function hello(
  * Pluriply MCP 도구를 등록한다.
  * @param {import('@modelcontextprotocol/sdk/server/mcp.js').McpServer} server
  * @param {import('./hub-client.js').HubClient} hub
- * @param {{agent: string, instanceId: string}} identity
+ * @param {{agent: string, instanceId: string|null}} identity instanceId 는 구버전 허브로 시작하면 null 이다.
  */
 export function registerTools(server, hub, { agent, instanceId }) {
-  const state = { currentChannel: null };
+  /** instanceId 는 재접속 hello 가 돌려준 값으로 갱신된다(구버전 허브로 시작해 null 이었던 경우). */
+  const state = { currentChannel: null, instanceId };
   const worker = Boolean(process.env.PLURIPLY_WORKER_TASK);
   /** 워커는 자기 태스크 깊이 + 1, 대화형 세션은 0 */
   const delegationDepth = () =>
@@ -192,7 +200,12 @@ export function registerTools(server, hub, { agent, instanceId }) {
         // join_channel이 도구를 다시 시작할 때까지 "say hello first"로 막힌다.
         // (hello는 인증이 필요한 요청이라, 토큰이 틀린 연결은 여기서 unauthorized를
         // 받아 hub-client의 재접속 판정이 그것을 본다.)
-        await hello(hub, { agent, worker, instanceId, duringReconnect: true });
+        state.instanceId = await hello(hub, {
+          agent,
+          worker,
+          instanceId: state.instanceId,
+          duringReconnect: true,
+        });
         if (!state.currentChannel) return;
         await hub.request(
           "channel.join",
@@ -226,7 +239,7 @@ export function registerTools(server, hub, { agent, instanceId }) {
           channelCode: code,
         });
         state.currentChannel = code;
-        return ok({ channelCode: code, peers, me: instanceId });
+        return ok({ channelCode: code, peers, me: state.instanceId });
       } catch (err) {
         return fail(err.message);
       }
@@ -260,7 +273,7 @@ export function registerTools(server, hub, { agent, instanceId }) {
       });
       const { tasks } = await hub.request("task.list", {
         channelCode: code,
-        to: instanceId,
+        to: state.instanceId,
         status: "submitted",
       });
       const { running } = await hub.request("worker.status", {
@@ -591,8 +604,8 @@ export function registerTools(server, hub, { agent, instanceId }) {
       ({ status, mine_only = true, sent_by_me = false, kind }, code) =>
         hub.request("task.list", {
           channelCode: code,
-          to: sent_by_me ? undefined : mine_only ? instanceId : undefined,
-          from: sent_by_me ? instanceId : undefined,
+          to: sent_by_me ? undefined : mine_only ? state.instanceId : undefined,
+          from: sent_by_me ? state.instanceId : undefined,
           status,
           kind,
         }),

package/src/setup/clients.js

@@ -104,6 +104,13 @@ export function isSkipped(env) {
  */
 export const TOOL_TIMEOUT_SEC = 600;
 
+/**
+ * Codex MCP 서버 기동 타임아웃(초). Codex 기본값은 10초인데, 커넥터는 MCP 핸드셰이크 전에 허브를
+ * 기다린다 — 멎은 허브가 락을 쥐면 인수 유예(10초)를 포함해 spawnHub 가 최대 20초 남짓 걸려 기본값으로는
+ * 기동에 실패한다(Plan 4g 최종 리뷰). 스폰 대기보다 길어야 한다(test/hub/timing.test.js 가 고정).
+ */
+export const CODEX_STARTUP_TIMEOUT_SEC = 30;
+
 /**
  * @param {"claude-desktop"|"antigravity-ide"} id
  * Antigravity 는 2.x 부터 IDE(`~/.gemini/antigravity-ide`) 와 허브(`~/.gemini/antigravity`) 로
@@ -297,7 +304,19 @@ function cliClient({
     register(env) {
       if (isSkipped(env)) return "skipped";
       const st = this.status(env);
-      if (st === "present") return "present";
+      if (st === "present") {
+        // 예전 버전으로 등록돼 타임아웃 키가 빠졌을 수 있다(예: 0.5.1 에 생긴 Codex startup_timeout_sec).
+        // 없는 키만 채우고 있는 값은 보존한다. 이미 동작하는 등록이므로 채우지 못해도 되돌리지 않고
+        // 안내만 한다 — 되돌리면 멀쩡한 등록을 지우게 된다.
+        if (afterAdd) {
+          const r = afterAdd(env);
+          if (!r.ok)
+            env.log(`hint: ${label}: ${r.reason}${r.fix ? ` — ${r.fix}` : ""}`);
+          else if (r.changed)
+            env.log(`updated ${label}: added missing timeout settings`);
+        }
+        return "present";
+      }
       if (typeof st === "object") {
         env.log(
           `hint: make sure ${label} has pluriply registered: ${hint(env.binPath)}`,
@@ -475,11 +494,14 @@ export const CLIENTS = [
     // codex mcp add 는 타임아웃 플래그가 없어 config.toml 을 직접 편집한다(스펙 §5)
     afterAdd(env) {
       const path = codexConfigPath(env);
+      // present 경로(이미 등록됨)에서 채우지 못했을 때 사용자에게 보여 줄 손 수정 방법
+      const fix = `add \`tool_timeout_sec = ${TOOL_TIMEOUT_SEC}\` and \`startup_timeout_sec = ${CODEX_STARTUP_TIMEOUT_SEC}\` under [mcp_servers.pluriply] by hand in ${path}`;
       try {
         if (!env.fs.existsSync(path))
           return {
             ok: false,
-            reason: `tool_timeout_sec not written (${path} missing)`,
+            fix,
+            reason: `timeout settings not written (${path} missing)`,
           };
         const r = insertTomlKey(
           env.fs.readFileSync(path, "utf8"),
@@ -490,19 +512,31 @@ export const CLIENTS = [
         if (r.reason === "no-header")
           return {
             ok: false,
-            reason: `tool_timeout_sec not written ([mcp_servers.pluriply] not found in ${path})`,
+            fix,
+            reason: `timeout settings not written ([mcp_servers.pluriply] not found in ${path})`,
           };
         if (r.reason === "unsupported")
           return {
             ok: false,
-            reason: `tool_timeout_sec not written (${path} contains triple-quoted strings; add \`tool_timeout_sec = ${TOOL_TIMEOUT_SEC}\` under [mcp_servers.pluriply] by hand)`,
+            fix,
+            reason: `timeout settings not written (${path} contains triple-quoted strings)`,
           };
-        if (r.changed) writeFileAtomic(env, path, r.text);
-        return { ok: true };
+        // 헤더·트리플쿼트 판정은 파일 단위라 위에서 통과했으면 두 번째 키도 같은 이유로는 실패하지 않는다.
+        // 이미 있는 값(사용자가 고친 값 포함)은 건드리지 않는다.
+        const s = insertTomlKey(
+          r.text,
+          "mcp_servers.pluriply",
+          "startup_timeout_sec",
+          String(CODEX_STARTUP_TIMEOUT_SEC),
+        );
+        const changed = r.changed || s.changed;
+        if (changed) writeFileAtomic(env, path, s.text);
+        return { ok: true, changed };
       } catch (err) {
         return {
           ok: false,
-          reason: `tool_timeout_sec not written (${err.message})`,
+          fix,
+          reason: `timeout settings not written (${err.message})`,
         };
       }
     },
@@ -535,22 +569,26 @@ export const CLIENTS = [
     // agy mcp add 도 타임아웃 플래그가 없다. agy 는 JSONC 를 읽지만 우리는 JSON.parse 만 쓴다(스펙 §11).
     afterAdd(env) {
       const path = agyConfigPath(env);
+      const fix = `add \`"timeoutSeconds": ${TOOL_TIMEOUT_SEC}\` to mcpServers.pluriply by hand in ${path}`;
       try {
         const doc = readJsonDoc(env, path);
         const entry = doc.mcpServers?.pluriply;
         if (!entry || typeof entry !== "object")
           return {
             ok: false,
+            fix,
             reason: `timeoutSeconds not written (mcpServers.pluriply not found in ${path})`,
           };
-        if (entry.timeoutSeconds === undefined) {
+        const changed = entry.timeoutSeconds === undefined;
+        if (changed) {
           entry.timeoutSeconds = TOOL_TIMEOUT_SEC;
           writeJsonAtomic(env, path, doc);
         }
-        return { ok: true };
+        return { ok: true, changed };
       } catch (err) {
         return {
           ok: false,
+          fix,
           reason: `timeoutSeconds not written (${err.message})`,
         };
       }

package/src/setup/run-setup.js

@@ -109,7 +109,8 @@ function walkHooks({ targets, rows, e, dryRun, hooks, remove }) {
  * `remove` 면 반대로 등록을 풀고 워커 설정·허브·(purge 시) 데이터까지 정리한다.
  * `--purge` 는 pluriply 홈이 심볼릭 링크면 **링크만 끊고** 링크가 가리키는 디렉터리는 남긴다
  * (그 안의 내용까지 지우려면 실제 경로를 직접 지워야 한다).
- * env.stopHub / env.rm / env.lstat 은 테스트가 주입한다.
+ * env.stopHub / env.ensureHub / env.rm / env.lstat 은 테스트가 주입한다(주입하지 않은 ensureHub 는 진짜
+ * detached 허브를 띄운다 — 테스트가 넣지 않으면 임시 홈에 허브가 남는다).
  * `hooks`(기본 true)는 Claude Code·Codex 의 Stop 훅 등록 여부다(스펙 §6). `--remove` 는 이 값과
  * 무관하게 항상 훅을 제거한다.
  * `hooksOnly`(스펙 §4)는 MCP 등록·워커·허브를 건너뛰고 Stop 훅만 설치(`remove` 면 제거)한다.
@@ -153,7 +154,8 @@ export async function runSetup({
   };
   if (!dryRun) {
     try {
-      out.hub = { port: (await (await hubClient()).ensureHub({ home })).port };
+      const ensure = e.ensureHub ?? (await hubClient()).ensureHub;
+      out.hub = { port: (await ensure({ home })).port };
     } catch (err) {
       out.hubError = err.message;
     }