fix: knowledge tool permission (#2501)

Author: dingfeli

crates/chat-cli/src/cli/chat/tools/knowledge.rs

@@ -13,6 +13,10 @@ use super::{
     InvokeOutput,
     OutputKind,
 };
+use crate::cli::agent::{
+    Agent,
+    PermissionEvalResult,
+};
 use crate::database::settings::Setting;
 use crate::os::Os;
 use crate::util::knowledge_store::KnowledgeStore;
@@ -472,6 +476,15 @@ impl Knowledge {
         })
     }
 
+    pub fn eval_perm(&self, agent: &Agent) -> PermissionEvalResult {
+        _ = self;
+        if agent.allowed_tools.contains("knowledge") {
+            PermissionEvalResult::Allow
+        } else {
+            PermissionEvalResult::Ask
+        }
+    }
+
     /// Format status data for display (UI rendering responsibility)
     fn format_status_display(status: &semantic_search_client::SystemStatus) -> String {
         let mut status_lines = Vec::new();

crates/chat-cli/src/cli/chat/tools/mod.rs

@@ -102,7 +102,7 @@ impl Tool {
             Tool::Custom(custom_tool) => custom_tool.eval_perm(agent),
             Tool::GhIssue(_) => PermissionEvalResult::Allow,
             Tool::Thinking(_) => PermissionEvalResult::Allow,
-            Tool::Knowledge(_) => PermissionEvalResult::Ask,
+            Tool::Knowledge(knowledge) => knowledge.eval_perm(agent),
         }
     }