Claude Code: Fix Docker path resolution in error_handlers.py - remove restrictive writable checks that fail at import time

error_handlers.py

@@ -76,31 +76,33 @@ def _get_base_dir() -> Path:
 
 def _resolve_status_file() -> Path:
     """Find cain_status.json by searching common locations."""
-    # Priority 1: /app/cain_status.json (Docker WORKDIR - primary location)
-    app_path = Path("/app/cain_status.json")
-    if app_path.parent.exists() and os.access(app_path.parent, os.W_OK):
-        return app_path
-
-    # Priority 2: CAIN_STATUS_PATH env var (direct override)
+    # Priority 1: CAIN_STATUS_PATH env var (set by app.py from OPENCLAW_DATA_DIR)
     env_path = os.environ.get('CAIN_STATUS_PATH')
     if env_path:
         return Path(env_path)
 
-    # Priority 3: OPENCLAW_DATA_DIR with writable check
-    base_dir = _get_base_dir()
-    candidate = base_dir / "cain_status.json"
-    if candidate.parent.exists() and os.access(candidate.parent, os.W_OK):
-        return candidate
+    # Priority 2: /app/cain_status.json (Docker WORKDIR - primary location)
+    # Don't check writability here - directory may not exist yet but can be created
+    app_path = Path("/app/cain_status.json")
+    if app_path.parent.exists():
+        return app_path
 
-    # Priority 4: Script directory as final fallback
+    # Priority 3: OPENCLAW_DATA_DIR
+    data_dir = os.environ.get('OPENCLAW_DATA_DIR')
+    if data_dir:
+        return Path(data_dir) / "cain_status.json"
+
+    # Priority 4: /data (standard Docker volume mount)
+    data_path = Path("/data/cain_status.json")
+    if data_path.parent.exists():
+        return data_path
+
+    # Priority 5: Script directory as final fallback
     script_dir = Path(os.path.dirname(__file__))
     return script_dir / "cain_status.json"
 
 STATUS_FILE = _resolve_status_file()
 
-# Ensure parent directory exists with proper permissions
-STATUS_FILE.parent.mkdir(parents=True, exist_ok=True)
-
 
 def handle_status_file_read() -> Dict[str, Any]:
     """