coxpit 5.27.1 → 5.27.5

package/README.md

@@ -151,14 +151,14 @@ One daemon, one SQLite file, zero external services. Machines are reached over S
 
 ## Status
 
-`v4.5` — **greenfield + remote access**. Point Coxpit at an empty folder and a fleet scaffolds a brand-new project across N agents on an empty initial commit — compare the foundations, keep the best (existing folders are never touched). Remote access detects your Tailscale and puts the board on `https://<machine>.<tailnet>.ts.net` in one click (Funnel for public, behind a warning; copy-paste Cloudflare/Caddy recipes otherwise) — Coxpit drives the tool, never hosts a relay. Builds on v4.3's Active-first board + Archive. All shipped and e2e-tested (45 checks). Roadmap: ROADMAP.md.
+`v4.5` — **greenfield + remote access**. Point Coxpit at an empty folder and a fleet scaffolds a brand-new project across N agents on an empty initial commit — compare the foundations, keep the best (existing folders are never touched). Remote access detects your Tailscale and puts the board on `https://<machine>.<tailnet>.ts.net` in one click (Funnel for public, behind a warning; copy-paste Cloudflare/Caddy recipes otherwise) — Coxpit drives the tool, never hosts a relay. Builds on v4.3's Active-first board + Archive. All shipped and e2e-tested (45 checks). Roadmap: docs/ROADMAP.md.
 
 ## Contributing
 
 Issues and PRs are welcome. Start with **[CONTRIBUTING.md](CONTRIBUTING.md)** for
 dev setup (`COXPIT_AUTH_DISABLED=1 npm run dev`), the verify gate (`npm run
 typecheck` + `bash test/e2e.sh`), and the house rules. Please read the
