fix(copilot): bound tool span agent labels

Author: j15z

apps/sim/lib/copilot/request/metrics.test.ts

@@ -21,7 +21,7 @@ vi.mock('@opentelemetry/api', () => ({
 }))
 
 import { TraceAttr } from '@/lib/copilot/generated/trace-attributes-v1'
-import { recordSimToolMetric } from '@/lib/copilot/request/metrics'
+import { normalizeToolAgentId, recordSimToolMetric } from '@/lib/copilot/request/metrics'
 
 describe('recordSimToolMetric', () => {
   beforeEach(() => {
@@ -66,4 +66,13 @@ describe('recordSimToolMetric', () => {
       [TraceAttr.GenAiAgentName]: 'other',
     })
   })
+
+  it.each([
+    { agentId: 'main', expected: 'main' },
+    { agentId: 'workflow', expected: 'workflow' },
+    { agentId: 'tenant-defined-agent', expected: 'other' },
+    { agentId: '', expected: 'other' },
+  ])('normalizes $agentId to $expected for every telemetry signal', ({ agentId, expected }) => {
+    expect(normalizeToolAgentId(agentId)).toBe(expected)
+  })
 })

apps/sim/lib/copilot/request/metrics.ts

@@ -73,7 +73,7 @@ const REGISTERED_AGENT_IDS = new Set([
   ...Object.values(TOOL_CATALOG).flatMap(({ subagentId }) => (subagentId ? [subagentId] : [])),
 ])
 
-function cappedAgentId(agentId: string): string {
+export function normalizeToolAgentId(agentId: string): string {
   return REGISTERED_AGENT_IDS.has(agentId) ? agentId : 'other'
 }
 
@@ -91,7 +91,7 @@ export function recordSimToolMetric(
     [TraceAttr.ToolName]: cappedToolName(name),
     [TraceAttr.ToolExecutor]: 'sim',
     [TraceAttr.ToolOutcome]: outcome,
-    [TraceAttr.GenAiAgentName]: cappedAgentId(agentId),
+    [TraceAttr.GenAiAgentName]: normalizeToolAgentId(agentId),
   }
   toolCalls.add(1, attrs)
   if (durationMs >= 0) toolDuration.record(durationMs, attrs)

apps/sim/lib/copilot/request/otel.ts

@@ -22,6 +22,7 @@ import {
 import { TraceAttr } from '@/lib/copilot/generated/trace-attributes-v1'
 import { TraceSpan } from '@/lib/copilot/generated/trace-spans-v1'
 import { contextFromRequestHeaders } from '@/lib/copilot/request/go/propagation'
+import { normalizeToolAgentId } from '@/lib/copilot/request/metrics'
 import { isExplicitStopReason } from '@/lib/copilot/request/session/abort-reason'
 
 // OTel GenAI content-capture env var (spec:
@@ -300,7 +301,7 @@ export async function withCopilotToolSpan<T>(
         [TraceAttr.ToolName]: input.toolName,
         [TraceAttr.ToolCallId]: input.toolCallId,
         [TraceAttr.ToolExecutor]: 'sim',
-        [TraceAttr.GenAiAgentName]: input.agentName,
+        [TraceAttr.GenAiAgentName]: normalizeToolAgentId(input.agentName),
         ...(input.runId ? { [TraceAttr.RunId]: input.runId } : {}),
         ...(input.chatId ? { [TraceAttr.ChatId]: input.chatId } : {}),
         ...(typeof input.argsBytes === 'number'