From 883299d070f788712f746db709a8ea0cbab634de Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E2=80=9CBeeRad=E2=80=9D?= Date: Thu, 29 Jan 2026 15:43:27 +1100 Subject: [PATCH] docs: update prd.json and progress.txt for story 13 Co-Authored-By: Claude Opus 4.5 --- ralph/prd.json | 4 ++-- ralph/progress.txt | 23 +++++++++++++++++++++++ 2 files changed, 25 insertions(+), 2 deletions(-) diff --git a/ralph/prd.json b/ralph/prd.json index dedac36..e51004b 100644 --- a/ralph/prd.json +++ b/ralph/prd.json @@ -174,8 +174,8 @@ "Both server.js and stdio-server.js updated consistently", "npm run type-check passes" ], - "passes": false, - "notes": "" + "passes": true, + "notes": "Removed 3 extraction tools (rah_extract_url, rah_extract_youtube, rah_extract_pdf) that called non-existent API endpoints. MCP servers now expose 11 working tools: node CRUD (add, search, update, get), edge CRUD (create, query, update), dimension CRUD (create, update, delete), semantic search. Both servers updated consistently." }, { "id": "14", diff --git a/ralph/progress.txt b/ralph/progress.txt index 6a647fb..3bce66e 100644 --- a/ralph/progress.txt +++ b/ralph/progress.txt @@ -200,3 +200,26 @@ Goal: Strip ra-h_os to lightweight 2-panel UI + MCP server - These were only used by contextBuilder.ts which was deleted in Story 8 - Type files are well-organized - each file has a clear purpose +## Story 13: Audit and clean MCP server +- Completed: Thu 29 Jan 2026 +- Files modified: + - apps/mcp-server/server.js (removed extraction tools) + - apps/mcp-server/stdio-server.js (removed extraction tools) +- Tools removed (called non-existent API routes): + - rah_extract_url (called /api/extract/url - doesn't exist) + - rah_extract_youtube (called /api/extract/youtube - doesn't exist) + - rah_extract_pdf (called /api/extract/pdf - doesn't exist) +- Remaining 11 tools (all verified working): + - Node CRUD: rah_add_node, rah_search_nodes, rah_update_node, rah_get_nodes + - Edge CRUD: rah_create_edge, rah_query_edges, rah_update_edge + - Dimension CRUD: rah_create_dimension, rah_update_dimension, rah_delete_dimension + - Semantic search: rah_search_embeddings +- Analysis: + - MCP servers are pure API clients - no direct code dependencies + - All tools use callRaHApi() to call Next.js API routes + - No chat/agent/delegation dependencies + - Both servers were kept in sync +- Learnings: + - Extraction tools were dead code - the API routes never existed in the open source version + - The MCP architecture is clean - just HTTP calls to local API +