fix(langchain): incorrect role on generations

Author: ianchi

packages/opentelemetry-instrumentation-langchain/opentelemetry/instrumentation/langchain/span_utils.py

@@ -211,11 +211,6 @@ def set_chat_response(span: Span, response: LLMResult) -> None:
     for generations in response.generations:
         for generation in generations:
             prefix = f"{GenAIAttributes.GEN_AI_COMPLETION}.{i}"
-            _set_span_attribute(
-                span,
-                f"{prefix}.role",
-                _message_type_to_role(generation.type),
-            )
 
             # Try to get content from various sources
             content = None
@@ -268,12 +263,19 @@ def set_chat_response(span: Span, response: LLMResult) -> None:
                     else generation.message.additional_kwargs.get("tool_calls")
                 )
                 if tool_calls and isinstance(tool_calls, list):
-                    _set_span_attribute(
-                        span,
-                        f"{prefix}.role",
-                        "assistant",
-                    )
                     _set_chat_tool_calls(span, prefix, tool_calls)
+
+                _set_span_attribute(
+                    span,
+                    f"{prefix}.role",
+                    _message_type_to_role(generation.message.type),
+                )
+            else:
+                _set_span_attribute(
+                    span,
+                    f"{prefix}.role",
+                    "assistant",
+                )
             i += 1