Merge pull request #484 from easyops-cn/steve/chat-panel

fix(): reset finished when send message again

Author: weareoutman

bricks/ai-portal/src/cruise-canvas/reducers/conversation.ts

@@ -26,6 +26,12 @@ export const conversation: Reducer<
     case "reset":
       return null;
 
+    case "started":
+      if (state) {
+        return { ...state, finished: false };
+      }
+      return state;
+
     case "finished":
       if (state) {
         return { ...state, finished: true };

bricks/ai-portal/src/cruise-canvas/reducers/interfaces.ts

@@ -16,6 +16,7 @@ export interface CruiseCanvasState {
 export type CruiseCanvasAction =
   | ConversationSSEAction
   | ConversationResetAction
+  | ConversationStartedAction
   | ConversationFinishedAction;
 
 export interface ConversationSSEAction {
@@ -28,6 +29,10 @@ export interface ConversationResetAction {
   type: "reset";
 }
 
+export interface ConversationStartedAction {
+  type: "started";
+}
+
 export interface ConversationFinishedAction {
   type: "finished";
 }

bricks/ai-portal/src/cruise-canvas/useConversationDetail.ts

@@ -104,6 +104,7 @@ export function useConversationDetail(
 
       requesting = true;
       ctrl = new AbortController();
+      dispatch({ type: "started" });
       try {
         const sseRequest = await (content === null && !action
           ? createSSEStream<ConversationPatch>(
@@ -157,13 +158,13 @@ export function useConversationDetail(
           dispatch({ type: "sse", payload: value, workspace: conversationId });
           isInitial = false;
         }
-        dispatch({ type: "finished" });
       } catch (e) {
         // eslint-disable-next-line no-console
         console.error("sse failed", e);
         handleHttpError(e);
       } finally {
         requesting = false;
+        dispatch({ type: "finished" });
       }
     };