Remove option to initialize SentrySDK using AIOptions

Author: alexsohn1126

samples/Sentry.Samples.ME.AI.Console/Program.cs

@@ -14,26 +14,28 @@
     throw new InvalidOperationException($"Environment variable for OpenAI API key '{varName}' is not set.");
 }
 
+// Initialize Sentry SDK
+SentrySdk.Init(options =>
+{
+#if !SENTRY_DSN_DEFINED_IN_ENV
+    // A DSN is required. You can set here in code, or you can set it in the SENTRY_DSN environment variable.
+    // See https://docs.sentry.io/product/sentry-basics/dsn-explainer/
+    options.Dsn = SamplesShared.Dsn;
+#endif
+    options.Debug = true;
+    options.DiagnosticLevel = SentryLevel.Debug;
+    options.SampleRate = 1;
+    options.TracesSampleRate = 1;
+});
+
 // Create OpenAI API client and wrap it with Sentry instrumentation
 var openAiClient = new OpenAI.Chat.ChatClient("gpt-4o-mini", openAiApiKey)
     .AsIChatClient()
     .AddSentry(options =>
     {
-#if !SENTRY_DSN_DEFINED_IN_ENV
-        // A DSN is required. You can set here in code, or you can set it in the SENTRY_DSN environment variable.
-        // See https://docs.sentry.io/product/sentry-basics/dsn-explainer/
-        options.Dsn = SamplesShared.Dsn;
-#endif
-        options.Debug = true;
-        options.DiagnosticLevel = SentryLevel.Debug;
-        options.SampleRate = 1;
-        options.TracesSampleRate = 1;
-
         // AI-specific settings
         options.RecordInputs = true;
         options.RecordOutputs = true;
-        // Since this is a simple console app without Sentry already set up, we need to initialize our SDK
-        options.InitializeSdk = true;
     });
 
 var client = new ChatClientBuilder(openAiClient)

src/Sentry.Extensions.AI/SentryAIOptions.cs

@@ -3,8 +3,7 @@ namespace Sentry.Extensions.AI;
 /// <summary>
 /// Sentry AI instrumentation options
 /// </summary>
-/// <inheritdoc />
-public class SentryAIOptions : SentryOptions
+public class SentryAIOptions
 {
     /// <summary>
     /// Whether to include request messages in spans.
@@ -20,12 +19,4 @@ public class SentryAIOptions : SentryOptions
     /// Name of the AI Agent
     /// </summary>
     public string AgentName { get; set; } = "Agent";
-
-    /// <summary>
-    /// Whether to initialize the Sentry SDK through this integration.
-    /// </summary>
-    /// <remarks>
-    /// If you have already set up Sentry in your application, there is no need to re-initialize the Sentry SDK
-    /// </remarks>
-    public bool InitializeSdk { get; set; } = false;
 }

src/Sentry.Extensions.AI/SentryChatClient.cs

@@ -11,12 +11,6 @@ public SentryChatClient(IChatClient client, Action<SentryAIOptions>? configure =
     {
         _sentryAIOptions = new SentryAIOptions();
         configure?.Invoke(_sentryAIOptions);
-
-        // If user requested to initialize the SDK, and SDK is not enabled already, then use the options to init Sentry
-        if (_sentryAIOptions.InitializeSdk && !SentrySdk.IsEnabled)
-        {
-            SentrySdk.Init(_sentryAIOptions);
-        }
     }
 
     /// <inheritdoc cref="IChatClient"/>

test/Sentry.Extensions.AI.Tests/SentryAIOptionsTests.cs

@@ -13,7 +13,6 @@ public void Constructor_SetsDefaultValues()
         // Assert
         Assert.True(options.RecordInputs);
         Assert.True(options.RecordOutputs);
-        Assert.False(options.InitializeSdk);
     }
 
     [Fact]
@@ -44,69 +43,23 @@ public void IncludeResponseContent_CanBeSet()
         Assert.False(options.RecordOutputs);
     }
 
-    [Fact]
-    public void InitializeSdk_CanBeSet()
-    {
-        // Arrange
-        var options = new SentryAIOptions();
-
-        // Act
-        options.InitializeSdk = true;
-
-        // Assert
-        Assert.True(options.InitializeSdk);
-    }
-
-    [Fact]
-    public void InheritsFromSentryOptions()
-    {
-        // Arrange & Act
-        var options = new SentryAIOptions();
-
-        // Assert
-        Assert.IsType<SentryOptions>(options, exactMatch: false);
-    }
-
-    [Fact]
-    public void CanSetSentryOptionsProperties()
-    {
-        // Arrange
-        var options = new SentryAIOptions();
-
-        // Act
-        options.Dsn = "https://key@sentry.io/project";
-        options.Environment = "test";
-        options.Release = "1.0.0";
-
-        // Assert
-        Assert.Equal("https://key@sentry.io/project", options.Dsn);
-        Assert.Equal("test", options.Environment);
-        Assert.Equal("1.0.0", options.Release);
-    }
-
     [Theory]
-    [InlineData(true, true, true)]
-    [InlineData(true, true, false)]
-    [InlineData(true, false, true)]
-    [InlineData(true, false, false)]
-    [InlineData(false, true, true)]
-    [InlineData(false, true, false)]
-    [InlineData(false, false, true)]
-    [InlineData(false, false, false)]
-    public void AllPropertyCombinations_WorkCorrectly(bool includeRequest, bool includeResponse, bool initializeSdk)
+    [InlineData(true, true)]
+    [InlineData(true, false)]
+    [InlineData(false, true)]
+    [InlineData(false, false)]
+    public void AllPropertyCombinations_WorkCorrectly(bool includeRequest, bool includeResponse)
     {
         // Arrange
         var options = new SentryAIOptions
         {
             // Act
             RecordInputs = includeRequest,
-            RecordOutputs = includeResponse,
-            InitializeSdk = initializeSdk
+            RecordOutputs = includeResponse
         };
 
         // Assert
         Assert.Equal(includeRequest, options.RecordInputs);
         Assert.Equal(includeResponse, options.RecordOutputs);
-        Assert.Equal(initializeSdk, options.InitializeSdk);
     }
 }