fix

Author: somarsai

crates/q_chat/src/tool_manager.rs

@@ -496,7 +496,11 @@ impl ToolManager {
         let tx = self.loading_status_sender.take();
         let display_task = self.loading_display_task.take();
         let tool_specs = {
-            let tool_specs = serde_json::from_str::<HashMap<String, ToolSpec>>(include_str!("tools/tool_index.json"))?;
+            let mut tool_specs = serde_json::from_str::<HashMap<String, ToolSpec>>(include_str!("tools/tool_index.json"))?;
+            if !crate::tools::think::Think::is_enabled() {
+                tool_specs.remove("q_think_tool");
+            }
+            
             Arc::new(Mutex::new(tool_specs))
         };
         let conversation_id = self.conversation_id.clone();
@@ -671,6 +675,7 @@ impl ToolManager {
             "execute_bash" => Tool::ExecuteBash(serde_json::from_value::<ExecuteBash>(value.args).map_err(map_err)?),
             "use_aws" => Tool::UseAws(serde_json::from_value::<UseAws>(value.args).map_err(map_err)?),
             "report_issue" => Tool::GhIssue(serde_json::from_value::<GhIssue>(value.args).map_err(map_err)?),
+            "q_think_tool" => Tool::Think(serde_json::from_value::<crate::tools::think::Think>(value.args).map_err(map_err)?),
             // Note that this name is namespaced with server_name{DELIMITER}tool_name
             name => {
                 let name = self.tn_map.get(name).map_or(name, String::as_str);

crates/q_chat/src/tools/mod.rs

@@ -55,7 +55,7 @@ impl Tool {
             Tool::UseAws(_) => "use_aws",
             Tool::Custom(custom_tool) => &custom_tool.name,
             Tool::GhIssue(_) => "gh_issue",
-            Tool::Think(_) => "model_think_tool",
+            Tool::Think(_) => "q_think_tool",
         }
         .to_owned()
     }
@@ -69,7 +69,7 @@ impl Tool {
             Tool::UseAws(use_aws) => use_aws.requires_acceptance(),
             Tool::Custom(_) => true,
             Tool::GhIssue(_) => false,
-            Tool::Think(_) => false,
+            Tool::Think(_) => true,
         }
     }
 
@@ -182,6 +182,7 @@ impl ToolPermissions {
             "execute_bash" => "trust read-only commands".dark_grey(),
             "use_aws" => "trust read-only commands".dark_grey(),
             "report_issue" => "trusted".dark_green().bold(),
+            "q_think_tool" => "trusted".dark_green().bold(),
             _ => "not trusted".dark_grey(),
         };
 

crates/q_chat/src/tools/think.rs

@@ -28,7 +28,7 @@ pub struct Think {
 
 impl Think {
     /// Checks if the thinking feature is enabled in settings
-    pub fn is_think_tool_enabled() -> bool {
+    pub fn is_enabled() -> bool {
         // Default to enabled if setting doesn't exist or can't be read
         settings::get_bool_or("chat.enableThinking", true)
     }

crates/q_chat/src/tools/tool_index.json

@@ -173,8 +173,8 @@
       "required": ["title"]
     }
   },
-  "think":{
-    "name": "think",
+  "q_think_tool":{
+    "name": "q_think_tool",
     "description": "The Think Tool is an internal reasoning mechanism enabling the model to systematically approach complex tasks by logically breaking them down before responding or acting; use it specifically for multi-step problems requiring step-by-step dependencies, reasoning through multiple constraints, synthesizing results from previous tool calls, planning intricate sequences of actions, troubleshooting complex errors, or making decisions involving multiple trade-offs. Avoid using it for straightforward tasks, basic information retrieval, summaries, always clearly define the reasoning challenge, structure thoughts explicitly, consider multiple perspectives, and summarize key insights before important decisions or complex tool interactions.",
     "input_schema": {
       "type": "object",