Make the tool name non-optional in ToolMessage (#43)

ToolCall has a required str property, and all our tools have a name,
thus I do not see a way to get a tool result (ToolMessage) that does
not have a name.

Author: mateusz834

splunklib/ai/engines/langchain.py

@@ -314,6 +314,11 @@ def _map_message_from_langchain(message: LC_BaseMessage) -> BaseMessage:
                 status=message.status,
             )
         case LC_ToolMessage():
+            # If this is reached, this likely means that we passed an invalid
+            # tool name to langchain.
+            assert message.name is not None, (
+                "langchain responded with a tool call that does not have a name"
+            )
             return ToolMessage(
                 name=message.name,
                 content=str(message.content),

splunklib/ai/messages.py

@@ -85,7 +85,7 @@ class ToolMessage(BaseMessage):
     """
 
     role: Literal["tool"] = "tool"
-    name: str | None = field(default=None)
+    name: str = field(default="")
     call_id: str = field(default="")
     status: Literal["success", "error"] = "success"