@bastani/atomic 0.6.8 → 0.7.0-2

package/src/commands/cli/chat/index.test.ts

@@ -1,252 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import {
-  buildLauncherEnv,
-  buildLauncherScript,
-  buildSpawnEnv,
-  resolveChatCommand,
-} from "./index.ts";
-import { atomicTempEnv } from "../../../lib/atomic-temp.ts";
-
-const TERMINAL_ENV_KEYS = ["LANG", "LC_ALL", "LC_CTYPE", "TERM", "COLORTERM"] as const;
-
-const sampleTerminalEnv: Record<string, string> = {
-  LANG: "en_US.UTF-8",
-  LC_ALL: "en_US.UTF-8",
-  LC_CTYPE: "en_US.UTF-8",
-  TERM: "xterm-256color",
-  COLORTERM: "truecolor",
-};
-
-function withMockPlatform<T>(platform: NodeJS.Platform, fn: () => T): T {
-  const originalPlatform = process.platform;
-  Object.defineProperty(process, "platform", {
-    configurable: true,
-    value: platform,
-  });
-  try {
-    return fn();
-  } finally {
-    Object.defineProperty(process, "platform", {
-      configurable: true,
-      value: originalPlatform,
-    });
-  }
-}
-
-function withEnvVar<T>(key: string, value: string, fn: () => T): T {
-  const originalValue = process.env[key];
-  process.env[key] = value;
-
-  try {
-    return fn();
-  } finally {
-    if (originalValue === undefined) {
-      delete process.env[key];
-    } else {
-      process.env[key] = originalValue;
-    }
-  }
-}
-
-describe("buildLauncherScript", () => {
-  test("builds a PowerShell launcher with cwd, env, args, and exit code", () => {
-    const { script, ext } = withMockPlatform("win32", () =>
-      buildLauncherScript(
-        "copilot",
-        ["--debug"],
-        "C:\\repo",
-        { ATOMIC_AGENT: "copilot" },
-      )
-    );
-
-    expect(ext).toBe("ps1");
-    expect(script).toContain('Set-Location "C:\\repo"');
-    expect(script).toContain('${env:ATOMIC_AGENT} = "copilot"');
-    expect(script).toContain('& "copilot" @("--debug")');
-    expect(script).toContain('if ($LASTEXITCODE -is [int]) { $atomicExitCode = $LASTEXITCODE }');
-    expect(script).toContain("exit $atomicExitCode");
-    expect(script).not.toContain("Invoke-AtomicSessionCleanup");
-  });
-
-  test("builds a bash launcher without tmux input suppression", () => {
-    const { script, ext } = withMockPlatform("linux", () =>
-      buildLauncherScript(
-        "claude",
-        ["--dangerously-skip-permissions"],
-        "/repo",
-        { ATOMIC_AGENT: "claude" },
-      )
-    );
-
-    expect(ext).toBe("sh");
-    expect(script).toContain('cd "/repo"');
-    expect(script).toContain('export ATOMIC_AGENT="claude"');
-    expect(script).toContain('"claude" "--dangerously-skip-permissions"');
-    expect(script).toContain("atomic_exit_code=$?");
-    expect(script).not.toContain("exec ");
-    expect(script).not.toContain("stty -echo -icanon");
-    expect(script).not.toContain("atomic_original_tty_state");
-    expect(script).not.toContain("trap atomic_cleanup");
-  });
-
-  describe("terminal env key exports", () => {
-    test("bash launcher exports LANG, LC_ALL, LC_CTYPE, TERM, COLORTERM", () => {
-      const { script, ext } = withMockPlatform("linux", () =>
-        buildLauncherScript("claude", [], "/repo", sampleTerminalEnv)
-      );
-
-      expect(ext).toBe("sh");
-      for (const key of TERMINAL_ENV_KEYS) {
-        expect(script).toContain(`export ${key}="${sampleTerminalEnv[key]}"`);
-      }
-    });
-
-    test("PowerShell launcher sets LANG, LC_ALL, LC_CTYPE, TERM, COLORTERM", () => {
-      const { script, ext } = withMockPlatform("win32", () =>
-        buildLauncherScript("claude", [], "C:\\repo", sampleTerminalEnv)
-      );
-
-      expect(ext).toBe("ps1");
-      for (const key of TERMINAL_ENV_KEYS) {
-        expect(script).toContain(`\${env:${key}} = "${sampleTerminalEnv[key]}"`);
-      }
-    });
-
-    test("bash launcher emits export lines for all five terminal env keys when merged with other env", () => {
-      const envVars = { ...sampleTerminalEnv, ATOMIC_AGENT: "claude" };
-      const { script } = withMockPlatform("linux", () =>
-        buildLauncherScript("claude", [], "/repo", envVars)
-      );
-
-      for (const key of TERMINAL_ENV_KEYS) {
-        expect(script).toContain(`export ${key}=`);
-      }
-      expect(script).toContain('export ATOMIC_AGENT="claude"');
-    });
-
-    test("PowerShell launcher emits braced env assignments for all five terminal env keys when merged with other env", () => {
-      const envVars = { ...sampleTerminalEnv, ATOMIC_AGENT: "copilot" };
-      const { script } = withMockPlatform("win32", () =>
-        buildLauncherScript("copilot", [], "C:\\repo", envVars)
-      );
-
-      for (const key of TERMINAL_ENV_KEYS) {
-        expect(script).toContain(`\${env:${key}} =`);
-      }
-      expect(script).toContain('${env:ATOMIC_AGENT} = "copilot"');
-    });
-
-    test("PowerShell launcher handles env keys with punctuation", () => {
-      const { script } = withMockPlatform("win32", () =>
-        buildLauncherScript("copilot", [], "C:\\repo", {
-          "ProgramFiles(x86)": "C:\\Program Files (x86)",
-        })
-      );
-
-      expect(script).toContain('${env:ProgramFiles(x86)} = "C:\\Program Files (x86)"');
-    });
-
-    test("bash launcher rejects invalid env keys", () => {
-      expect(() =>
-        withMockPlatform("linux", () =>
-          buildLauncherScript("claude", [], "/repo", {
-            "ProgramFiles(x86)": "C:\\Program Files (x86)",
-          })
-        )
-      ).toThrow('Invalid Bash env key "ProgramFiles(x86)"');
-    });
-  });
-});
-
-describe("chat env builders", () => {
-  const devcontainerEnv = {
-    GH_TOKEN: "gh-secret",
-    COPILOT_GITHUB_TOKEN: "copilot-secret",
-    ANTHROPIC_API_KEY: "anthropic-secret",
-    PATH: "/usr/local/bin",
-    HOME: "/home/dev",
-    LANG: "C",
-    TERM: "dumb",
-    "ProgramFiles(x86)": "C:\\Program Files (x86)",
-  };
-
-  test("buildLauncherEnv serializes only terminal defaults and explicit env", () => {
-    const env = buildLauncherEnv({ ATOMIC_AGENT: "copilot" }, devcontainerEnv);
-
-    expect(env).toEqual({
-      LANG: "en_US.UTF-8",
-      LC_ALL: "en_US.UTF-8",
-      LC_CTYPE: "en_US.UTF-8",
-      TERM: "xterm-256color",
-      COLORTERM: "truecolor",
-      ATOMIC_AGENT: "copilot",
-    });
-  });
-
-  test("Claude temp env is preserved when serialized into a launcher", () => {
-    const claudeTempEnv = atomicTempEnv("/home/dev/.atomic/tmp");
-    const env = buildLauncherEnv({ ...claudeTempEnv, ATOMIC_AGENT: "claude" }, devcontainerEnv);
-
-    expect(env.TMPDIR).toBe("/home/dev/.atomic/tmp");
-    expect(env.TMP).toBe("/home/dev/.atomic/tmp");
-    expect(env.TEMP).toBe("/home/dev/.atomic/tmp");
-  });
-
-  test("buildLauncherEnv does not include inherited secrets or platform keys", () => {
-    const env = buildLauncherEnv({ ATOMIC_AGENT: "copilot" }, devcontainerEnv);
-
-    expect(env.GH_TOKEN).toBeUndefined();
-    expect(env.COPILOT_GITHUB_TOKEN).toBeUndefined();
-    expect(env.ANTHROPIC_API_KEY).toBeUndefined();
-    expect(env.PATH).toBeUndefined();
-    expect(env.HOME).toBeUndefined();
-    expect(env["ProgramFiles(x86)"]).toBeUndefined();
-  });
-
-  test("buildSpawnEnv preserves inherited process env while normalizing terminal keys", () => {
-    const env = buildSpawnEnv({ ATOMIC_AGENT: "copilot" }, devcontainerEnv);
-
-    expect(env.GH_TOKEN).toBe("gh-secret");
-    expect(env.COPILOT_GITHUB_TOKEN).toBe("copilot-secret");
-    expect(env.ANTHROPIC_API_KEY).toBe("anthropic-secret");
-    expect(env.PATH).toBe("/usr/local/bin");
-    expect(env.HOME).toBe("/home/dev");
-    expect(env["ProgramFiles(x86)"]).toBe("C:\\Program Files (x86)");
-    expect(env.LANG).toBe("en_US.UTF-8");
-    expect(env.TERM).toBe("xterm-256color");
-    expect(env.ATOMIC_AGENT).toBe("copilot");
-  });
-
-  test("launcher script generated from buildLauncherEnv contains no inherited secret values", () => {
-    const env = buildLauncherEnv({ ATOMIC_AGENT: "copilot" }, devcontainerEnv);
-    const { script } = withMockPlatform("linux", () =>
-      buildLauncherScript("copilot", [], "/repo", env)
-    );
-
-    expect(script).not.toContain("gh-secret");
-    expect(script).not.toContain("copilot-secret");
-    expect(script).not.toContain("anthropic-secret");
-    expect(script).toContain('export LANG="en_US.UTF-8"');
-    expect(script).toContain('export ATOMIC_AGENT="copilot"');
-  });
-});
-
-describe("resolveChatCommand", () => {
-  test("uses COPILOT_CLI_PATH for Copilot even when command is outside PATH", () => {
-    withEnvVar("COPILOT_CLI_PATH", "/custom/bin/copilot-custom", () => {
-      expect(resolveChatCommand("copilot")).toBe("/custom/bin/copilot-custom");
-    });
-  });
-
-  test("launcher script accepts resolved Copilot path", () => {
-    const resolvedCopilotPath = "/custom/bin/copilot-custom";
-    const { script } = withMockPlatform("linux", () =>
-      buildLauncherScript(resolvedCopilotPath, ["--debug"], "/repo", {
-        ATOMIC_AGENT: "copilot",
-      })
-    );
-
-    expect(script).toContain('"/custom/bin/copilot-custom" "--debug"');
-    expect(script).not.toContain('"copilot" "--debug"');
-  });
-});

