Claude Code: Fix path resolution in error_handlers.py - remove hardcoded /app paths

- Replace hardcoded /app/* paths with dynamic directory resolution
- Add _get_base_dir() function to determine runtime directory dynamically
- Respect OPENCLAW_DATA_DIR environment variable for Docker environment
- Simplify _resolve_status_file() to trust CAIN_STATUS_PATH env var

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

error_handlers.py

@@ -58,37 +58,50 @@ async def generic_exception_handler(request: Request, exc: Exception) -> JSONRes
 
 
 # Configuration path - cain_status.json location
-# Dynamically search common locations for cain_status.json
+def _get_base_dir() -> Path:
+    """Determine the correct base directory dynamically."""
+    # Priority 1: OPENCLAW_DATA_DIR (set by Docker environment)
+    data_dir = os.environ.get('OPENCLAW_DATA_DIR')
+    if data_dir:
+        return Path(data_dir)
+
+    # Priority 2: /data (standard Docker volume mount)
+    data_path = Path("/data")
+    if data_path.exists():
+        return data_path
+
+    # Priority 3: Use this script's directory as fallback
+    return Path(os.path.dirname(__file__))
+
+
 def _resolve_status_file() -> Path:
     """Find cain_status.json by searching common locations."""
     # Priority 1: CAIN_STATUS_PATH env var (direct path or set by app.py)
+    # Trust this completely if set - don't second-guess it
     env_path = os.environ.get('CAIN_STATUS_PATH')
     if env_path:
-        candidate = Path(env_path)
-        if candidate.exists() or candidate.parent.exists():
-            return candidate
+        return Path(env_path)
 
-    # Priority 2: Search relative to this script's location
-    # Docker WORKDIR is /app, so script_dir will be /app
+    # Priority 2: Use OPENCLAW_DATA_DIR (set in Docker)
+    base_dir = _get_base_dir()
+    candidate = base_dir / "cain_status.json"
+    if candidate.parent.exists():
+        return candidate
+
+    # Priority 3: Search relative to this script's location
     script_dir = Path(os.path.dirname(__file__))
     relative_paths = [
-        script_dir / "cain_status.json",               # /app/cain_status.json
-        script_dir / ".openclaw" / "cain_status.json", # /app/.openclaw/cain_status.json
-        script_dir / ".openclaw" / "agents" / "cain_status.json",  # /app/.openclaw/agents/cain_status.json
+        script_dir / "cain_status.json",               # Script directory
+        script_dir / ".openclaw" / "cain_status.json", # .openclaw subdir
+        script_dir / ".openclaw" / "agents" / "cain_status.json",  # agents subdir
         script_dir.parent / "cain_status.json",
     ]
     for candidate in relative_paths:
         if candidate.exists():
             return candidate
 
-    # Priority 3: Common absolute locations
-    for base in ["./cain_status.json", "/app/cain_status.json", "/app/.openclaw/cain_status.json", "/data/cain_status.json"]:
-        candidate = Path(base)
-        if candidate.exists():
-            return candidate
-
-    # Default to script_dir/cain_status.json (will be created if needed)
-    return script_dir / "cain_status.json"
+    # Default: use base_dir/cain_status.json (will be created if needed)
+    return _get_base_dir() / "cain_status.json"
 
 STATUS_FILE = _resolve_status_file()