fix(mcp): resolve project_id by name for MCP clients

wcatz · wcatz · commit 341170bc5ffd · 2026-03-15T21:14:28.000-04:00
MCP clients pass project name ("ghost") but DB stores project_id as
sha256(abs_path)[:12]. Every MCP query returned empty results.

Add ResolveProjectByName to Store and resolveProjectID helper to MCP
server. All tool handlers now resolve name→hash before querying.
diff --git a/internal/mcpserver/mcpserver.go b/internal/mcpserver/mcpserver.go
@@ -47,6 +47,30 @@ func (s *Server) Run(ctx context.Context) error {
 	return s.mcp.Run(ctx, &mcp.StdioTransport{})
 }
 
+// resolveProjectID resolves a project_id that may be a name (e.g. "ghost")
+// into the actual hash ID (e.g. "6bdc098af7f5") stored in the database.
+// Tries direct ID match first, then falls back to name lookup.
+func (s *Server) resolveProjectID(ctx context.Context, input string) string {
+	// Check if input directly matches a project ID.
+	projects, err := s.store.ListProjects(ctx)
+	if err == nil {
+		for _, p := range projects {
+			if p.ID == input {
+				return input
+			}
+		}
+	}
+
+	// Try name lookup.
+	resolved, err := s.store.ResolveProjectByName(ctx, input)
+	if err == nil && resolved != "" {
+		return resolved
+	}
+
+	// Return as-is if nothing matched — let downstream handle the miss.
+	return input
+}
+
 func (s *Server) registerTools() {
 	// ghost_memory_search — search memories by keyword or semantic query.
 	type searchArgs struct {
@@ -66,6 +90,8 @@ func (s *Server) registerTools() {
 			args.Limit = 10
 		}
 
+		args.ProjectID = s.resolveProjectID(ctx, args.ProjectID)
+
 		memories, err := s.store.SearchFTS(ctx, args.ProjectID, args.Query, args.Limit)
 		if err != nil {
 			return nil, nil, fmt.Errorf("search failed: %w", err)
@@ -118,6 +144,8 @@ func (s *Server) registerTools() {
 			args.Tags = []string{}
 		}
 
+		args.ProjectID = s.resolveProjectID(ctx, args.ProjectID)
+
 		if err := s.store.EnsureProject(ctx, args.ProjectID, "", args.ProjectID); err != nil {
 			return nil, nil, fmt.Errorf("ensure project: %w", err)
 		}
@@ -155,6 +183,8 @@ func (s *Server) registerTools() {
 			args.Limit = 20
 		}
 
+		args.ProjectID = s.resolveProjectID(ctx, args.ProjectID)
+
 		var sb strings.Builder
 
 		memories, err := s.store.GetTopMemories(ctx, args.ProjectID, args.Limit)
@@ -203,6 +233,8 @@ func (s *Server) registerTools() {
 			args.Limit = 30
 		}
 
+		args.ProjectID = s.resolveProjectID(ctx, args.ProjectID)
+
 		var memories []memory.Memory
 		var err error
 
diff --git a/internal/memory/store.go b/internal/memory/store.go
@@ -97,6 +97,23 @@ func (s *Store) EnsureProject(ctx context.Context, id, path, name string) error
 	return err
 }
 
+// ResolveProjectByName looks up a project by name and returns its hash ID.
+// Returns empty string if no project with that name exists.
+func (s *Store) ResolveProjectByName(ctx context.Context, name string) (string, error) {
+	s.mu.RLock()
+	defer s.mu.RUnlock()
+
+	var id string
+	err := s.db.QueryRowContext(ctx, `SELECT id FROM projects WHERE name = ? LIMIT 1`, name).Scan(&id)
+	if err == sql.ErrNoRows {
+		return "", nil
+	}
+	if err != nil {
+		return "", fmt.Errorf("resolve project by name: %w", err)
+	}
+	return id, nil
+}
+
 // Create inserts a new memory and returns its ID.
 func (s *Store) Create(ctx context.Context, projectID string, m Memory) (string, error) {
 	s.mu.Lock()
diff --git a/internal/provider/provider.go b/internal/provider/provider.go
@@ -66,6 +66,7 @@ type MemoryStore interface {
 	// Project management
 	ListProjects(ctx context.Context) ([]memory.Project, error)
 	EnsureProject(ctx context.Context, id, path, name string) error
+	ResolveProjectByName(ctx context.Context, name string) (string, error)
 
 	// Conversation persistence
 	CreateConversation(ctx context.Context, projectID, mode string) (string, error)
diff --git a/internal/server/server_test.go b/internal/server/server_test.go
@@ -80,6 +80,9 @@ func (m *mockStore) ReplaceNonManual(_ context.Context, _ string, _ []memory.Mem
 }
 func (m *mockStore) ListProjects(_ context.Context) ([]memory.Project, error) { return nil, nil }
 func (m *mockStore) EnsureProject(_ context.Context, _, _, _ string) error    { return nil }
+func (m *mockStore) ResolveProjectByName(_ context.Context, _ string) (string, error) {
+	return "", nil
+}
 func (m *mockStore) CreateConversation(_ context.Context, _, _ string) (string, error) {
 	return "", nil
 }

Original file line number	Diff line number	Diff line change
`@@ -80,6 +80,9 @@ func (m *mockStore) ReplaceNonManual(_ context.Context, _ string, _ []memory.Mem`
`80`	`80`	`}`
`81`	`81`	`func (m *mockStore) ListProjects(_ context.Context) ([]memory.Project, error) { return nil, nil }`
`82`	`82`	`func (m *mockStore) EnsureProject(_ context.Context, _, _, _ string) error { return nil }`
	`83`	`+func (m *mockStore) ResolveProjectByName(_ context.Context, _ string) (string, error) {`
	`84`	`+ return "", nil`
	`85`	`+}`
`83`	`86`	`func (m *mockStore) CreateConversation(_ context.Context, _, _ string) (string, error) {`
`84`	`87`	`return "", nil`
`85`	`88`	`}`