fix: update FastMCP initialization for API changes

- Remove deprecated auth_server_provider parameter
- Use auth parameter correctly with OAuthProvider instead of AuthSettings
- Fixes type error after dependency updates

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: phernandez

src/basic_memory/alembic/versions/cc7172b46608_update_search_index_schema.py

@@ -57,7 +57,6 @@ def upgrade() -> None:
     """)
 
 
-
 def downgrade() -> None:
     """Downgrade database schema to use old search index."""
     # Drop the updated search_index table

src/basic_memory/api/routers/knowledge_router.py

@@ -274,4 +274,4 @@ async def delete_entities(
         background_tasks.add_task(search_service.delete_by_permalink, permalink)
 
     result = DeleteEntitiesResponse(deleted=deleted)
-    return result
+    return result

src/basic_memory/api/routers/project_router.py

@@ -145,7 +145,9 @@ async def remove_project(
     try:
         old_project = await project_service.get_project(name)
         if not old_project:  # pragma: no cover
-            raise HTTPException(status_code=404, detail=f"Project: '{name}' does not exist")  # pragma: no cover
+            raise HTTPException(
+                status_code=404, detail=f"Project: '{name}' does not exist"
+            )  # pragma: no cover
 
         await project_service.remove_project(name)
 
@@ -186,7 +188,9 @@ async def set_default_project(
         # get the new project
         new_default_project = await project_service.get_project(name)
         if not new_default_project:  # pragma: no cover
-            raise HTTPException(status_code=404, detail=f"Project: '{name}' does not exist")  # pragma: no cover
+            raise HTTPException(
+                status_code=404, detail=f"Project: '{name}' does not exist"
+            )  # pragma: no cover
 
         await project_service.set_default_project(name)
 

src/basic_memory/config.py

@@ -179,7 +179,7 @@ def load_config(self) -> BasicMemoryConfig:
 
     def save_config(self, config: BasicMemoryConfig) -> None:
         """Save configuration to file."""
-        try:            
+        try:
             self.config_file.write_text(json.dumps(config.model_dump(), indent=2))
         except Exception as e:  # pragma: no cover
             logger.error(f"Failed to save config: {e}")

src/basic_memory/mcp/server.py

@@ -106,6 +106,5 @@ def create_auth_config() -> tuple[AuthSettings | None, Any | None]:
 mcp = FastMCP(
     name="Basic Memory",
     log_level="DEBUG",
-    auth_server_provider=auth_provider,
-    auth=auth_settings,
+    auth=auth_provider,
 )

src/basic_memory/mcp/tools/move_note.py

@@ -31,7 +31,7 @@ async def move_note(
 
     Examples:
         - Move to new folder: move_note("My Note", "work/notes/my-note.md")
-        - Move by permalink: move_note("my-note-permalink", "archive/old-notes/my-note.md") 
+        - Move by permalink: move_note("my-note-permalink", "archive/old-notes/my-note.md")
         - Specify project: move_note("My Note", "archive/my-note.md", project="work-project")
 
     Note: This operation moves notes within the specified project only. Moving notes

src/basic_memory/services/project_service.py

@@ -22,6 +22,7 @@
 from basic_memory.utils import generate_permalink
 from basic_memory.config import config_manager
 
+
 class ProjectService:
     """Service for managing Basic Memory projects."""
 
@@ -545,4 +546,4 @@ def get_system_status(self) -> SystemStatus:
             database_size=db_size_readable,
             watch_status=watch_status,
             timestamp=datetime.now(),
-        )
+        )

src/basic_memory/services/search_service.py

@@ -148,7 +148,7 @@ def _extract_entity_tags(self, entity: Entity) -> List[str]:
                 # If parsing fails, treat as single tag
                 return [tags] if tags.strip() else []
 
-        return [] # pragma: no cover 
+        return []  # pragma: no cover
 
     async def index_entity(
         self,

src/basic_memory/sync/sync_service.py

@@ -379,7 +379,9 @@ async def handle_move(self, old_path, new_path):
             updates = {"file_path": new_path}
 
             # If configured, also update permalink to match new path
-            if self.app_config.update_permalinks_on_move and self.file_service.is_markdown(new_path):
+            if self.app_config.update_permalinks_on_move and self.file_service.is_markdown(
+                new_path
+            ):
                 # generate new permalink value
                 new_permalink = await self.entity_service.resolve_permalink(new_path)
 
@@ -505,4 +507,4 @@ async def scan_directory(self, directory: Path) -> ScanResult:
             f"duration_ms={duration_ms}"
         )
 
-        return result
+        return result

test-int/conftest.py

@@ -49,10 +49,8 @@ async def test_my_mcp_tool(mcp_server, app):
 The `app` fixture ensures FastAPI dependency overrides are active, and
 `mcp_server` provides the MCP server with proper project session initialization.
 """
-import os
+
 from typing import AsyncGenerator
-from unittest import mock
-from unittest.mock import patch
 
 import pytest
 import pytest_asyncio
@@ -110,21 +108,29 @@ async def test_project(tmp_path, engine_factory) -> Project:
     project = await project_repository.create(project_data)
     return project
 
+
 @pytest.fixture
 def config_home(tmp_path, monkeypatch) -> Path:
     monkeypatch.setenv("HOME", str(tmp_path))
     return tmp_path
 
+
 @pytest.fixture(scope="function")
 def app_config(config_home, test_project, tmp_path, monkeypatch) -> BasicMemoryConfig:
     """Create test app configuration."""
     projects = {test_project.name: str(test_project.path)}
-    app_config = BasicMemoryConfig(env="test", projects=projects, default_project=test_project.name, update_permalinks_on_move=True)
+    app_config = BasicMemoryConfig(
+        env="test",
+        projects=projects,
+        default_project=test_project.name,
+        update_permalinks_on_move=True,
+    )
 
     # Set the module app_config instance project list (like regular tests)
     monkeypatch.setattr("basic_memory.config.app_config", app_config)
     return app_config
 
+
 @pytest.fixture
 def config_manager(app_config: BasicMemoryConfig, config_home, monkeypatch) -> ConfigManager:
     config_manager = ConfigManager()
@@ -145,6 +151,7 @@ def config_manager(app_config: BasicMemoryConfig, config_home, monkeypatch) -> C
 
     return config_manager
 
+
 @pytest.fixture
 def project_session(test_project: Project):
     # initialize the project session with the test project
@@ -166,9 +173,10 @@ def project_config(test_project, monkeypatch):
     return project_config
 
 
-
 @pytest.fixture(scope="function")
-def app(app_config, project_config, engine_factory, test_project, project_session, config_manager) -> FastAPI:
+def app(
+    app_config, project_config, engine_factory, test_project, project_session, config_manager
+) -> FastAPI:
     """Create test FastAPI application with single project."""
 
     app = fastapi_app
@@ -228,4 +236,4 @@ def mcp_server(app_config, search_service):
 async def client(app: FastAPI) -> AsyncGenerator[AsyncClient, None]:
     """Create test client that both MCP and tests will use."""
     async with AsyncClient(transport=ASGITransport(app=app), base_url="http://test") as client:
-        yield client
+        yield client