@gooddata/sdk-ui-gen-ai 11.35.0-alpha.6 → 11.35.0

package/esm/store/messages/messagesSlice.js

@@ -2,6 +2,7 @@
 import { createSlice } from "@reduxjs/toolkit";
 import { isAssistantMessage, isChatConversationLocalItem, isUserMessage, isVisualizationContents, makeErrorContent, makeErrorContents, } from "../../model.js";
 import { convertMessageToChatConversation } from "../sideEffects/utils.js";
+import { createEmptyConversation, getConversationData, getConversationLocalId, isConversationWithLocalId, } from "../utils.js";
 export const LS_VERBOSE_KEY = "gd-gen-ai-verbose";
 export const messagesSliceName = "messages";
 /**
@@ -16,17 +17,16 @@ const getInitialVerboseState = () => {
 };
 const initialState = {
     // Start with loading state to avoid re-render from empty state on startup
-    asyncProcess: "loading",
     loaded: false,
     verbose: getInitialVerboseState(),
     //old messages
+    messageAsyncProcess: "loading",
     messageOrder: [],
     messages: {},
     //conversations
     conversations: undefined,
     currentConversation: undefined,
-    conversationItems: {},
-    conversationItemsOrder: [],
+    conversationsData: {},
 };
 const setNormalizedMessages = (state, messages) => {
     state.messages = messages.reduce((acc, message) => {
@@ -41,16 +41,37 @@ const setNormalizedConversations = (state, conversations) => {
 };
 const setNormalizedConversation = (state, conversation, items = []) => {
     state.currentConversation = conversation;
-    state.conversationItems = items.reduce((acc, message) => {
+    state.loaded = true;
+    const data = getConversationData(state.conversationsData, conversation.localId);
+    const normalizedItems = items.reduce((acc, message) => {
         acc[message.localId] = message;
         return acc;
     }, {});
-    state.conversationItemsOrder = items.map((message) => message.localId);
-    state.loaded = true;
+    const normalizedOrder = items.map((message) => message.localId);
+    if (data) {
+        data.order = normalizedOrder;
+        data.items = normalizedItems;
+    }
+};
+const setConversationInProgress = (state, inProgress, localConversationId) => {
+    if (!localConversationId) {
+        return;
+    }
+    state.conversations = state.conversations?.map((conversation) => conversation.localId === localConversationId ? { ...conversation, inProgress } : conversation);
+    if (isConversationWithLocalId(state.currentConversation, localConversationId)) {
+        state.currentConversation = {
+            ...state.currentConversation,
+            inProgress,
+        };
+    }
 };
-const getAssistantMessageStrict = (state, assistantMessageId) => {
+const getAssistantMessageStrict = (state, assistantMessageId, conversationId) => {
     if (state.currentConversation) {
-        const message = state.conversationItems[assistantMessageId];
+        const data = getConversationData(state.conversationsData, conversationId);
+        if (!data) {
+            throw new Error(`Unexpected error during message evaluation.`);
+        }
+        const message = data.items[assistantMessageId];
         if (message.role !== "assistant") {
             throw new Error(`Unexpected error during message evaluation.`);
         }
@@ -64,9 +85,13 @@ const getAssistantMessageStrict = (state, assistantMessageId) => {
         return message;
     }
 };
-const getUserMessageStrict = (state, assistantMessageId) => {
+const getUserMessageStrict = (state, assistantMessageId, conversationId) => {
     if (state.currentConversation) {
-        const message = state.conversationItems[assistantMessageId];
+        const data = getConversationData(state.conversationsData, conversationId);
+        if (!data) {
+            throw new Error(`Unexpected error during message evaluation.`);
+        }
+        const message = data.items[assistantMessageId];
         if (message.role !== "user") {
             throw new Error(`Unexpected error during message evaluation.`);
         }
@@ -80,9 +105,13 @@ const getUserMessageStrict = (state, assistantMessageId) => {
         return message;
     }
 };
-const getMessageExists = (state, assistantMessageId) => {
+const getMessageExists = (state, assistantMessageId, conversationId) => {
     if (state.currentConversation) {
-        return !!state.conversationItems[assistantMessageId];
+        if (!conversationId) {
+            return false;
+        }
+        const data = getConversationData(state.conversationsData, conversationId);
+        return !!data?.items[assistantMessageId];
     }
     return !!state.messages[assistantMessageId];
 };
@@ -90,13 +119,17 @@ const getMessageExists = (state, assistantMessageId) => {
  * Get the user message before the assistant message, or undefined if there isn't one.
  * This is useful for dynamically created assistant messages that don't have a preceding user message.
  */
