🐛 Knowledge base summary failed. #1619

Author: jiangpeiling

backend/utils/document_vector_utils.py

@@ -425,7 +425,10 @@ def summarize_document(document_content: str, filename: str, language: str = LAN
             ]
 
             # Call LLM
-            response = llm(messages, max_tokens=max_words * 2)  # Allow more tokens for generation
+            response = llm(messages, max_tokens=max_words * 2)
+            # Allow more tokens for generation
+            if not response or not response.content:
+                return ""
             return response.content.strip()
         else:
             # Fallback to placeholder if no model configuration
@@ -504,6 +507,8 @@ def summarize_cluster(document_summaries: List[str], language: str = LANGUAGE["Z
 
             # Call LLM
             response = llm(messages, max_tokens=max_words * 2)  # Allow more tokens for generation
+            if not response or not response.content:
+                return ""
             return response.content.strip()
         else:
             # Fallback to placeholder if no model configuration

frontend/app/[locale]/setup/knowledges/components/document/DocumentList.tsx

@@ -245,15 +245,21 @@ const DocumentListContainer = forwardRef<DocumentListRef, DocumentListProps>(
       setSummary("");
 
       try {
-        await knowledgeBaseService.summaryIndex(
+        const result = await knowledgeBaseService.summaryIndex(
           knowledgeBaseName,
           1000,
           (newText) => {
             setSummary((prev) => prev + newText);
           },
           selectedModel
         );
-        message.success(t("document.summary.completed"));
+        // Only show success message if summary was actually generated
+        if (result && result.trim()) {
+          message.success(t("document.summary.completed"));
+        } else {
+          // If no summary was generated, show error message
+          message.error(t("knowledgeBase.summary.notGenerated"));
+        }
       } catch (error) {
         message.error(t("document.summary.error"));
         log.error(t("document.summary.error"), error);

frontend/public/locales/en/common.json

@@ -474,6 +474,7 @@
   "knowledgeBase.empty.description": "Please select a knowledge base from the list on the left, or create a new one",
   "knowledgeBase.error.createUpload": "Failed to create knowledge base or upload files:",
   "knowledgeBase.error.getSummary": "Failed to generate knowledge base summary:",
+  "knowledgeBase.summary.notGenerated": "Knowledge base summary was not generated, please change model configuration and retry",
   "knowledgeBase.name.new": "new_base",
   "knowledgeBase.message.getDocumentsFailed": "Failed to get documents",
 

frontend/public/locales/zh/common.json

@@ -475,6 +475,7 @@
   "knowledgeBase.empty.description": "请在左侧列表选择一个知识库，或创建新的知识库",
   "knowledgeBase.error.createUpload": "知识库创建或上传失败：",
   "knowledgeBase.error.getSummary": "获取知识库总结失败：",
+  "knowledgeBase.summary.notGenerated": "未生成知识库总结，请更换模型配置重试",
   "knowledgeBase.name.new": "新知识库",
   "knowledgeBase.message.getDocumentsFailed": "获取文档列表失败",
 

frontend/services/knowledgeBaseService.ts

@@ -1,5 +1,7 @@
 // Unified encapsulation of knowledge base related API calls
 
+import i18n from "i18next";
+
 import { API_ENDPOINTS } from "./api";
 
 import { NAME_CHECK_STATUS } from "@/const/agentConfig";
@@ -466,6 +468,19 @@ class KnowledgeBaseService {
                 if (onProgress) {
                   onProgress(data.message);
                 }
+              } else if (data.status === "completed") {
+                // Check if message field exists when status is completed
+                if (!data.message || data.message.trim() === "") {
+                  // No summary was generated, throw internationalized error
+                  const errorMessage = i18n.t(
+                    "knowledgeBase.summary.notGenerated"
+                  );
+                  throw new Error(errorMessage);
+                }
+                summary += data.message;
+                if (onProgress) {
+                  onProgress(data.message);
+                }
               } else if (data.status === "error") {
                 throw new Error(data.message);
               }