Fix dotnet format issues: add UTF-8 BOM and remove unused using

kshyju · Copilot · kshyju · commit ecc41ea8e8ed · 2026-03-03T17:16:01.000-08:00
- Add UTF-8 BOM to 20 .cs files across DurableTask, AzureFunctions,
  unit tests, and workflow samples
- Remove unnecessary using directive in 07_SubWorkflows/Executors.cs

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/dotnet/samples/04-hosting/DurableWorkflows/ConsoleApps/07_SubWorkflows/Executors.cs b/dotnet/samples/04-hosting/DurableWorkflows/ConsoleApps/07_SubWorkflows/Executors.cs
@@ -1,7 +1,6 @@
 ﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.Workflows;
-using Microsoft.Agents.AI.Workflows.Checkpointing;
 
 namespace SubWorkflows;
 
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/DurableOptions.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/DurableOptions.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using System.Diagnostics;
 using Microsoft.Agents.AI.DurableTask.Workflows;
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Logs.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Logs.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.DurableTask;
 using Microsoft.Extensions.AI;
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/ServiceCollectionExtensions.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/ServiceCollectionExtensions.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.DurableTask.Workflows;
 using Microsoft.Agents.AI.Workflows;
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableExecutorDispatcher.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableExecutorDispatcher.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 // ConfigureAwait Usage in Orchestration Code:
 // This file uses ConfigureAwait(true) because it runs within orchestration context.
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableRunStatus.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableRunStatus.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 namespace Microsoft.Agents.AI.DurableTask.Workflows;
 
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowContext.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowContext.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using System.Diagnostics;
 using System.Diagnostics.CodeAnalysis;
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowJsonContext.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowJsonContext.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using System.Text.Json.Serialization;
 
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowLiveStatus.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowLiveStatus.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.Workflows;
 
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowOptions.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowOptions.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using System.Diagnostics;
 using Microsoft.Agents.AI.Workflows;
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowRunner.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowRunner.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 // ConfigureAwait Usage in Orchestration Code:
 // This file uses ConfigureAwait(true) because it runs within orchestration context.
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowWaitingForInputEvent.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/DurableWorkflowWaitingForInputEvent.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using System.Diagnostics;
 using System.Diagnostics.CodeAnalysis;
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/IStreamingWorkflowRun.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/IStreamingWorkflowRun.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.Workflows;
 
diff --git a/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/PendingRequestPortStatus.cs b/dotnet/src/Microsoft.Agents.AI.DurableTask/Workflows/PendingRequestPortStatus.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 namespace Microsoft.Agents.AI.DurableTask.Workflows;
 
diff --git a/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/BuiltInFunctionExecutor.cs b/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/BuiltInFunctionExecutor.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Azure.Functions.Worker;
 using Microsoft.Azure.Functions.Worker.Context.Features;
diff --git a/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/FunctionMetadataFactory.cs b/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/FunctionMetadataFactory.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.DurableTask;
 using Microsoft.Azure.Functions.Worker.Core.FunctionMetadata;
diff --git a/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/FunctionsApplicationBuilderExtensions.cs b/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/FunctionsApplicationBuilderExtensions.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.DurableTask;
 using Microsoft.Agents.AI.DurableTask.Workflows;
diff --git a/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/FunctionsDurableOptions.cs b/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/FunctionsDurableOptions.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.DurableTask;
 
diff --git a/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/Workflows/DurableWorkflowOptionsExtensions.cs b/dotnet/src/Microsoft.Agents.AI.Hosting.AzureFunctions/Workflows/DurableWorkflowOptionsExtensions.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.DurableTask.Workflows;
 using Microsoft.Agents.AI.Workflows;
diff --git a/dotnet/tests/Microsoft.Agents.AI.DurableTask.UnitTests/Workflows/DurableActivityExecutorTests.cs b/dotnet/tests/Microsoft.Agents.AI.DurableTask.UnitTests/Workflows/DurableActivityExecutorTests.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using System.Text.Json;
 using Microsoft.Agents.AI.DurableTask.Workflows;
