fix: refactor config to always load from disk

Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

justfile

@@ -179,4 +179,4 @@ beta version:
 
 # List all available recipes
 default:
-    @just --list
+    @just --list

pyproject.toml

@@ -52,7 +52,7 @@ build-backend = "hatchling.build"
 
 [tool.pytest.ini_options]
 pythonpath = ["src", "tests"]
-addopts = "--cov=basic_memory --cov-report term-missing -ra -q"
+addopts = "--cov=basic_memory --cov-report term-missing"
 testpaths = ["tests"]
 asyncio_mode = "strict"
 asyncio_default_fixture_loop_scope = "function"

src/basic_memory/alembic/env.py

@@ -8,17 +8,19 @@
 
 from alembic import context
 
+from basic_memory.config import ConfigManager
+
 # set config.env to "test" for pytest to prevent logging to file in utils.setup_logging()
 os.environ["BASIC_MEMORY_ENV"] = "test"
 
 # Import after setting environment variable  # noqa: E402
-from basic_memory.config import app_config  # noqa: E402
 from basic_memory.models import Base  # noqa: E402
 
 # this is the Alembic Config object, which provides
 # access to the values within the .ini file in use.
 config = context.config
 
+app_config = ConfigManager().config
 # Set the SQLAlchemy URL from our app config
 sqlalchemy_url = f"sqlite:///{app_config.database_path}"
 config.set_main_option("sqlalchemy.url", sqlalchemy_url)

src/basic_memory/api/app.py

@@ -20,13 +20,15 @@
     search,
     prompt_router,
 )
-from basic_memory.config import app_config
+from basic_memory.config import ConfigManager
 from basic_memory.services.initialization import initialize_app, initialize_file_sync
 
 
 @asynccontextmanager
 async def lifespan(app: FastAPI):  # pragma: no cover
     """Lifecycle manager for the FastAPI app."""
+
+    app_config = ConfigManager().config
     # Initialize app and database
     logger.info("Starting Basic Memory API")
     await initialize_app(app_config)

src/basic_memory/api/routers/management_router.py

@@ -6,7 +6,7 @@
 from loguru import logger
 from pydantic import BaseModel
 
-from basic_memory.config import app_config
+from basic_memory.config import ConfigManager
 from basic_memory.deps import SyncServiceDep, ProjectRepositoryDep
 
 router = APIRouter(prefix="/management", tags=["management"])
