• feat: add install command for one-click MCP client setup #177 👈 (View in Graphite)
• build: copier update from template 0.29.1 #176
• main

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

Greptile Summary

Adds a codereviewbuddy install <client> CLI command for one-click MCP client registration, supporting Claude Desktop, Claude Code, Cursor, Windsurf, Windsurf Next, and generic JSON output. The implementation reuses FastMCP's StdioMCPServer model and update_config_file() for config-file clients, shells out to claude mcp add for Claude Code, and opens deeplinks for Cursor.

• install.py: New 365-line module with well-factored shared helpers (_build_server_config, _write_config_file) and per-client subcommands. Supports --env and --env-file flags for env var injection. Optional pyperclip dependency for clipboard support.
• cli.py: Registers the install_app subcommand group on the main cyclopts app — minimal, clean integration.
• tests/test_install.py: Comprehensive test suite with 38 tests covering all clients, env handling, error paths, and edge cases with proper mocking.
• README.md: New "Quick setup" section with install examples added before the existing manual config instructions.
• Two minor style suggestions: the Cursor deeplink payload includes an empty env dict (unlike the mcp-json output which omits it), and _find_claude_command only checks stdout for the version string.

Confidence Score: 4/5

• This PR is safe to merge — well-tested new feature with no functional bugs, only minor style suggestions.
• Score of 4 reflects a clean, well-structured feature addition with comprehensive test coverage (38 tests). The two style comments are non-blocking improvements. The code correctly handles cross-platform paths, error conditions, and graceful degradation for optional dependencies (dotenv, pyperclip).
• src/codereviewbuddy/install.py deserves the most attention as the only new production code file (365 lines), though it is well-covered by tests.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["codereviewbuddy install <client>"] --> B{"Client type?"}
    B -->|claude-desktop| C["_get_claude_desktop_config_path()"]
    B -->|windsurf / windsurf-next| D["_get_windsurf_config_path()"]
    B -->|claude-code| E["_find_claude_command()"]
    B -->|cursor| F["Build deeplink URL"]
    B -->|mcp-json| G["Generate JSON"]

    C --> H["_build_server_config()"]
    D --> H
    H --> I["_write_config_file()"]
    I --> J["update_config_file() from FastMCP"]

    E --> K{"Claude CLI found?"}
    K -->|Yes| L["_build_server_config()"]
    L --> M["subprocess: claude mcp add"]
    K -->|No| N["Exit with error"]

    F --> O["_build_server_config()"]
    O --> P["base64 encode config"]
    P --> Q["_open_deeplink()"]

    G --> R["_build_server_config()"]
    R --> S{"--copy flag?"}
    S -->|Yes| T["pyperclip.copy()"]
    S -->|No| U["print JSON to stdout"]

_{Last reviewed commit: 72bb818}

Merge activity

• Feb 18, 7:17 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Feb 18, 7:18 AM UTC: Graphite rebased this pull request as part of a merge.
• Feb 18, 7:19 AM UTC: @ichoosetoaccept merged this pull request with Graphite.

Found 9 test failures on Blacksmith runners:

Failures