From 53314d9c3898b95a4768da2ffc7baabe2f72d44c Mon Sep 17 00:00:00 2001
From: 7Sageer <12210216@mail.sustech.edu.cn>
Date: Thu, 11 Dec 2025 05:28:06 +0800
Subject: [PATCH] Fix: Return complete MCP response object instead of content
 array in SearchRoutes (#223)

All search route handlers were returning only result.content (array) instead
of the complete result object containing {content, isError}. This caused the
MCP server to receive an invalid response format, resulting in all search
tools hanging indefinitely.

Changes:
- Updated 14 route handlers to return full result object
- Ensures MCP protocol compatibility by providing expected response structure

Fixes search functionality for all claude-mem search tools including:
- Unified search, timeline, decisions, changes
- Observations, sessions, prompts search
- Concept, file, type filtering
- Recent context and timeline queries
---
 .../worker/http/routes/SearchRoutes.ts        | 28 +++++++++----------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/services/worker/http/routes/SearchRoutes.ts b/src/services/worker/http/routes/SearchRoutes.ts
index bb30af8c..9d693788 100644
--- a/src/services/worker/http/routes/SearchRoutes.ts
+++ b/src/services/worker/http/routes/SearchRoutes.ts
@@ -49,7 +49,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleUnifiedSearch = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.search(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -58,7 +58,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleUnifiedTimeline = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.timeline(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -67,7 +67,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleDecisions = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.decisions(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -76,7 +76,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleChanges = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.changes(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -85,7 +85,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleHowItWorks = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.howItWorks(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -94,7 +94,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleSearchObservations = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.searchObservations(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -103,7 +103,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleSearchSessions = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.searchSessions(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -112,7 +112,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleSearchPrompts = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.searchUserPrompts(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -121,7 +121,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleSearchByConcept = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.findByConcept(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -130,7 +130,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleSearchByFile = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.findByFile(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -139,7 +139,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleSearchByType = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.findByType(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -148,7 +148,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleGetRecentContext = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.getRecentContext(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -157,7 +157,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleGetContextTimeline = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.getContextTimeline(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**
@@ -234,7 +234,7 @@ export class SearchRoutes extends BaseRouteHandler {
    */
   private handleGetTimelineByQuery = this.wrapHandler(async (req: Request, res: Response): Promise<void> => {
     const result = await this.searchManager.getTimelineByQuery(req.query);
-    res.json(result.content);
+    res.json(result);
   });
 
   /**