Skip to content

Implement to support with claude desktop - #4

Merged
hungpham10 merged 1 commit into
mainfrom
feature/fix-issue-cannot-working-with-claude-desktop
Aug 10, 2026
Merged

Implement to support with claude desktop#4
hungpham10 merged 1 commit into
mainfrom
feature/fix-issue-cannot-working-with-claude-desktop

Conversation

@hungpham10

@hungpham10 hungpham10 commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features

    • Added session-based MCP workspace management with initialization, indexing, reinitialization, and deinitialization tools.
    • Added MCP serving over standard input/output with clearer tool discovery and client-visible errors.
    • Simplified the CLI to focus on init, deinit, and serve --mcp.
    • Added optional indexing during workspace initialization and explicit full reindexing.
  • Documentation

    • Updated setup, lifecycle, available tools, architecture, and re-indexing guidance.
    • Documented the recommended MCP session workflow.

@hungpham10
hungpham10 merged commit 40c4ac9 into main Aug 10, 2026
5 of 9 checks passed
@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 99cb9a90-7c0a-4c16-851c-27e6468f983b

📥 Commits

Reviewing files that changed from the base of the PR and between a029afd and 5095bf9.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (11)
  • README.md
  • crates/codegraph-mcp/Cargo.toml
  • crates/codegraph-mcp/src/http.rs
  • crates/codegraph-mcp/src/lib.rs
  • crates/codegraph-mcp/src/protocol.rs
  • crates/codegraph-mcp/src/server-instructions.md
  • crates/codegraph-mcp/src/session.rs
  • crates/codegraph-mcp/src/stdio.rs
  • crates/codegraph-mcp/src/tools.rs
  • crates/codegraph/Cargo.toml
  • crates/codegraph/src/main.rs

Walkthrough

The MCP crate now uses rmcp transports and session-bound workspace operations. The CLI supports asynchronous init, deinit, and serve --mcp commands. Documentation describes the new lifecycle and expanded tool set.

Changes

MCP session and CLI lifecycle

Layer / File(s) Summary
rmcp transports and tool contracts
crates/codegraph-mcp/Cargo.toml, crates/codegraph-mcp/src/http.rs, crates/codegraph-mcp/src/stdio.rs, crates/codegraph-mcp/src/tools.rs
The crate adds rmcp-based tool definitions and stdio serving. It adds a feature-gated HTTP transport stub.
Workspace session lifecycle
crates/codegraph-mcp/src/session.rs, crates/codegraph-mcp/src/tools.rs, crates/codegraph-mcp/src/server-instructions.md
Sessions now bind, initialize, deinitialize, validate, reload, and reindex workspaces. Initialization can return extraction statistics.
Session-aware server dispatch
crates/codegraph-mcp/src/lib.rs
CodegraphServer replaces the fixed-root server. Tool calls use session state and rmcp responses.
Reduced CLI and usage documentation
crates/codegraph/src/main.rs, crates/codegraph/Cargo.toml, README.md
The CLI removes legacy commands and supports asynchronous lifecycle commands and MCP serving. README instructions reflect the new workflow and 30 tools.

Estimated code review effort: 4 (Complex) | ~45 minutes

Sequence Diagram(s)

sequenceDiagram
  participant MCPClient
  participant serve_stdio
  participant CodegraphServer
  participant Session
  MCPClient->>serve_stdio: Send MCP tool request
  serve_stdio->>CodegraphServer: Forward rmcp call
  CodegraphServer->>Session: Apply session operation
  Session-->>CodegraphServer: Return state or tool data
  CodegraphServer-->>serve_stdio: Return rmcp result
  serve_stdio-->>MCPClient: Send MCP response
Loading

Possibly related PRs

Suggested reviewers: cleboost, hungpham7-tiki

Poem

I bound a workspace, said the rabbit with glee,
Then indexed its symbols beneath the MCP tree.
rmcp carried each call,
Sessions guarded them all,
And the CLI grew small as can be.

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/fix-issue-cannot-working-with-claude-desktop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codspeed-hq

codspeed-hq Bot commented Aug 10, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 28 untouched benchmarks


Comparing feature/fix-issue-cannot-working-with-claude-desktop (5095bf9) with main (a029afd)

Open in CodSpeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant