From 219cea0c538fd4ecd05fddc920d50a5ddc4cc788 Mon Sep 17 00:00:00 2001 From: kangfenmao Date: Sun, 9 Mar 2025 21:23:40 +0800 Subject: [PATCH] fix(MessageTools): Improve empty tool responses check Use lodash's isEmpty for more robust null/undefined handling when checking MCP tool responses --- src/renderer/src/pages/home/Messages/MessageTools.tsx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/renderer/src/pages/home/Messages/MessageTools.tsx b/src/renderer/src/pages/home/Messages/MessageTools.tsx index 21dcce81..d5aad2e3 100644 --- a/src/renderer/src/pages/home/Messages/MessageTools.tsx +++ b/src/renderer/src/pages/home/Messages/MessageTools.tsx @@ -2,6 +2,7 @@ import { CheckOutlined, ExpandOutlined, LoadingOutlined } from '@ant-design/icon import { useSettings } from '@renderer/hooks/useSettings' import { MCPToolResponse, Message } from '@renderer/types' import { Collapse, message as antdMessage, Modal, Tooltip } from 'antd' +import { isEmpty } from 'lodash' import { FC, useMemo, useState } from 'react' import { useTranslation } from 'react-i18next' import styled from 'styled-components' @@ -24,7 +25,7 @@ const MessageTools: FC = ({ message }) => { const toolResponses = message.metadata?.mcpTools || [] - if (!toolResponses.length && !message.reasoning_content) { + if (isEmpty(toolResponses)) { return null }