mirror of
https://github.com/basicmachines-co/basic-memory
synced 2026-06-21 13:47:35 +00:00
phernandez
94bdfe77e4
BASIC_MEMORY_CLOUD_MODE env var was never checked in resolve_runtime_mode(), so cloud deployments always ran as LOCAL mode. This caused file sync to start in the cloud container, which then failed with "DATABASE_URL must be set when using Postgres backend" because there's no local DB in cloud mode. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: phernandez <paul@basicmachines.co>
61 lines
1.7 KiB
Python
61 lines
1.7 KiB
Python
"""Runtime mode resolution for Basic Memory.
|
|
|
|
This module centralizes runtime mode detection, ensuring cloud/local/test
|
|
determination happens in one place rather than scattered across modules.
|
|
|
|
Composition roots (containers) read ConfigManager and use this module
|
|
to resolve the runtime mode, then pass the result downstream.
|
|
"""
|
|
|
|
import os
|
|
from enum import Enum, auto
|
|
|
|
|
|
class RuntimeMode(Enum):
|
|
"""Runtime modes for Basic Memory."""
|
|
|
|
LOCAL = auto() # Local standalone mode (default)
|
|
CLOUD = auto() # Cloud mode with remote sync
|
|
TEST = auto() # Test environment
|
|
|
|
@property
|
|
def is_cloud(self) -> bool:
|
|
return self == RuntimeMode.CLOUD
|
|
|
|
@property
|
|
def is_local(self) -> bool:
|
|
return self == RuntimeMode.LOCAL
|
|
|
|
@property
|
|
def is_test(self) -> bool:
|
|
return self == RuntimeMode.TEST
|
|
|
|
|
|
def resolve_runtime_mode(
|
|
is_test_env: bool,
|
|
) -> RuntimeMode:
|
|
"""Resolve the runtime mode from configuration flags.
|
|
|
|
This is the single source of truth for mode resolution.
|
|
Composition roots call this with config values they've read.
|
|
|
|
Args:
|
|
is_test_env: Whether running in test environment
|
|
|
|
Returns:
|
|
The resolved RuntimeMode
|
|
"""
|
|
if is_test_env:
|
|
return RuntimeMode.TEST
|
|
|
|
# Trigger: BASIC_MEMORY_CLOUD_MODE env var is set
|
|
# Why: cloud deployments must not start local file sync — cloud handles
|
|
# file storage via S3/Tigris, and the local sync tries to open a
|
|
# SQLite/Postgres DB that doesn't exist in the cloud container
|
|
# Outcome: returns CLOUD mode, skipping file sync initialization
|
|
cloud_mode = os.getenv("BASIC_MEMORY_CLOUD_MODE", "").lower() in ("1", "true")
|
|
if cloud_mode:
|
|
return RuntimeMode.CLOUD
|
|
|
|
return RuntimeMode.LOCAL
|