Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions python/packages/mem0/agent_framework_mem0/_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,10 @@ async def invoking(self, messages: ChatMessage | MutableSequence[ChatMessage], *
messages_list = [messages] if isinstance(messages, ChatMessage) else list(messages)
input_text = "\n".join(msg.text for msg in messages_list if msg and msg.text and msg.text.strip())

# Validate input text is not empty before searching (possible for function approval responses)
if not input_text.strip():
return Context(messages=None)

search_response: MemorySearchResponse_v1_1 | MemorySearchResponse_v2 = await self.mem0_client.search( # type: ignore[misc]
query=input_text,
user_id=self.user_id,
Expand Down
26 changes: 26 additions & 0 deletions python/packages/mem0/tests/test_mem0_context_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -406,6 +406,32 @@ async def test_model_invoking_no_memories_returns_none_instructions(self, mock_m
assert isinstance(context, Context)
assert not context.messages

async def test_model_invoking_function_approval_response_returns_none_instructions(
self, mock_mem0_client: AsyncMock
) -> None:
"""Test invoking with function approval response content messages returns context with None instructions."""
from agent_framework import FunctionApprovalResponseContent, FunctionCallContent

provider = Mem0Provider(user_id="user123", mem0_client=mock_mem0_client)
function_call = FunctionCallContent(call_id="1", name="test_func", arguments='{"arg1": "value1"}')
message = ChatMessage(
role=Role.USER,
contents=[
FunctionApprovalResponseContent(
id="approval_1",
function_call=function_call,
approved=True,
)
],
)

mock_mem0_client.search.return_value = []

context = await provider.invoking(message)

assert isinstance(context, Context)
assert not context.messages

async def test_model_invoking_filters_empty_message_text(self, mock_mem0_client: AsyncMock) -> None:
"""Test that empty message text is filtered out from query."""
provider = Mem0Provider(user_id="user123", mem0_client=mock_mem0_client)
Expand Down
Loading