impact-chatbot 2.3.84 → 2.3.86

package/dist/index.cjs.js

@@ -675,8 +675,11 @@ const handleMessageLike = async (question, liked, setLoadingState, displaySnackM
     try {
         let request;
         if (currentMode === "agent") {
+            // Use the session stored on this specific message. The live sessionId is
+            // reset once a flow completes / awaits follow-up, so relying on it would
+            // send an empty session_id or the session of a newer chat.
             const agentPayload = {
-                session_id: sessionId,
+                session_id: activeMessage?.chatSessionId || activeMessage?.sessionId || sessionId,
                 liked,
             };
             request = await likeDislikeCommentForAgent(agentPayload, baseUrl);
@@ -1439,31 +1442,50 @@ const ThinkinHeaderInfo = (props) => {
     });
     const { streamStartTime, isStreamCompleted, finalElapsedSeconds, thinkingHeaderMessage } = thinkingContext;
     const [elapsed, setElapsed] = React.useState(0);
+    const [frozenText, setFrozenText] = React.useState(null);
     const intervalRef = React.useRef(null);
     // Once this instance sees "completed", freeze it permanently so a future
     // chat's streamStartTime doesn't cause it to start ticking again.
     const frozenRef = React.useRef(false);
     const frozenTextRef = React.useRef(null);
-    const streamStartTimeRef = React.useRef(null); // Track streamStartTime changes to unfreeze
+    // The streamStartTime this instance latched onto. Used to detect when a
+    // different stream has taken over (e.g. the user asked a new question while
+    // this message's step form was still waiting for input).
+    const ownStartTimeRef = React.useRef(null);
+    /**
+     * Freezes this instance permanently on a final label and elapsed value.
+     */
+    const freeze = (text, seconds) => {
+        frozenRef.current = true;
+        frozenTextRef.current = text;
+        setFrozenText(text);
+        setElapsed(seconds);
+    };
     React.useEffect(() => {
         // If already frozen (stream completed), NEVER unfreeze.
         // This prevents a new chat's streamStartTime from restarting this timer.
         if (frozenRef.current)
             return;
-        // Track streamStartTime changes for non-frozen instances
-        if (streamStartTimeRef.current !== streamStartTime) {
-            streamStartTimeRef.current = streamStartTime;
+        // Adopt the first non-null streamStartTime we see as this instance's own
+        if (ownStartTimeRef.current === null && streamStartTime) {
+            ownStartTimeRef.current = streamStartTime;
         }
+        const ownStartTime = ownStartTimeRef.current;
         // Clear any existing interval
         if (intervalRef.current) {
             clearInterval(intervalRef.current);
             intervalRef.current = null;
         }
         if (isStreamCompleted && typeof finalElapsedSeconds === 'number') {
-            // Freeze this instance — it will never tick again
-            frozenRef.current = true;
-            frozenTextRef.current = `Completed in ${formatElapsedTime(finalElapsedSeconds)}`;
-            setElapsed(finalElapsedSeconds);
+            // Freeze this instance - it will never tick again
+            freeze(`Completed in ${formatElapsedTime(finalElapsedSeconds)}`, finalElapsedSeconds);
+        }
+        else if (ownStartTime !== null && streamStartTime !== ownStartTime) {
+            // A different stream has started (new question) while this one never
+            // completed - typically an abandoned step form. Freeze on this
+            // instance's OWN elapsed time instead of following the new stream.
+            const ownElapsed = Math.floor((Date.now() - ownStartTime) / 1000);
+            freeze(`Completed in ${formatElapsedTime(ownElapsed)}`, ownElapsed);
         }
         else if (streamStartTime && !isStreamCompleted) {
             // Live timer — update every second
@@ -1484,8 +1506,8 @@ const ThinkinHeaderInfo = (props) => {
     }, [streamStartTime, isStreamCompleted, finalElapsedSeconds]);
     // Determine display text
     let displayText = thinkingHeaderMessage || '';
-    if (frozenRef.current && frozenTextRef.current) {
-        displayText = frozenTextRef.current;
+    if (frozenText || (frozenRef.current && frozenTextRef.current)) {
+        displayText = frozenText || frozenTextRef.current;
     }
     else if (streamStartTime && !isStreamCompleted) {
         displayText = `Working for ${formatElapsedTime(elapsed)}`;
@@ -2479,6 +2501,20 @@ const useChatFlow = (chatDataRef, setLoader, setFlowType, setScreenName, setUser
         }
         if (data.flow_type === "agent" && data.actionType === "direct") {
             data.baseUrl = baseUrl;
+            // Reset the timer before the request goes out (same as index.jsx
+            // handleSendMessage). Without this the new ThinkinHeaderInfo mounts while
+            // the previous response's "completed" state is still in Redux and freezes
+            // itself on that stale elapsed time.
+            // streamStartTime stays null here: processStream sets it, so a question
+            // has exactly one start time (two would make the new header look like a
+            // superseded one and freeze it at 0m:00s).
+            dispatch(smartBotActions.setThinkingContext({
+                thinkingContent: "",
+                thinkingHeaderMessage: "Working for 0m:00s",
+                streamStartTime: null,
+                isStreamCompleted: false,
+                finalElapsedSeconds: null,
+            }));
             prepareDataAndSendToAgent(data, false, {
                 chatbotContext: chatbotContext,
                 setChatbotContext: smartBotActions.setChatbotContext,
@@ -5337,6 +5373,13 @@ const sseevent = (message, messageToStoreRef) => {
                 ? parsedData.chat_id
                 : "";
             messageToStoreRef.current.sessionId = parsedData?.session_id;
+            // Sticky copy of the session id this message belongs to. sessionId gets
+            // cleared on completed/follow-up so the next user message starts a fresh
+            // session, but message level actions (like/dislike) still need to refer
+            // to the session that actually produced this response.
+            if (parsedData?.session_id) {
+                messageToStoreRef.current.chatSessionId = parsedData.session_id;
+            }
         }
         if (messageToStoreRef.current.currentMode === "navigation" &&
             parsedData?.session_id) {
@@ -8685,7 +8728,7 @@ const StreamedContent = ({ botData, botProps }) => {
     const retryTimerRef = React.useRef(null); // Interval ref for countdown
     const retryCountdownRef = React.useRef(0); // Non-state countdown for interval callback
     const thinkingContentRef = React.useRef("");
-    reactRedux.useSelector((state) => {
+    const thinkingContext = reactRedux.useSelector((state) => {
         return state.smartBotReducer.thinkingContext;
     });
     const stepRef = React.useRef(_isAbortedRemount
@@ -8708,7 +8751,10 @@ const StreamedContent = ({ botData, botProps }) => {
     const thinkingStartTimeRef = React.useRef(null);
     const thinkingTimeFinalRef = React.useRef(0);
     const thinkingHeaderMessageRef = React.useRef("Working for 0m:00s");
-    const streamStartTimeRef = React.useRef(null);
+    // Inherit the in-flight start time so a remount (e.g. tab switch, where
+    // processStream is not re-run) keeps the original one instead of resetting.
+    // Null for a fresh question - processStream sets it.
+    const streamStartTimeRef = React.useRef(thinkingContext?.isStreamCompleted ? null : thinkingContext?.streamStartTime || null);
     const thinkingDoneRef = React.useRef(false);
     const thinkingStartedRef = React.useRef(false);
     const setThinkingContentRef = React.useRef(setThinkingContent); // Store current setThinkingContent function
@@ -9683,7 +9729,12 @@ const StreamedContent = ({ botData, botProps }) => {
                     },
                 };
                 const hasStepFormWidgets = !isEmpty(stepFormDataMapRef.current);
-                processResponse(response, botData.inputBody, currentMode, botData.utilityObject.customChatConfig, {
+                // response.session_id is intentionally blank once the flow completes, so
+                // keep the retained session separately and stamp it on the stored message.
+                const messageChatSessionId = messageToStoreRef.current.chatSessionId ||
+                    messageToStoreRef.current.sessionId ||
+                    "";
+                Promise.resolve(processResponse(response, botData.inputBody, currentMode, botData.utilityObject.customChatConfig, {
                     newChatData: chatDataInfoRef,
                     isTabEnabled: true,
                     steps: stepRef.current.map(s => ({ ...s })),
@@ -9691,7 +9742,15 @@ const StreamedContent = ({ botData, botProps }) => {
                     questions: [...questionsRef.current],
                     questionsStepsMap: { ...questionsStepsMapRef.current },
                     stepFormDataMap: { ...stepFormDataMapRef.current },
-                }, activeConversationId);
+                }, activeConversationId)).then(() => {
+                    if (!messageChatSessionId)
+                        return;
+                    const storedMessages = chatDataInfoRef.current?.[currentMode]?.conversations?.[activeConversationId]?.messages;
+                    if (storedMessages?.length) {
+                        storedMessages[storedMessages.length - 1].chatSessionId =
+                            messageChatSessionId;
+                    }
+                });
                 // [
                 //   {
                 //     header: "Finding relevant information",
@@ -15855,10 +15914,14 @@ const SmartBot = (props) => {
                     answerMode: answerMode,
                 };
                 // if(!isEmpty(userInput)) {
+                // Clear the previous response's completed state so the incoming
+                // ThinkinHeaderInfo doesn't freeze on it. streamStartTime stays null
+                // here: processStream sets it, so a question has exactly one start
+                // time (two would make the new header look like a superseded one).
                 dispatch(smartBotActions.setThinkingContext({
                     thinkingContent: "",
                     thinkingHeaderMessage: "Working for 0m:00s",
-                    streamStartTime: Date.now(),
+                    streamStartTime: null,
                     isStreamCompleted: false,
                     finalElapsedSeconds: null,
                 }));