Address GitHub Copilot review comments.

Signed-off-by: currantw <[email protected]>

Author: currantw

README.md

@@ -101,18 +101,19 @@ Console.WriteLine($"User: {user}");
 var passwordConfig = new StandaloneClientConfigurationBuilder()
     .WithAddress(host, port)
     .WithAuthentication("username", "password")
-    .WithTls(false)
+    .WithTls()
     .Build();
 
-using var client = await GlideClient.CreateClient(passwordConfig);
+using var passwordClient = await GlideClient.CreateClient(passwordConfig);
 
 // IAM authentication.
+var iamAuthConfig = new IamAuthConfig("my-cluster", ServiceType.ElastiCache, "us-east-1");
 var iamConfig = new ClusterClientConfigurationBuilder()
     .WithAddress(host, port)
-    .WithAuthentication("username", "my-cluster", ServiceType.ElastiCache, "us-east-1")
+    .WithAuthentication("username", iamAuthConfig)
     .Build();
 
-using var client = await GlideClient.CreateClient(iamConfig);
+using var iamClient = await GlideClient.CreateClient(iamConfig);
 ```
 
 ## Core API Examples

sources/Valkey.Glide/BaseClient.cs

@@ -45,8 +45,8 @@ public void Dispose()
     /// <returns>A task that completes when the refresh attempt finishes.</returns>
     public async Task RefreshIamTokenAsync()
     {
-        Message message = _messageContainer.GetMessageForCall();
-        RefreshIamTokenFfi(_clientPointer, (ulong)message.Index);
+        Message message = MessageContainer.GetMessageForCall();
+        RefreshIamTokenFfi(ClientPointer, (ulong)message.Index);
         IntPtr response = await message;
         try
         {

sources/Valkey.Glide/Internals/FFI.structs.cs

@@ -786,50 +786,50 @@ internal struct NodeAddress
     }
 
     [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Ansi)]
-    internal struct AuthenticationInfo(string? username, string? password, IamCredentials? iamCredentials)
+    internal readonly struct AuthenticationInfo(string? username, string? password, IamCredentials? iamCredentials)
     {
         /// <summary>
         /// Username for authentication.
         /// </summary>
         [MarshalAs(UnmanagedType.LPStr)]
-        public string? Username = username;
+        public readonly string? Username = username;
 
         /// <summary>
         /// Password for authentication.
         /// </summary>
         [MarshalAs(UnmanagedType.LPStr)]
-        public string? Password = password;
+        public readonly string? Password = password;
 
         /// <summary>
         /// IAM credentials for authentication.
         /// </summary>
-        public IamCredentials? IamCredentials = iamCredentials;
+        public readonly IamCredentials? IamCredentials = iamCredentials;
     }
 
     [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Ansi)]
-    internal struct IamCredentials(string clusterName, string region, ServiceType serviceType, uint? refreshIntervalSeconds)
+    internal readonly struct IamCredentials(string clusterName, string region, ServiceType serviceType, uint? refreshIntervalSeconds)
     {
         /// <summary>
         /// The name of the cluster for IAM authentication.
         /// </summary>
         [MarshalAs(UnmanagedType.LPStr)]
-        public string ClusterName = clusterName;
+        public readonly string ClusterName = clusterName;
 
         /// <summary>
         /// The AWS region for IAM authentication.
         /// </summary>
         [MarshalAs(UnmanagedType.LPStr)]
-        public string Region = region;
+        public readonly string Region = region;
 
         /// <summary>
         /// The AWS service type for IAM authentication.
         /// </summary>
-        public ServiceType ServiceType = serviceType;
+        public readonly ServiceType ServiceType = serviceType;
 
         /// <summary>
         /// The refresh interval in seconds for IAM authentication.
         /// </summary>
-        public uint? RefreshIntervalSeconds = refreshIntervalSeconds;
+        public readonly uint? RefreshIntervalSeconds = refreshIntervalSeconds;
     }
 
     internal enum ServiceType : uint

tests/Valkey.Glide.UnitTests/ConnectionConfigurationTests.cs

@@ -77,7 +77,7 @@ public void WithAuthentication_MultipleCalls_LastWins()
     {
         // Password-based authentication last.
         var builder = new StandaloneClientConfigurationBuilder();
-        var iamConfig = new IamAuthConfig(ClusterName, ServiceType.ElastiCache, Region, RefreshIntervalSeconds);
+        var iamConfig = new IamAuthConfig(ClusterName, ServiceType.MemoryDB, Region);
         builder.WithAuthentication(Username, iamConfig);
         builder.WithAuthentication(Username, Password);