fix(SDK + Agno): set gen_ai.tool.name on @tool() decorator and Agno tool spans (#3904)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: OzBenSimhonTraceloop

packages/opentelemetry-instrumentation-agno/opentelemetry/instrumentation/agno/_tool_wrappers.py

@@ -40,6 +40,7 @@ def __call__(self, wrapped, instance, args, kwargs):
                 span.set_attribute(SpanAttributes.TRACELOOP_SPAN_KIND,
                                    TraceloopSpanKindValues.TOOL.value)
                 span.set_attribute(SpanAttributes.TRACELOOP_ENTITY_NAME, function_name)
+                span.set_attribute(GenAIAttributes.GEN_AI_TOOL_NAME, function_name)
 
                 if hasattr(instance.function, 'description') and instance.function.description:
                     span.set_attribute("tool.description", instance.function.description)
@@ -109,6 +110,7 @@ async def __call__(self, wrapped, instance, args, kwargs):
                 span.set_attribute(SpanAttributes.TRACELOOP_SPAN_KIND,
                                    TraceloopSpanKindValues.TOOL.value)
                 span.set_attribute(SpanAttributes.TRACELOOP_ENTITY_NAME, function_name)
+                span.set_attribute(GenAIAttributes.GEN_AI_TOOL_NAME, function_name)
 
                 if hasattr(instance.function, 'description') and instance.function.description:
                     span.set_attribute("tool.description", instance.function.description)

packages/opentelemetry-instrumentation-agno/tests/test_agent.py

@@ -88,6 +88,7 @@ def add_numbers(a: int, b: int) -> int:
     # so we just check that if they exist, they have the right attributes
     for tool_span in tool_spans:
         assert tool_span.attributes.get(SpanAttributes.TRACELOOP_ENTITY_NAME) == "add_numbers"
+        assert tool_span.attributes.get(GenAIAttributes.GEN_AI_TOOL_NAME) == "add_numbers"
         assert tool_span.attributes.get(GenAIAttributes.GEN_AI_SYSTEM) == "agno"
 
 
@@ -179,6 +180,7 @@ def multiply(a: int, b: int) -> int:
     tool_spans = [s for s in spans if s.name == "multiply.tool"]
     for tool_span in tool_spans:
         assert tool_span.attributes.get(SpanAttributes.TRACELOOP_ENTITY_NAME) == "multiply"
+        assert tool_span.attributes.get(GenAIAttributes.GEN_AI_TOOL_NAME) == "multiply"
         assert tool_span.attributes.get(GenAIAttributes.GEN_AI_SYSTEM) == "agno"
 
 
@@ -244,4 +246,5 @@ def multiply(a: int, b: int) -> int:
     tool_spans = [s for s in spans if s.name == "multiply.tool"]
     for tool_span in tool_spans:
         assert tool_span.attributes.get(SpanAttributes.TRACELOOP_ENTITY_NAME) == "multiply"
+        assert tool_span.attributes.get(GenAIAttributes.GEN_AI_TOOL_NAME) == "multiply"
         assert tool_span.attributes.get(GenAIAttributes.GEN_AI_SYSTEM) == "agno"

packages/traceloop-sdk/traceloop/sdk/decorators/base.py

@@ -16,6 +16,9 @@
 from opentelemetry.trace.status import Status, StatusCode
 from opentelemetry import context as context_api
 from opentelemetry.semconv_ai import SpanAttributes, TraceloopSpanKindValues
+from opentelemetry.semconv._incubating.attributes.gen_ai_attributes import (
+    GEN_AI_TOOL_NAME,
+)
 
 from traceloop.sdk.tracing import get_tracer, set_workflow_name, set_agent_name
 from traceloop.sdk.tracing.tracing import (
@@ -141,7 +144,6 @@ def _setup_span(entity_name, tlp_span_kind, version):
         set_workflow_name(entity_name)
 
     if tlp_span_kind == TraceloopSpanKindValues.AGENT:
-        print(f"Setting agent name: {entity_name}")
         set_agent_name(entity_name)
 
     span_name = f"{entity_name}.{tlp_span_kind.value}"
@@ -160,6 +162,8 @@ def _setup_span(entity_name, tlp_span_kind, version):
 
         span.set_attribute(SpanAttributes.TRACELOOP_SPAN_KIND, tlp_span_kind.value)
         span.set_attribute(SpanAttributes.TRACELOOP_ENTITY_NAME, entity_name)
+        if tlp_span_kind == TraceloopSpanKindValues.TOOL:
+            span.set_attribute(GEN_AI_TOOL_NAME, entity_name)
         if version:
             span.set_attribute(SpanAttributes.TRACELOOP_ENTITY_VERSION, version)