diff --git a/dotnet/tests/Microsoft.Agents.AI.DurableTask.UnitTests/Workflows/DurableStreamingWorkflowRunTests.cs b/dotnet/tests/Microsoft.Agents.AI.DurableTask.UnitTests/Workflows/DurableStreamingWorkflowRunTests.cs
@@ -624,7 +624,7 @@ public async Task SendResponseAsync_NullRequestEvent_ThrowsAsync()
 
         // Act & Assert
         await Assert.ThrowsAsync<ArgumentNullException>(() =>
-            run.SendResponseAsync<string>(null!, "response").AsTask());
+            run.SendResponseAsync(null!, "response").AsTask());
     }
 
     #endregion
diff --git a/dotnet/tests/Microsoft.Agents.AI.DurableTask.UnitTests/Workflows/DurableWorkflowContextTests.cs b/dotnet/tests/Microsoft.Agents.AI.DurableTask.UnitTests/Workflows/DurableWorkflowContextTests.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+﻿// Copyright (c) Microsoft. All rights reserved.
 
 using Microsoft.Agents.AI.DurableTask.Workflows;
 using Microsoft.Agents.AI.Workflows;
@@ -13,7 +13,7 @@ private static FunctionExecutor<string> CreateTestExecutor(string id = "test-exe
     #region ReadStateAsync
 
     [Fact]
-    public async Task ReadStateAsync_KeyExistsInInitialState_ReturnsValue()
+    public async Task ReadStateAsync_KeyExistsInInitialState_ReturnsValueAsync()
     {
         // Arrange
         Dictionary<string, string> state = new() { ["__default__:counter"] = "42" };
@@ -27,7 +27,7 @@ public async Task ReadStateAsync_KeyExistsInInitialState_ReturnsValue()
     }
 
     [Fact]
-    public async Task ReadStateAsync_KeyDoesNotExist_ReturnsNull()
+    public async Task ReadStateAsync_KeyDoesNotExist_ReturnsNullAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -40,7 +40,7 @@ public async Task ReadStateAsync_KeyDoesNotExist_ReturnsNull()
     }
 
     [Fact]
-    public async Task ReadStateAsync_LocalUpdateTakesPriorityOverInitialState()
+    public async Task ReadStateAsync_LocalUpdateTakesPriorityOverInitialStateAsync()
     {
         // Arrange
         Dictionary<string, string> state = new() { ["__default__:key"] = "\"old\"" };
@@ -55,7 +55,7 @@ public async Task ReadStateAsync_LocalUpdateTakesPriorityOverInitialState()
     }
 
     [Fact]
-    public async Task ReadStateAsync_ScopeCleared_IgnoresInitialState()
+    public async Task ReadStateAsync_ScopeCleared_IgnoresInitialStateAsync()
     {
         // Arrange
         Dictionary<string, string> state = new() { ["__default__:key"] = "\"value\"" };
@@ -70,7 +70,7 @@ public async Task ReadStateAsync_ScopeCleared_IgnoresInitialState()
     }
 
     [Fact]
