Просмотр исходного кода

Refactor AiAssistantSidebar to remove unused threadData references and improve headerText logic

Shun Miyazawa 9 месяцев назад
Родитель
Сommit
68fc932a0d

+ 1 - 4
apps/app/src/features/openai/client/components/AiAssistant/AiAssistantSidebar/AiAssistantSidebar.tsx

@@ -360,13 +360,10 @@ const AiAssistantSidebarSubstance: React.FC<AiAssistantSidebarSubstanceProps> =
   }, [headerIconForEditorAssistant, headerIconForKnowledgeAssistant, isEditorAssistant]);
   }, [headerIconForEditorAssistant, headerIconForKnowledgeAssistant, isEditorAssistant]);
 
 
   const headerText = useMemo(() => {
   const headerText = useMemo(() => {
-    if (threadData?.title) {
-      return threadData.title;
-    }
     return isEditorAssistant
     return isEditorAssistant
       ? headerTextForEditorAssistant
       ? headerTextForEditorAssistant
       : headerTextForKnowledgeAssistant;
       : headerTextForKnowledgeAssistant;
-  }, [threadData?.title, isEditorAssistant, headerTextForEditorAssistant, headerTextForKnowledgeAssistant]);
+  }, [isEditorAssistant, headerTextForEditorAssistant, headerTextForKnowledgeAssistant]);
 
 
   const placeHolder = useMemo(() => {
   const placeHolder = useMemo(() => {
     if (form.formState.isSubmitting) {
     if (form.formState.isSubmitting) {

+ 3 - 8
apps/app/src/features/openai/client/services/knowledge-assistant.tsx

@@ -67,7 +67,7 @@ type UseKnowledgeAssistant = () => {
 export const useKnowledgeAssistant: UseKnowledgeAssistant = () => {
 export const useKnowledgeAssistant: UseKnowledgeAssistant = () => {
   // Hooks
   // Hooks
   const { data: aiAssistantSidebarData } = useAiAssistantSidebar();
   const { data: aiAssistantSidebarData } = useAiAssistantSidebar();
-  const { aiAssistantData } = aiAssistantSidebarData ?? {};
+  const { aiAssistantData, threadData } = aiAssistantSidebarData ?? {};
   const { mutate: mutateRecentThreads } = useSWRINFxRecentThreads();
   const { mutate: mutateRecentThreads } = useSWRINFxRecentThreads();
   const { trigger: mutateThreadData } = useSWRMUTxThreads(aiAssistantData?._id);
   const { trigger: mutateThreadData } = useSWRMUTxThreads(aiAssistantData?._id);
   const { t } = useTranslation();
   const { t } = useTranslation();
@@ -80,9 +80,6 @@ export const useKnowledgeAssistant: UseKnowledgeAssistant = () => {
     },
     },
   });
   });
 
 
-  // States
-  const [currentThreadTitle, setCurrentThreadTitle] = useState<string | undefined>(undefined);
-
   // Functions
   // Functions
   const resetForm = useCallback(() => {
   const resetForm = useCallback(() => {
     const summaryMode = form.getValues('summaryMode');
     const summaryMode = form.getValues('summaryMode');
@@ -98,8 +95,6 @@ export const useKnowledgeAssistant: UseKnowledgeAssistant = () => {
     });
     });
     const thread = response.data;
     const thread = response.data;
 
 
-    setCurrentThreadTitle(thread.title);
-
     // No need to await because data is not used
     // No need to await because data is not used
     mutateThreadData();
     mutateThreadData();
 
 
@@ -140,8 +135,8 @@ export const useKnowledgeAssistant: UseKnowledgeAssistant = () => {
   }, []);
   }, []);
 
 
   const headerText = useMemo(() => {
   const headerText = useMemo(() => {
-    return <>{currentThreadTitle ?? aiAssistantData?.name}</>;
-  }, [aiAssistantData?.name, currentThreadTitle]);
+    return <>{threadData?.title ?? aiAssistantData?.name}</>;
+  }, [aiAssistantData?.name, threadData?.title]);
 
 
   const placeHolder = useMemo(() => { return 'sidebar_ai_assistant.knowledge_assistant_placeholder' }, []);
   const placeHolder = useMemo(() => { return 'sidebar_ai_assistant.knowledge_assistant_placeholder' }, []);