feat(cli): adopt @outfitter/cli with createCLI() entry point

[galligan]

Summary

• Replace raw new Command() with createCLI() from @outfitter/cli
• Global --json flag with OUTFITTER_JSON env var bridge (no per-command --json needed)
• Subcommands registered via cli.register() (chainable)
• Upgrade Commander from v13 to v14 (required by @outfitter/cli types)
• Update shouldOutputJson() to check OUTFITTER_JSON env var (FIRE-9)

Test plan

• bun run check passes
• bun test passes (298 tests)
• bun apps/cli/bin/fw.ts status --short smoke test

🤘🏻 In-collaboration-with: Claude Code

Greptile Summary

This PR successfully migrates the CLI from raw Commander.js to the @outfitter/cli wrapper with createCLI(). The changes introduce a global --json flag that sets OUTFITTER_JSON env var, eliminating the need for per-command --json options on the root command. The implementation correctly updates shouldOutputJson() to check this new env var while maintaining backwards compatibility with legacy FIREWATCH_JSONL and FIREWATCH_JSON variables.

Key changes:

• Adopted createCLI() entry point with chainable cli.register() for subcommands
• Upgraded Commander from v13 to v14 (required by @outfitter/cli types)
• Removed hidden --json option from root command (now provided globally by createCLI)
• Updated shouldOutputJson() with proper precedence: explicit flags → OUTFITTER_JSON → legacy env vars → TTY detection
• Changed entry point from program.parseAsync() to cli.parse()

The refactoring is clean, maintains API compatibility, and follows the described migration plan.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The refactoring is well-structured, maintains backwards compatibility, and aligns with the stated test plan (bun run check passes, bun test passes, smoke test completed). The changes are isolated to CLI initialization and flag handling, with proper env var bridging for the global --json flag.
• No files require special attention

Important Files Changed

Filename	Overview
apps/cli/package.json	Added @outfitter/cli and upgraded Commander v13 -> v14, plus supporting packages (@outfitter/config, @outfitter/schema, @outfitter/types, zod)
apps/cli/src/index.ts	Replaced new Command() with createCLI(), removed hidden --json from root (now global), chainable cli.register() for subcommands, updated to cli.parse()
apps/cli/src/utils/tty.ts	Updated shouldOutputJson() to check OUTFITTER_JSON env var (set by createCLI global --json), with proper precedence and backwards compatibility for legacy env vars

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User runs: fw --json query] --> B[createCLI preAction hook]
    B --> C[Sets OUTFITTER_JSON=1]
    C --> D[cli.parse executes command]
    D --> E[Command calls shouldOutputJson]
    E --> F{Check precedence}
    F -->|1. Explicit flag| G[--jsonl or --no-jsonl?]
    G -->|Yes| H[Return true/false]
    G -->|No| I{Check options.json === true?}
    I -->|Yes| J[Return true]
    I -->|No| K{Check OUTFITTER_JSON?}
    K -->|=1| L[Return true]
    K -->|=0| M[Return false]
    K -->|Not set| N{Check legacy env vars}
    N -->|FIREWATCH_JSONL/JSON| O[Return based on value]
    N -->|Not set| P{Check TTY}
    P -->|Is TTY| Q[Return false - human]
    P -->|Not TTY| R[Return true - JSONL]

Loading

_{Last reviewed commit: abcf1dd}

[linear]

FIRE-9 Adopt `@outfitter/cli` with `createCLI()` and flag presets

[galligan]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• docs: update architecture docs for Outfitter stack adoption #172
• feat(mcp): adopt @outfitter/mcp with createMcpServer() and defineTool() #171
• refactor(cli): wire exitWithError() across key commands #170
• feat(cli): adopt @outfitter/cli with createCLI() entry point #169 👈 (View in Graphite)
• feat(core): extract sync handler #168
• feat(core): extract PR mutation handlers (approve, reject, edit) #167
• feat(core): extract feedback handlers (reply, comment, ack, close) #166
• feat(core): extract read handlers (doctor, query) #165
• feat(core): define handler contract and extract statusHandler #164
• chore(core): upgrade @outfitter/contracts to ^0.4.1 #163
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.