Default OTEL service name to agent name for multi-instance dashboards

When Telemetry.ServiceName is not explicitly overridden, it now defaults
to the Agent:Name value (e.g. "RockBot", "RockBot-Bob"). This lets
Grafana dashboards differentiate between instances without extra config.

If Telemetry__ServiceName is set explicitly in the ConfigMap it still
takes precedence — the default only applies when it equals "rockbot".

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: rockfordlhotka

src/RockBot.Agent/Program.cs

@@ -40,13 +40,21 @@
         builder.Configuration.AddJsonFile(Path.Combine(pvcBase, "appsettings.json"), optional: true, reloadOnChange: false);
 }
 
+var agentName = builder.Configuration["Agent:Name"] ?? "RockBot";
+
 builder.Services.AddRockBotRabbitMq(opts => builder.Configuration.GetSection("RabbitMq").Bind(opts));
 
 // OpenTelemetry — enabled via Telemetry:Enabled config key (set in k8s ConfigMap)
+// ServiceName defaults to the agent name so multi-instance deployments are
+// distinguishable in Grafana dashboards without extra config.
 if (builder.Configuration.GetValue<bool>("Telemetry:Enabled"))
 {
     builder.Services.AddRockBotTelemetry(opts =>
-        builder.Configuration.GetSection("Telemetry").Bind(opts));
+    {
+        builder.Configuration.GetSection("Telemetry").Bind(opts);
+        if (opts.ServiceName == "rockbot")
+            opts.ServiceName = agentName;
+    });
 }
 
 // ── LLM configuration — three-tier (Low / Balanced / High) ──────────────────
@@ -253,8 +261,6 @@ async Task<IChatClient> BuildClientForTierAsync(LlmTierConfig config, string tie
 builder.Services.AddSingleton<IToolSkillProvider, MemoryToolSkillProvider>();
 builder.Services.AddSingleton<IToolSkillProvider, SkillToolSkillProvider>();
 
-var agentName = builder.Configuration["Agent:Name"] ?? "RockBot";
-
 builder.Services.AddRockBotHost(agent =>
 {
     agent.WithIdentity(agentName);