Update

database.py

@@ -3,54 +3,66 @@
 """
 Модуль для работы с файловой базой данных (JSON) для хранения истории эмоций.
 
-Ключевые особенности:
-- Адаптация под Hugging Face Spaces: пути к файлу БД и файлу блокировки
-  разделены для решения проблем с правами доступа.
-- Отказоустойчивость: код автоматически создает необходимую директорию для
-  базы данных при первом запуске, предотвращая `FileNotFoundError`.
-- Потокобезопасность: используется `filelock` для предотвращения гонки
-  состояний при одновременной записи.
+ФИНАЛЬНАЯ ВЕРСИЯ: Решает проблему `PermissionError` на платформе
+Hugging Face Spaces, корректно обрабатывая права на примонтированную
+директорию `/data`.
+
+Принцип работы:
+1. При запуске код проверяет, существует ли директория `/data` и доступна ли
+   она для записи.
+2. Если да, то база данных будет размещена в ней (постоянное хранилище).
+3. Если нет (например, Persistent Storage не подключен), используется
+   временная директория /tmp, чтобы избежать падения приложения.
+4. В функции сохранения `save_emotion` более не делается попытка создать
+   саму директорию `/data`, что и было причиной ошибки.
 """
 import json
 import os
+import logging
 from datetime import datetime
 from typing import Any, Dict, List, Optional
 from filelock import FileLock
 
-# --- Конфигурация путей к файлам ---
+logger = logging.getLogger(__name__)
+
+# --- Динамическая конфигурация путей к файлам ---
+
+DB_FILE = "history.json"
+LOCK_FILE = "history.json.lock"
 
 # Проверяем, запущено ли приложение на Hugging Face Spaces.
 if os.environ.get('SPACE_ID'):
-    # На сервере HF Spaces:
-    # 1. Файл базы данных (постоянный) храним в /data для персистентности.
-    DB_FILE = "/data/history.json"
-    # 2. Файл блокировки (временный) храним в /tmp, где всегда есть права на запись.
-    LOCK_FILE = "/tmp/history.json.lock"
-else:
-    # При локальном запуске оба файла находятся в текущей директории.
-    DB_FILE = "history.json"
-    LOCK_FILE = "history.json.lock"
+    persistent_storage_path = "/data"
+    
+    # Проверяем, существует ли путь и есть ли у нас права на запись в него.
+    if os.path.exists(persistent_storage_path) and os.access(persistent_storage_path, os.W_OK):
+        # Если да, используем постоянное хранилище.
+        DB_FILE = os.path.join(persistent_storage_path, "history.json")
+        LOCK_FILE = "/tmp/history.json.lock"
+        logger.info(f"Обнаружено доступное постоянное хранилище. База данных: {DB_FILE}")
+    else:
+        # Если нет, выводим предупреждение и используем временное хранилище.
+        logger.warning("Постоянное хранилище в /data не найдено или недоступно. "
+                       "История будет теряться при перезапусках.")
+        DB_FILE = "/tmp/history.json"
+        LOCK_FILE = "/tmp/history.json.lock"
 
 
 def _read_data() -> Dict[str, Any]:
-    """
-    Вспомогательная функция для безопасного чтения данных из JSON-файла.
-    Возвращает пустой словарь, если файл не существует, пуст или поврежден.
-    """
-    if not os.path.exists(DB_FILE) or os.path.getsize(DB_FILE) == 0:
+    """Вспомогательная функция для безопасного чтения данных из JSON-файла."""
+    if not os.path.exists(DB_FILE):
         return {}
     try:
-        with open(DB_FILE, "r", encoding="utf-8") as f:
-            return json.load(f)
+        if os.path.getsize(DB_FILE) > 0:
+            with open(DB_FILE, "r", encoding="utf-8") as f:
+                return json.load(f)
     except (json.JSONDecodeError, FileNotFoundError):
         return {}
+    return {}
 
 
 def save_emotion(user_id: int, emotion: str) -> None:
-    """
-    Сохраняет запись об эмоции пользовател�� в базу данных.
-    Ограничивает максимальный размер истории для одного пользователя.
-    """
+    """Сохраняет запись об эмоции пользователя в базу данных."""
     lock = FileLock(LOCK_FILE, timeout=10)
     with lock:
         data = _read_data()
@@ -65,37 +77,23 @@ def save_emotion(user_id: int, emotion: str) -> None:
         }
         data[user_id_str].append(record)
 
-        # Ограничиваем историю, чтобы файл не разрастался бесконечно.
         MAX_HISTORY_PER_USER = 200
         if len(data[user_id_str]) > MAX_HISTORY_PER_USER:
             data[user_id_str] = data[user_id_str][-MAX_HISTORY_PER_USER:]
 
-        # --- ИСПРАВЛЕНИЕ: Гарантируем, что директория для файла БД существует ---
-        # `open()` в режиме 'w' не создает родительские директории,
-        # что приводит к FileNotFoundError при первом запуске на "чистой" системе.
-        db_directory = os.path.dirname(DB_FILE)
-        # os.makedirs создаст директорию, только если ее нет (благодаря exist_ok=True).
-        # Проверяем, что путь к директории не пустой (для локального запуска).
-        if db_directory:
-            os.makedirs(db_directory, exist_ok=True)
-        # --- КОНЕЦ ИСПРАВЛЕНИЯ ---
-
-        with open(DB_FILE, "w", encoding="utf-8") as f:
-            json.dump(data, f, ensure_ascii=False, indent=4)
+        try:
+            # ИСПРАВЛЕНИЕ: Мы больше не пытаемся создать саму директорию /data.
+            # Наша логика при запуске уже подтвердила, что она существует
+            # и доступна. Мы можем сразу открывать файл для записи.
+            # Функция open() сама создаст файл history.json, если его нет.
+            with open(DB_FILE, "w", encoding="utf-8") as f:
+                json.dump(data, f, ensure_ascii=False, indent=4)
+        except Exception as e:
+            logger.error(f"КРИТИЧЕСКАЯ ОШИБКА: Не удалось сохранить данные в файл {DB_FILE}: {e}", exc_info=True)
 
 
 def get_user_history(user_id: int, limit: Optional[int] = 10) -> List[Dict[str, str]]:
-    """
-    Возвращает историю эмоций пользователя из базы данных.
-
-    Args:
-        user_id: ID пользователя в Telegram.
-        limit: Максимальное количество последних записей. Если None,
-               возвращается вся история пользователя.
-
-    Returns:
-        Список словарей с записями об эмоциях.
-    """
+    """Возвращает историю эмоций пользователя из базы данных."""
     lock = FileLock(LOCK_FILE, timeout=10)
     with lock:
         data = _read_data()