tower-studio 0.1.3 → 0.1.6

package/src/lib/platform.ts

@@ -1,518 +0,0 @@
-/**
- * platform.ts — Unified cross-platform utilities.
- *
- * Consolidates duplicate command-resolution and path-normalization logic that
- * previously lived in both cli-test.ts (async, for child_process.spawn) and
- * session-store.ts (sync, for node-pty).  Both consumers now import from here.
- *
- * Design constraints:
- *   - Must support both sync (PTY) and async (spawn) callers.
- *   - Must be testable with platform mocks (win32 / darwin / linux).
- *   - No side-effects on import — all state is passed as arguments.
- */
-
-import { constants as fsConstants, promises as fs } from "node:fs";
-import { execSync } from "node:child_process";
-import path from "node:path";
-
-// ---------------------------------------------------------------------------
-// Types
-// ---------------------------------------------------------------------------
-
-export interface SpawnTarget {
-  command: string;
-  args: string[];
-}
-
-export interface ResolveCommandOptions {
-  /** Working directory for resolving relative paths */
-  cwd?: string;
-  /** Environment variables (defaults to process.env) */
-  env?: NodeJS.ProcessEnv;
-  /** Override platform for testing (defaults to process.platform) */
-  platform?: NodeJS.Platform;
-}
-
-// ---------------------------------------------------------------------------
-// Platform detection
-// ---------------------------------------------------------------------------
-
-export function isWindows(platform: NodeJS.Platform = process.platform): boolean {
-  return platform === "win32";
-}
-
-// ---------------------------------------------------------------------------
-// Path normalization
-// ---------------------------------------------------------------------------
-
-/**
- * Normalize a file path for the target platform:
- *   - Runs path.normalize (collapses .. and .)
- *   - On Windows: converts forward slashes to backslashes
- *   - On Unix: converts backslashes to forward slashes
- *
- * Useful when comparing user-provided paths that may contain mixed separators
- * (e.g. "D:\project/foo/bar").
- */
-export function normalizePath(
-  p: string,
-  platform: NodeJS.Platform = process.platform,
-): string {
-  const normalized = path.normalize(p);
-  if (isWindows(platform)) {
-    return normalized.replace(/\//g, "\\");
-  }
-  return normalized.replace(/\\/g, "/");
-}
-
-/**
- * Convert a path to forward-slash form for cross-platform comparison.
- * Useful when comparing paths from different OS representations
- * (e.g. git worktree list output on Windows vs stored paths).
- */
-export function toForwardSlash(p: string): string {
-  return p.replace(/\\/g, "/");
-}
-
-// ---------------------------------------------------------------------------
-// Command quoting (Windows cmd.exe)
-// ---------------------------------------------------------------------------
-
-/**
- * Quote/escape a single argument for use inside a `cmd.exe /c` command line.
- *
- * Rules:
- *   - Flatten newlines (CR/LF → space) — CreateProcess does not accept them.
- *   - Double `"` as required by cmd.exe escaping (`""` inside a quoted arg).
- *   - Wrap in `"…"` when the argument contains whitespace or shell meta-chars.
- *   - Empty string → `""`.
- */
-export function quoteForCmd(arg: string): string {
-  const flat = arg.replace(/\r\n|\r|\n/g, " ");
-  if (!flat.length) return '""';
-  const escaped = flat.replace(/"/g, '""');
-  return /[\s"&<>|^()]/.test(escaped) ? `"${escaped}"` : escaped;
-}
-
-// ---------------------------------------------------------------------------
-// Environment utilities
-// ---------------------------------------------------------------------------
-
-/** Default PATH values when the env has none (prevents "command not found"). */
-function defaultPathForPlatform(platform: NodeJS.Platform = process.platform): string {
-  if (isWindows(platform)) {
-    return "C:\\Windows\\System32;C:\\Windows;C:\\Windows\\System32\\Wbem";
-  }
-  return "/usr/local/bin:/opt/homebrew/bin:/usr/local/sbin:/usr/bin:/bin:/usr/sbin:/sbin";
-}
-
-/**
- * Ensure env has a non-empty PATH. If missing/empty, merges sensible defaults.
- * Returns a new object — never mutates the input.
- */
-export function ensurePathInEnv(
-  env: Record<string, string | undefined>,
-  platform: NodeJS.Platform = process.platform,
-): Record<string, string | undefined> {
-  const key = isWindows(platform) ? "Path" : "PATH";
-  const current = env.PATH ?? env.Path ?? "";
-  if (current.length > 0) return env;
-  return { ...env, [key]: defaultPathForPlatform(platform) };
-}
-
-/**
- * Claude Code nesting prevention — strip env vars that block launching
- * a new Claude CLI instance from within an existing Claude session.
- * Returns a new object.
- */
-const CLAUDE_NESTING_VARS = [
-  "CLAUDECODE",
-  "CLAUDE_CODE_ENTRYPOINT",
-  "CLAUDE_CODE_SESSION",
-  "CLAUDE_CODE_PARENT_SESSION",
-];
-
-export function stripClaudeNestingEnv(
-  env: Record<string, string | undefined>,
-): Record<string, string | undefined> {
-  const cleaned = { ...env };
-  for (const key of CLAUDE_NESTING_VARS) {
-    delete cleaned[key];
-  }
-  return cleaned;
-}
-
-/**
- * Redact sensitive values from an env object for safe logging.
- * Matches keys containing: key, token, secret, password, authorization, cookie.
- */
-const SENSITIVE_KEY_RE = /key|token|secret|password|passwd|authorization|cookie/i;
-
-export function redactEnvForLogs(
-  env: Record<string, string | undefined>,
-): Record<string, string> {
-  const result: Record<string, string> = {};
-  for (const [k, v] of Object.entries(env)) {
-    if (v === undefined) continue;
-    result[k] = SENSITIVE_KEY_RE.test(k) ? "***REDACTED***" : v;
-  }
-  return result;
-}
-
-// ---------------------------------------------------------------------------
-// Command resolution
-// ---------------------------------------------------------------------------
-
-/** Windows PATHEXT extensions, parsed from environment or sensible default. */
-function windowsPathExts(env: NodeJS.ProcessEnv): string[] {
-  return (env.PATHEXT ?? ".EXE;.CMD;.BAT;.COM").split(";").filter(Boolean);
-}
-
-/**
- * Async command resolution — walks PATH directories and probes file existence.
- * Suitable for child_process.spawn callers that can afford a few fs.access calls.
- *
- * @returns Absolute path to the executable, or `null` if not found.
- */
-export async function resolveCommandPath(
-  command: string,
-  opts: ResolveCommandOptions = {},
-): Promise<string | null> {
-  const cwd = opts.cwd ?? process.cwd();
-  const env = opts.env ?? process.env;
-  const platform = opts.platform ?? process.platform;
-
-  const hasPathSeparator = command.includes("/") || command.includes("\\");
-  if (hasPathSeparator) {
-    const absolute = path.isAbsolute(command) ? command : path.resolve(cwd, command);
-    return (await pathExists(absolute, platform)) ? absolute : null;
-  }
-
-  const pathValue = env.PATH ?? env.Path ?? "";
-  const delimiter = isWindows(platform) ? ";" : ":";
-  const dirs = pathValue.split(delimiter).filter(Boolean);
-  const exts = isWindows(platform) ? windowsPathExts(env) : [""];
-  const hasExtension = isWindows(platform) && path.extname(command).length > 0;
-
-  for (const dir of dirs) {
-    const candidates = isWindows(platform)
-      ? hasExtension
-        ? [path.join(dir, command)]
-        : exts.map((ext) => path.join(dir, `${command}${ext}`))
-      : [path.join(dir, command)];
-
-    for (const candidate of candidates) {
-      if (await pathExists(candidate, platform)) return candidate;
-    }
-  }
-
-  return null;
-}
-
-/** Function signature for the shell exec used by resolveCommandPathSync (injectable for tests). */
-export type ExecSyncFn = (cmd: string) => string;
-
-const defaultExecSync: ExecSyncFn = (cmd) =>
-  execSync(cmd, { encoding: "utf-8", timeout: 5000 });
-
-/**
- * Sync command resolution — uses `where` (Windows) or `which` (Unix).
- * Suitable for node-pty callers that need a synchronous result.
- *
- * On Windows, prefers .cmd/.bat/.exe over extension-less shims.
- *
- * @param command  - The command name (e.g. "claude")
- * @param platform - Override for testing
- * @param exec     - Override shell exec for testing
- * @returns Resolved path, or the original command if resolution fails.
- */
-export function resolveCommandPathSync(
-  command: string,
-  platform: NodeJS.Platform = process.platform,
-  exec: ExecSyncFn = defaultExecSync,
-): string {
-  // If it already has an extension or path separator, return as-is
-  if (path.extname(command) || command.includes("/") || command.includes("\\")) {
-    return command;
-  }
-
-  if (!isWindows(platform)) {
-    return command; // Unix callers rely on PATH lookup at spawn time
-  }
-
-  try {
-    const result = exec(`where ${command}`);
-    const foundPaths = result.trim().split("\n").map((p) => p.trim()).filter(Boolean);
-
-    // Prefer .cmd/.bat/.exe over extension-less shims
-    const withExt = foundPaths.find((p) => /\.(cmd|bat|exe)$/i.test(p));
-    if (withExt) return withExt;
-    if (foundPaths.length > 0) return foundPaths[0];
-  } catch {
-    // Command not found in PATH — return as-is, let caller handle
-  }
-
-  return command;
-}
-
-// ---------------------------------------------------------------------------
-// Spawn target resolution
-// ---------------------------------------------------------------------------
-
-/**
- * Async spawn target — resolves command + wraps .cmd/.bat for Windows.
- * Use with child_process.spawn.
- */
-export async function resolveSpawnTarget(
-  command: string,
-  args: string[],
-  opts: ResolveCommandOptions = {},
-): Promise<SpawnTarget> {
-  const env = opts.env ?? process.env;
-  const platform = opts.platform ?? process.platform;
-
-  const resolved = await resolveCommandPath(command, opts);
-  const executable = resolved ?? command;
-
-  return wrapForPlatform(executable, args, env, platform);
-}
-
-/**
- * Sync spawn target — resolves command + wraps .cmd/.bat for Windows.
- * Use with node-pty.
- */
-export function resolveSpawnTargetSync(
-  command: string,
-  args: string[],
-  platform: NodeJS.Platform = process.platform,
-  exec?: ExecSyncFn,
-): SpawnTarget {
-  const resolved = resolveCommandPathSync(command, platform, exec);
-  return wrapForPlatform(resolved, args, process.env, platform);
-}
-
-// ---------------------------------------------------------------------------
-// Internal helpers
-// ---------------------------------------------------------------------------
-
-/** Wrap a resolved executable for the target platform's shell if needed. */
-function wrapForPlatform(
-  executable: string,
-  args: string[],
-  env: NodeJS.ProcessEnv,
-  platform: NodeJS.Platform,
-): SpawnTarget {
-  if (!isWindows(platform)) {
-    return { command: executable, args };
-  }
-
-  const ext = path.extname(executable).toLowerCase();
-  if (ext === ".cmd" || ext === ".bat" || ext === ".com") {
-    const shell = env.ComSpec || "cmd.exe";
-    const commandLine = [quoteForCmd(executable), ...args.map(quoteForCmd)].join(" ");
-    return {
-      command: shell,
-      args: ["/d", "/s", "/c", commandLine],
-    };
-  }
-
-  return { command: executable, args };
-}
-
-async function pathExists(
-  candidate: string,
-  platform: NodeJS.Platform = process.platform,
-): Promise<boolean> {
-  try {
-    await fs.access(candidate, isWindows(platform) ? fsConstants.F_OK : fsConstants.X_OK);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-// ---------------------------------------------------------------------------
-// Shell detection (for PTY spawning, AI CLI testing)
-// ---------------------------------------------------------------------------
-
-export interface DetectedShell {
-  /** Display name, e.g. "zsh", "Git Bash" */
-  name: string;
-  /** Absolute path to the shell binary */
-  path: string;
-}
-
-/**
- * Detect available shells on the current platform.
- *
- * macOS/Linux: reads /etc/shells (same approach as VS Code).
- * Windows: probes well-known Git Bash / PowerShell / WSL paths
- *          (adapted from VS Code's terminal profile detection).
- */
-export async function detectShells(
-  platform: NodeJS.Platform = process.platform,
-): Promise<DetectedShell[]> {
-  if (!isWindows(platform)) {
-    return detectUnixShells();
-  }
-  return detectWindowsShells();
-}
-
-/** Read /etc/shells — the OS-maintained list of valid login shells. */
-async function detectUnixShells(): Promise<DetectedShell[]> {
-  try {
-    const content = await fs.readFile("/etc/shells", "utf-8");
-    const shells: DetectedShell[] = [];
-    const seen = new Map<string, number>();
-
-    for (const raw of content.split("\n")) {
-      const line = raw.replace(/#.*/, "").trim();
-      if (!line || !line.startsWith("/")) continue;
-      if (!(await pathExists(line))) continue;
-
-      let name = path.basename(line);
-      // Handle duplicate names (e.g. /bin/bash and /usr/local/bin/bash)
-      const count = seen.get(name) ?? 0;
-      seen.set(name, count + 1);
-      if (count > 0) name = `${name} (${count + 1})`;
-
-      shells.push({ name, path: line });
-    }
-    return shells;
-  } catch {
-    // /etc/shells not readable — return sensible defaults
-    const defaults = ["/bin/zsh", "/bin/bash", "/bin/sh"];
-    const shells: DetectedShell[] = [];
-    for (const p of defaults) {
-      if (await pathExists(p)) shells.push({ name: path.basename(p), path: p });
-    }
-    return shells;
-  }
-}
-
-/** Probe well-known Windows shell locations (VS Code approach). */
-async function detectWindowsShells(): Promise<DetectedShell[]> {
-  const shells: DetectedShell[] = [];
-
-  // --- Git Bash ---
-  // Strategy: find git.exe on PATH → derive bash.exe path, then probe ProgramFiles
-  const gitBashCandidates: string[] = [];
-
-  // 1. From git.exe on PATH (most reliable)
-  const gitPath = await resolveCommandPath("git", { platform: "win32" });
-  if (gitPath) {
-    const gitRoot = path.resolve(path.dirname(gitPath), "..", "..");
-    gitBashCandidates.push(
-      path.join(gitRoot, "bin", "bash.exe"),
-      path.join(gitRoot, "usr", "bin", "bash.exe"),
-    );
-  }
-
-  // 2. Standard install directories
-  const programDirs = [
-    process.env.ProgramW6432,
-    process.env.ProgramFiles,
-    process.env["ProgramFiles(x86)"],
-    process.env.LocalAppData ? path.join(process.env.LocalAppData, "Programs") : null,
-  ].filter(Boolean) as string[];
-
-  for (const dir of programDirs) {
-    gitBashCandidates.push(
-      path.join(dir, "Git", "bin", "bash.exe"),
-      path.join(dir, "Git", "usr", "bin", "bash.exe"),
-    );
-  }
-
-  // 3. Scoop
-  const home = process.env.USERPROFILE ?? "";
-  if (home) {
-    gitBashCandidates.push(
-      path.join(home, "scoop", "apps", "git", "current", "bin", "bash.exe"),
-    );
-  }
-
-  // 4. CLAUDE_CODE_GIT_BASH_PATH env var
-  if (process.env.CLAUDE_CODE_GIT_BASH_PATH) {
-    gitBashCandidates.unshift(process.env.CLAUDE_CODE_GIT_BASH_PATH);
-  }
-
-  for (const candidate of gitBashCandidates) {
-    if (await pathExists(candidate, "win32")) {
-      shells.push({ name: "Git Bash", path: candidate });
-      break;
-    }
-  }
-
-  // --- PowerShell Core (pwsh.exe) ---
-  const pwshPath = await resolveCommandPath("pwsh", { platform: "win32" });
-  if (pwshPath) {
-    shells.push({ name: "PowerShell", path: pwshPath });
-  }
-
-  // --- Windows PowerShell (legacy) ---
-  const winDir = process.env.windir ?? "C:\\WINDOWS";
-  const legacyPs = path.join(winDir, "System32", "WindowsPowerShell", "v1.0", "powershell.exe");
-  if (await pathExists(legacyPs, "win32")) {
-    shells.push({ name: "Windows PowerShell", path: legacyPs });
-  }
-
-  // --- Command Prompt ---
-  const cmd = process.env.ComSpec ?? path.join(winDir, "System32", "cmd.exe");
-  if (await pathExists(cmd, "win32")) {
-    shells.push({ name: "Command Prompt", path: cmd });
-  }
-
-  return shells;
-}
-
-// ---------------------------------------------------------------------------
-// Terminal app detection (for "Open in Terminal" GUI action)
-// ---------------------------------------------------------------------------
-
-export interface DetectedTerminalApp {
-  /** Display name, e.g. "iTerm2", "Windows Terminal" */
-  name: string;
-  /** Value for macOS `open -a` or Windows executable path */
-  value: string;
-}
-
-/** Well-known macOS terminal GUI applications. */
-const MAC_TERMINAL_APPS: Array<{ name: string; bundleName: string }> = [
-  { name: "Terminal", bundleName: "Terminal" },
-  { name: "iTerm2", bundleName: "iTerm" },
-  { name: "Warp", bundleName: "Warp" },
-  { name: "Alacritty", bundleName: "Alacritty" },
-  { name: "WezTerm", bundleName: "WezTerm" },
-  { name: "kitty", bundleName: "kitty" },
-  { name: "Hyper", bundleName: "Hyper" },
-];
-
-/**
- * Detect available GUI terminal applications.
- * macOS: checks /Applications/*.app
- * Windows: checks for Windows Terminal (wt.exe) in PATH
- */
-export async function detectTerminalApps(
-  platform: NodeJS.Platform = process.platform,
-): Promise<DetectedTerminalApp[]> {
-  if (platform === "darwin") {
-    const results: DetectedTerminalApp[] = [];
-    for (const t of MAC_TERMINAL_APPS) {
-      if (await pathExists(`/Applications/${t.bundleName}.app`, platform)) {
-        results.push({ name: t.name, value: t.bundleName });
-      }
-    }
-    return results;
-  }
-
-  if (isWindows(platform)) {
-    const results: DetectedTerminalApp[] = [];
-    const wtPath = await resolveCommandPath("wt", { platform });
-    if (wtPath) {
-      results.push({ name: "Windows Terminal", value: wtPath });
-    }
-    return results;
-  }
-
-  return [];
-}

package/src/lib/preview-process.ts

@@ -1,40 +0,0 @@
-import { type ChildProcess } from "node:child_process";
-
-// Module-level singleton — persists across requests in the same Node.js process
-const previewProcesses = new Map<string, ChildProcess>();
-
-export function registerPreviewProcess(taskId: string, child: ChildProcess): void {
-  // Clean up stale entry if exists
-  killPreviewProcess(taskId);
-  // Auto-remove from map when process exits
-  child.on("exit", () => {
-    if (previewProcesses.get(taskId) === child) {
-      previewProcesses.delete(taskId);
-    }
-  });
-  previewProcesses.set(taskId, child);
-}
-
-export function killPreviewProcess(taskId: string): boolean {
-  const child = previewProcesses.get(taskId);
-  if (!child) return false;
-  previewProcesses.delete(taskId);
-  if (child.killed) return false;
-  child.kill("SIGTERM");
-  return true;
-}
-
-export function killAllPreviewProcesses(): void {
-  for (const taskId of [...previewProcesses.keys()]) {
-    killPreviewProcess(taskId);
-  }
-}
-
-export function isPreviewRunning(taskId: string): boolean {
-  const child = previewProcesses.get(taskId);
-  return !!child && !child.killed;
-}
-
-// Cleanup on process exit
-process.on("SIGTERM", () => killAllPreviewProcesses());
-process.on("SIGINT", () => killAllPreviewProcesses());

package/src/lib/pty/__tests__/ws-server-assistant.test.ts

@@ -1,7 +0,0 @@
-import { describe, it } from "vitest";
-
-describe("ws-server assistant handling", () => {
-  it.todo("immediately destroys __assistant__ session on WS close (BE-05)");
-  it.todo("does not set keepalive timer for __assistant__ sessions");
-  it.todo("still applies keepalive for non-assistant sessions");
-});