-const getUserMessageBeforeSafe = (state, assistantMessageId) => {
+const getUserMessageBeforeSafe = (state, assistantMessageId, conversationId) => {
     if (state.currentConversation) {
-        const messageIndex = state.conversationItemsOrder.indexOf(assistantMessageId);
+        const data = getConversationData(state.conversationsData, conversationId);
+        if (!data) {
+            return undefined;
+        }
+        const messageIndex = data.order.indexOf(assistantMessageId);
         if (messageIndex <= 0) {
             return undefined;
         }
-        const message = state.conversationItems[state.conversationItemsOrder[messageIndex - 1]];
+        const message = data.items[data.order[messageIndex - 1]];
         return message.role === "user" ? message : undefined;
     }
     const messageIndex = state.messageOrder.indexOf(assistantMessageId);
@@ -111,20 +144,36 @@ const messagesSlice = createSlice({
     initialState,
     reducers: {
         loadThreadAction: (state) => {
-            // do not override loading if new conversation landing is active
-            if (state.currentConversation === "new") {
-                return;
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+                if (data) {
+                    data.asyncProcess = "loading";
+                }
+            }
+            else {
+                state.messageAsyncProcess = "loading";
             }
-            state.asyncProcess = "loading";
         },
         loadThreadErrorAction: (state, { payload: { error } }) => {
             state.globalError = errorToObject(error);
-            delete state.asyncProcess;
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+                delete data?.asyncProcess;
+            }
+            else {
+                delete state.messageAsyncProcess;
+            }
         },
         loadThreadSuccessAction: (state, { payload: { messages, threadId } }) => {
             setNormalizedMessages(state, messages);
             state.threadId = threadId;
-            delete state.asyncProcess;
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+                delete data?.asyncProcess;
+            }
+            else {
+                delete state.messageAsyncProcess;
+            }
         },
         /**
          * Restore previously cached messages immediately while the backend is still loading.
@@ -135,14 +184,12 @@ const messagesSlice = createSlice({
          * - The input remains disabled (Input checks !!asyncProcess, and "restoring" is truthy).
          */
         restoreCachedMessagesAction: (state, { payload: { messages } }) => {
-            //TODO: s.hacker New needs to be changed
-            const normalized = messages.reduce((acc, message) => {
+            state.messages = messages.reduce((acc, message) => {
                 acc[message.localId] = message;
                 return acc;
             }, {});
-            state.messages = normalized;
             state.messageOrder = messages.map((message) => message.localId);
-            state.asyncProcess = "restoring";
+            state.messageAsyncProcess = "restoring";
         },
         loadConversationsSuccessAction: (state, { payload: { conversations }, }) => {
             setNormalizedConversations(state, conversations);
@@ -150,31 +197,56 @@ const messagesSlice = createSlice({
         loadConversationSuccessAction: (state, { payload: { currentConversation, conversationItems, threadId }, }) => {
             setNormalizedConversation(state, currentConversation, conversationItems);
             state.threadId = threadId;
-            delete state.asyncProcess;
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+                delete data?.asyncProcess;
+            }
+            else {
+                delete state.messageAsyncProcess;
+            }
         },
         clearThreadAction: (state) => {
-            state.asyncProcess = "clearing";
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+                if (data) {
+                    data.asyncProcess = "clearing";
+                }
+            }
+            else {
+                state.messageAsyncProcess = "clearing";
+            }
         },
         clearThreadErrorAction: (state, { payload: { error } }) => {
             state.globalError = errorToObject(error);
-            delete state.asyncProcess;
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+                delete data?.asyncProcess;
+            }
+            else {
+                delete state.messageAsyncProcess;
+            }
         },
         clearThreadSuccessAction: (state) => {
             state.messages = {};
             state.messageOrder = [];
             state.loaded = false;
-            delete state.asyncProcess;
             delete state.globalError;
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+                delete data?.asyncProcess;
+            }
+            else {
+                delete state.messageAsyncProcess;
+            }
         },
         clearConversationSuccessAction: (state, { payload: { conversation, threadId }, }) => {
             state.conversations = [conversation, ...(state.conversations ?? [])];
             state.currentConversation = conversation;
-            state.conversationItems = {};
-            state.conversationItemsOrder = [];
             state.threadId = threadId;
             state.loaded = false;
-            delete state.asyncProcess;
             delete state.globalError;
+            const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+            delete data?.asyncProcess;
         },
         /**
          * Add message to the stack
@@ -189,20 +261,21 @@ const messagesSlice = createSlice({
                     throw new Error("Working with conversation message but thread mode is active.");
                 }
                 const currentConversation = state.currentConversation;
-                if (currentConversation !== "new") {
-                    currentConversation.updatedAt = new Date().toISOString();
-                    state.conversations = state.conversations?.map((conversation) => {
-                        if (conversation.id === currentConversation.id) {
-                            return {
-                                ...conversation,
-                                updatedAt: currentConversation.updatedAt,
-                            };
-                        }
-                        return conversation;
-                    });
+                currentConversation.updatedAt = new Date().toISOString();
+                state.conversations = state.conversations?.map((conversation) => {
+                    if (conversation.localId === currentConversation.localId) {
+                        return {
+                            ...conversation,
+                            updatedAt: currentConversation.updatedAt,
+                        };
+                    }
+                    return conversation;
+                });
+                const data = getConversationData(state.conversationsData, currentConversation.localId);
+                if (data) {
+                    data.items[message.localId] = message;
+                    data.order.push(message.localId);
                 }
-                state.conversationItems[message.localId] = message;
-                state.conversationItemsOrder.push(message.localId);
             }
             else {
                 if (state.currentConversation) {
@@ -216,13 +289,17 @@ const messagesSlice = createSlice({
         /**
          * Start the message evaluation, adding new assistant message as an incomplete placeholder
          */
-        evaluateMessageAction: (state, { payload: { message }, }) => {
+        evaluateMessageAction: (state, { payload: { message, conversationId }, }) => {
             if (isChatConversationLocalItem(message)) {
-                if (!state.currentConversation) {
+                if (!conversationId) {
                     throw new Error("Working with conversation message but thread mode is active.");
                 }
-                state.conversationItems[message.localId] = message;
-                state.conversationItemsOrder.push(message.localId);
+                const data = getConversationData(state.conversationsData, conversationId);
+                if (data) {
+                    data.items[message.localId] = message;
+                    data.order.push(message.localId);
+                }
+                setConversationInProgress(state, true, conversationId);
             }
             else {
                 if (state.currentConversation) {
@@ -231,23 +308,33 @@ const messagesSlice = createSlice({
                 state.messages[message.localId] = message;
                 state.messageOrder.push(message.localId);
             }
-            state.asyncProcess = "evaluating";
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, conversationId);
+                if (data) {
+                    data.asyncProcess = "evaluating";
+                }
+            }
+            else {
+                state.messageAsyncProcess = "evaluating";
+            }
         },
         /**
          * The evaluation failed, need to update the assistant message.
          */
         evaluateMessageErrorAction: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, payload.conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 assistantMessage.complete = true;
                 assistantMessage.streaming = false;
                 assistantMessage.content = makeErrorContent(payload.error);
+                const data = getConversationData(state.conversationsData, payload.conversationId);
+                delete data?.asyncProcess;
             }
             else {
                 assistantMessage.complete = true;
                 assistantMessage.content.push(makeErrorContents(payload.error));
+                delete state.messageAsyncProcess;
             }
-            delete state.asyncProcess;
         },
         /**
          * Received new chunk from server over SSE.
@@ -255,12 +342,12 @@ const messagesSlice = createSlice({
         evaluateMessageStreamingAction: (state, { payload, }) => {
             //NOTE: During streaming a message, user can choose to close or reset a chat
             // and without this check, we would get an unwanted error
-            const exists = getMessageExists(state, payload.assistantMessageId);
+            const exists = getMessageExists(state, payload.assistantMessageId, payload.conversationId);
             if (!exists) {
                 return;
             }
             // Update assistant message
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, payload.conversationId);
             assistantMessage.id = payload.interactionId ?? assistantMessage.id;
             if (isChatConversationLocalItem(assistantMessage)) {
                 assistantMessage.content = payload.content ?? {
@@ -279,21 +366,20 @@ const messagesSlice = createSlice({
             // Also update the interaction id in the relevant user message (if one exists)
             // Note: dynamically created assistant messages (for multi-interaction streams)
             // may not have a preceding user message
-            const userMessage = getUserMessageBeforeSafe(state, payload.assistantMessageId);
+            const userMessage = getUserMessageBeforeSafe(state, payload.assistantMessageId, payload.conversationId);
             if (userMessage) {
                 userMessage.id = payload.interactionId ?? userMessage.id;
             }
         },
         evaluateConversationTitleAction: (state, { payload, }) => {
-            const conversation = state.conversations?.find((c) => c.id === payload.conversation.id);
+            const conversation = state.conversations?.find((c) => c.localId === payload.conversation.localId);
             if (!conversation) {
                 return;
             }
             conversation.generatingTitle = payload.generatingTitle;
             if (payload.title) {
                 conversation.title = payload.title;
-                if (state.currentConversation !== "new" &&
-                    state.currentConversation?.id === conversation.id) {
+                if (isConversationWithLocalId(state.currentConversation, conversation.localId)) {
                     state.currentConversation.title = payload.title;
                 }
             }
@@ -301,12 +387,12 @@ const messagesSlice = createSlice({
         evaluateMessageUpdateAction: (state, { payload, }) => {
             //NOTE: During streaming a message, user can choose to close or reset a chat
             // and without this check, we would get an unwanted error
-            const exists = getMessageExists(state, payload.userMessageId);
+            const exists = getMessageExists(state, payload.userMessageId, payload.conversationId);
             if (!exists) {
                 return;
             }
             // Update assistant message
-            const userMessage = getUserMessageStrict(state, payload.userMessageId);
+            const userMessage = getUserMessageStrict(state, payload.userMessageId, payload.conversationId);
             userMessage.id = payload.interactionId ?? userMessage.id;
             if (isChatConversationLocalItem(userMessage)) {
                 const message = payload.message;
@@ -320,12 +406,15 @@ const messagesSlice = createSlice({
             }
         },
         evaluateMessageCompleteAction: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, payload.conversationId);
             assistantMessage.complete = true;
-            delete state.asyncProcess;
+            setConversationInProgress(state, false, payload.conversationId);
+            const data = getConversationData(state.conversationsData, payload.conversationId);
+            delete data?.asyncProcess;
+            delete state.messageAsyncProcess;
         },
         evaluateMessageSuggestionsAction: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, payload.conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 assistantMessage.suggestions = payload.item;
             }
@@ -344,31 +433,23 @@ const messagesSlice = createSlice({
         },
         startNewConversationAction: (state) => {
             // keep list of conversations, but clear current conversation data in the view
-            state.currentConversation = "new";
-            state.conversationItems = {};
-            state.conversationItemsOrder = [];
+            state.currentConversation = createEmptyConversation();
             // mark as loaded to prevent auto-loading last conversation while on landing
             state.loaded = true;
-            delete state.asyncProcess;
         },
         setCurrentConversationAction: (state, { payload }) => {
-            const existing = state.conversations?.find((c) => c.id === payload.conversation.id);
+            const existing = state.conversations?.find((c) => c.localId === payload.conversation.localId);
             if (existing) {
-                //Same conversation as current, do nothing
-                if (state.currentConversation !== "new" && state.currentConversation?.id === existing.id) {
-                    return;
-                }
+                const data = getConversationData(state.conversationsData, existing.localId);
                 state.currentConversation = existing;
-                state.threadId = existing.id;
-                state.messages = {};
-                state.messageOrder = [];
-                state.loaded = false;
-                //TODO: Existing conversation
+                state.loaded = !!data?.order.length;
+                state.threadId = payload.conversation.id;
+                existing.id = payload.conversation.id;
             }
             else {
                 state.conversations = [payload.conversation, ...(state.conversations ?? [])];
                 state.currentConversation = payload.conversation;
-                state.threadId = payload.conversation.id;
+                state.threadId = payload.conversation.localId;
                 state.loaded = true;
             }
         },
@@ -376,10 +457,17 @@ const messagesSlice = createSlice({
             state.globalError = errorToObject(error);
         },
         cancelAsyncAction: (state) => {
-            delete state.asyncProcess;
+            if (state.currentConversation) {
+                const data = getConversationData(state.conversationsData, state.currentConversation.localId);
+                delete data?.asyncProcess;
+            }
+            else {
+                delete state.messageAsyncProcess;
+            }
         },
         setUserFeedback: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const conversationId = getConversationLocalId(state.currentConversation);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 const original = assistantMessage.feedback ?? {
                     type: "feedback",
@@ -397,7 +485,7 @@ const messagesSlice = createSlice({
         },
         setUserFeedbackError: (state, { payload, }) => {
             // Reset feedback to NONE and set error when submission fails
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, payload.conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 const original = assistantMessage.feedback ?? {
                     type: "feedback",
@@ -417,7 +505,7 @@ const messagesSlice = createSlice({
         },
         clearUserFeedbackError: (state, { payload, }) => {
             // Clear feedback error after showing toast
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, payload.conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 delete assistantMessage.feedback?.error;
             }
@@ -426,7 +514,8 @@ const messagesSlice = createSlice({
             }
         },
         saveVisualizationAction: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const conversationId = getConversationLocalId(state.currentConversation);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 if (assistantMessage.content.type !== "multipart") {
                     throw new Error("Unexpected message type");
@@ -453,7 +542,8 @@ const messagesSlice = createSlice({
             }
         },
         savedVisualizationAction: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const conversationId = getConversationLocalId(state.currentConversation);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 if (assistantMessage.content.type !== "multipart") {
                     throw new Error("Unexpected message type");
@@ -467,7 +557,7 @@ const messagesSlice = createSlice({
             }
         },
         saveVisualizationErrorAction: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, payload.conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 if (assistantMessage.content.type !== "multipart") {
                     throw new Error("Unexpected message type");
@@ -494,7 +584,7 @@ const messagesSlice = createSlice({
             }
         },
         saveVisualizationSuccessAction: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, payload.conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 if (assistantMessage.content.type !== "multipart") {
                     throw new Error("Unexpected message type");
@@ -523,7 +613,8 @@ const messagesSlice = createSlice({
         },
         saveVisualisationRenderStatusAction: (state, _action) => state,
         saveVisualisationRenderStatusSuccessAction: (state, { payload, }) => {
-            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId);
+            const conversationId = getConversationLocalId(state.currentConversation);
+            const assistantMessage = getAssistantMessageStrict(state, payload.assistantMessageId, conversationId);
             if (isChatConversationLocalItem(assistantMessage)) {
                 if (assistantMessage.content.type !== "multipart") {
                     throw new Error("Unexpected message type");
@@ -548,14 +639,13 @@ const messagesSlice = createSlice({
         visualizationErrorAction: (state, _action) => state,
         pinConversationAction: (state, _action) => state,
         pinConversationSuccessAction: (state, { payload }) => {
-            state.conversations = state.conversations?.map((conversation) => conversation.id === payload.conversationId
+            state.conversations = state.conversations?.map((conversation) => conversation.localId === payload.conversationId
                 ? {
                     ...conversation,
                     pinned: payload.pinned,
                 }
                 : conversation);
-            if (state.currentConversation !== "new" &&
-                state.currentConversation?.id === payload.conversationId) {
+            if (isConversationWithLocalId(state.currentConversation, payload.conversationId)) {
                 state.currentConversation = {
                     ...state.currentConversation,
                     pinned: payload.pinned,
@@ -563,14 +653,13 @@ const messagesSlice = createSlice({
             }
         },
         pinConversationFailureAction: (state, { payload }) => {
-            state.conversations = state.conversations?.map((conversation) => conversation.id === payload.conversationId
+            state.conversations = state.conversations?.map((conversation) => conversation.localId === payload.conversationId
                 ? {
                     ...conversation,
                     pinned: payload.pinned,
                 }
                 : conversation);
-            if (state.currentConversation !== "new" &&
-                state.currentConversation?.id === payload.conversationId) {
+            if (isConversationWithLocalId(state.currentConversation, payload.conversationId)) {
                 state.currentConversation = {
                     ...state.currentConversation,
                     pinned: payload.pinned,
@@ -579,14 +668,12 @@ const messagesSlice = createSlice({
         },
         deleteConversationAction: (state, _action) => state,
         deleteConversationStartAction: (state, { payload }) => {
-            if (state.currentConversation !== "new" &&
-                payload.conversationId === state.currentConversation?.id) {
+            if (isConversationWithLocalId(state.currentConversation, payload.conversationId)) {
                 // keep list of conversations, but clear current conversation data in the view
-                state.currentConversation = "new";
-                state.conversationItems = {};
-                state.conversationItemsOrder = [];
+                state.currentConversation = createEmptyConversation();
             }
-            state.conversations = state.conversations?.filter((conversation) => conversation.id !== payload.conversationId);
+            delete state.conversationsData[payload.conversationId];
+            state.conversations = state.conversations?.filter((conversation) => conversation.localId !== payload.conversationId);
         },
         deleteConversationSuccessAction: (state, _action) => state,
         deleteConversationFailureAction: (state, { payload }) => {

package/esm/store/sideEffects/index.js

@@ -23,7 +23,7 @@ import { onVisualizationSuccessSave } from "./onVisualizationSuccessSave.js";
 export function* rootSaga() {
     yield takeLatest(loadThreadAction.type, onThreadLoad);
     yield takeLatest(clearThreadAction.type, onThreadClear);
-    yield takeLatest(newMessageAction.type, onUserMessage);
+    yield takeEvery(newMessageAction.type, onUserMessage);
     //messages API
     yield takeEvery(setUserFeedback.type, onUserFeedback);
     yield takeEvery(saveVisualizationAction.type, onVisualizationSave);

package/esm/store/sideEffects/onConversationDelete.d.ts

@@ -20,5 +20,7 @@ export declare function onConversationDelete({ payload }: PayloadAction<{
     };
     type: "messages/deleteConversationFailureAction";
 }> | import("redux-saga/effects").SelectEffect, void, IAnalyticalBackend & string & import("@gooddata/sdk-backend-spi").IChatConversation & {
+    localId: string;
     generatingTitle?: boolean | undefined;
+    inProgress?: boolean | undefined;
 }>;

package/esm/store/sideEffects/onThreadLoad.js

@@ -5,6 +5,7 @@ import { settingsSelector } from "../chatWindow/chatWindowSelectors.js";
 import { loadMessages } from "../localStorage.js";
 import { conversationSelector, conversationsSelector } from "../messages/messagesSelectors.js";
 import { cancelAsyncAction, loadConversationSuccessAction, loadConversationsSuccessAction, loadThreadErrorAction, loadThreadSuccessAction, restoreCachedMessagesAction, } from "../messages/messagesSlice.js";
+import { createEmptyConversation } from "../utils.js";
 import { interactionsToMessages } from "./converters/interactionsToMessages.js";
 import { convertToLocalContent } from "./converters/toLocalContent.js";
 /**
@@ -124,7 +125,12 @@ function* fetchAllConversations() {
     if (cancelledConversations) {
         return;
     }
-    const conversationItems = resultsConversations.items;
+    const conversationItems = resultsConversations.items.map((item) => {
+        return {
+            ...item,
+            localId: item.id,
+        };
+    });
     yield put(loadConversationsSuccessAction({
         conversations: conversationItems,
     }));
@@ -136,16 +142,20 @@ function* fetchCurrentConversation() {
     const conversations = yield select(conversationsSelector);
     const conversation = yield select(conversationSelector);
     // New conversation selected
-    if (conversation === "new") {
+    if (conversation?.localId && !conversation.id) {
         yield put(loadConversationSuccessAction({
-            currentConversation: "new",
+            currentConversation: conversation,
             conversationItems: [],
         }));
         return;
     }
     const api = backend.workspace(workspace).genAI().getChatConversations({ isPreview });
+    //Sort by createion date, because normally there are pinned on top
+    const sortedConv = conversations
+        ?.slice()
+        .sort((a, b) => new Date(b.updatedAt).getTime() - new Date(a.updatedAt).getTime());
     //Load the first conversation if there are any or create a new one
-    const selectedConversation = conversation ?? conversations?.[0] ?? undefined;
+    const selectedConversation = conversation ?? sortedConv?.[0] ?? undefined;
     if (selectedConversation) {
         const preparedThread = api.getConversationThread(selectedConversation.id);
         const [resultsItems, cancelledItems] = yield race([call(fetchConversationHistory, preparedThread), take(cancelAsyncAction.type)]);
@@ -166,7 +176,7 @@ function* fetchCurrentConversation() {
     }
     else {
         yield put(loadConversationSuccessAction({
-            currentConversation: "new",
+            currentConversation: createEmptyConversation(),
             conversationItems: [],
         }));
     }