fix: disable file sync in integration tests to prevent hanging

The MCP server lifespan starts a background file sync task when
sync_changes=True (the default). This task runs indefinitely, causing
integration tests to hang.

Fix: Set sync_changes=False in the test app_config fixture. This is the
proper solution because:
- It's test configuration, not production code hacking
- Integration tests don't need file sync - they test MCP tools/API
- The lifespan works correctly, it just won't start the blocking task

Also includes minor formatting cleanup in app.py.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

src/basic_memory/cli/app.py

@@ -38,7 +38,11 @@ def app_callback(
     # Skip for 'mcp' command - it has its own lifespan that handles initialization
     # Skip for API-using commands (status, sync, etc.) - they handle initialization via deps.py
     api_commands = {"mcp", "status", "sync", "project", "tools"}
-    if not version and ctx.invoked_subcommand is not None and ctx.invoked_subcommand not in api_commands:
+    if (
+        not version
+        and ctx.invoked_subcommand is not None
+        and ctx.invoked_subcommand not in api_commands
+    ):
         from basic_memory.services.initialization import ensure_initialization
 
         app_config = ConfigManager().config

test-int/conftest.py

@@ -236,6 +236,7 @@ def app_config(
         default_project_mode=False,  # Match real-world usage - tools must pass explicit project
         update_permalinks_on_move=True,
         cloud_mode=False,  # Explicitly disable cloud mode
+        sync_changes=False,  # Disable file sync in tests - prevents lifespan from starting blocking task
         database_backend=database_backend,
         database_url=database_url,
     )

test-int/mcp/test_lifespan_shutdown_sync_task_cancellation_integration.py

@@ -66,5 +66,3 @@ async def _run_client_once():
     # Use asyncio.run to match the CLI/MCP execution model where loop teardown
     # would hang if a background task is left running.
     asyncio.run(_run_client_once())
-
-