@@ -40,7 +40,9 @@ async def start_watch_service(
     if request.app.state.watch_task is not None and not request.app.state.watch_task.done():
         # Watch service is already running
         return WatchStatusResponse(running=True)
-
+    
+    app_config = ConfigManager().config
+    
     # Create and start a new watch service
     logger.info("Starting watch service via management API")
 

src/basic_memory/cli/app.py

@@ -2,16 +2,16 @@
 
 import typer
 
-from basic_memory.config import get_project_config
+from basic_memory.config import get_project_config, ConfigManager
 from basic_memory.mcp.project_session import session
 
 
 def version_callback(value: bool) -> None:
     """Show version and exit."""
     if value:  # pragma: no cover
         import basic_memory
-        from basic_memory.config import config
 
+        config = get_project_config()
         typer.echo(f"Basic Memory version: {basic_memory.__version__}")
         typer.echo(f"Current project: {config.project}")
         typer.echo(f"Project path: {config.home}")
@@ -44,9 +44,9 @@ def app_callback(
 
     # Run initialization for every command unless --version was specified
     if not version and ctx.invoked_subcommand is not None:
-        from basic_memory.config import app_config
         from basic_memory.services.initialization import ensure_initialization
 
+        app_config = ConfigManager().config
         ensure_initialization(app_config)
 
         # Initialize MCP session with the specified project or default

src/basic_memory/cli/commands/db.py

@@ -8,7 +8,7 @@
 
 from basic_memory import db
 from basic_memory.cli.app import app
-from basic_memory.config import app_config, config_manager
+from basic_memory.config import ConfigManager, BasicMemoryConfig, save_basic_memory_config
 
 
 @app.command()
@@ -18,6 +18,8 @@ def reset(
     """Reset database (drop all tables and recreate)."""
     if typer.confirm("This will delete all data in your db. Are you sure?"):
         logger.info("Resetting database...")
+        config_manager = ConfigManager()
+        app_config = config_manager.config
         # Get database path
         db_path = app_config.app_database_path
 
@@ -27,9 +29,8 @@ def reset(
             logger.info(f"Database file deleted: {db_path}")
 
         # Reset project configuration
-        config_manager.config.projects = {"main": str(Path.home() / "basic-memory")}
-        config_manager.config.default_project = "main"
-        config_manager.save_config(config_manager.config)
+        config = BasicMemoryConfig()
+        save_basic_memory_config(config_manager.config_file, config)
         logger.info("Project configuration reset to default")
 
         # Create a new empty database

src/basic_memory/cli/commands/import_chatgpt.py

@@ -7,7 +7,7 @@
 
 import typer
 from basic_memory.cli.app import import_app
-from basic_memory.config import config
+from basic_memory.config import get_project_config
 from basic_memory.importers import ChatGPTImporter
 from basic_memory.markdown import EntityParser, MarkdownProcessor
 from loguru import logger
@@ -19,6 +19,7 @@
 
 async def get_markdown_processor() -> MarkdownProcessor:
     """Get MarkdownProcessor instance."""
+    config = get_project_config()
     entity_parser = EntityParser(config.home)
     return MarkdownProcessor(entity_parser)
 
@@ -49,7 +50,7 @@ def import_chatgpt(
 
         # Get markdown processor
         markdown_processor = asyncio.run(get_markdown_processor())
-
+        config = get_project_config()
         # Process the file
         base_path = config.home / folder
         console.print(f"\nImporting chats from {conversations_json}...writing to {base_path}")

src/basic_memory/cli/commands/import_claude_conversations.py

@@ -7,7 +7,7 @@
 
 import typer
 from basic_memory.cli.app import claude_app
-from basic_memory.config import config
+from basic_memory.config import get_project_config
 from basic_memory.importers.claude_conversations_importer import ClaudeConversationsImporter
 from basic_memory.markdown import EntityParser, MarkdownProcessor
 from loguru import logger
@@ -19,6 +19,7 @@
 
 async def get_markdown_processor() -> MarkdownProcessor:
     """Get MarkdownProcessor instance."""
+    config = get_project_config()
     entity_parser = EntityParser(config.home)
     return MarkdownProcessor(entity_parser)
 
@@ -42,6 +43,7 @@ def import_claude(
     After importing, run 'basic-memory sync' to index the new files.
     """
 
+    config = get_project_config()
     try:
         if not conversations_json.exists():
             typer.echo(f"Error: File not found: {conversations_json}", err=True)

src/basic_memory/cli/commands/import_claude_projects.py

@@ -7,7 +7,7 @@
 
 import typer
 from basic_memory.cli.app import claude_app
-from basic_memory.config import config
+from basic_memory.config import get_project_config
 from basic_memory.importers.claude_projects_importer import ClaudeProjectsImporter
 from basic_memory.markdown import EntityParser, MarkdownProcessor
 from loguru import logger
@@ -19,6 +19,7 @@
 
 async def get_markdown_processor() -> MarkdownProcessor:
     """Get MarkdownProcessor instance."""
+    config = get_project_config()
     entity_parser = EntityParser(config.home)
     return MarkdownProcessor(entity_parser)
 
@@ -41,6 +42,7 @@ def import_projects(
 
     After importing, run 'basic-memory sync' to index the new files.
     """
+    config = get_project_config()
     try:
         if not projects_json.exists():
             typer.echo(f"Error: File not found: {projects_json}", err=True)