diff options
| author | Paul Buetow <paul@buetow.org> | 2026-02-10 19:28:27 +0200 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-02-10 19:28:27 +0200 |
| commit | 5551695f3b0d10c9a22cfacdb10c2cf7bd572421 (patch) | |
| tree | 282611eacf1fd4c38d54d5cea87decdf2b1cbdb7 /internal/mcp | |
| parent | ec745129258ae800065e302a2a40b54488cbca08 (diff) | |
Add MCP server implementation with comprehensive test coverage
Implements a full Model Context Protocol (MCP) server for managing and serving prompts
to LLM applications. The server provides CRUD operations for prompts with automatic
backups and template rendering support.
Key additions:
- cmd/hexai-mcp-server: Main MCP server binary entrypoint
- internal/hexaimcp: Server orchestrator with configuration and setup
- internal/mcp: Core MCP protocol implementation (JSON-RPC 2.0)
- internal/promptstore: Prompt storage with JSONL backend and automatic backups
- Comprehensive test suites achieving 80%+ coverage for all MCP packages
- Magefile targets for building and installing the MCP server
- Complete documentation for setup, API, prompts, and backups
Test coverage:
- internal/hexaimcp: 84.3%
- internal/mcp: 80.3%
- internal/promptstore: 81.2%
- Overall project: 81.5%
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Diffstat (limited to 'internal/mcp')
| -rw-r--r-- | internal/mcp/handlers_test.go | 955 | ||||
| -rw-r--r-- | internal/mcp/server.go | 494 | ||||
| -rw-r--r-- | internal/mcp/server_test.go | 505 | ||||
| -rw-r--r-- | internal/mcp/transport.go | 69 | ||||
| -rw-r--r-- | internal/mcp/types.go | 187 |
5 files changed, 2210 insertions, 0 deletions
diff --git a/internal/mcp/handlers_test.go b/internal/mcp/handlers_test.go new file mode 100644 index 0000000..79c567e --- /dev/null +++ b/internal/mcp/handlers_test.go @@ -0,0 +1,955 @@ +// Summary: Tests for MCP prompt management handlers +package mcp + +import ( + "bytes" + "encoding/json" + "fmt" + "io" + "log" + "testing" + "time" + + "codeberg.org/snonux/hexai/internal/promptstore" +) + +func TestServer_PromptsCreate(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/create request + params := CreatePromptRequest{ + Name: "test_create", + Title: "Test Create Prompt", + Description: "A test prompt", + Arguments: []PromptArgument{ + {Name: "input", Description: "Test input", Required: true}, + }, + Messages: []PromptMessage{ + { + Role: "user", + Content: MessageContent{ + Type: "text", + Text: "Test: {{input}}", + }, + }, + }, + Tags: []string{"test"}, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 10, + Method: "prompts/create", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } + + // Parse result + resultBytes, _ := json.Marshal(resp.Result) + var result PromptOperationResult + if err := json.Unmarshal(resultBytes, &result); err != nil { + t.Fatalf("Unmarshal result error = %v", err) + } + + if !result.Success { + t.Errorf("Success = false, want true") + } +} + +func TestServer_PromptsUpdate(t *testing.T) { + now := time.Now() + store := &mockPromptStore{ + prompts: map[string]*promptstore.Prompt{ + "test_update": { + Name: "test_update", + Title: "Original Title", + Created: now, + Updated: now, + Messages: []promptstore.PromptMessage{ + { + Role: "user", + Content: promptstore.MessageContent{ + Type: "text", + Text: "Original text", + }, + }, + }, + }, + }, + } + + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/update request + params := UpdatePromptRequest{ + Name: "test_update", + Title: "Updated Title", + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 11, + Method: "prompts/update", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } + + // Parse result + resultBytes, _ := json.Marshal(resp.Result) + var result PromptOperationResult + if err := json.Unmarshal(resultBytes, &result); err != nil { + t.Fatalf("Unmarshal result error = %v", err) + } + + if !result.Success { + t.Errorf("Success = false, want true") + } +} + +func TestServer_PromptsDelete(t *testing.T) { + now := time.Now() + store := &mockPromptStore{ + prompts: map[string]*promptstore.Prompt{ + "test_delete": { + Name: "test_delete", + Title: "To Be Deleted", + Created: now, + Updated: now, + Messages: []promptstore.PromptMessage{}, + }, + }, + } + + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/delete request + params := DeletePromptRequest{ + Name: "test_delete", + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 12, + Method: "prompts/delete", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } + + // Parse result + resultBytes, _ := json.Marshal(resp.Result) + var result PromptOperationResult + if err := json.Unmarshal(resultBytes, &result); err != nil { + t.Fatalf("Unmarshal result error = %v", err) + } + + if !result.Success { + t.Errorf("Success = false, want true") + } +} + +func TestServer_PromptsCreate_MissingName(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/create request without name + params := CreatePromptRequest{ + Title: "No Name", + Messages: []PromptMessage{ + {Role: "user", Content: MessageContent{Type: "text", Text: "Test"}}, + }, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 13, + Method: "prompts/create", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for missing name") + } + + if resp.Error.Code != ErrCodeInvalidParams { + t.Errorf("Error code = %d, want %d", resp.Error.Code, ErrCodeInvalidParams) + } +} + +// Update mockPromptStore to support Create, Update, Delete +func (m *mockPromptStore) Create(prompt *promptstore.Prompt) error { + if _, exists := m.prompts[prompt.Name]; exists { + return fmt.Errorf("prompt already exists: %s", prompt.Name) + } + m.prompts[prompt.Name] = prompt + return nil +} + +func (m *mockPromptStore) Update(prompt *promptstore.Prompt) error { + if _, exists := m.prompts[prompt.Name]; !exists { + return fmt.Errorf("prompt not found: %s", prompt.Name) + } + m.prompts[prompt.Name] = prompt + return nil +} + +func (m *mockPromptStore) Delete(name string) error { + if _, exists := m.prompts[name]; !exists { + return fmt.Errorf("prompt not found: %s", name) + } + delete(m.prompts, name) + return nil +} + +func TestServer_PromptsUpdate_NotFound(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/update request for non-existent prompt + params := UpdatePromptRequest{ + Name: "nonexistent", + Title: "Updated Title", + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 20, + Method: "prompts/update", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for non-existent prompt") + } +} + +func TestServer_PromptsUpdate_MissingName(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/update request without name + params := UpdatePromptRequest{ + Title: "Updated Title", + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 21, + Method: "prompts/update", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for missing name") + } +} + +func TestServer_PromptsDelete_NotFound(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/delete request for non-existent prompt + params := DeletePromptRequest{ + Name: "nonexistent", + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 22, + Method: "prompts/delete", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for non-existent prompt") + } +} + +func TestServer_PromptsDelete_MissingName(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/delete request without name + params := DeletePromptRequest{} + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 23, + Method: "prompts/delete", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for missing name") + } +} + +func TestServer_PromptsCreate_AlreadyExists(t *testing.T) { + now := time.Now() + store := &mockPromptStore{ + prompts: map[string]*promptstore.Prompt{ + "existing": { + Name: "existing", + Title: "Existing Prompt", + Created: now, + Updated: now, + Messages: []promptstore.PromptMessage{}, + }, + }, + } + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/create request with existing name + params := CreatePromptRequest{ + Name: "existing", + Title: "Duplicate", + Messages: []PromptMessage{ + {Role: "user", Content: MessageContent{Type: "text", Text: "Test"}}, + }, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 24, + Method: "prompts/create", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for duplicate prompt") + } +} + +func TestServer_PromptsGet_NotFound(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/get request for non-existent prompt + params := GetPromptRequest{ + Name: "nonexistent", + Arguments: map[string]string{}, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 25, + Method: "prompts/get", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for non-existent prompt") + } +} + +func TestServer_PromptsList_WithError(t *testing.T) { + store := &mockPromptStore{ + listFn: func(cursor string, limit int) ([]promptstore.Prompt, string, error) { + return nil, "", fmt.Errorf("store error") + }, + } + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/list request + req := Request{ + JSONRPC: "2.0", + ID: 26, + Method: "prompts/list", + Params: json.RawMessage(`{}`), + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error from store") + } +} + +func TestServer_PromptsUpdate_WithMessages(t *testing.T) { + now := time.Now() + store := &mockPromptStore{ + prompts: map[string]*promptstore.Prompt{ + "test": { + Name: "test", + Title: "Original", + Created: now, + Updated: now, + Messages: []promptstore.PromptMessage{ + { + Role: "user", + Content: promptstore.MessageContent{ + Type: "text", + Text: "Original", + }, + }, + }, + }, + }, + } + + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/update request with new messages + params := UpdatePromptRequest{ + Name: "test", + Title: "Updated", + Messages: []PromptMessage{ + { + Role: "user", + Content: MessageContent{ + Type: "text", + Text: "Updated message", + }, + }, + }, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 27, + Method: "prompts/update", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } +} + +func TestServer_PromptsCreate_WithAllFields(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/create request with all fields + params := CreatePromptRequest{ + Name: "complete", + Title: "Complete Prompt", + Description: "Full description", + Arguments: []PromptArgument{ + {Name: "arg1", Description: "First arg", Required: true}, + {Name: "arg2", Description: "Second arg", Required: false}, + }, + Messages: []PromptMessage{ + { + Role: "user", + Content: MessageContent{ + Type: "text", + Text: "Test {{arg1}} and {{arg2}}", + }, + }, + { + Role: "assistant", + Content: MessageContent{ + Type: "text", + Text: "Response", + }, + }, + }, + Tags: []string{"tag1", "tag2"}, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 28, + Method: "prompts/create", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } + + // Parse result + resultBytes, _ := json.Marshal(resp.Result) + var result PromptOperationResult + if err := json.Unmarshal(resultBytes, &result); err != nil { + t.Fatalf("Unmarshal result error = %v", err) + } + + if !result.Success { + t.Errorf("Success = false, want true") + } +} + +func TestServer_PromptsList_WithCursorAndLimit(t *testing.T) { + now := time.Now() + store := &mockPromptStore{ + prompts: map[string]*promptstore.Prompt{ + "test1": { + Name: "test1", + Title: "Test 1", + Created: now, + Updated: now, + }, + "test2": { + Name: "test2", + Title: "Test 2", + Created: now, + Updated: now, + }, + }, + } + + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/list request with cursor and limit + params := map[string]interface{}{ + "cursor": "test1", + "limit": 10, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 30, + Method: "prompts/list", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } +} + +func TestServer_PromptsUpdate_WithDescription(t *testing.T) { + now := time.Now() + store := &mockPromptStore{ + prompts: map[string]*promptstore.Prompt{ + "test": { + Name: "test", + Title: "Original", + Created: now, + Updated: now, + Messages: []promptstore.PromptMessage{ + { + Role: "user", + Content: promptstore.MessageContent{ + Type: "text", + Text: "Original", + }, + }, + }, + }, + }, + } + + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/update request with description + params := UpdatePromptRequest{ + Name: "test", + Description: "Updated description", + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 31, + Method: "prompts/update", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } +} + +func TestServer_PromptsUpdate_WithArguments(t *testing.T) { + now := time.Now() + store := &mockPromptStore{ + prompts: map[string]*promptstore.Prompt{ + "test": { + Name: "test", + Title: "Original", + Created: now, + Updated: now, + Messages: []promptstore.PromptMessage{ + { + Role: "user", + Content: promptstore.MessageContent{ + Type: "text", + Text: "Original", + }, + }, + }, + }, + }, + } + + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/update request with arguments + params := UpdatePromptRequest{ + Name: "test", + Arguments: []PromptArgument{ + {Name: "newarg", Description: "New argument", Required: true}, + }, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 32, + Method: "prompts/update", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } +} + +func TestServer_PromptsUpdate_WithTags(t *testing.T) { + now := time.Now() + store := &mockPromptStore{ + prompts: map[string]*promptstore.Prompt{ + "test": { + Name: "test", + Title: "Original", + Created: now, + Updated: now, + Messages: []promptstore.PromptMessage{ + { + Role: "user", + Content: promptstore.MessageContent{ + Type: "text", + Text: "Original", + }, + }, + }, + }, + }, + } + + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/update request with tags + params := UpdatePromptRequest{ + Name: "test", + Tags: []string{"newtag1", "newtag2"}, + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 33, + Method: "prompts/update", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error != nil { + t.Fatalf("Error = %v, want nil", resp.Error) + } +} + +func TestServer_Run_InvalidJSON(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + inBuf := &bytes.Buffer{} + outBuf := &bytes.Buffer{} + logger := log.New(io.Discard, "", 0) + server := NewServer(inBuf, outBuf, logger, store) + + // Write invalid JSON + msg := []byte(`{invalid json}`) + header := fmt.Sprintf("Content-Length: %d\r\n\r\n", len(msg)) + inBuf.WriteString(header) + inBuf.Write(msg) + + // Run in background + done := make(chan error, 1) + go func() { + done <- server.Run() + }() + + // Give time for processing + time.Sleep(50 * time.Millisecond) + + // Should have written error response + if outBuf.Len() == 0 { + t.Error("Expected error response to be written") + } +} + +func TestServer_PromptsCreate_MissingMessages(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Initialize server + server.mu.Lock() + server.initialized = true + server.mu.Unlock() + + // Send prompts/create request without messages + params := CreatePromptRequest{ + Name: "test", + Title: "Test", + } + paramsBytes, _ := json.Marshal(params) + req := Request{ + JSONRPC: "2.0", + ID: 34, + Method: "prompts/create", + Params: paramsBytes, + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for missing messages") + } +} + +func TestServer_HandleInitialize_InvalidParams(t *testing.T) { + store := &mockPromptStore{prompts: make(map[string]*promptstore.Prompt)} + server, _, outBuf := createTestServer(t, store) + + // Send initialize request with invalid params + req := Request{ + JSONRPC: "2.0", + ID: 35, + Method: "initialize", + Params: json.RawMessage(`{invalid}`), + } + + server.handle(req) + + // Read response + resp, err := readResponse(outBuf) + if err != nil { + t.Fatalf("readResponse() error = %v", err) + } + + if resp.Error == nil { + t.Fatal("Expected error for invalid params") + } +} diff --git a/internal/mcp/server.go b/internal/mcp/server.go new file mode 100644 index 0000000..f6479d9 --- /dev/null +++ b/internal/mcp/server.go @@ -0,0 +1,494 @@ +// Summary: MCP server over stdio; manages prompt store, dispatches requests, and handles protocol. +package mcp + +import ( + "bufio" + "encoding/json" + "fmt" + "io" + "log" + "strings" + "sync" + "time" + + "codeberg.org/snonux/hexai/internal" + "codeberg.org/snonux/hexai/internal/promptstore" +) + +// Server implements an MCP server over stdio using JSON-RPC 2.0. +// Follows the same pattern as the LSP server with dispatch table and thread safety. +type Server struct { + in *bufio.Reader + out io.Writer + outMu sync.Mutex + logger *log.Logger + store promptstore.PromptStore + initialized bool + mu sync.RWMutex + + // Dispatch table for JSON-RPC methods + handlers map[string]func(Request) +} + +// NewServer creates a new MCP server with the given store and I/O streams. +// The store provides access to prompts; logger is used for debugging. +func NewServer(r io.Reader, w io.Writer, logger *log.Logger, store promptstore.PromptStore) *Server { + s := &Server{ + in: bufio.NewReader(r), + out: w, + logger: logger, + store: store, + } + + // Initialize dispatch table + s.handlers = map[string]func(Request){ + "initialize": s.handleInitialize, + "initialized": s.handleInitialized, + "prompts/list": s.handlePromptsList, + "prompts/get": s.handlePromptsGet, + "prompts/create": s.handlePromptsCreate, + "prompts/update": s.handlePromptsUpdate, + "prompts/delete": s.handlePromptsDelete, + "notifications/initialized": s.handleInitialized, + } + + return s +} + +// Run starts the server main loop, reading and dispatching requests. +// Returns on EOF or fatal error. +func (s *Server) Run() error { + for { + body, err := s.readMessage() + if err == io.EOF { + return nil + } + if err != nil { + return fmt.Errorf("read message: %w", err) + } + + var req Request + if err := json.Unmarshal(body, &req); err != nil { + s.logger.Printf("invalid JSON: %v", err) + s.sendError(nil, ErrCodeParseError, "Parse error") + continue + } + + if req.Method == "" { + // Response from client; ignore + continue + } + + // Dispatch request + go s.handle(req) + } +} + +// handle dispatches a request to the appropriate handler. +func (s *Server) handle(req Request) { + handler, ok := s.handlers[req.Method] + if !ok { + s.logger.Printf("method not found: %s", req.Method) + s.sendError(req.ID, ErrCodeMethodNotFound, fmt.Sprintf("Method not found: %s", req.Method)) + return + } + + handler(req) +} + +// handleInitialize processes the initialize request and returns server capabilities. +func (s *Server) handleInitialize(req Request) { + var params InitializeRequest + if err := json.Unmarshal(req.Params, ¶ms); err != nil { + s.sendError(req.ID, ErrCodeInvalidParams, "Invalid initialize params") + return + } + + s.logger.Printf("initialize from client: %s %s (protocol: %s)", + params.ClientInfo.Name, params.ClientInfo.Version, params.ProtocolVersion) + + // Validate protocol version (accept both old and new versions for compatibility) + if params.ProtocolVersion != "2024-11-05" && params.ProtocolVersion != "2025-06-18" { + s.logger.Printf("warning: unsupported protocol version: %s", params.ProtocolVersion) + } + + result := InitializeResult{ + ProtocolVersion: "2025-06-18", + Capabilities: ServerCapabilities{ + Prompts: &PromptsCapability{ + ListChanged: false, + Mutable: true, // Advertise that we support create/update/delete + }, + }, + ServerInfo: ServerInfo{ + Name: "hexai-mcp-server", + Version: internal.Version, + }, + } + + s.mu.Lock() + s.initialized = true + s.mu.Unlock() + + s.sendResponse(req.ID, result) +} + +// handleInitialized processes the initialized notification. +// This is sent by the client after receiving initialize response. +func (s *Server) handleInitialized(_ Request) { + s.logger.Printf("client sent initialized notification") + // No response required for notifications +} + +// handlePromptsList processes the prompts/list request. +func (s *Server) handlePromptsList(req Request) { + s.mu.RLock() + if !s.initialized { + s.mu.RUnlock() + s.sendError(req.ID, ErrCodeInvalidRequest, "Server not initialized") + return + } + s.mu.RUnlock() + + var params ListPromptsRequest + if req.Params != nil && len(req.Params) > 0 { + if err := json.Unmarshal(req.Params, ¶ms); err != nil { + s.sendError(req.ID, ErrCodeInvalidParams, "Invalid prompts/list params") + return + } + } + + // List prompts from store + prompts, nextCursor, err := s.store.List(params.Cursor, 100) + if err != nil { + s.logger.Printf("store list error: %v", err) + s.sendError(req.ID, ErrCodeInternalError, "Failed to list prompts") + return + } + + // Convert to PromptInfo + var infos []PromptInfo + for _, p := range prompts { + args := make([]PromptArgument, len(p.Arguments)) + for i, a := range p.Arguments { + args[i] = PromptArgument{ + Name: a.Name, + Description: a.Description, + Required: a.Required, + } + } + infos = append(infos, PromptInfo{ + Name: p.Name, + Title: p.Title, + Description: p.Description, + Arguments: args, + }) + } + + result := ListPromptsResult{ + Prompts: infos, + NextCursor: nextCursor, + } + + s.sendResponse(req.ID, result) +} + +// handlePromptsGet processes the prompts/get request. +func (s *Server) handlePromptsGet(req Request) { + s.mu.RLock() + if !s.initialized { + s.mu.RUnlock() + s.sendError(req.ID, ErrCodeInvalidRequest, "Server not initialized") + return + } + s.mu.RUnlock() + + var params GetPromptRequest + if err := json.Unmarshal(req.Params, ¶ms); err != nil { + s.sendError(req.ID, ErrCodeInvalidParams, "Invalid prompts/get params") + return + } + + if params.Name == "" { + s.sendError(req.ID, ErrCodeInvalidParams, "Missing prompt name") + return + } + + // Get prompt from store + prompt, err := s.store.Get(params.Name) + if err != nil { + s.logger.Printf("store get error: %v", err) + s.sendError(req.ID, ErrCodeInvalidParams, fmt.Sprintf("Prompt not found: %s", params.Name)) + return + } + + // Render prompt with arguments + messages, err := s.renderPrompt(prompt, params.Arguments) + if err != nil { + s.logger.Printf("render error: %v", err) + s.sendError(req.ID, ErrCodeInvalidParams, err.Error()) + return + } + + result := GetPromptResult{ + Description: prompt.Description, + Messages: messages, + } + + s.sendResponse(req.ID, result) +} + +// renderPrompt substitutes template arguments in prompt messages. +// Returns error if required arguments are missing. +func (s *Server) renderPrompt(prompt *promptstore.Prompt, args map[string]string) ([]PromptMessage, error) { + // Validate required arguments + for _, arg := range prompt.Arguments { + if arg.Required { + if _, ok := args[arg.Name]; !ok { + return nil, fmt.Errorf("missing required argument: %s", arg.Name) + } + } + } + + // Render each message + var rendered []PromptMessage + for _, msg := range prompt.Messages { + text := msg.Content.Text + + // Simple template substitution: {{arg}} -> value + for key, val := range args { + placeholder := "{{" + key + "}}" + text = strings.ReplaceAll(text, placeholder, val) + } + + rendered = append(rendered, PromptMessage{ + Role: msg.Role, + Content: MessageContent{ + Type: msg.Content.Type, + Text: text, + }, + }) + } |
