fix: run migrations when using CLI entry point

- Add ensure_migrations function to run migrations on startup
- Make CLI callback run migrations for every command
- Add tests to verify migration behavior
- Fixes issue #87 where migrations weren't run when using pip-installed CLI

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

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

Author: phernandez

src/basic_memory/cli/main.py

@@ -3,6 +3,7 @@
 import asyncio
 
 import typer
+from loguru import logger
 
 from basic_memory.cli.app import app  # pragma: no cover
 
@@ -23,6 +24,18 @@
 from basic_memory.db import run_migrations as db_run_migrations
 
 
+# Helper function to run database migrations
+def ensure_migrations():  # pragma: no cover
+    """Ensure database migrations are run before executing commands."""
+    try:
+        logger.info("Running database migrations on startup...")
+        asyncio.run(db_run_migrations(config))
+    except Exception as e:
+        logger.error(f"Error running migrations: {e}")
+        # Continue execution even if migrations fail
+        # The actual command might still work or will fail with a more specific error
+
+
 # Version command
 @app.callback(invoke_without_command=True)
 def main(
@@ -58,6 +71,10 @@ def main(
 
         os.environ["BASIC_MEMORY_PROJECT"] = project
 
+    # Run migrations for every command unless --version was specified
+    if not version and ctx.invoked_subcommand is not None:
+        ensure_migrations()
+
 
 if __name__ == "__main__":  # pragma: no cover
     # Run database migrations

tests/cli/test_cli_tools.py

@@ -2,6 +2,9 @@
 
 These tests use real MCP tools with the test environment instead of mocks.
 """
+# Import the ensure_migrations function from main.py for testing
+import asyncio
+from basic_memory.cli.main import ensure_migrations
 
 import io
 from datetime import datetime, timedelta
@@ -410,3 +413,36 @@ def test_continue_conversation_no_results(cli_env):
     # Check result contains expected content for no results
     assert "Continuing conversation on: NonexistentTopic" in result.stdout
     assert "The supplied query did not return any information" in result.stdout
+
+
+@patch("basic_memory.cli.main.db_run_migrations")
+def test_ensure_migrations_runs_migrations(mock_run_migrations, test_config, monkeypatch):
+    """Test that ensure_migrations runs migrations."""
+    # Configure mock
+    async def mock_async_success(*args, **kwargs):
+        return True
+    
+    mock_run_migrations.return_value = mock_async_success()
+    
+    # Call the function
+    ensure_migrations()
+    
+    # Check that run_migrations was called
+    mock_run_migrations.assert_called_once()
+
+
+@patch("basic_memory.cli.main.db_run_migrations")
+@patch("basic_memory.cli.main.logger")
+def test_ensure_migrations_handles_errors(mock_logger, mock_run_migrations, test_config, monkeypatch):
+    """Test that ensure_migrations handles errors gracefully."""
+    # Configure mock to raise an exception when awaited
+    async def mock_async_error(*args, **kwargs):
+        raise Exception("Test error")
+    
+    mock_run_migrations.side_effect = mock_async_error
+    
+    # Call the function
+    ensure_migrations()
+    
+    # Check that error was logged
+    mock_logger.error.assert_called_once()