fix(openai): streaming tool_call + logging multiple tool_call

[guillaumefontan]

Fixes #464 and #465

[CLAassistant]

All committers have signed the CLA.

[nirga]

Thanks! Left a small nit comment

[nirga]

nit: remove?

[guillaumefontan]

All done thanks!

[guillaumefontan]

@nirga just a note, this will break anyone's code which uses
span.attributes['${SpanAttributes.LLM_COMPLETIONS}.0.function_call.name'] (will return undefined)
as the new format will become
span.attributes['${SpanAttributes.LLM_COMPLETIONS}.0.function_call.0.name'].
To preserve backward compatibility, we could make it so that the first tool call remains
span.attributes['${SpanAttributes.LLM_COMPLETIONS}.0.function_call.name']
while all other tool calls are indexed:
span.attributes['${SpanAttributes.LLM_COMPLETIONS}.0.function_call.1.name'].
However this may be confusing.

[nirga]

@guillaumefontan I think we should match the way we've implemented this on Python -
https://github.com/traceloop/openllmetry/blob/a1b82d8b272ce3861d88f5a2898d3170aa9a6df0/packages/opentelemetry-instrumentation-openai/opentelemetry/instrumentation/openai/shared/chat_wrappers.py#L448

So it should be tool_calls (plural)

Also note the CI is broken on your PR

[guillaumefontan]

Hi @nirga I've updated to tool_calls and fixed the lint warnings. However the build error:
Run pip3 install chromadb error: externally-managed-environment
does not appear to be related to my changes. Let me know if there is anything I can do.

[nirga]

Sorry for the delay here, working on fixing the CI now. LGTM except for that!