Add setup for claude code#29
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdate README to describe both the MCP server and new privacy-focused web client, and clarify configuration instructions for using the server with Claude Code and other MCP clients, including default certificate paths and optional environment configuration. Flow diagram for configuring Keylime MCP with Claude Code and Claude Desktopflowchart TD
A[Install and build Keylime MCP server<br/>make build-server] --> B{Which MCP client?}
B --> C[Claude Code]
B --> D[Claude Desktop or other MCP client]
C --> C1[Run<br/>claude mcp add keylime --scope user /full/path/to/keylime-mcp/bin/server]
C1 --> E[Server uses default cert path<br/>/var/lib/keylime/cv_ca]
D --> D1[Edit<br/>~/.config/Claude/claude_desktop_config.json]
D1 --> D2[Add mcpServers.keylime entry<br/>with command and args]
D2 --> E
E --> F[Optionally configure env block<br/>for non-default cert paths]
F --> G[Ensure certs readable<br/>make setup-certs]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- In the Claude Code section, consider explicitly tying the
claude mcp addcommand to the build step (e.g., noting that/full/path/to/keylime-mcp/bin/serverrefers to the binary produced bymake build-server) to avoid ambiguity about which path to use. - You might want to clarify that the JSON config snippet applies only to config-file–driven MCP clients (like Claude Desktop) and is not needed when using the
claude mcp addCLI, to help users choose the correct setup path.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In the Claude Code section, consider explicitly tying the `claude mcp add` command to the build step (e.g., noting that `/full/path/to/keylime-mcp/bin/server` refers to the binary produced by `make build-server`) to avoid ambiguity about which path to use.
- You might want to clarify that the JSON config snippet applies only to config-file–driven MCP clients (like Claude Desktop) and is not needed when using the `claude mcp add` CLI, to help users choose the correct setup path.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Signed-off-by: Marek Safarik <msafarik@redhat.com>
b978328 to
bb1ed44
Compare
Summary by Sourcery
Document Claude Code setup and update client configuration guidance for the Keylime MCP server and web client.
Documentation: