@minimalcorp/tsunagi 0.3.1 → 0.3.3

package/dist/server/pty-manager.d.ts

@@ -35,6 +35,10 @@ export interface PtySession {
     /** GCインターバルタイマー */
     gcTimer: ReturnType<typeof setInterval> | null;
 }
+export interface TerminalSize {
+    cols: number;
+    rows: number;
+}
 declare class PtyManager {
     private sessions;
     /** sessionId → 最後に input を送信した socket.id */
@@ -42,6 +46,14 @@ declare class PtyManager {
     setActiveSocket(sessionId: string, socketId: string): void;
     getActiveSocket(sessionId: string): string | undefined;
     clearActiveSocket(sessionId: string, socketId: string): void;
+    /** sessionId → (socket.id → その socket の xterm サイズ)。挿入順 = 最後に resize/claim した順 */
+    private terminalSizes;
+    setTerminalSize(sessionId: string, socketId: string, size: TerminalSize): void;
+    /** socket の登録を外し、残っている socket のうち最後に使われたものを返す */
+    removeTerminalSocket(sessionId: string, socketId: string): {
+        socketId: string;
+        size: TerminalSize;
+    } | undefined;
     /**
      * @param unsetKeys 外側 Terminal や DB 由来でも内部ターミナルに渡さない環境変数キー
      *   （例: Ollama タブでは Anthropic の認証トークンを取り除く）

package/dist/server/pty-manager.js

@@ -59,6 +59,31 @@ class PtyManager {
             this.activeSocketIds.delete(sessionId);
         }
     }
+    /** sessionId → (socket.id → その socket の xterm サイズ)。挿入順 = 最後に resize/claim した順 */
+    terminalSizes = new Map();
+    setTerminalSize(sessionId, socketId, size) {
+        let sizes = this.terminalSizes.get(sessionId);
+        if (!sizes) {
+            sizes = new Map();
+            this.terminalSizes.set(sessionId, sizes);
+        }
+        // 末尾に移して「最後に使われた socket」を末尾で表す
+        sizes.delete(socketId);
+        sizes.set(socketId, size);
+    }
+    /** socket の登録を外し、残っている socket のうち最後に使われたものを返す */
+    removeTerminalSocket(sessionId, socketId) {
+        const sizes = this.terminalSizes.get(sessionId);
+        if (!sizes)
+            return undefined;
+        sizes.delete(socketId);
+        const last = Array.from(sizes.entries()).at(-1);
+        if (!last) {
+            this.terminalSizes.delete(sessionId);
+            return undefined;
+        }
+        return { socketId: last[0], size: last[1] };
+    }
     /**
      * @param unsetKeys 外側 Terminal や DB 由来でも内部ターミナルに渡さない環境変数キー
      *   （例: Ollama タブでは Anthropic の認証トークンを取り除く）

package/dist/server/routes/terminal.js

@@ -27,8 +27,34 @@ function stripTerminalQueries(s) {
         .replace(/\x1b\[[?>]?[0-9;]*n/g, '') // DSR: ESC[?6n(主犯) / ESC[6n / ESC[5n など
         .replace(/\x1b\[[?>=]?[0-9;]*c/g, ''); // DA: ESC[c / ESC[0c / ESC[>c / ESC[=c
 }
+/**
+ * 表示中のクライアントがいなくなった PTY の cols をこの幅に縮める。
+ * 一定の幅を下回ると Claude(Ink) が full-frame redraw を行い <Static> を再 emit するという
+ * 観測に基づくしきい値。元 cols が既にこの値の場合のみ -1 にして必ず cols を変化させる。
+ *
+ * 再表示時は replay 後にクライアントが実サイズへ resize するため、必ず cols が変わり
+ * SIGWINCH が飛んで Claude が現在の画面を描き直す（同一サイズの resize は no-op で
+ * SIGWINCH が飛ばない）。縮める処理を切断時に済ませておくことで、縮めた時の再描画が
+ * 再表示より前に scrollback に入り、replay と描き直しの順序がタイミングに依存しない。
+ */
+const REDRAW_BUMP_COLS = 64;
+function bumpColsForRedraw(session) {
+    const { pty: ptyProcess } = session;
+    const cols = ptyProcess.cols === REDRAW_BUMP_COLS ? REDRAW_BUMP_COLS - 1 : REDRAW_BUMP_COLS;
+    try {
+        ptyProcess.resize(cols, ptyProcess.rows);
+    }
+    catch {
+        // 終了済みの PTY は無視する
+    }
+}
 export async function terminalRoutes(fastify) {
     const io = fastify.io;
+    // 作成中のセッション（sessionId → 作成処理）。存在確認から PTY 作成までの間に await を挟むため、
+    // 同じ sessionId の作成要求が並行すると両方が存在確認を通過し、後発が "Session already exists" で
+    // 失敗する（開発時の StrictMode による effect の二重実行で必ず起きる）。後発は作成中の処理を待って
+    // 再利用として返す。
+    const pendingCreations = new Map();
     initPtyEnvSync(io);
     // socket.io イベントハンドラ
     io.on('connection', (socket) => {
@@ -52,6 +78,10 @@ export async function terminalRoutes(fastify) {
                 socket.join(room);
                 return;
             }
+            // 別セッションへ join し直した場合は、前のセッションのサイズ登録を外す
+            if (boundSessionId && boundSessionId !== sessionId) {
+                ptyManager.removeTerminalSocket(boundSessionId, socket.id);
+            }
             boundSessionId = sessionId;
             const session = ptyManager.getSession(sessionId);
             if (!session) {
@@ -66,6 +96,7 @@ export async function terminalRoutes(fastify) {
             dataHandlerDispose = null;
             exitHandlerDispose = null;
             socket.removeAllListeners('resize');
+            socket.removeAllListeners('claim');
             socket.removeAllListeners('input');
             socket.join(room);
             // 環境変数の変更が未反映ならクライアントに伝える（反映待ちバッジ）
@@ -82,16 +113,19 @@ export async function terminalRoutes(fastify) {
             ptyManager.setActiveSocket(sessionId, socket.id);
             const { pty: ptyProcess } = session;
             const isReused = session.scrollback.length > 0;
-            // リングバッファの内容を一括送信（再接続時の画面復元）
-            if (isReused) {
-                // 端末クエリ列（ESC[?6n / ESC[c 等）を除去してから replay する。
-                // これを怠ると xterm.js が応答を生成し PTY へ注入され、入力欄に "1;2c64;3R..." が
-                // 勝手に入力される（stripTerminalQueries のコメント参照）。
-                const buffered = stripTerminalQueries(session.scrollback.join(''));
+            // リングバッファの内容を一括送信（画面復元）。初回接続・再接続を問わず join のたびに送る。
+            // クライアントは replay を受けたら画面をクリアし、xterm を PTY と同じサイズにしてから
+            // 書き込み、完了後に実サイズへ resize する（新規セッションは data が空）。
+            // 端末クエリ列（ESC[?6n / ESC[c 等）を除去してから replay する。
+            // これを怠ると xterm.js が応答を生成し PTY へ注入され、入力欄に "1;2c64;3R..." が
+            // 勝手に入力される（stripTerminalQueries のコメント参照）。
+            const buffered = isReused ? stripTerminalQueries(session.scrollback.join('')) : '';
+            socket.emit('replay', {
                 // 末尾の \r\n / \n / \r をトリムする。
-                const trimmed = buffered.replace(/[\r\n]+$/, '');
-                socket.emit('output', { data: trimmed });
-            }
+                data: buffered.replace(/[\r\n]+$/, ''),
+                cols: ptyProcess.cols,
+                rows: ptyProcess.rows,
+            });
             // reused の場合、最初の resize まで onData 出力をバッファリング
             let initialResizeHandled = !isReused;
             const pendingOutput = [];
@@ -125,11 +159,16 @@ export async function terminalRoutes(fastify) {
                 ptyManager.deleteSession(sessionId);
             });
             exitHandlerDispose = () => exitHandler.dispose();
-            // resize イベント: PTYリサイズ
+            // resize イベント: この socket のサイズを記録し、所有者 socket の場合のみ PTY へ反映する。
+            // 同一タスクを複数ウィンドウで開いた場合に、フォアグラウンドでない側のサイズで
+            // PTY が変わり Claude の表示幅がずれるのを防ぐ（所有者の切り替えは claim で行う）。
             socket.on('resize', ({ sessionId: sid, cols, rows }) => {
                 if (sid !== sessionId)
                     return;
-                ptyProcess.resize(cols, rows);
+                ptyManager.setTerminalSize(sessionId, socket.id, { cols, rows });
+                if (ptyManager.getActiveSocket(sessionId) === socket.id) {
+                    ptyProcess.resize(cols, rows);
+                }
                 // reused時: 初回 resize 後にバッファリングしていた出力をフラッシュ
                 if (!initialResizeHandled) {
                     initialResizeHandled = true;
@@ -140,6 +179,17 @@ export async function terminalRoutes(fastify) {
                     }
                 }
             });
+            // claim イベント: フォアグラウンドになった socket が所有者になり、PTY を自分のサイズに合わせる。
+            // サイズが変われば SIGWINCH で Claude が描き直す。
+            socket.on('claim', ({ sessionId: sid, cols, rows }) => {
+                if (sid !== sessionId)
+                    return;
+                ptyManager.setTerminalSize(sessionId, socket.id, { cols, rows });
+                ptyManager.setActiveSocket(sessionId, socket.id);
+                if (ptyProcess.cols !== cols || ptyProcess.rows !== rows) {
+                    ptyProcess.resize(cols, rows);
+                }
+            });
             // input イベント: 所有者 socket からの入力のみ PTY へ書き込む。
             // ゾンビ/非所有 socket（再接続前の旧 socket・誤バインド socket 等）からの重複 input は
             // 破棄し、1 キーストロークの多重書き込み（例: /clear が連続入力される）を防ぐ。
@@ -170,8 +220,26 @@ export async function terminalRoutes(fastify) {
                 // 遅延切断では GC を張らない（現役セッションが誤って GC 対象になるのを防ぐ）。
                 const wasOwner = ptyManager.getActiveSocket(boundSessionId) === socket.id;
                 ptyManager.clearActiveSocket(boundSessionId, socket.id);
-                if (wasOwner) {
+                const next = ptyManager.removeTerminalSocket(boundSessionId, socket.id);
+                if (wasOwner && next) {
+                    // 同じ PTY を表示している socket が残っている → 最後に使われた socket に所有権と
+                    // サイズを引き継ぐ（表示中なので GC も cols の縮小も行わない）
+                    ptyManager.setActiveSocket(boundSessionId, next.socketId);
+                    const session = ptyManager.getSession(boundSessionId);
+                    try {
+                        session?.pty.resize(next.size.cols, next.size.rows);
+                    }
+                    catch {
+                        // 終了済みの PTY は無視する
+                    }
+                }
+                else if (wasOwner) {
                     ptyManager.scheduleGc(boundSessionId);
+                    // 次に表示されたとき Claude に描き直させるため、表示中のクライアントがいない間に
+                    // cols を縮めておく（REDRAW_BUMP_COLS 参照）
+                    const session = ptyManager.getSession(boundSessionId);
+                    if (session)
+                        bumpColsForRedraw(session);
                 }
             }
         });
@@ -190,14 +258,27 @@ export async function terminalRoutes(fastify) {
             fastify.log.info({ sessionId }, 'Reusing existing PTY session');
             return reply.status(200).send({ sessionId, reused: true });
         }
-        const launchRequest = { cwd, env };
+        const pending = pendingCreations.get(sessionId);
+        const creation = pending ??
+            (async () => {
+                const launchRequest = { cwd, env };
+                const launch = await resolvePtyLaunch(sessionId, launchRequest);
+                await startPtySession(sessionId, launch, {
+                    request: launchRequest,
+                    launchClaude: Boolean(claude),
+                    command,
+                });
+            })();
+        if (!pending) {
+            pendingCreations.set(sessionId, creation);
+            void creation.catch(() => undefined).finally(() => pendingCreations.delete(sessionId));
+        }
         try {
-            const launch = await resolvePtyLaunch(sessionId, launchRequest);
-            await startPtySession(sessionId, launch, {
-                request: launchRequest,
-                launchClaude: Boolean(claude),
-                command,
-            });
+            await creation;
+            if (pending) {
+                fastify.log.info({ sessionId }, 'Reusing PTY session created by a concurrent request');
+                return reply.status(200).send({ sessionId, reused: true });
+            }
             return reply.status(201).send({ sessionId, reused: false });
         }
         catch (err) {

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "@minimalcorp/tsunagi",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@minimalcorp/tsunagi",
-      "version": "0.3.1",
+      "version": "0.3.3",
       "license": "SEE LICENSE IN LICENSE",
       "os": [
         "darwin",
@@ -107,9 +107,9 @@
       "license": "MIT"
     },
     "node_modules/@fastify/cors": {
-      "version": "11.3.0",
-      "resolved": "https://registry.npmjs.org/@fastify/cors/-/cors-11.3.0.tgz",
-      "integrity": "sha512-ggQGua+xHv1MvePbPr0v//xLYEsCXbWspquXCJS9Ot5YoRXq8J8ZWzHnxDBVnbtXosvistXo6LtNzOJswf64Fw==",
+      "version": "11.3.1",
+      "resolved": "https://registry.npmjs.org/@fastify/cors/-/cors-11.3.1.tgz",
+      "integrity": "sha512-gZXNzd6FLHzOIfvoK4+Az3Wko2V6Y0MIsLxpyo4AEYl7a9q1JBPwPaxLea4nkr++8FCiJiPpYYo6E02uZW0dqw==",
       "funding": [
         {
           "type": "github",
@@ -127,9 +127,9 @@
       }
     },
     "node_modules/@fastify/deepmerge": {
-      "version": "3.2.1",
-      "resolved": "https://registry.npmjs.org/@fastify/deepmerge/-/deepmerge-3.2.1.tgz",
-      "integrity": "sha512-N5Oqvltoa2r9z1tbx4xjky0oRR60v+T47Ic4J1ukoVQcptLOrIdRnCSdTGmOmajZuHVKlTnfcmrjyqsGEW1ztA==",
+      "version": "3.2.2",
+      "resolved": "https://registry.npmjs.org/@fastify/deepmerge/-/deepmerge-3.2.2.tgz",
+      "integrity": "sha512-155eqA/WMQdN6gUD4RrhS2K0Pcz2Ou9pQ86L0sipOpP9ONbTA6oYw/hW4VdBjR4egcDHqmWUK9x80oUX7S1mDQ==",
       "funding": [
         {
           "type": "github",
@@ -194,9 +194,9 @@
       "license": "MIT"
     },
     "node_modules/@fastify/http-proxy": {
-      "version": "11.6.3",
-      "resolved": "https://registry.npmjs.org/@fastify/http-proxy/-/http-proxy-11.6.3.tgz",
-      "integrity": "sha512-yrPdZD9vdBoMtuRhzlTuVpvoyUrqW6/VXqFrSbRBrbUN+x5+PjddnBVbooY7DILZ0QixSuU7a9sG96JVbAbESg==",
+      "version": "11.6.4",
+      "resolved": "https://registry.npmjs.org/@fastify/http-proxy/-/http-proxy-11.6.4.tgz",
+      "integrity": "sha512-JPsbLX9D5X5yES1A6M2yx1i2PGf8eNyKT48j+6pgLIwxfRYpj1MEzR1CwASRvqdZOWhcPmdsZiIKehSgioW6ww==",
       "funding": [
         {
           "type": "github",
@@ -303,15 +303,15 @@
       }
     },
     "node_modules/@hono/node-server": {
-      "version": "2.1.3",
-      "resolved": "https://registry.npmjs.org/@hono/node-server/-/node-server-2.1.3.tgz",
-      "integrity": "sha512-TA//nWMqPhbfdfneACk6t5a9eqbS9lABEPyKn0/xZTah3H3U2XaVg85rJFl0/Fyit0I552YDHgXGVSf3GwqbUw==",
+      "version": "2.1.4",
+      "resolved": "https://registry.npmjs.org/@hono/node-server/-/node-server-2.1.4.tgz",
+      "integrity": "sha512-2g1qeHl4BduwDn6L8nkAvDrPUWPMlkQ//a5NhP/zIyaXKMjZNRV+cG9Csu+BiJFf6+68pqzjf4Hi5sJGKHuS1g==",
       "license": "MIT",
       "engines": {
         "node": ">=20"
       },
       "peerDependencies": {
-        "hono": "^4"
+        "hono": "^4 || ^5.0.0-0"
       }
     },
     "node_modules/@kwsites/file-exists": {
@@ -500,9 +500,9 @@
       ]
     },
     "node_modules/@modelcontextprotocol/sdk": {
-      "version": "1.32.0",
-      "resolved": "https://registry.npmjs.org/@modelcontextprotocol/sdk/-/sdk-1.32.0.tgz",
-      "integrity": "sha512-8BviX/hK4Gd2eL1KdTwm0gfl4d3wdoErW0Jd00h6nomUAtk6IJk5vwQJc6kfWtJTzN5OEV/lgIiJKI8fiDGAlA==",
+      "version": "1.32.1",
+      "resolved": "https://registry.npmjs.org/@modelcontextprotocol/sdk/-/sdk-1.32.1.tgz",
+      "integrity": "sha512-2DdE+SJDtzLEEWzY1ZjY7Q+VcPhcV1KisD3zI4u0XZyktsjHum1mwbMI+JaulUBi2OZk+KJAi2uPXzxichPkdw==",
       "license": "MIT",
       "dependencies": {
         "@hono/node-server": "^1.19.9 || ^2.0.5",
@@ -2233,9 +2233,9 @@
       }
     },
     "node_modules/express-rate-limit": {
-      "version": "8.7.0",
-      "resolved": "https://registry.npmjs.org/express-rate-limit/-/express-rate-limit-8.7.0.tgz",
-      "integrity": "sha512-hOwV7WOxXfjRpAM1DSJWZDXx3GhplwD8IfwuwvogD8i1Qnkgosw/H45s4ZnFAUHDAhPjlY9hLBvJhKmGMyY26g==",
+      "version": "8.7.1",
+      "resolved": "https://registry.npmjs.org/express-rate-limit/-/express-rate-limit-8.7.1.tgz",
+      "integrity": "sha512-2FXbqkygil3mZiXzEtM0IjE/UBmBNot0J1GEVrJT4JbrvwOSoEtqtO+wHbiMQELWe1eNvHoYRg2dkjOid+6JkA==",
       "license": "MIT",
       "dependencies": {
         "debug": "^4.4.3",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@minimalcorp/tsunagi",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Multi-repo GitHub project management with Claude AI integration, designed for visualizing and controlling AI-driven development locally.",
   "keywords": [
     "tsunagi",