Change ToolResult.content to str

mateusz834 · mateusz834 · commit dfbf85863b00 · 2026-04-16T10:54:55.000+02:00
diff --git a/splunklib/ai/engines/langchain.py b/splunklib/ai/engines/langchain.py
@@ -1354,10 +1354,7 @@ async def _tool_call(
                 "ToolException from LangChain should not be raised in tool.func"
             )
 
-        # TODO: Should we change the splunklib.ai.tools.ToolResult.content to a str, instead of list[str]?
-        text_content = "\n".join(result.content)
-
-        artifact = ToolResult(text_content, result.structured_content)
+        artifact = ToolResult(result.content, result.structured_content)
 
         if result.structured_content:
             # For both local tools and remote tools (Splunk MCP Server App), the primary
@@ -1371,7 +1368,7 @@ async def _tool_call(
             # this assumption may need to be revisited. For now, this approach is fine.
             # Worst-case scenario is the same information is provided to the LLM twice.
             return asdict(result), artifact  # both content + structured_content
-        return text_content, artifact
+        return result.content, artifact
 
     return StructuredTool(
         name=_normalize_tool_name(tool.name, tool.type),
diff --git a/splunklib/ai/tools.py b/splunklib/ai/tools.py
@@ -40,7 +40,7 @@ class ToolException(Exception):
 
 @dataclass(frozen=True)
 class ToolResult:
-    content: list[str]
+    content: str
     structured_content: dict[str, Any] | None
 
 
@@ -243,7 +243,7 @@ def _convert_tool_result(
             text_contents.append(content.text)
 
     return ToolResult(
-        content=text_contents, structured_content=result.structuredContent
+        content="\n".join(text_contents), structured_content=result.structuredContent
     )
 
 
diff --git a/tests/unit/ai/test_tool_settings.py b/tests/unit/ai/test_tool_settings.py
@@ -7,7 +7,7 @@
 
 
 async def no_op() -> ToolResult:
-    return ToolResult(content=[], structured_content={})
+    return ToolResult(content="", structured_content={})
 
 
 LOCAL_TOOL_1 = Tool(
