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

- 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>

Author: groksrc

src/basic_memory/mcp/tools/__init__.py

@@ -18,24 +18,24 @@
 from basic_memory.mcp.tools.edit_note import edit_note
 from basic_memory.mcp.tools.move_note import move_note
 from basic_memory.mcp.tools.project_management import (
-    list_projects,
+    list_memory_projects,
     switch_project,
     get_current_project,
     set_default_project,
-    create_project,
+    create_memory_project,
     delete_project,
 )
 
 __all__ = [
     "build_context",
     "canvas",
-    "create_project",
+    "create_memory_project",
     "delete_note",
     "delete_project",
     "edit_note",
     "get_current_project",
     "list_directory",
-    "list_projects",
+    "list_memory_projects",
     "move_note",
     "read_content",
     "read_note",

src/basic_memory/mcp/tools/project_management.py

@@ -16,8 +16,8 @@
 from basic_memory.schemas.project_info import ProjectList, ProjectStatusResponse, ProjectInfoRequest
 
 
-@mcp.tool()
-async def list_projects(ctx: Context | None = None) -> str:
+@mcp.tool("list_memory_projects")
+async def list_memory_projects(ctx: Context | None = None) -> str:
     """List all available projects with their status.
 
     Shows all Basic Memory projects that are available, indicating which one
@@ -27,7 +27,7 @@ async def list_projects(ctx: Context | None = None) -> str:
         Formatted list of projects with status indicators
 
     Example:
-        list_projects()
+        list_memory_projects()
     """
     if ctx:  # pragma: no cover
         await ctx.info("Listing all available projects")
@@ -186,8 +186,8 @@ async def set_default_project(project_name: str, ctx: Context | None = None) ->
     return add_project_metadata(result, session.get_current_project())
 
 
-@mcp.tool()
-async def create_project(
+@mcp.tool("create_memory_project")
+async def create_memory_project(
     project_name: str, project_path: str, set_default: bool = False, ctx: Context | None = None
 ) -> str:
     """Create a new Basic Memory project.
@@ -204,8 +204,8 @@ async def create_project(
         Confirmation message with project details
 
     Example:
-        create_project("my-research", "~/Documents/research")
-        create_project("work-notes", "/home/user/work", set_default=True)
+        create_memory_project("my-research", "~/Documents/research")
+        create_memory_project("work-notes", "/home/user/work", set_default=True)
     """
     if ctx:  # pragma: no cover
         await ctx.info(f"Creating project: {project_name} at {project_path}")

test-int/mcp/test_project_management_integration.py

@@ -15,7 +15,7 @@ async def test_list_projects_basic_operation(mcp_server, app):
     async with Client(mcp_server) as client:
         # List all available projects
         list_result = await client.call_tool(
-            "list_projects",
+            "list_memory_projects",
             {},
         )
 
@@ -248,7 +248,7 @@ async def test_project_management_workflow(mcp_server, app):
         assert "test-project" in current_result[0].text
 
         # 2. List all projects
-        list_result = await client.call_tool("list_projects", {})
+        list_result = await client.call_tool("list_memory_projects", {})
         assert "Available projects:" in list_result[0].text
         assert "test-project" in list_result[0].text
 
@@ -269,7 +269,7 @@ async def test_project_metadata_consistency(mcp_server, app):
         # Test all project management tools and verify they include project metadata
 
         # list_projects
-        list_result = await client.call_tool("list_projects", {})
+        list_result = await client.call_tool("list_memory_projects", {})
         assert "Project: test-project" in list_result[0].text
 
         # get_current_project
@@ -350,7 +350,7 @@ async def test_create_project_basic_operation(mcp_server, app):
     async with Client(mcp_server) as client:
         # Create a new project
         create_result = await client.call_tool(
-            "create_project",
+            "create_memory_project",
             {
                 "project_name": "test-new-project",
                 "project_path": "/tmp/test-new-project",
@@ -370,7 +370,7 @@ async def test_create_project_basic_operation(mcp_server, app):
         assert "Project: test-project" in create_text  # Should still show current project
 
         # Verify project appears in project list
-        list_result = await client.call_tool("list_projects", {})
+        list_result = await client.call_tool("list_memory_projects", {})
         list_text = list_result[0].text
         assert "test-new-project" in list_text
 
@@ -382,7 +382,7 @@ async def test_create_project_with_default_flag(mcp_server, app):
     async with Client(mcp_server) as client:
         # Create a new project and set as default
         create_result = await client.call_tool(
-            "create_project",
+            "create_memory_project",
             {
                 "project_name": "test-default-project",
                 "project_path": "/tmp/test-default-project",
@@ -412,7 +412,7 @@ async def test_create_project_duplicate_name(mcp_server, app):
     async with Client(mcp_server) as client:
         # First create a project
         await client.call_tool(
-            "create_project",
+            "create_memory_project",
             {
                 "project_name": "duplicate-test",
                 "project_path": "/tmp/duplicate-test-1",
@@ -422,7 +422,7 @@ async def test_create_project_duplicate_name(mcp_server, app):
         # Try to create another project with same name
         with pytest.raises(Exception) as exc_info:
             await client.call_tool(
-                "create_project",
+                "create_memory_project",
                 {
                     "project_name": "duplicate-test",
                     "project_path": "/tmp/duplicate-test-2",
@@ -431,7 +431,7 @@ async def test_create_project_duplicate_name(mcp_server, app):
 
         # Should show error about duplicate name
         error_message = str(exc_info.value)
-        assert "create_project" in error_message
+        assert "create_memory_project" in error_message
         assert (
             "duplicate-test" in error_message
             or "already exists" in error_message
@@ -446,15 +446,15 @@ async def test_delete_project_basic_operation(mcp_server, app):
     async with Client(mcp_server) as client:
         # First create a project to delete
         await client.call_tool(
-            "create_project",
+            "create_memory_project",
             {
                 "project_name": "to-be-deleted",
                 "project_path": "/tmp/to-be-deleted",
             },
         )
 
         # Verify it exists
-        list_result = await client.call_tool("list_projects", {})
+        list_result = await client.call_tool("list_memory_projects", {})
         assert "to-be-deleted" in list_result[0].text
 
         # Delete the project
@@ -478,7 +478,7 @@ async def test_delete_project_basic_operation(mcp_server, app):
         assert "Project: test-project" in delete_text  # Should show current project
 
         # Verify project no longer appears in list
-        list_result_after = await client.call_tool("list_projects", {})
+        list_result_after = await client.call_tool("list_memory_projects", {})
         assert "to-be-deleted" not in list_result_after[0].text
 
 
@@ -540,7 +540,7 @@ async def test_project_lifecycle_workflow(mcp_server, app):
 
         # 1. Create new project
         create_result = await client.call_tool(
-            "create_project",
+            "create_memory_project",
             {
                 "project_name": project_name,
                 "project_path": project_path,
@@ -595,7 +595,7 @@ async def test_project_lifecycle_workflow(mcp_server, app):
         assert "removed successfully" in delete_result[0].text
 
         # 7. Verify project is gone from list
-        list_result = await client.call_tool("list_projects", {})
+        list_result = await client.call_tool("list_memory_projects", {})
         assert project_name not in list_result[0].text
 
 
@@ -609,7 +609,7 @@ async def test_create_delete_project_edge_cases(mcp_server, app):
 
         # Create project with special characters
         create_result = await client.call_tool(
-            "create_project",
+            "create_memory_project",
             {
                 "project_name": special_name,
                 "project_path": f"/tmp/{special_name}",
@@ -619,7 +619,7 @@ async def test_create_delete_project_edge_cases(mcp_server, app):
         assert special_name in create_result[0].text
 
         # Verify it appears in list
-        list_result = await client.call_tool("list_projects", {})
+        list_result = await client.call_tool("list_memory_projects", {})
         assert special_name in list_result[0].text
 
         # Delete it
@@ -633,5 +633,5 @@ async def test_create_delete_project_edge_cases(mcp_server, app):
         assert special_name in delete_result[0].text
 
         # Verify it's gone
-        list_result_after = await client.call_tool("list_projects", {})
+        list_result_after = await client.call_tool("list_memory_projects", {})
         assert special_name not in list_result_after[0].text

tests/mcp/test_server.py

@@ -93,7 +93,7 @@ def test_create_auth_config_supabase_provider_missing_anon_key(self):
             # Missing SUPABASE_ANON_KEY
         }
 
-        with patch.dict(os.environ, env_vars):
+        with patch.dict(os.environ, env_vars, clear=True):
             with pytest.raises(ValueError, match="SUPABASE_URL and SUPABASE_ANON_KEY must be set"):
                 create_auth_config()