-    public async Task ReadStateAsync_WithNamedScope_ReadsFromCorrectScope()
+    public async Task ReadStateAsync_WithNamedScope_ReadsFromCorrectScopeAsync()
     {
         // Arrange
         Dictionary<string, string> state = new()
@@ -92,7 +92,7 @@ public async Task ReadStateAsync_WithNamedScope_ReadsFromCorrectScope()
     [Theory]
     [InlineData(null)]
     [InlineData("")]
-    public async Task ReadStateAsync_NullOrEmptyKey_ThrowsArgumentException(string? key)
+    public async Task ReadStateAsync_NullOrEmptyKey_ThrowsArgumentExceptionAsync(string? key)
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -106,7 +106,7 @@ public async Task ReadStateAsync_NullOrEmptyKey_ThrowsArgumentException(string?
     #region ReadOrInitStateAsync
 
     [Fact]
-    public async Task ReadOrInitStateAsync_KeyDoesNotExist_CallsFactoryAndQueuesUpdate()
+    public async Task ReadOrInitStateAsync_KeyDoesNotExist_CallsFactoryAndQueuesUpdateAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -120,7 +120,7 @@ public async Task ReadOrInitStateAsync_KeyDoesNotExist_CallsFactoryAndQueuesUpda
     }
 
     [Fact]
-    public async Task ReadOrInitStateAsync_KeyExists_ReturnsExistingValue()
+    public async Task ReadOrInitStateAsync_KeyExists_ReturnsExistingValueAsync()
     {
         // Arrange
         Dictionary<string, string> state = new() { ["__default__:key"] = "\"existing\"" };
@@ -142,7 +142,7 @@ public async Task ReadOrInitStateAsync_KeyExists_ReturnsExistingValue()
     [Theory]
     [InlineData(null)]
     [InlineData("")]
-    public async Task ReadOrInitStateAsync_NullOrEmptyKey_ThrowsArgumentException(string? key)
+    public async Task ReadOrInitStateAsync_NullOrEmptyKey_ThrowsArgumentExceptionAsync(string? key)
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -153,7 +153,7 @@ await Assert.ThrowsAnyAsync<ArgumentException>(
     }
 
     [Fact]
-    public async Task ReadOrInitStateAsync_ValueType_MissingKey_CallsFactory()
+    public async Task ReadOrInitStateAsync_ValueType_MissingKey_CallsFactoryAsync()
     {
         // Arrange
         // Validates that ReadStateAsync<int> returns null (not 0) for missing keys,
@@ -170,7 +170,7 @@ public async Task ReadOrInitStateAsync_ValueType_MissingKey_CallsFactory()
     }
 
     [Fact]
-    public async Task ReadOrInitStateAsync_NullFactory_ThrowsArgumentNullException()
+    public async Task ReadOrInitStateAsync_NullFactory_ThrowsArgumentNullExceptionAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -185,7 +185,7 @@ await Assert.ThrowsAsync<ArgumentNullException>(
     #region QueueStateUpdateAsync
 
     [Fact]
-    public async Task QueueStateUpdateAsync_SetsValue_VisibleToSubsequentRead()
+    public async Task QueueStateUpdateAsync_SetsValue_VisibleToSubsequentReadAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -199,7 +199,7 @@ public async Task QueueStateUpdateAsync_SetsValue_VisibleToSubsequentRead()
     }
 
     [Fact]
-    public async Task QueueStateUpdateAsync_NullValue_RecordsDeletion()
+    public async Task QueueStateUpdateAsync_NullValue_RecordsDeletionAsync()
     {
         // Arrange
         Dictionary<string, string> state = new() { ["__default__:key"] = "\"value\"" };
@@ -216,7 +216,7 @@ public async Task QueueStateUpdateAsync_NullValue_RecordsDeletion()
     [Theory]
     [InlineData(null)]
     [InlineData("")]
-    public async Task QueueStateUpdateAsync_NullOrEmptyKey_ThrowsArgumentException(string? key)
+    public async Task QueueStateUpdateAsync_NullOrEmptyKey_ThrowsArgumentExceptionAsync(string? key)
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -231,7 +231,7 @@ await Assert.ThrowsAnyAsync<ArgumentException>(
     #region QueueClearScopeAsync
 
     [Fact]
-    public async Task QueueClearScopeAsync_DefaultScope_ClearsStateAndPendingUpdates()
+    public async Task QueueClearScopeAsync_DefaultScope_ClearsStateAndPendingUpdatesAsync()
     {
         // Arrange
         Dictionary<string, string> state = new() { ["__default__:key"] = "\"value\"" };
@@ -247,7 +247,7 @@ public async Task QueueClearScopeAsync_DefaultScope_ClearsStateAndPendingUpdates
     }
 
     [Fact]
-    public async Task QueueClearScopeAsync_NamedScope_OnlyClearsThatScope()
+    public async Task QueueClearScopeAsync_NamedScope_OnlyClearsThatScopeAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -267,7 +267,7 @@ public async Task QueueClearScopeAsync_NamedScope_OnlyClearsThatScope()
     #region ReadStateKeysAsync
 
     [Fact]
-    public async Task ReadStateKeysAsync_ReturnsKeysFromInitialState()
+    public async Task ReadStateKeysAsync_ReturnsKeysFromInitialStateAsync()
     {
         // Arrange
         Dictionary<string, string> state = new()
@@ -287,7 +287,7 @@ public async Task ReadStateKeysAsync_ReturnsKeysFromInitialState()
     }
 
     [Fact]
-    public async Task ReadStateKeysAsync_MergesLocalUpdatesAndDeletions()
+    public async Task ReadStateKeysAsync_MergesLocalUpdatesAndDeletionsAsync()
     {
         // Arrange
         Dictionary<string, string> state = new()
@@ -309,7 +309,7 @@ public async Task ReadStateKeysAsync_MergesLocalUpdatesAndDeletions()
     }
 
     [Fact]
-    public async Task ReadStateKeysAsync_AfterClearScope_ExcludesInitialState()
+    public async Task ReadStateKeysAsync_AfterClearScope_ExcludesInitialStateAsync()
     {
         // Arrange
         Dictionary<string, string> state = new() { ["__default__:old"] = "\"val\"" };
@@ -326,7 +326,7 @@ public async Task ReadStateKeysAsync_AfterClearScope_ExcludesInitialState()
     }
 
     [Fact]
-    public async Task ReadStateKeysAsync_WithNamedScope_OnlyReturnsKeysFromThatScope()
+    public async Task ReadStateKeysAsync_WithNamedScope_OnlyReturnsKeysFromThatScopeAsync()
     {
         // Arrange
         Dictionary<string, string> state = new()
@@ -349,7 +349,7 @@ public async Task ReadStateKeysAsync_WithNamedScope_OnlyReturnsKeysFromThatScope
     #region AddEventAsync
 
     [Fact]
-    public async Task AddEventAsync_AddsEventToCollection()
+    public async Task AddEventAsync_AddsEventToCollectionAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -364,7 +364,7 @@ public async Task AddEventAsync_AddsEventToCollection()
     }
 
     [Fact]
-    public async Task AddEventAsync_NullEvent_DoesNotAdd()
+    public async Task AddEventAsync_NullEvent_DoesNotAddAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -383,7 +383,7 @@ public async Task AddEventAsync_NullEvent_DoesNotAdd()
     #region SendMessageAsync
 
     [Fact]
-    public async Task SendMessageAsync_SerializesMessageWithTypeName()
+    public async Task SendMessageAsync_SerializesMessageWithTypeNameAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -398,7 +398,7 @@ public async Task SendMessageAsync_SerializesMessageWithTypeName()
     }
 
     [Fact]
-    public async Task SendMessageAsync_NullMessage_DoesNotAdd()
+    public async Task SendMessageAsync_NullMessage_DoesNotAddAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -417,7 +417,7 @@ public async Task SendMessageAsync_NullMessage_DoesNotAdd()
     #region YieldOutputAsync
 
     [Fact]
-    public async Task YieldOutputAsync_AddsWorkflowOutputEvent()
+    public async Task YieldOutputAsync_AddsWorkflowOutputEventAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -432,7 +432,7 @@ public async Task YieldOutputAsync_AddsWorkflowOutputEvent()
     }
 
     [Fact]
-    public async Task YieldOutputAsync_NullOutput_DoesNotAdd()
+    public async Task YieldOutputAsync_NullOutput_DoesNotAddAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -451,7 +451,7 @@ public async Task YieldOutputAsync_NullOutput_DoesNotAdd()
     #region RequestHaltAsync
 
     [Fact]
-    public async Task RequestHaltAsync_SetsHaltRequestedAndAddsEvent()
+    public async Task RequestHaltAsync_SetsHaltRequestedAndAddsEventAsync()
     {
         // Arrange
         DurableWorkflowContext context = new(null, CreateTestExecutor());
@@ -490,7 +490,7 @@ public void ConcurrentRunsEnabled_ReturnsFalse()
     }
 
     [Fact]
-    public async Task Constructor_NullInitialState_CreatesEmptyState()
+    public async Task Constructor_NullInitialState_CreatesEmptyStateAsync()
     {
         // Arrange & Act
         DurableWorkflowContext context = new(null, CreateTestExecutor());

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-// Copyright (c) Microsoft. All rights reserved.`
	`1`	`+// Copyright (c) Microsoft. All rights reserved.`
`2`	`2`
`3`	`3`	`using System.Diagnostics;`
`4`	`4`	`using Microsoft.Agents.AI.DurableTask.Workflows;`