@minimalcorp/tsunagi 0.1.31 → 0.1.32

package/dist/cli.js

@@ -184,10 +184,25 @@ verifyArtifact(NEXT_STANDALONE_ENTRY, 'Next.js standalone artifact');
 // Fastify が単一の公開ポート。Next.js は内部ポートで動かし Fastify からプロキシする。
 const PORT = process.env.PORT ?? '2791';
 const NEXT_PORT = '2792';
+// Fastify サーバー自身の待受ポートは TSUNAGI_SERVER_PORT で伝える（generic な PORT は上書きしない）。
+// PORT を直接上書きすると、ユーザーが Terminal で明示的に PORT=xxxx と指定していた場合でも
+// tsunagi 内部ターミナル側でその値と tsunagi 自身の値を区別できなくなるため。
+//
+// NODE_ENV は Fastify/Next 等が直接参照するため PORT と同じ専用キー方式が使えず、サーバー
+// プロセス自身には強制的に 'production' を渡す必要がある。上書きする前に外側 Terminal 由来の
+// 元の値（未設定なら undefined）を TSUNAGI_OUTER_NODE_ENV に退避し、tsunagi 内部ターミナル側
+// （apps/server/src/pty-manager.ts）で復元できるようにする。
+const outerNodeEnv = process.env.NODE_ENV;
 const fastifyChild = spawn(process.execPath, [FASTIFY_ENTRY_JS], {
     stdio: ['inherit', 'pipe', 'pipe'],
     cwd: PACKAGE_ROOT,
-    env: { ...process.env, NODE_ENV: 'production', PORT, TSUNAGI_NEXT_PORT: NEXT_PORT },
+    env: {
+        ...process.env,
+        NODE_ENV: 'production',
+        ...(outerNodeEnv !== undefined ? { TSUNAGI_OUTER_NODE_ENV: outerNodeEnv } : {}),
+        TSUNAGI_SERVER_PORT: PORT,
+        TSUNAGI_NEXT_PORT: NEXT_PORT,
+    },
 });
 const nextChild = spawn(process.execPath, [NEXT_STANDALONE_ENTRY], {
     stdio: ['inherit', 'pipe', 'pipe'],

package/dist/server/index.js

@@ -22,7 +22,10 @@ import { llmRoutes } from './routes/llm.js';
 import { stopLlmServerOnExit } from './lib/llm-process.js';
 import { createBasicAuth } from './basic-auth.js';
 // Fastify は単一の公開エンドポイント。Next.js は内部ポートで動かしプロキシする。
-const PORT = Number(process.env.PORT) || 2791;
+// PORT ではなく TSUNAGI_SERVER_PORT を見る（apps/cli が注入する専用キー）。generic な PORT を
+// 使うと、ユーザーが Terminal で明示的に設定した PORT と衝突し、内部ターミナルへの伝播を
+// 区別できなくなるため（pty-manager.ts 参照）。
+const PORT = Number(process.env.TSUNAGI_SERVER_PORT) || 2791;
 const NEXT_PORT = Number(process.env.TSUNAGI_NEXT_PORT) || 2792;
 const extraOrigins = (process.env.TSUNAGI_EXTRA_CORS_ORIGINS ?? '')
     .split(',')

package/dist/server/pty-manager.js

@@ -2,6 +2,26 @@ import * as pty from 'node-pty';
 import * as os from 'os';
 const SCROLLBACK_MAX_BYTES = 256 * 1024; // 256KB
 const SESSION_GC_INTERVAL_MS = 30 * 60 * 1000; // 30分
+/**
+ * apps/cli が Fastify サーバー起動時に自分で注入する環境変数
+ * （TSUNAGI_SERVER_PORT, NODE_ENV, TSUNAGI_NEXT_PORT, TSUNAGI_OUTER_NODE_ENV）。
+ * サーバープロセス自身の制御用に後から足された値のため、内部ターミナルには継承しない。
+ * 外側 Terminal 由来の環境変数はこのリストと無関係に、これまで通り全て継承する
+ * （PORT はここでは扱わない。apps/cli は generic な PORT を上書きせず、専用キー
+ * TSUNAGI_SERVER_PORT で自身の待受ポートを伝えるため、ユーザーが Terminal で明示した
+ * PORT はそのまま内部ターミナルまで届く）。
+ *
+ * NODE_ENV は Fastify/Next 等が直接参照するため、PORT と違って専用キーへ逃がせず、
+ * サーバープロセス自身には強制的に 'production' が入る。apps/cli はサーバー起動前の
+ * 元の値（外側 Terminal 由来、未設定なら undefined）を TSUNAGI_OUTER_NODE_ENV に退避して
+ * 渡してくるので、ここで NODE_ENV を一旦除外した後、退避値があれば NODE_ENV として復元する。
+ */
+const TSUNAGI_INJECTED_ENV_KEYS = [
+    'TSUNAGI_SERVER_PORT',
+    'NODE_ENV',
+    'TSUNAGI_NEXT_PORT',
+    'TSUNAGI_OUTER_NODE_ENV',
+];
 class PtyManager {
     sessions = new Map();
     /** sessionId → 最後に input を送信した socket.id */
@@ -22,13 +42,25 @@ class PtyManager {
             throw new Error(`Session already exists: ${sessionId}`);
         }
         const shell = process.env.SHELL || (os.platform() === 'win32' ? 'cmd.exe' : 'bash');
+        // tsunagi 自身が注入した環境変数は内部ターミナルに継承しない（TSUNAGI_INJECTED_ENV_KEYS 参照）。
+        // /settings 等で明示的に同名キーを設定した場合は env（dbEnv 経由）が後勝ちで上書きするため、
+        // カスケードの優先順位・挙動は変わらない。
+        const baseEnv = { ...process.env };
+        for (const key of TSUNAGI_INJECTED_ENV_KEYS) {
+            delete baseEnv[key];
+        }
+        // 外側 Terminal に元々 NODE_ENV があった場合は復元する。なければ未設定のままにする
+        // （tsunagi が強制した 'production' を内部ターミナルに漏らさないため）。
+        if (process.env.TSUNAGI_OUTER_NODE_ENV !== undefined) {
+            baseEnv.NODE_ENV = process.env.TSUNAGI_OUTER_NODE_ENV;
+        }
         const ptyProcess = pty.spawn(shell, [], {
             name: 'xterm-256color',
             cols: 80,
             rows: 24,
             cwd,
             env: {
-                ...process.env,
+                ...baseEnv,
                 ...env,
                 TERM: 'xterm-256color',
                 COLORTERM: 'truecolor',

package/dist/server/routes/terminal.js

@@ -7,9 +7,9 @@ import { getEnv } from '../lib/repositories/environment.js';
 import { ensureClaudeOnboardingCompleted } from '../lib/claude-config-guard.js';
 // サーバーはプロジェクトルートから起動されるため process.cwd() でルートを取得
 const TSUNAGI_EDITOR_PATH = path.resolve(process.cwd(), 'scripts/monaco-editor.sh');
-// Fastify(API) の公開ポート。index.ts と同じ既定値。monaco-editor.sh など PTY 内の
-// プロセスが API を叩く際の同一ホスト向けベース URL に使う。
-const SERVER_PORT = Number(process.env.PORT) || 2791;
+// Fastify(API) の公開ポート。index.ts と同じ既定値・同じ環境変数(TSUNAGI_SERVER_PORT)を見る。
+// monaco-editor.sh など PTY 内のプロセスが API を叩く際の同一ホスト向けベース URL に使う。
+const SERVER_PORT = Number(process.env.TSUNAGI_SERVER_PORT) || 2791;
 /**
  * 再接続時の画面復元用 scrollback から「端末への問い合わせ（クエリ）」シーケンスを除去する。
  *