Updated docker for /data issue

.dockerignore

@@ -6,4 +6,3 @@ __pycache__/
 node_modules/
 web/node_modules/
 web/.vite/
-data/

Dockerfile

@@ -14,6 +14,10 @@ ENV PYTHONUNBUFFERED=1 \
     DATA_DIR=/data
 WORKDIR /app
 
+ENV PORT=7860 DATA_DIR=/data
+
+RUN mkdir -p /data/uploads
+
 # (optional) if you hit build issues with some libs
 RUN apt-get update && apt-get install -y --no-install-recommends build-essential && rm -rf /var/lib/apt/lists/*
 

backend/app/config/settings.py

@@ -1,15 +1,34 @@
 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
+# Resolve the repo root when running outside Docker
 REPO_ROOT = Path(__file__).resolve().parents[3]
 
-def _default_data_dir() -> Path:
-    return (REPO_ROOT / "data").resolve()
+def _writable_dir(candidates: list[Path]) -> Path:
+    for p in candidates:
+        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
+        except Exception:
+            continue
+    raise RuntimeError(f"No writable data dir from: {candidates!r}")
 
-def _default_uploads_dir() -> Path:
-    return (_default_data_dir() / "uploads").resolve()
+def _default_data_dir() -> Path:
+    # 1) Respect env if provided
+    if os.getenv("DATA_DIR"):
+        return Path(os.getenv("DATA_DIR")).resolve()
+    # 2) Prefer /data inside containers (Docker/HF Spaces)
+    candidates = [Path("/data")]
+    # 3) Repo local ./data for local dev
+    candidates.append((REPO_ROOT / "data").resolve())
+    # 4) Last resort: /tmp
+    candidates.append(Path(tempfile.gettempdir()) / "pulsemaps" / "data")
+    return _writable_dir(candidates)
 
 def _default_frontend_dist() -> Path:
     return (REPO_ROOT / "web" / "dist").resolve()
@@ -22,16 +41,16 @@ class Settings(BaseSettings):
         populate_by_name=True,
     )
 
-    # Models / API keys
+    # API / Models
     OPENAI_API_KEY: str | None = None
     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,10 +64,20 @@ class Settings(BaseSettings):
     nvidia_api_key: str | None = None
 
     def ensure_dirs(self) -> None:
-        # Resolve in case env provided relative strings
+        # Fill path fields if not set, then ensure they exist
+        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")
+
+        # Absolute, existing
         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()
-        # Create everything robustly
+
         self.DATA_DIR.mkdir(parents=True, exist_ok=True)
         self.UPLOADS_DIR.mkdir(parents=True, exist_ok=True)