refactor(pubsub): Simplify synchronization primitives in PubSub message handling

- Replace `Lock` with standard `object` for synchronization
- Remove unnecessary blank lines in namespace declarations

Signed-off-by: Joe Brinkman <[email protected]>

Author: jbrinkman

sources/Valkey.Glide/PubSubMessageHandler.cs

@@ -1,6 +1,5 @@
 // Copyright Valkey GLIDE Project Contributors - SPDX Identifier: Apache-2.0
 
-
 namespace Valkey.Glide;
 
 /// <summary>
@@ -19,7 +18,7 @@ internal sealed class PubSubMessageHandler : IDisposable
     private readonly MessageCallback? _callback;
     private readonly object? _context;
     private readonly PubSubMessageQueue _queue;
-    private readonly Lock _lock = new();
+    private readonly object _lock = new();
     private volatile bool _disposed;
 
     /// <summary>

sources/Valkey.Glide/PubSubMessageQueue.cs

@@ -13,7 +13,7 @@ public sealed class PubSubMessageQueue : IDisposable
 {
     private readonly ConcurrentQueue<PubSubMessage> _messages;
     private readonly SemaphoreSlim _messageAvailable;
-    private readonly Lock _lock = new();
+    private readonly object _lock = new();
     private volatile bool _disposed;
 
     /// <summary>