Remove defunct AgentExtensions.WithOpenTelemetry (#1033)

dotnet/samples/GettingStarted/AgentOpenTelemetry/Program.cs

@@ -97,22 +97,23 @@ static async Task<string> GetWeatherAsync([Description("The location to get the
     return $"The weather in {location} is cloudy with a high of 15°C.";
 }
 
-// To ensure chat client's function calling is captured in the open telemetry, the chat client needs to have UseOpenTelemetry after UseFunctionInvocation
 using var instrumentedChatClient = new AzureOpenAIClient(new Uri(endpoint), new AzureCliCredential())
     .GetChatClient(deploymentName)
         .AsIChatClient() // Converts a native OpenAI SDK ChatClient into a Microsoft.Extensions.AI.IChatClient
         .AsBuilder()
         .UseFunctionInvocation()
-        .UseOpenTelemetry(sourceName: SourceName, configure: (cfg) => cfg.EnableSensitiveData = true)
+        .UseOpenTelemetry(sourceName: SourceName, configure: (cfg) => cfg.EnableSensitiveData = true) // enable telemetry at the chat client level
         .Build();
 
 appLogger.LogInformation("Creating Agent with OpenTelemetry instrumentation");
 // Create the agent with the instrumented chat client
-using var agent = new ChatClientAgent(instrumentedChatClient,
-            name: "OpenTelemetryDemoAgent",
-            instructions: "You are a helpful assistant that provides concise and informative responses.",
-            tools: [AIFunctionFactory.Create(GetWeatherAsync)])
-        .WithOpenTelemetry(SourceName); // Enable telemetry on the agent
+var agent = new ChatClientAgent(instrumentedChatClient,
+    name: "OpenTelemetryDemoAgent",
+    instructions: "You are a helpful assistant that provides concise and informative responses.",
+    tools: [AIFunctionFactory.Create(GetWeatherAsync)])
+    .AsBuilder()
+    .UseOpenTelemetry(SourceName) // enable telemetry at the agent level
+    .Build();
 
 var thread = agent.GetNewThread();
 

dotnet/samples/GettingStarted/Agents/Agent_Step08_Observability/Program.cs

@@ -25,12 +25,12 @@ using var tracerProvider = Sdk.CreateTracerProviderBuilder()
     .Build();
 
 // Create the agent, and enable OpenTelemetry instrumentation.
-AIAgent agent = new AzureOpenAIClient(
-    new Uri(endpoint),
-    new AzureCliCredential())
-     .GetChatClient(deploymentName)
-     .CreateAIAgent(JokerInstructions, JokerName)
-     .WithOpenTelemetry(sourceName: sourceName);
+AIAgent agent = new AzureOpenAIClient(new Uri(endpoint), new AzureCliCredential())
+    .GetChatClient(deploymentName)
+    .CreateAIAgent(JokerInstructions, JokerName)
+    .AsBuilder()
+    .UseOpenTelemetry(sourceName: sourceName)
+    .Build();
 
 // Invoke the agent and output the text result.
 Console.WriteLine(await agent.RunAsync("Tell me a joke about a pirate."));

dotnet/src/Microsoft.Agents.AI/AgentExtensions.cs

@@ -13,25 +13,6 @@ namespace Microsoft.Agents.AI;
 /// </summary>
 public static class AgentExtensions
 {
-    /// <summary>
-    /// Wraps the agent with OpenTelemetry instrumentation.
-    /// </summary>
-    /// <param name="agent">The agent to wrap.</param>
-    /// <param name="sourceName">An optional source name that will be used on the telemetry data.</param>
-    /// <param name="enableSensitiveData">When <see langword="true"/> indicates whether potentially sensitive information should be included in telemetry. Default is <see langword="false"/></param>
-    /// <returns>An <see cref="OpenTelemetryAgent"/> that wraps the original agent with telemetry.</returns>
-    public static OpenTelemetryAgent WithOpenTelemetry(this AIAgent agent, string? sourceName = null, bool? enableSensitiveData = null)
-    {
-        OpenTelemetryAgent otel = new(agent, sourceName);
-
-        if (enableSensitiveData is not null)
-        {
-            otel.EnableSensitiveData = enableSensitiveData.Value;
-        }
-
-        return otel;
-    }
-
     /// <summary>
     /// Creates a <see cref="AIFunction"/> that will invoke the provided Agent.
     /// </summary>