Refactor ChatEditor to track the current conversation ID and guard

message callbacks

* Added a new member `m_viewingConvId` to `ChatEditor` to keep the ID of
the conversation currently being displayed.
* Updated the `currentEditorChanged` signal handler to set
`m_viewingConvId` and use it instead of a local `convId`. The handler
now early‑returns when the signal originates from a different editor
instance.
* Updated all callbacks that receive a `Message` (`onMessageAppended`,
`onPendingMessageChanged`) and the follow‑up widget creation to ignore
messages whose `convId` does not match `m_viewingConvId`.

These changes ensure that each editor instance only processes messages
for its own conversation, preventing cross‑talk between editors when the
user switches tabs. This also makes the code clearer by explicitly
storing the conversation context.

Author: cristianadam

llamachateditor.cpp

@@ -156,24 +156,25 @@ ChatEditor::ChatEditor()
             &EditorManager::currentEditorChanged,
             this,
             [this](Core::IEditor *editor) {
-                if (editor == this) {
-                    const QString convId = QString::fromUtf8(m_document->contents());
-                    ViewingChat chat = ChatManager::instance().getViewingChat(convId);
-
-                    // A new conversation has one root message
-                    if (chat.messages.size() > 1) {
-                        refreshMessages(chat.messages, chat.conv.currNode);
-                    } else {
-                        ChatManager::instance().refreshServerProps();
-                    }
-
-                    ChatManager::instance().setCurrentConversation(convId);
-                    m_document->setPreferredDisplayName(chat.conv.name);
+                if (editor != this)
+                    return;
 
-                    EditorManager::instance()->updateWindowTitles();
+                m_viewingConvId = QString::fromUtf8(m_document->contents());
+                ViewingChat chat = ChatManager::instance().getViewingChat(m_viewingConvId);
 
-                    m_input->setFocus();
+                // A new conversation has one root message
+                if (chat.messages.size() > 1) {
+                    refreshMessages(chat.messages, chat.conv.currNode);
+                } else {
+                    ChatManager::instance().refreshServerProps();
                 }
+
+                ChatManager::instance().setCurrentConversation(m_viewingConvId);
+                m_document->setPreferredDisplayName(chat.conv.name);
+
+                EditorManager::instance()->updateWindowTitles();
+
+                m_input->setFocus();
             });
 
     // Make sure we have the markdown content before saving
@@ -288,6 +289,9 @@ void ChatEditor::createFollowUpWidget(const QString &convId,
                                       qint64 leafNodeId,
                                       const QStringList &questions)
 {
+    if (convId != m_viewingConvId)
+        return;
+
     if (m_followUpWidget) {
         m_followUpWidget->deleteLater();
         m_followUpWidget = nullptr;
@@ -426,6 +430,9 @@ void ChatEditor::refreshMessages(const QVector<Message> &messages, qint64 leafNo
 
 void ChatEditor::onMessageAppended(const Message &msg, qint64 pendingId)
 {
+    if (msg.convId != m_viewingConvId)
+        return;
+
     ViewingChat chat = ChatManager::instance().getViewingChat(msg.convId);
     if (pendingId < 0) {
         refreshMessages(chat.messages, msg.id);
@@ -513,6 +520,9 @@ void ChatEditor::onMessageAppended(const Message &msg, qint64 pendingId)
 
 void ChatEditor::onPendingMessageChanged(const Message &pm)
 {
+    if (pm.convId != m_viewingConvId)
+        return;
+
     ChatMessage *w = nullptr;
 
     auto it = std::find_if(m_messageWidgets.begin(),

llamachateditor.h

@@ -91,6 +91,7 @@ public slots:
     bool m_searchActive{false};
 
     SearchToolbar *m_searchToolbar{nullptr};
+    QString m_viewingConvId;
 };
 
 void setupChatEditor();