fix: anthropic response data should include tool call id (#1636)

Author: mathislucka

logfire/_internal/integrations/llm_providers/anthropic.py

@@ -120,10 +120,11 @@ def on_response(response: ResponseT, span: LogfireSpan) -> ResponseT:
             elif block.type == 'tool_use':  # pragma: no branch
                 message.setdefault('tool_calls', []).append(
                     {
+                        'id': block.id,
                         'function': {
                             'arguments': block.model_dump_json(include={'input'}),
                             'name': block.name,
-                        }
+                        },
                     }
                 )
         span.set_attribute('response_data', {'message': message, 'usage': response.usage})

tests/otel_integrations/test_anthropic.py

@@ -545,7 +545,9 @@ def test_tool_messages(instrumented_client: anthropic.Anthropic, exporter: TestE
                     'response_data': {
                         'message': {
                             'role': 'assistant',
-                            'tool_calls': [{'function': {'arguments': '{"input":{"param":"param"}}', 'name': 'tool'}}],
+                            'tool_calls': [
+                                {'id': 'id', 'function': {'arguments': '{"input":{"param":"param"}}', 'name': 'tool'}}
+                            ],
                         },
                         'usage': IsPartialDict(
                             {