[Repo Assist] feat(tracing): add GenAIOperationName and GenAIAgentName to mcp.tool_call spans

[github-actions]

🤖 This PR was created by Repo Assist, an automated AI assistant.

Summary

Adds two missing [OTel GenAI semantic convention]((opentelemetry.io/redacted) attributes to StartToolCallSpan:

Attribute	Value	Why
gen_ai.operation.name	"execute_tool"	Primary grouping key in Honeycomb/Grafana Tempo for GenAI tool-call spans
gen_ai.agent.name	"mcp-gateway"	Identifies the gateway component in traces

Both constants (GenAIOperationName, GenAIAgentName) were already defined in internal/tracing/genai_attrs.go but never referenced — a gap flagged in the Go Fan module review (#7972, items 1 and 2).

Changes

internal/tracing/span_helpers.go — StartToolCallSpan:

// Before
GenAISystem.String("mcp"),
GenAIAgentID.String(serverID),
MCPMethod.String("tools/call"),
GenAIToolName.String(toolName),

// After
GenAISystem.String("mcp"),
GenAIOperationName.String("execute_tool"),  // ← new
GenAIAgentName.String("mcp-gateway"),       // ← new
GenAIAgentID.String(serverID),
MCPMethod.String("tools/call"),
GenAIToolName.String(toolName),

internal/tracing/span_helpers_test.go — TestStartToolCallSpan: two new hasAttr assertions for the added attributes.

Rationale

• gen_ai.operation.name is the standard grouping dimension for GenAI spans. Observability dashboards that follow OTel semconv use it as the primary filter/grouping field.
• gen_ai.agent.name completes the standard GenAI identity attributes (alongside the already-set gen_ai.agent.id) so traces are self-describing.
• No new dependencies, no behaviour change — purely additive attribute enrichment.

Test Status

Format check: gofmt ✅ — no formatting issues.

⚠️ Build/test: make agent-finished could not run locally — proxy.golang.org is blocked in the sandbox environment (infrastructure constraint). The CI pipeline will validate the build and tests. The change is a trivially correct attribute addition with no API changes.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• proxy.golang.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "proxy.golang.org"

See Network Configuration for more information.

Generated by Repo Assist · 188.5 AIC · ⊞ 12.2K · ◷
_{Comment /repo-assist to run again}

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@851905c06e905bf362a9f6cc54f912e3df747d55

[Copilot]

Pull request overview

This PR enriches mcp.tool_call OpenTelemetry spans with two missing GenAI semantic convention attributes so tool-call traces are easier to group and identify in downstream observability tools.

Changes:

• Add gen_ai.operation.name = "execute_tool" to StartToolCallSpan.
• Add gen_ai.agent.name = "mcp-gateway" to StartToolCallSpan.
• Extend TestStartToolCallSpan to assert both new attributes are present.

Show a summary per file

File	Description
internal/tracing/span_helpers.go	Adds GenAI operation + agent name attributes to the tool-call span attributes set.
internal/tracing/span_helpers_test.go	Adds assertions verifying the new span attributes are emitted.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 2/2 changed files
• Comments generated: 0