refactor: Change handling finished tools and thinking blocks

ChatView/ChatRootView.cpp

@@ -217,11 +217,6 @@ ChatRootView::ChatRootView(QQuickItem *parent)
         &Utils::BaseAspect::changed,
         this,
         &ChatRootView::isThinkingSupportChanged);
-    connect(
-        &Settings::toolsSettings().debugToolsAndThinkingComponent,
-        &Utils::BaseAspect::changed,
-        this,
-        &ChatRootView::isToolDebugging);
 }
 
 ChatModel *ChatRootView::chatModel() const
@@ -1131,9 +1126,5 @@ bool ChatRootView::isThinkingSupport() const
     return provider && provider->supportThinking();
 }
 
-bool ChatRootView::isToolDebugging() const
-{
-    return Settings::toolsSettings().debugToolsAndThinkingComponent();
-}
 
 } // namespace QodeAssist::Chat

ChatView/ChatRootView.hpp

@@ -58,7 +58,6 @@ class ChatRootView : public QQuickItem
     Q_PROPERTY(int currentMessagePendingEdits READ currentMessagePendingEdits NOTIFY currentMessageEditsStatsChanged FINAL)
     Q_PROPERTY(int currentMessageRejectedEdits READ currentMessageRejectedEdits NOTIFY currentMessageEditsStatsChanged FINAL)
     Q_PROPERTY(bool isThinkingSupport READ isThinkingSupport NOTIFY isThinkingSupportChanged FINAL)
-    Q_PROPERTY(bool isToolDebugging READ isToolDebugging NOTIFY isToolDebuggingChanged FINAL)
 
     QML_ELEMENT
 
@@ -143,8 +142,6 @@ public:
 
     bool isThinkingSupport() const;
 
-    bool isToolDebugging() const;
-
 public slots:
     void sendMessage(const QString &message);
     void copyToClipboard(const QString &text);
@@ -180,8 +177,6 @@ signals:
 
     void isThinkingSupportChanged();
 
-    void isToolDebuggingChanged();
-
 private:
     void updateFileEditStatus(const QString &editId, const QString &status);
     QString getChatsHistoryDir() const;

ChatView/qml/RootItem.qml

@@ -135,7 +135,7 @@ ChatRootView {
                 }
 
                 onLoaded: {
-                    if (componentLoader.sourceComponent == chatItemComponent && !root.isToolDebugging) {
+                    if (componentLoader.sourceComponent == chatItemComponent) {
                         chatListView.hideServiceComponents(index)
                     }
                 }
@@ -194,15 +194,11 @@ ChatRootView {
                     width: parent.width
                     toolContent: model.content
 
-                    FadeListItemAnimation{
-                        id: toolFadeAnimation
-                    }
-
                     Connections {
                         target: chatListView
                         function onHideServiceComponents(itemIndex) {
                             if (index !== itemIndex) {
-                                toolFadeAnimation.start()
+                                toolsItem.headerOpacity = 0.5
                             }
                         }
                     }
@@ -238,6 +234,8 @@ ChatRootView {
                 id: thinkingMessageComponent
 
                 ThinkingStatusItem {
+                    id: thinking
+
                     width: parent.width
                     thinkingContent: {
                         let content = model.content
@@ -249,15 +247,11 @@ ChatRootView {
                     }
                     isRedacted: model.isRedacted !== undefined ? model.isRedacted : false
 
-                    FadeListItemAnimation{
-                        id: thinkingFadeAnimation
-                    }
-
                     Connections {
                         target: chatListView
                         function onHideServiceComponents(itemIndex) {
                             if (index !== itemIndex) {
-                                thinkingFadeAnimation.start()
+                                thinking.headerOpacity = 0.5
                             }
                         }
                     }

ChatView/qml/ThinkingStatusItem.qml

@@ -28,6 +28,8 @@ Rectangle {
     property bool isRedacted: false
     property bool expanded: false
     
+    property alias headerOpacity: headerRow.opacity
+
     radius: 6
     color: palette.base
     clip: true

ChatView/qml/ToolStatusItem.qml

@@ -26,6 +26,8 @@ Rectangle {
     property string toolContent: ""
     property bool expanded: false
 
+    property alias headerOpacity: headerRow.opacity
+
     readonly property int firstNewline: toolContent.indexOf('\n')
     readonly property string toolName: firstNewline > 0 ? toolContent.substring(0, firstNewline) : toolContent
     readonly property string toolResult: firstNewline > 0 ? toolContent.substring(firstNewline + 1) : ""

settings/SettingsConstants.hpp

@@ -91,7 +91,6 @@ const char CA_ALLOW_FILE_SYSTEM_WRITE[] = "QodeAssist.caAllowFileSystemWrite";
 const char CA_ALLOW_ACCESS_OUTSIDE_PROJECT[] = "QodeAssist.caAllowAccessOutsideProject";
 const char CA_ENABLE_EDIT_FILE_TOOL[] = "QodeAssist.caEnableEditFileTool";
 const char CA_ENABLE_BUILD_PROJECT_TOOL[] = "QodeAssist.caEnableBuildProjectTool";
-const char CA_DEBUG_TOOLS_AND_THINKING_COMPONENT[] = "QodeAssist.caDebugToolsAndThinkingComponent";
 
 const char QODE_ASSIST_GENERAL_OPTIONS_ID[] = "QodeAssist.GeneralOptions";
 const char QODE_ASSIST_GENERAL_SETTINGS_PAGE_ID[] = "QodeAssist.1GeneralSettingsPageId";

settings/ToolsSettings.cpp

@@ -89,12 +89,6 @@ ToolsSettings::ToolsSettings()
                "project. This feature is under testing and may have unexpected behavior."));
     enableBuildProjectTool.setDefaultValue(false);
 
-    debugToolsAndThinkingComponent.setSettingsKey(Constants::CA_DEBUG_TOOLS_AND_THINKING_COMPONENT);
-    debugToolsAndThinkingComponent.setLabelText(Tr::tr("Always show Tools and Thinking Components in chat"));
-    debugToolsAndThinkingComponent.setToolTip(
-        Tr::tr("Disable disapearing tools and thinking component from chat"));
-    debugToolsAndThinkingComponent.setDefaultValue(false);
-
     resetToDefaults.m_buttonText = Tr::tr("Reset Page to Defaults");
 
     readSettings();
@@ -114,8 +108,7 @@ ToolsSettings::ToolsSettings()
                     Space{8},
                     allowFileSystemRead,
                     allowFileSystemWrite,
-                    allowAccessOutsideProject,
+                    allowAccessOutsideProject
-                    debugToolsAndThinkingComponent
                 }},
             Space{8},
             Group{
@@ -151,7 +144,6 @@ void ToolsSettings::resetSettingsToDefaults()
         resetAspect(autoApplyFileEdits);
         resetAspect(enableEditFileTool);
         resetAspect(enableBuildProjectTool);
-        resetAspect(debugToolsAndThinkingComponent);
         writeSettings();
     }
 }

settings/ToolsSettings.hpp

@@ -36,7 +36,6 @@ public:
     Utils::BoolAspect allowFileSystemRead{this};
     Utils::BoolAspect allowFileSystemWrite{this};
     Utils::BoolAspect allowAccessOutsideProject{this};
-    Utils::BoolAspect debugToolsAndThinkingComponent{this};
 
     // Experimental features
     Utils::BoolAspect enableEditFileTool{this};