package/src/commands/cli/chat/index.ts

@@ -1,430 +0,0 @@
-#!/usr/bin/env bun
-/**
- * Chat CLI command for atomic
- *
- * Spawns the native agent CLI in tmux/psmux with an OpenTUI footer pane.
- *
- * All extra arguments after `-a <agent>` are forwarded to the native CLI.
- *
- * Usage:
- *   atomic chat -a <agent> [native-args...]
- */
-
-import { dirname, join } from "node:path";
-import { homedir } from "node:os";
-import { mkdir, writeFile, rm } from "node:fs/promises";
-import { AGENT_CONFIG, type AgentKey } from "../../../services/config/index.ts";
-import { getProviderOverrides } from "../../../services/config/atomic-config.ts";
-import { getCopilotScmDisableFlags } from "../../../services/config/scm-sync.ts";
-import {
-  resolveAdditionalInstructionsPath,
-} from "../../../services/config/additional-instructions.ts";
-import { ensureProjectSetup } from "../init/index.ts";
-import { COLORS } from "../../../theme/colors.ts";
-import {
-  getCommandPath,
-} from "../../../services/system/detect.ts";
-import { checkAgentAuth, printAuthError } from "../../../services/system/auth.ts";
-import {
-  ensureAtomicGlobalAgentConfigs,
-} from "../../../services/config/atomic-global-config.ts";
-import { getConfigRoot } from "../../../services/config/config-path.ts";
-import {
-  isInsideAtomicSocket,
-  isInsideTmux,
-  isTmuxInstalled,
-  resetMuxBinaryCache,
-  createSession,
-  detachAndAttachAtomic,
-  killSessionOnPaneExit,
-  killSession,
-  spawnMuxAttach,
-  switchClient,
-} from "../../../sdk/runtime/tmux.ts";
-import { spawnAttachedFooter } from "../../../sdk/runtime/attached-footer.ts";
-import { ensureTmuxInstalled } from "../../../lib/spawn.ts";
-import {
-  buildLauncherEnv,
-  buildSpawnEnv,
-  buildTmuxEnv,
-} from "../../../lib/terminal-env.ts";
-import { atomicTempEnv } from "../../../lib/atomic-temp.ts";
-import {
-  type CommandPathResolver,
-  resolveCopilotCliPath,
-} from "../../../sdk/providers/copilot.ts";
-
-export {
-  buildLauncherEnv,
-  buildSpawnEnv,
-  buildTmuxEnv,
-  TERMINAL_ENV_KEYS,
-  type TerminalEnvKey,
-} from "../../../lib/terminal-env.ts";
-
-// ============================================================================
-// Types
-// ============================================================================
-
-export type AgentType = AgentKey;
-
-/**
- * Options for the chat command.
- */
-export interface ChatCommandOptions {
-  /** Agent type to use (claude, opencode, copilot) */
-  agentType?: AgentType;
-  /** Extra args/options forwarded verbatim to the native agent CLI */
-  passthroughArgs?: string[];
-}
-
-// ============================================================================
-// Helpers
-// ============================================================================
-
-export function getAgentDisplayName(agentType: AgentType): string {
-  return AGENT_CONFIG[agentType].name;
-}
-
-/**
- * Build the argv array for spawning the agent CLI.
- *
- * Starts with the agent's default chat_flags (or replaces them entirely
- * when the user sets `chatFlags` in `.atomic/settings.json`), then
- * appends any extra args the user passed after `-a <agent>`.
- */
-export async function buildAgentArgs(
-  agentType: AgentType,
-  passthroughArgs: string[] = [],
-  projectRoot: string = process.cwd(),
-): Promise<string[]> {
-  const config = AGENT_CONFIG[agentType];
-  const overrides = await getProviderOverrides(agentType, projectRoot);
-
-  const flags = overrides.chatFlags ?? [...config.chat_flags];
-
-  // Copilot has no on-disk MCP toggle — `--disable-mcp-server <name>` is
-  // the equivalent of flipping `enabled: false` in .opencode/opencode.json
-  // or adding to `disabledMcpjsonServers` in .claude/settings.json.
-  const scmFlags =
-    agentType === "copilot" ? await getCopilotScmDisableFlags(projectRoot) : [];
-
-  // Claude Code is the only one with a flag that takes an instructions file.
-  // OpenCode and Copilot CLI consume the file via config (.opencode/opencode.json
-  // `instructions` array) and env var (`COPILOT_CUSTOM_INSTRUCTIONS_DIRS`)
-  // respectively — see `applyManagedOnboardingFiles` and `chatCommand`'s env
-  // build. Skipped silently when no file resolves so the CLI still spawns
-  // even on a fresh checkout that hasn't run `autoSyncIfStale` yet.
-  const instructionsFlags: string[] = [];
-  if (agentType === "claude") {
-    const path = resolveAdditionalInstructionsPath(projectRoot);
-    if (path) instructionsFlags.push("--append-system-prompt-file", path);
-  }
-
-  return [...flags, ...scmFlags, ...instructionsFlags, ...passthroughArgs];
-}
-
-/**
- * Directory containing the resolved additional-instructions `AGENTS.md`,
- * or `undefined` if no file resolves. Used to set
- * `COPILOT_CUSTOM_INSTRUCTIONS_DIRS` on Copilot spawns — Copilot loads
- * `AGENTS.md` from each dir on that list.
- */
-export function getAdditionalInstructionsDir(
-  projectRoot: string,
-): string | undefined {
-  const path = resolveAdditionalInstructionsPath(projectRoot);
-  return path ? dirname(path) : undefined;
-}
-
-export function resolveChatCommand(
-  agentType: AgentType,
-  resolveCommandPath: CommandPathResolver = getCommandPath,
-): string | undefined {
-  if (agentType === "copilot") {
-    return resolveCopilotCliPath(resolveCommandPath);
-  }
-
-  const config = AGENT_CONFIG[agentType];
-  return resolveCommandPath(config.cmd) ?? undefined;
-}
-
-function generateChatId(): string {
-  return crypto.randomUUID().slice(0, 8);
-}
-
-/** Escape a string for safe interpolation inside a bash double-quoted string. */
-function escBash(s: string): string {
-  return s.replace(/[\\"$`!]/g, "\\$&");
-}
-
-/** Escape a string for safe interpolation inside a PowerShell double-quoted string. */
-function escPwsh(s: string): string {
-  return s.replace(/[`"$]/g, "`$&");
-}
-
-const POSIX_ENV_KEY_RE = /^[A-Za-z_][A-Za-z0-9_]*$/;
-
-function assertBashEnvKey(key: string): void {
-  if (!POSIX_ENV_KEY_RE.test(key)) {
-    throw new Error(
-      `Invalid Bash env key "${key}": must match /^[A-Za-z_][A-Za-z0-9_]*$/`,
-    );
-  }
-}
-
-function escPwshEnvKey(key: string): string {
-  return key.replace(/}/g, "`}");
-}
-
-async function removeLauncher(path: string): Promise<void> {
-  try {
-    await rm(path, { force: true });
-  } catch {
-    // Cleanup best effort; attach/fallback result should remain authoritative.
-  }
-}
-
-/**
- * Build a launcher script that preserves cwd and properly quotes args.
- * This avoids shell-injection risks from passthrough args.
- */
-export function buildLauncherScript(
-  cmd: string,
-  args: string[],
-  projectRoot: string,
-  envVars: Record<string, string> = {},
-): { script: string; ext: string } {
-  const isWin = process.platform === "win32";
-  const envEntries = Object.entries(envVars);
-
-  if (isWin) {
-    // PowerShell: use array splatting for safe arg passing
-    const argList = args.map((a) => `"${escPwsh(a)}"`).join(", ");
-    const envLines = envEntries.map(
-      ([key, value]) => `\${env:${escPwshEnvKey(key)}} = "${escPwsh(value)}"`,
-    );
-    const script = [
-      `Set-Location "${escPwsh(projectRoot)}"`,
-      ...envLines,
-      argList.length > 0
-        ? `& "${escPwsh(cmd)}" @(${argList})`
-        : `& "${escPwsh(cmd)}"`,
-      "$atomicExitCode = 0",
-      "if ($LASTEXITCODE -is [int]) { $atomicExitCode = $LASTEXITCODE }",
-      "exit $atomicExitCode",
-    ].join("\n");
-    return { script, ext: "ps1" };
-  }
-
-  const quotedCommand = [
-    `"${escBash(cmd)}"`,
-    ...args.map((arg) => `"${escBash(arg)}"`),
-  ].join(" ");
-  const envLines = envEntries.map(([key, value]) => {
-    assertBashEnvKey(key);
-    return `export ${key}="${escBash(value)}"`;
-  });
-  const script = [
-    "#!/bin/bash",
-    `cd "${escBash(projectRoot)}"`,
-    ...envLines,
-    quotedCommand,
-    "atomic_exit_code=$?",
-    'exit "$atomic_exit_code"',
-  ].join("\n");
-  return { script, ext: "sh" };
-}
-
-// ============================================================================
-// Chat Command Implementation
-// ============================================================================
-
-/**
- * Spawn the native agent CLI as an interactive subprocess.
- *
- * Always creates a new session in the atomic tmux socket and attaches
- * to it, regardless of whether the user is already inside tmux.
- * Falls back to direct spawn only when no TTY is available or tmux
- * cannot be installed.
- *
- * @param options - Chat command configuration options
- * @returns Exit code from the agent process
- */
-export async function chatCommand(options: ChatCommandOptions = {}): Promise<number> {
-  const { agentType, passthroughArgs } = options;
-
-  if (!agentType) {
-    throw new Error("agentType is required. Start chat with `atomic chat -a <agent>`.");
-  }
-
-  const config = AGENT_CONFIG[agentType];
-
-  const executable = resolveChatCommand(agentType);
-
-  // Check the agent CLI is installed
-  if (!executable) {
-    console.error(
-      `${COLORS.red}Error: '${config.cmd}' is not installed or not in PATH.${COLORS.reset}`
-    );
-    console.error(`Install it from: ${config.install_url}`);
-    return 1;
-  }
-
-  // ── Preflight: authentication ──
-  // Copilot and Claude expose SDK-level login checks; run them now so
-  // users get a short actionable error instead of being dropped into a
-  // native CLI that immediately redirects them to /login.
-  const auth = await checkAgentAuth(agentType);
-  if (!auth.loggedIn) {
-    printAuthError(agentType, auth);
-    return 1;
-  }
-
-  // ── Preflight: global config sync ──
-  const projectRoot = process.cwd();
-  const configRoot = getConfigRoot();
-
-  await ensureAtomicGlobalAgentConfigs(configRoot);
-
-  // ── Preflight: project setup (onboarding files, skills) ──
-  await ensureProjectSetup(agentType, projectRoot);
-
-  // ── Build argv ──
-  const args = await buildAgentArgs(agentType, passthroughArgs, projectRoot);
-  const cmd = [executable, ...args];
-  const overrides = await getProviderOverrides(agentType, projectRoot);
-  const claudeTempEnv = agentType === "claude" ? atomicTempEnv() : {};
-  // ATOMIC_AGENT must be baked into the launcher env so the agent CLI
-  // and anything it spawns can read it from process start.
-  const envVars: Record<string, string> = {
-    ...config.env_vars,
-    ...claudeTempEnv,
-    ...overrides.envVars,
-    ATOMIC_AGENT: agentType,
-  };
-
-  // Copilot CLI loads `AGENTS.md` from any directory listed in
-  // `COPILOT_CUSTOM_INSTRUCTIONS_DIRS` (comma-separated). Point it at the
-  // resolved AGENTS.md's parent dir so our additional instructions get
-  // appended to the persona without touching the project's `AGENTS.md`.
-  // Skip dirs containing a comma — Copilot CLI has no documented escape
-  // syntax for the list separator, so a comma in the path (rare on POSIX,
-  // possible in Windows usernames) would be misparsed as a list boundary.
-  if (agentType === "copilot") {
-    const dir = getAdditionalInstructionsDir(projectRoot);
-    if (dir && dir.includes(",")) {
-      console.error(
-        `${COLORS.yellow}Warning: skipping COPILOT_CUSTOM_INSTRUCTIONS_DIRS entry because the path contains a comma, which Copilot CLI cannot escape: ${dir}${COLORS.reset}`,
-      );
-    } else if (dir) {
-      const existing = envVars.COPILOT_CUSTOM_INSTRUCTIONS_DIRS;
-      envVars.COPILOT_CUSTOM_INSTRUCTIONS_DIRS = existing
-        ? `${existing},${dir}`
-        : dir;
-    }
-  }
-
-  const spawnEnv = buildSpawnEnv(envVars);
-  const launcherEnv = buildLauncherEnv(envVars);
-  const tmuxEnv = buildTmuxEnv(envVars);
-
-  // ── No TTY: tmux attach requires a real terminal ──
-  if (!process.stdin.isTTY) {
-    return spawnDirect(cmd, projectRoot, spawnEnv);
-  }
-
-  // ── Ensure tmux is available ──
-  if (!isTmuxInstalled()) {
-    console.log("Terminal multiplexer not found. Installing...");
-    try {
-      await ensureTmuxInstalled();
-      resetMuxBinaryCache();
-    } catch {
-      // Fall through to check below
-    }
-    if (!isTmuxInstalled()) {
-      // No tmux available — fall back to direct spawn
-      return spawnDirect(cmd, projectRoot, spawnEnv);
-    }
-  }
-
-  // ── Build launcher script for safe arg/cwd handling ──
-  const chatId = generateChatId();
-  const windowName = `atomic-chat-${agentType}-${chatId}`;
-
-  const sessionsDir = join(homedir(), ".atomic", "sessions", "chat");
-  await mkdir(sessionsDir, { recursive: true });
-  const { script, ext } = buildLauncherScript(
-    executable,
-    args,
-    projectRoot,
-    launcherEnv,
-  );
-  const launcherPath = join(sessionsDir, `${windowName}.${ext}`);
-  await writeFile(launcherPath, script, { mode: 0o755 });
-
-  const shellCmd = process.platform === "win32"
-    ? `pwsh -NoProfile -File "${launcherPath}"`
-    : `bash "${launcherPath}"`;
-
-  // ── Create session on the atomic socket and attach ──
-  try {
-    const paneId = createSession(windowName, shellCmd, undefined, projectRoot, tmuxEnv);
-    spawnAttachedFooter(windowName, paneId, agentType);
-    killSessionOnPaneExit(windowName, paneId);
-
-    if (isInsideAtomicSocket()) {
-      // Already on the atomic server — just switch to the new session.
-      switchClient(windowName);
-      await removeLauncher(launcherPath);
-      return 0;
-    }
-
-    if (isInsideTmux()) {
-      // Inside a different tmux server — detach and replace the client
-      // with an attach to the atomic socket (no nesting).
-      detachAndAttachAtomic(windowName);
-      await removeLauncher(launcherPath);
-      return 0;
-    }
-
-    const attachProc = spawnMuxAttach(windowName);
-    const exitCode = await attachProc.exited;
-
-    await removeLauncher(launcherPath);
-
-    // If tmux attach itself failed (e.g. lost TTY), clean up and fall back
-    if (exitCode !== 0) {
-      try { killSession(windowName); } catch {}
-      return spawnDirect(cmd, projectRoot, spawnEnv);
-    }
-
-    return exitCode;
-  } catch (error) {
-    await removeLauncher(launcherPath);
-    const message = error instanceof Error ? error.message : String(error);
-    console.error(
-      `${COLORS.yellow}Warning: Failed to create tmux session (${message}). Falling back to direct spawn.${COLORS.reset}`
-    );
-    return spawnDirect(cmd, projectRoot, spawnEnv);
-  }
-}
-
-/**
- * Spawn the agent CLI directly with inherited stdio.
- * Used when not inside tmux.
- */
-async function spawnDirect(
-  cmd: string[],
-  projectRoot: string,
-  env: Record<string, string> = {},
-): Promise<number> {
-  const proc = Bun.spawn(cmd, {
-    stdio: ["inherit", "inherit", "inherit"],
-    cwd: projectRoot,
-    env,
-  });
-
-  return await proc.exited;
-}

package/src/commands/cli/chat.ts

@@ -1,8 +0,0 @@
-#!/usr/bin/env bun
-/**
- * Compatibility barrel for the chat CLI command.
- *
- * The implementation lives under `commands/cli/chat/`.
- */
-
-export * from "./chat/index.ts";