• feat: add list_recent_unresolved tool to surface comments on merged PRs #183
• fix: check stderr for Claude Code detection and exclude empty env from deeplink #181 👈 (View in Graphite)
• main

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

Greptile Summary

This PR makes two targeted fixes to the MCP client installation system:

1. Claude Code CLI detection now checks stderr in addition to stdout when running claude --version, fixing detection for versions that output version info to stderr.
2. Cursor deeplink excludes empty env by adding exclude_defaults=True to model_dump_json(), producing shorter/cleaner URLs when no environment variables are configured.

• Both changes are well-tested with new test cases (test_found_via_stderr, test_deeplink_excludes_empty_env).
• The existing test_not_claude_code test was updated to explicitly set stderr="" on the mock, aligning with the new code path.
• Changes are minimal and focused. No issues found in the new code.

Confidence Score: 4/5

• This PR is safe to merge — the changes are minimal, well-tested, and address clear behavioral gaps.
• Score of 4 reflects small, focused fixes with good test coverage. No logic issues found in the new code. The only minor concern is that the existing test_found_in_path test doesn't explicitly set stderr on the mock (relying on MagicMock auto-attributes), which is a pre-existing test hygiene concern but not a blocker.
• No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["_find_claude_command()"] --> B{"shutil.which('claude')"}
    B -->|found| C["subprocess.run\n[claude, --version]"]
    C --> D{"'Claude Code' in stdout\nOR stderr?"}
    D -->|yes| E["Return path"]
    D -->|no| F["Check potential_paths"]
    B -->|not found| F
    F --> G{"path.exists()?"}
    G -->|yes| H["subprocess.run\n[path, --version]"]
    H --> I{"'Claude Code' in stdout\nOR stderr?"}
    I -->|yes| J["Return path"]
    I -->|no| G
    G -->|no more paths| K["Return None"]

    L["cmd_cursor()"] --> M["_build_server_config()"]
    M --> N["model_dump_json\n(exclude_none, exclude_defaults)"]
    N --> O{"env == {}?"}
    O -->|yes| P["Exclude env from JSON"]
    O -->|no| Q["Include env in JSON"]
    P --> R["Base64 encode → deeplink URL"]
    Q --> R

_{Last reviewed commit: 56fa49f}

Merge activity

• Feb 18, 10:46 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Feb 18, 10:46 AM UTC: @ichoosetoaccept merged this pull request with Graphite.