fix(chat): hide tool context for ACP chats

Author: meijieru

lua/codecompanion/interactions/chat/tool_registry.lua

@@ -56,6 +56,11 @@ end
 ---@param opts? table Optional parameters for the context_item
 ---@return nil
 local function add_context(chat, id, opts)
+  opts = opts or {}
+  if chat.adapter and chat.adapter.type == "acp" and opts.visible == nil then
+    opts.visible = false
+  end
+
   chat.context:add({
     source = "tool",
     name = "tool",
@@ -121,7 +126,7 @@ end
 function ToolRegistry:add_single_tool(tool, opts)
   opts = opts or {}
   if opts.visible == nil then
-    opts.visible = true
+    opts.visible = self.chat.adapter and self.chat.adapter.type ~= "acp"
   end
 
   local tool_config = opts.config or config.interactions.chat.tools[tool]

tests/interactions/chat/tools/test_tool_registry.lua

@@ -60,6 +60,63 @@ T["ToolRegistry"][":add"]["renders tool in the chat buffer"] = function()
   h.expect_contains("func", content)
 end
 
+T["ToolRegistry"][":add"]["hides tool context for ACP chats"] = function()
+  child.lua([[
+    _G.chat = h.setup_chat_buffer({}, {
+      name = "test_acp",
+      config = {
+        name = "test_acp",
+        type = "acp",
+        roles = { user = "user", assistant = "assistant" },
+        handlers = {
+          form_messages = function()
+            return {}
+          end,
+        },
+      },
+    })
+    _G.chat.tool_registry:add("func")
+    _G.chat.context:render()
+    _G.buf_lines = h.get_buf_lines(_G.chat.bufnr)
+  ]])
+
+  local lines = child.lua_get([[_G.buf_lines]])
+  local content = table.concat(lines, "\n")
+
+  h.expect_tbl_contains("func", child.lua_get([[_G.chat.tool_registry.in_use]]))
+  h.eq(nil, content:find("func", 1, true))
+end
+
+T["ToolRegistry"][":add"]["hides tool group context for ACP chats"] = function()
+  child.lua([[
+    _G.chat = h.setup_chat_buffer({}, {
+      name = "test_acp",
+      config = {
+        name = "test_acp",
+        type = "acp",
+        roles = { user = "user", assistant = "assistant" },
+        handlers = {
+          form_messages = function()
+            return {}
+          end,
+        },
+      },
+    })
+    _G.chat.tool_registry:add("senior_dev")
+    _G.chat.context:render()
+    _G.buf_lines = h.get_buf_lines(_G.chat.bufnr)
+  ]])
+
+  local lines = child.lua_get([[_G.buf_lines]])
+  local content = table.concat(lines, "\n")
+
+  h.expect_tbl_contains("func", child.lua_get([[_G.chat.tool_registry.in_use]]))
+  h.expect_tbl_contains("cmd", child.lua_get([[_G.chat.tool_registry.in_use]]))
+  h.eq(nil, content:find("senior_dev", 1, true))
+  h.eq(nil, content:find("func", 1, true))
+  h.eq(nil, content:find("cmd", 1, true))
+end
+
 T["ToolRegistry"][":add"]["returns nil for unknown name"] = function()
   local result = child.lua_get([[_G.chat.tool_registry:add("nonexistent_tool_xyz")]])