From 7261bd8c3fab95e6ea628803ad20bcf5e97238a1 Mon Sep 17 00:00:00 2001 From: Tony Date: Fri, 13 Mar 2026 14:43:47 +0800 Subject: [PATCH] feat(feishu): display tool calls in code block messages - Tool hint messages with _tool_hint metadata now render as formatted code blocks - Uses Feishu interactive card message type with markdown code fences - Shows "Tool Call" header followed by code in a monospace block - Adds comprehensive unit tests for the new functionality Co-Authorship-Bot: Claude Opus 4.6 --- nanobot/channels/feishu.py | 20 ++++++++++++------- tests/test_feishu_tool_hint_code_block.py | 24 +++++++++++++---------- 2 files changed, 27 insertions(+), 17 deletions(-) diff --git a/nanobot/channels/feishu.py b/nanobot/channels/feishu.py index 2122d97..cfc3de0 100644 --- a/nanobot/channels/feishu.py +++ b/nanobot/channels/feishu.py @@ -822,18 +822,24 @@ class FeishuChannel(BaseChannel): receive_id_type = "chat_id" if msg.chat_id.startswith("oc_") else "open_id" loop = asyncio.get_running_loop() - # Handle tool hint messages as code blocks + # Handle tool hint messages as code blocks in interactive cards if msg.metadata.get("_tool_hint"): if msg.content and msg.content.strip(): - code_content = { - "title": "Tool Call", - "code": msg.content.strip(), - "language": "text" + # Create a simple card with a code block + code_text = msg.content.strip() + card = { + "config": {"wide_screen_mode": True}, + "elements": [ + { + "tag": "markdown", + "content": f"**Tool Call**\n\n```\n{code_text}\n```" + } + ] } await loop.run_in_executor( None, self._send_message_sync, - receive_id_type, msg.chat_id, "code", - json.dumps(code_content, ensure_ascii=False), + receive_id_type, msg.chat_id, "interactive", + json.dumps(card, ensure_ascii=False), ) return diff --git a/tests/test_feishu_tool_hint_code_block.py b/tests/test_feishu_tool_hint_code_block.py index c10c322..2c84060 100644 --- a/tests/test_feishu_tool_hint_code_block.py +++ b/tests/test_feishu_tool_hint_code_block.py @@ -24,7 +24,7 @@ def mock_feishu_channel(): def test_tool_hint_sends_code_message(mock_feishu_channel): - """Tool hint messages should be sent as code blocks.""" + """Tool hint messages should be sent as interactive cards with code blocks.""" msg = OutboundMessage( channel="feishu", chat_id="oc_123456", @@ -37,20 +37,23 @@ def test_tool_hint_sends_code_message(mock_feishu_channel): import asyncio asyncio.run(mock_feishu_channel.send(msg)) - # Verify code message was sent + # Verify interactive message with card was sent assert mock_send.call_count == 1 call_args = mock_send.call_args[0] receive_id_type, receive_id, msg_type, content = call_args assert receive_id_type == "chat_id" assert receive_id == "oc_123456" - assert msg_type == "code" + assert msg_type == "interactive" - # Parse content to verify structure - content_dict = json.loads(content) - assert content_dict["title"] == "Tool Call" - assert content_dict["code"] == 'web_search("test query")' - assert content_dict["language"] == "text" + # Parse content to verify card structure + card = json.loads(content) + assert card["config"]["wide_screen_mode"] is True + assert len(card["elements"]) == 1 + assert card["elements"][0]["tag"] == "markdown" + # Check that code block is properly formatted + expected_md = "**Tool Call**\n\n```\nweb_search(\"test query\")\n```" + assert card["elements"][0]["content"] == expected_md def test_tool_hint_empty_content_does_not_send(mock_feishu_channel): @@ -105,6 +108,7 @@ def test_tool_hint_multiple_tools_in_one_message(mock_feishu_channel): asyncio.run(mock_feishu_channel.send(msg)) call_args = mock_send.call_args[0] + msg_type = call_args[2] content = json.loads(call_args[3]) - assert content["code"] == 'web_search("query"), read_file("/path/to/file")' - assert "\n" not in content["code"] # Single line as intended + assert msg_type == "interactive" + assert "web_search(\"query\"), read_file(\"/path/to/file\")" in content["elements"][0]["content"]