Update backend/app/config/settings.py

backend/app/config/settings.py

@@ -1,12 +1,48 @@
+# backend/app/config/settings.py
+from __future__ import annotations
 from pathlib import Path
 from pydantic import Field
 from pydantic_settings import BaseSettings, SettingsConfigDict
+import os, tempfile
 
 # Resolve repo root no matter where uvicorn is launched from
 REPO_ROOT = Path(__file__).resolve().parents[3]
 
+def _try_writable(p: Path) -> Path | None:
+    try:
+        p.mkdir(parents=True, exist_ok=True)
+        t = p / ".write_test"
+        t.write_text("ok", encoding="utf-8")
+        t.unlink(missing_ok=True)
+        return p.resolve()
+    except Exception:
+        return None
+
 def _default_data_dir() -> Path:
-    return (REPO_ROOT / "data").resolve()
+    """
+    HF Spaces:
+      - /data is writable & persisted across restarts
+      - /app is read-only (repo checkout)
+    Order:
+      1) DATA_DIR env (if set)
+      2) /data
+      3) <repo>/data
+      4) temp dir
+    """
+    candidates: list[Path] = []
+    env = os.getenv("DATA_DIR")
+    if env:
+        candidates.append(Path(env))
+    candidates.append(Path("/data"))
+    candidates.append(REPO_ROOT / "data")
+    candidates.append(Path(tempfile.gettempdir()) / "pulsemaps" / "data")
+
+    for p in candidates:
+        w = _try_writable(p)
+        if w:
+            return w
+    # As a last resort, raise a clear error
+    raise RuntimeError(f"No writable DATA_DIR found. Tried: {candidates!r}")
 
 def _default_uploads_dir() -> Path:
     return (_default_data_dir() / "uploads").resolve()
@@ -27,11 +63,11 @@ class Settings(BaseSettings):
     OPENAI_MODEL_AGENT: str = "gpt-4o"
     OPENAI_MODEL_CLASSIFIER: str = "gpt-4o-mini"
 
-    # Paths (env may override with absolute or relative; we resolve below)
+    # Paths
     DATA_DIR: Path = Field(default_factory=_default_data_dir)
-    REPORTS_DB: Path = Field(default_factory=lambda: _default_data_dir() / "pulsemaps_reports.db")
-    SESSIONS_DB: Path = Field(default_factory=lambda: _default_data_dir() / "pulsemap_sessions.db")
-    UPLOADS_DIR: Path = Field(default_factory=_default_uploads_dir)
+    REPORTS_DB: Path | None = None
+    SESSIONS_DB: Path | None = None
+    UPLOADS_DIR: Path | None = None
     FRONTEND_DIST: Path = Field(default_factory=_default_frontend_dist)
 
     # Defaults
@@ -45,12 +81,22 @@ class Settings(BaseSettings):
     nvidia_api_key: str | None = None
 
     def ensure_dirs(self) -> None:
-        # Resolve in case env provided relative strings
-        self.DATA_DIR = self.DATA_DIR.resolve()
-        self.UPLOADS_DIR = self.UPLOADS_DIR.resolve()
-        # Create everything robustly
+        # Fill derived paths if not explicitly provided
+        if self.REPORTS_DB is None:
+            self.REPORTS_DB = self.DATA_DIR / "pulsemaps_reports.db"
+        if self.SESSIONS_DB is None:
+            self.SESSIONS_DB = self.DATA_DIR / "pulsemap_sessions.db"
+        if self.UPLOADS_DIR is None:
+            self.UPLOADS_DIR = self.DATA_DIR / "uploads"
+
+        # Create & resolve
         self.DATA_DIR.mkdir(parents=True, exist_ok=True)
         self.UPLOADS_DIR.mkdir(parents=True, exist_ok=True)
 
+        self.DATA_DIR = self.DATA_DIR.resolve()
+        self.REPORTS_DB = self.REPORTS_DB.resolve()
+        self.SESSIONS_DB = self.SESSIONS_DB.resolve()
+        self.UPLOADS_DIR = self.UPLOADS_DIR.resolve()
+
 settings = Settings()
-settings.ensure_dirs()
+settings.ensure_dirs()