fix(ollama): tool maps and arguments (#738)

Author: sixlive

src/Providers/Ollama/Maps/MessageMap.php

@@ -62,6 +62,7 @@ protected function mapToolResultMessage(ToolResultMessage $message): void
         foreach ($message->toolResults as $toolResult) {
             $this->mappedMessages[] = [
                 'role' => 'tool',
+                'tool_name' => $toolResult->toolName,
                 'content' => is_string($toolResult->result)
                     ? $toolResult->result
                     : (json_encode($toolResult->result) ?: ''),
@@ -94,7 +95,7 @@ protected function mapAssistantMessage(AssistantMessage $message): void
             'tool_calls' => $message->toolCalls ? array_map(fn (ToolCall $toolCall): array => [
                 'function' => [
                     'name' => $toolCall->name,
-                    'arguments' => $toolCall->arguments(),
+                    'arguments' => (object) $toolCall->arguments(),
                 ],
             ], $message->toolCalls) : null,
         ]);

tests/Providers/Ollama/MessageMapTest.php

@@ -83,14 +83,14 @@
     $messageMap = new MessageMap([$assistantMessage]);
     $result = $messageMap->map();
 
-    expect($result)->toBe([
+    expect($result)->toEqual([
         [
             'role' => 'assistant',
             'content' => 'Assistant response',
             'tool_calls' => [[
                 'function' => [
                     'name' => 'search',
-                    'arguments' => [
+                    'arguments' => (object) [
                         'query' => 'What is Prism?',
                     ],
                 ],
@@ -114,6 +114,7 @@
     expect($result)->toBe([
         [
             'role' => 'tool',
+            'tool_name' => 'test-tool',
             'content' => 'Tool execution result',
         ],
     ]);
@@ -134,6 +135,7 @@
     expect($result)->toBe([
         [
             'role' => 'tool',
+            'tool_name' => 'test-tool',
             'content' => '{"key":"value"}',
         ],
     ]);