From d2362a3f1edb197f6c627d0e4357159d06bbb5ed Mon Sep 17 00:00:00 2001
From: dmitriylaukhin <laukhin@dementiy.com>
Date: Tue, 19 May 2026 13:47:35 +0500
Subject: [PATCH] Fix QuestionSplitView error handling, loading state, file key

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../questions/QuestionSplitView.tsx           | 32 ++++++++++++++++---
 1 file changed, 27 insertions(+), 5 deletions(-)

diff --git a/src/components/questions/QuestionSplitView.tsx b/src/components/questions/QuestionSplitView.tsx
index 8676505..a52adc5 100644
--- a/src/components/questions/QuestionSplitView.tsx
+++ b/src/components/questions/QuestionSplitView.tsx
@@ -61,12 +61,24 @@ export function QuestionSplitView({ currentUserId }: { currentUserId: string })
   const [sending, setSending] = useState(false);
   const [closing, setClosing] = useState(false);
   const [error, setError] = useState("");
+  const [listError, setListError] = useState("");
+  const [listLoading, setListLoading] = useState(true);
 
   const fetchList = useCallback(async () => {
-    const res = await fetch("/api/questions");
-    if (res.ok) {
+    setListError("");
+    setListLoading(true);
+    try {
+      const res = await fetch("/api/questions");
+      if (!res.ok) {
+        setListError("Не удалось загрузить вопросы");
+        return;
+      }
       const data = await res.json();
       setQuestions(data);
+    } catch {
+      setListError("Не удалось загрузить вопросы");
+    } finally {
+      setListLoading(false);
     }
   }, []);
 
@@ -166,7 +178,17 @@ export function QuestionSplitView({ currentUserId }: { currentUserId: string })
 
         {/* Question list */}
         <div className="flex-1 overflow-y-auto">
-          {filtered.length === 0 && (
+          {listError && (
+            <p className="text-xs p-4" style={{ color: "var(--destructive)" }}>
+              {listError}
+            </p>
+          )}
+          {listLoading && (
+            <p className="text-xs p-4" style={{ color: "var(--muted-foreground)" }}>
+              Загрузка...
+            </p>
+          )}
+          {!listLoading && filtered.length === 0 && (
             <p className="text-xs p-4" style={{ color: "var(--muted-foreground)" }}>
               Нет вопросов
             </p>
@@ -270,9 +292,9 @@ export function QuestionSplitView({ currentUserId }: { currentUserId: string })
                     <p style={{ color: "var(--foreground)" }}>{msg.text}</p>
                     {msg.files && msg.files.length > 0 && (
                       <div className="flex flex-col gap-1 mt-2">
-                        {msg.files.map((f, i) => (
+                        {msg.files.map((f) => (
                           <a
-                            key={i}
+                            key={f.url}
                             href={f.url}
                             target="_blank"
                             rel="noopener noreferrer"