Skip to content

fix: resolve MCP-Hub tool validation errors for project management functions#198

Closed
groksrc wants to merge 1 commit into
mainfrom
fix/mcp-hub
Closed

fix: resolve MCP-Hub tool validation errors for project management functions#198
groksrc wants to merge 1 commit into
mainfrom
fix/mcp-hub

Conversation

@groksrc

@groksrc groksrc commented Jul 1, 2025

Copy link
Copy Markdown
Member

Summary

  • Fixed FastMCP validation errors when using Basic Memory with MCP-Hub
  • Renamed functions to match tool decorator names for proper validation
  • Resolves issue [BUG] Problems with MCP #190

Problem

When using Basic Memory through MCP-Hub, the list_memory_projects and create_memory_project tools were failing with Pydantic validation errors. FastMCP requires that function names match the tool decorator names exactly.

Solution

  • Renamed list_projects() to list_memory_projects()
  • Renamed create_project() to create_memory_project()
  • Updated imports and exports accordingly

Test plan

  • Built Docker image with changes
  • Tested all 19 MCP tools through direct connection - all pass
  • Verified the specific failing tools now work correctly
  • No breaking changes to existing functionality

Fixes #190

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

…nctions

- Renamed list_projects() to list_memory_projects()
- Renamed create_project() to create_memory_project()
- Updated imports and exports accordingly
- Fixed test environment variable isolation

Fixes #190

Signed-off-by: Drew Cain <groksrc@gmail.com>
@groksrc groksrc closed this Jul 1, 2025
@groksrc groksrc deleted the fix/mcp-hub branch July 1, 2025 13:50
@groksrc groksrc restored the fix/mcp-hub branch July 1, 2025 13:50
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.

[BUG] Problems with MCP

1 participant