Skip to content

Conversation

@ishaksebsib
Copy link
Contributor

@ishaksebsib ishaksebsib commented Oct 20, 2025

Greptile Overview

Updated On: 2025-10-20 11:46:01 UTC

Greptile Summary

This PR fixes a broken documentation link in the MCP (Model Context Protocol) feature documentation. The link originally pointed to a deprecated installation guide at documentation/getting-started/installation and now correctly points to the CLI v2 getting started guide at documentation/cli-v2/getting-started. This aligns with the repository's current documentation structure where CLI v2 documentation has replaced the older getting-started/installation path. The change also adds a trailing newline to maintain proper file formatting. This is a straightforward documentation maintenance fix with no functional impact on the codebase.

Changed Files
Filename Score Overview
features/mcp/helix-mcp.mdx 5/5 Fixed broken internal documentation link from deprecated installation guide to CLI v2 getting started guide and added trailing newline

Confidence Score: 5/5

This PR is safe to merge. It only updates a documentation link to point to the correct current path and adds proper file formatting. No code logic, API contracts, or functionality is affected. The change improves user experience by ensuring documentation links work correctly.

Sequence Diagram

sequenceDiagram
    participant User
    participant LLM as "LLM Provider<br/>(OpenAI/Gemini/Anthropic)"
    participant MCP as "MCP Server"
    participant Client as "Helix Client"
    participant DB as "HelixDB Instance"

    User->>MCP: "Initialize MCPServer(name, client)"
    MCP->>MCP: "Configure ToolConfig & Embedder"
    User->>MCP: "mcp.run()"
    MCP-->>User: "Server running on http://localhost:8000/mcp/"

    User->>LLM: "Initialize Provider (OpenAI/Gemini/Anthropic)"
    User->>LLM: "llm.enable_mcps('helix-mcp')"
    LLM->>MCP: "Connect to MCP endpoint"
    MCP-->>LLM: "Available tools (init, next, search_vector, etc.)"

    User->>LLM: "Send query/prompt"
    LLM->>MCP: "Call MCP tool (e.g., search_vector)"
    MCP->>Client: "Execute HQL query"
    Client->>DB: "Query HelixDB"
    DB-->>Client: "Return results"
    Client-->>MCP: "Query results"
    MCP-->>LLM: "Tool response"
    LLM-->>User: "Final response with data"
Loading

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@Gand4lf Gand4lf merged commit 97986e4 into HelixDB:main Nov 27, 2025
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.

3 participants