-[non-goals](ROADMAP.md#non-goals) before proposing a feature, and report security
+[non-goals](docs/ROADMAP.md#non-goals) before proposing a feature, and report security
 issues privately per **[SECURITY.md](SECURITY.md)** (Coxpit exposes shells).
 
 ## License

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coxpit",
-  "version": "5.27.1",
+  "version": "5.27.5",
   "description": "Self-hosted cockpit for running a fleet of AI coding agents across your own machines — parallel worktree runs, live board, compare & merge, web terminal, design capture.",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE.md",

package/src/index.ts

@@ -5,8 +5,13 @@ import { machines } from './db/schema';
 import { acquireDaemonLock, updateLockPort } from './lock';
 import { reconcileOrphanRuns } from './orchestrator';
 import { buildServer } from './server';
+import { augmentPathForGuiLaunch } from './paths';
 import type { AddressInfo } from 'node:net';
 
+// macOS: GUI/launchd 로 뜨면 PATH 에 Homebrew 가 없어 로컬 tmux 를 못 찾는다(issue #10).
+// 어떤 로컬 spawn 보다 먼저 PATH 를 보강한다.
+augmentPathForGuiLaunch();
+
 // Windows 네이티브는 에이전트 실행 계층(sh·tmux·git worktree over sh)이 성립하지 않는다.
 // 보드/원격 머신 관제는 되지만 로컬 run 은 불가 — WSL 데몬을 안내한다.
 if (process.platform === 'win32') {

package/src/paths.ts

@@ -0,0 +1,33 @@
+import { execFileSync } from 'node:child_process';
+import { existsSync } from 'node:fs';
+
+// GUI/launchd 로 뜬 macOS 데몬은 PATH 가 `/usr/bin:/bin:/usr/sbin:/sbin` 뿐이라
+// Homebrew(`/opt/homebrew/bin`·`/usr/local/bin`) 가 빠진다. tmux 는 macOS 기본 제공이
+// 아니라 로컬 터미널이 `spawn tmux ENOENT` 로 죽는다(remote 는 ssh 가 /usr/bin 에 있어 동작).
+// → issue #10. 시작 시 한 번 PATH 를 보강하면 로컬 `pty().spawn('tmux')` 와
+// runShellOn 의 `sh -c` 프로브(둘 다 process.env.PATH 상속)가 함께 해결된다.
+// 시스템 경로 우선순위는 건드리지 않고 없는 디렉터리만 뒤에 덧붙인다(시스템 툴 그림자 방지).
+export function augmentPathForGuiLaunch(): void {
+  if (process.platform !== 'darwin') return;
+  const have = new Set((process.env.PATH ?? '').split(':').filter(Boolean));
+  const add: string[] = [];
+  const push = (d: string): void => {
+    const dir = d.trim();
+    if (dir && !have.has(dir) && existsSync(dir)) { have.add(dir); add.push(dir); }
+  };
+
+  // 1) 로그인 셸의 PATH — 사용자 환경(asdf·nvm·커스텀 tmux 위치)까지 포괄. best-effort.
+  try {
+    const shell = process.env.SHELL || '/bin/zsh';
+    const out = execFileSync(shell, ['-lc', 'printf %s "$PATH"'], { timeout: 4000, encoding: 'utf8' });
+    for (const d of out.split(':')) push(d);
+  } catch { /* 로그인 셸 실패 → 아래 알려진 경로로 보강 */ }
+
+  // 2) 알려진 Homebrew/local 경로 — 로그인 셸이 안 되는 환경 대비.
+  for (const d of ['/opt/homebrew/bin', '/opt/homebrew/sbin', '/usr/local/bin']) push(d);
+
+  if (add.length) {
+    process.env.PATH = `${process.env.PATH}:${add.join(':')}`;
+    console.log(`[coxpit] PATH augmented for GUI/launchd launch (+${add.length}: ${add.join(', ')})`);
+  }
+}

package/src/server.ts

@@ -4,6 +4,7 @@ import { homedir } from 'node:os';
 import { resolve as presolve, dirname as pdirname, join as pjoin, sep as psep } from 'node:path';
 import { createRequire } from 'node:module';
 import { randomBytes } from 'node:crypto';
+import { execFileSync } from 'node:child_process';
 import Fastify, { type FastifyInstance, type FastifyReply } from 'fastify';
 import websocket from '@fastify/websocket';
 import { eq, inArray, and, like, desc } from 'drizzle-orm';
@@ -185,8 +186,26 @@ function sharePageHTML(
 </div></body></html>`;
 }
 
+// macOS 의 machine-wide pty 상한(kern.tty.ptmx_max). 1회 캐시. 다른 OS 는 null.
+let cachedPtyMax: number | null | undefined;
+function ptyMax(): number | null {
+  if (cachedPtyMax !== undefined) return cachedPtyMax;
+  cachedPtyMax = null;
+  if (process.platform === 'darwin') {
+    try {
+      const n = Number(execFileSync('sysctl', ['-n', 'kern.tty.ptmx_max'], { timeout: 2000 }).toString().trim());
+      cachedPtyMax = Number.isFinite(n) && n > 0 ? n : null;
+    } catch { cachedPtyMax = null; }
+  }
+  return cachedPtyMax;
+}
+
 export async function buildServer(): Promise<FastifyInstance> {
   const app = Fastify({ logger: true });
+  // 살아있는 웹 터미널 수 — pty 압력 조기경보(/api/health)용. openTerm 성공 시 +1, 소켓 close 시 -1.
+  // 누수가 재발하면 이 값이 실제 열린 탭보다 커지지 않아도(닫을 때 감소), machine-wide ptmx 대비
+  // 이 데몬의 부하를 노출한다. leak 재발 자체는 회귀 테스트(test/pty-fd.mjs)가 잡는다.
+  let liveTerminals = 0;
   await app.register(websocket);
   // urlencoded 본문 파서(deps 0) — login/setup 폼이 real navigation POST 를 보내면
   // 브라우저가 그 응답의 Set-Cookie 를 확정 커밋한다(Safari fetch-then-replace 레이스 회피).
@@ -205,7 +224,10 @@ export async function buildServer(): Promise<FastifyInstance> {
   app.addHook('onRequest', authGate);
 
   // 무인증 헬스(외부 감시용)
-  app.get('/api/health', async () => ({ ok: true, name: 'coxpit', version: config.version }));
+  app.get('/api/health', async () => {
+    const max = ptyMax();
+    return { ok: true, name: 'coxpit', version: config.version, terminals: liveTerminals, ...(max ? { ptyMax: max } : {}) };
+  });
 
   // 플릿 보드(단일 페이지). 인증 게이트 적용됨(무인증 요청은 게이트가 login/setup 페이지로 응답).
   app.get('/', async (_req, reply) => reply.type('text/html').send(BOARD_HTML));
@@ -1603,10 +1625,25 @@ export async function buildServer(): Promise<FastifyInstance> {
     try {
       term = openTerm(info.machine, info.session, cols, rows);
     } catch (e) {
-      socket.send(JSON.stringify({ t: 'err', d: 'pty spawn failed: ' + String(e).slice(0, 200) }));
+      // 에러 문구를 실행 가능한 사유로 번역(원문은 원인을 가린다 — issue #9/#10).
+      const raw = String(e);
+      let d: string;
+      if (raw.includes('posix_spawnp failed')) {
+        d = 'no free pty on this machine (kern.tty.ptmx_max reached) — restart the coxpit daemon';
+        req.log.warn({ err: raw }, 'pty exhausted (posix_spawnp failed) — machine out of ptys');
+      } else if (raw.includes('ENOENT')) {
+        const bin = /spawn (\S+) ENOENT/.exec(raw)?.[1] ?? 'tmux';
+        d = `terminal binary "${bin}" not found on the daemon PATH — install it, or set COXPIT_TMUX / launch the app with Homebrew on PATH`;
+        req.log.warn({ err: raw }, 'terminal binary not on PATH (ENOENT)');
+      } else {
+        d = 'pty spawn failed: ' + raw.slice(0, 200);
+      }
+      socket.send(JSON.stringify({ t: 'err', d }));
       socket.close();
       return;
     }
+    liveTerminals++;   // pty 압력 지표(/api/health) — close 에서 정확히 1회 감소
+    let closed = false;
     // 백프레셔 — WS 송신 버퍼가 차면 pty 를 잠시 멈춰 폭주 방지
     let paused = false;
     term.onData((d) => {
@@ -1629,6 +1666,7 @@ export async function buildServer(): Promise<FastifyInstance> {
       } catch { /* ignore */ }
     });
     socket.on('close', () => {
+      if (!closed) { closed = true; liveTerminals = Math.max(0, liveTerminals - 1); }
       clearInterval(drain); clearInterval(keepalive);
       try { term.kill(); } catch { /* gone */ }
     });

package/src/term.ts

@@ -1,5 +1,5 @@
 import { createRequire } from 'node:module';
-import { chmodSync } from 'node:fs';
+import { chmodSync, closeSync, fstatSync, readdirSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import type { IPty } from 'node-pty';
 import { config } from './config';
@@ -13,8 +13,15 @@ function fixSpawnHelper(): void {
   try {
     const ptyPkg = require_.resolve('node-pty/package.json');
     const base = dirname(ptyPkg);
+    const candidates = [
+      join(base, 'build', 'Release', 'spawn-helper'), // 소스빌드(node-gyp) 경로
+      join(base, 'build', 'Debug', 'spawn-helper'),
+    ];
     for (const dir of [`darwin-${process.arch}`, `linux-${process.arch}`]) {
-      try { chmodSync(join(base, 'prebuilds', dir, 'spawn-helper'), 0o755); } catch { /* absent */ }
+      candidates.push(join(base, 'prebuilds', dir, 'spawn-helper'));
+    }
+    for (const p of candidates) {
+      try { chmodSync(p, 0o755); } catch { /* absent or read-only bundle */ }
     }
   } catch { /* node-pty missing — openTerm 에서 에러 */ }
 }
@@ -28,6 +35,39 @@ function pty(): PtyModule {
   return ptyMod;
 }
 
+// node-pty 1.1.0 의 macOS(posix_spawn) 경로는 spawn 마다 pty master 를 하나 더 열고 안 닫는다
+// (src/unix/pty.cc pty_posix_spawn: low_fds 정리 루프가 count==0 이면 아무것도 닫지 않음).
+// 터미널을 열 때마다 /dev/ptmx 가 하나씩 새어 kern.tty.ptmx_max(맥 기본 511) 가 마르면
+// 그 뒤로는 모든 spawn 이 "posix_spawnp failed." 로 죽는다 (2026-09-10 맥미니 데몬 256개 누수).
+// spawn 전후의 fd 를 비교해, 새로 생긴 pty master(term.fd 와 같은 major 의 문자 디바이스) 중
+// term.fd 가 아닌 것만 닫는다. 슬레이브·kqueue·/dev/null 은 major 가 달라 건드리지 않고,
+// 스레드풀이 동시에 여는 일반 파일도 문자 디바이스가 아니라 안전하다. Linux 는 forkpty 경로라 해당 없음.
+function liveFds(): Set<number> {
+  const s = new Set<number>();
+  for (const n of readdirSync('/dev/fd')) {
+    const fd = Number(n);
+    try { fstatSync(fd); s.add(fd); } catch { /* readdir 자신의 fd 등 이미 닫힌 것 */ }
+  }
+  return s;
+}
+export function spawnPty(file: string, args: string[], opts: Parameters<PtyModule['spawn']>[2]): IPty {
+  if (process.platform !== 'darwin') return pty().spawn(file, args, opts);
+  const before = liveFds();
+  const term = pty().spawn(file, args, opts);
+  const fd = (term as unknown as { fd?: number }).fd;
+  if (typeof fd !== 'number') return term;
+  let masterMajor: number;
+  try { masterMajor = (fstatSync(fd).rdev >> 24) & 0xff; } catch { return term; }
+  for (const n of liveFds()) {
+    if (n === fd || before.has(n)) continue;
+    try {
+      const st = fstatSync(n);
+      if (st.isCharacterDevice() && ((st.rdev >> 24) & 0xff) === masterMajor) closeSync(n);
+    } catch { /* 그 사이 닫힘 */ }
+  }
+  return term;
+}
+
 function isLocal(m: MachineTarget): boolean {
   return m.kind === 'local' || m.address === '';
 }
@@ -45,7 +85,7 @@ export function openTerm(m: MachineTarget, session: string, cols: number, rows:
     env: { ...process.env, TERM: 'xterm-256color', LANG: config.lang } as Record<string, string>,
   };
   if (isLocal(m)) {
-    return pty().spawn('tmux', ['attach-session', '-t', '=' + session], opts);
+    return spawnPty('tmux', ['attach-session', '-t', '=' + session], opts);
   }
   const args: string[] = [
     '-tt',
@@ -58,5 +98,5 @@ export function openTerm(m: MachineTarget, session: string, cols: number, rows:
   // 세션명은 우리가 만든 coxpit-rN 형식이라 셸 주입 여지 없음 — 그래도 인용.
   // 원격도 UTF-8 로케일 명시 (비대화 ssh 는 LANG 미설정이 보통)
   args.push(target, `export LANG='${config.lang.replace(/'/g, '')}'; tmux attach-session -t '=${session.replace(/'/g, "'\\''")}'`);
-  return pty().spawn('ssh', args, opts);
+  return spawnPty('ssh', args, opts);
 }