Address PR review comments

- Fix JsonDocument leak: clone RootElement and dispose document
- Make _playwright/_browser nullable, null-check in DisposeAsync
- Close page after test to avoid resource leak
- Use using on TcpListener

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: bruno-garcia

test/Sentry.AspNetCore.Blazor.WebAssembly.PlaywrightTests/BlazorWasmTestApp.cs

@@ -73,7 +73,7 @@ public async Task StartAsync()
 
     private static int GetFreePort()
     {
-        var listener = new TcpListener(IPAddress.Loopback, 0);
+        using var listener = new TcpListener(IPAddress.Loopback, 0);
         listener.Start();
         var port = ((IPEndPoint)listener.LocalEndpoint).Port;
         listener.Stop();

test/Sentry.AspNetCore.Blazor.WebAssembly.PlaywrightTests/NavigationBreadcrumbTests.cs

@@ -6,8 +6,8 @@ namespace Sentry.AspNetCore.Blazor.WebAssembly.PlaywrightTests;
 public class NavigationBreadcrumbTests : IAsyncLifetime
 {
     private readonly BlazorWasmTestApp _app = new();
-    private IPlaywright _playwright = null!;
-    private IBrowser _browser = null!;
+    private IPlaywright? _playwright;
+    private IBrowser? _browser;
 
     public async Task InitializeAsync()
     {
@@ -29,15 +29,18 @@ public async Task InitializeAsync()
 
     public async Task DisposeAsync()
     {
-        await _browser.DisposeAsync();
-        _playwright.Dispose();
+        if (_browser != null)
+        {
+            await _browser.DisposeAsync();
+        }
+        _playwright?.Dispose();
         await _app.DisposeAsync();
     }
 
     [Fact]
     public async Task Navigation_CreatesBreadcrumbs_WithCorrectFromAndTo()
     {
-        var page = await _browser.NewPageAsync();
+        var page = await _browser!.NewPageAsync();
 
         // Collect all intercepted envelopes
         var envelopeReceived = new TaskCompletionSource<string>();
@@ -98,5 +101,7 @@ await route.FulfillAsync(new RouteFulfillOptions
         var second = navBreadcrumbs[1];
         second.GetProperty("data").GetProperty("from").GetString().Should().Be("/second");
         second.GetProperty("data").GetProperty("to").GetString().Should().Be("/trigger-capture");
+
+        await page.CloseAsync();
     }
 }

test/Sentry.AspNetCore.Blazor.WebAssembly.PlaywrightTests/SentryEnvelopeParser.cs

@@ -23,7 +23,8 @@ internal static class SentryEnvelopeParser
             if (itemHeader.TryGetProperty("type", out var typeEl) &&
                 typeEl.GetString() == "event")
             {
-                return JsonDocument.Parse(lines[i + 1]).RootElement;
+                using var eventDoc = JsonDocument.Parse(lines[i + 1]);
+                return eventDoc.RootElement.Clone();
             }
         }