Fix comments

Author: torosent

src/Abstractions/TaskOptions.cs

@@ -100,7 +100,10 @@ public SubOrchestrationOptions(TaskOptions options, string? instanceId = null)
 /// The time when the orchestration instance should start executing. If not specified or if a date-time in the past
 /// is specified, the orchestration instance will be scheduled immediately.
 /// </param>
-/// <param name="Tags">
-/// A dictionary of tags to associate with the orchestration instance.
-/// </param>
-public record StartOrchestrationOptions(string? InstanceId = null, DateTimeOffset? StartAt = null, IDictionary<string, string>? Tags = null);
+public record StartOrchestrationOptions(string? InstanceId = null, DateTimeOffset? StartAt = null)
+{
+    /// <summary>
+    /// Gets or sets the tags to associate with the orchestration instance. Tags are key-value pairs that can be used.
+    /// </summary>
+    public IDictionary<string, string>? Tags { get; init; } = new Dictionary<string, string>();
+}

src/Client/Core/OrchestrationMetadata.cs

@@ -82,7 +82,7 @@ public OrchestrationMetadata(string name, string instanceId)
     /// <summary>
     /// Gets the tags associated with the orchestration instance.
     /// </summary>
-    public IDictionary<string, string>? Tags { get; init; }
+    public IReadOnlyDictionary<string, string> Tags { get; init; } = new Dictionary<string, string>();
 
     /// <summary>
     /// Gets the failure details, if any, for the orchestration instance.

src/Client/Grpc/GrpcDurableTaskClient.cs

@@ -97,7 +97,7 @@ public override async Task<string> ScheduleNewOrchestrationInstanceAsync(
         };
 
         // Add tags to the collection
-        if (options?.Tags != null)
+        if (request?.Tags != null && options?.Tags != null)
         {
             foreach (KeyValuePair<string, string> tag in options.Tags)
             {