fix: prevent users from setting a extremly short history that resulti…

…ng in no content being sent for the title summary
ChatGPTNextWeb · Sep 13, 2024 · a5679d4 · a5679d4
1 parent 1b869d9
commit a5679d4
Showing 1 changed file with 14 additions and 5 deletions.
diff --git a/app/store/chat.ts b/app/store/chat.ts
@@ -569,12 +569,21 @@ export const useChatStore = createPersistStore(
           session.topic === DEFAULT_TOPIC &&
           countMessages(messages) >= SUMMARIZE_MIN_LEN
         ) {
-          const topicMessages = messages.concat(
-            createMessage({
-              role: "user",
-              content: Locale.Store.Prompt.Topic,
-            }),
+          const startIndex = Math.max(
+            0,
+            messages.length - modelConfig.historyMessageCount,
           );
+          const topicMessages = messages
+            .slice(
+              startIndex < messages.length ? startIndex : messages.length - 1,
+              messages.length,
+            )
+            .concat(
+              createMessage({
+                role: "user",
+                content: Locale.Store.Prompt.Topic,
+              }),
+            );
           api.llm.chat({
             messages: topicMessages,
             config: {