update by comment and also fix some warnings

Author: nytian

src/Abstractions/TaskOrchestrationContext.cs

@@ -68,7 +68,7 @@ public abstract class TaskOrchestrationContext
     /// <summary>
     /// Gets the configuration settings for the orchestration context.
     /// </summary>
-    public virtual IDictionary<string, object?> Properties { get; } = new Dictionary<string, object?>();
+    public virtual IReadOnlyDictionary<string, object?> Properties { get; } = new Dictionary<string, object?>();
 
     /// <summary>
     /// Gets the entity feature, for interacting with entities.

src/ScheduledTasks/Client/ScheduledTaskClient.cs

@@ -5,7 +5,7 @@ namespace Microsoft.DurableTask.ScheduledTasks;
 
 /// <summary>
 /// Client for managing scheduled tasks.
-/// Provides methods to retrieve a ScheduleClient, list all schedules, 
+/// Provides methods to retrieve a ScheduleClient, list all schedules,
 /// and get details of a specific schedule.
 /// </summary>
 public abstract class ScheduledTaskClient

src/ScheduledTasks/Models/ScheduleState.cs

@@ -50,4 +50,4 @@ public void RefreshScheduleRunExecutionToken()
     {
         this.ExecutionToken = Guid.NewGuid().ToString("N");
     }
-}
+}

src/Shared/AzureManaged/GrpcRetryPolicyDefaults.cs

@@ -39,7 +39,6 @@ sealed class GrpcRetryPolicyDefaults
     /// </remarks>
     public static readonly ServiceConfig DefaultServiceConfig;
 
-
     /// <summary>
     /// The default retry policy for gRPC operations.
     /// </summary>

src/Worker/Core/Shims/TaskOrchestrationContextWrapper.cs

@@ -53,16 +53,12 @@ public TaskOrchestrationContextWrapper(
         OrchestrationContext innerContext,
         OrchestrationInvocationContext invocationContext,
         object? deserializedInput,
-        IEnumerable<KeyValuePair<string, object?>> properties)
+        Dictionary<string, object?> properties)
     {
         this.innerContext = Check.NotNull(innerContext);
         this.invocationContext = Check.NotNull(invocationContext);
-        if (properties is null)
-        {
-            throw new ArgumentNullException(nameof(properties));
-        }
+        this.Properties = Check.NotNull(properties);
 
-        this.Properties = properties.ToDictionary(pair => pair.Key, pair => pair.Value);
         this.logger = this.CreateReplaySafeLogger("Microsoft.DurableTask");
         this.deserializedInput = deserializedInput;
     }
@@ -85,7 +81,7 @@ public TaskOrchestrationContextWrapper(
     /// <summary>
     /// Gets the configuration settings for the orchestration.
     /// </summary>
-    public override IDictionary<string, object?> Properties { get; } = new Dictionary<string, object?>();
+    public override IReadOnlyDictionary<string, object?> Properties { get; } = new Dictionary<string, object?>();
 
     /// <inheritdoc/>
     public override TaskOrchestrationEntityFeature Entities