fix: address PR review feedback

- Add using declarations on TurnContext instances to ensure disposal
- Use ChannelId constructor instead of implicit string conversion for
  msteams and directline channel IDs for consistency
- Fix grammar in comment: 'we will required' -> 'we will require'

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: flaviocdc

src/Notification/Microsoft.Agents.A365.Notifications/AgentNotification.cs

@@ -54,7 +54,7 @@ public AgentNotification OnAgentNotification(string subChannelId, AgentNotificat
             {
                 // Wrap the activity in an AgentNotificationActivity
                 var agentNotificationActivity = new AgentNotificationActivity(turnContext.Activity);
-                // for now, we will required the handler to return the proper result.. we will change this later to return a structured result and handle the response back. 
+                // For now, we will require the handler to return the proper result. We will change this later to return a structured result and handle the response back.
                 await handler(turnContext, turnState, agentNotificationActivity, cancellationToken);
             };
             AddRoute(_app, routeSelector, routeHandler, isAgenticOnly: true, rank: rank, autoSignInHandlers: autoSignInHandlers);

src/Tests/Microsoft.Agents.A365.Notifications.Tests/AgentNotificationRoutingTests.cs

@@ -54,7 +54,7 @@ public async Task AgenticEmailNotification_RoutesToNotificationHandler_NotGeneri
             RouteRank.Last,
             isAgenticOnly: true);
 
-        var turnContext = CreateAgenticNotificationTurnContext(SubChannels.AgentsEmailSubChannel);
+        using var turnContext = CreateAgenticNotificationTurnContext(SubChannels.AgentsEmailSubChannel);
 
         // Act
         await app.OnTurnAsync(turnContext, CancellationToken.None);
@@ -94,7 +94,7 @@ public async Task AgenticNotification_WithWildcardHandler_RoutesToNotificationHa
             RouteRank.Last,
             isAgenticOnly: true);
 
-        var turnContext = CreateAgenticNotificationTurnContext(SubChannels.AgentsWordSubChannel);
+        using var turnContext = CreateAgenticNotificationTurnContext(SubChannels.AgentsWordSubChannel);
 
         // Act
         await app.OnTurnAsync(turnContext, CancellationToken.None);
@@ -131,7 +131,7 @@ public async Task LifecycleNotification_RoutesToLifecycleHandler_NotGenericEvent
             RouteRank.Last,
             isAgenticOnly: true);
 
-        var turnContext = CreateAgenticLifecycleNotificationTurnContext(Events.AgenticUserIdentityCreated);
+        using var turnContext = CreateAgenticLifecycleNotificationTurnContext(Events.AgenticUserIdentityCreated);
 
         // Act
         await app.OnTurnAsync(turnContext, CancellationToken.None);
@@ -179,7 +179,7 @@ public async Task NonAgenticMessage_RoutesToNonAgenticHandler()
             isAgenticOnly: false);
 
         // Create a normal (non-agentic) message — no agents channel, no agenticUser role
-        var turnContext = CreateNonAgenticMessageTurnContext();
+        using var turnContext = CreateNonAgenticMessageTurnContext();
 
         // Act
         await app.OnTurnAsync(turnContext, CancellationToken.None);
@@ -222,12 +222,12 @@ public async Task AgenticMessage_NonAgentsChannel_RoutesToGenericAgenticHandler(
         {
             Type = ActivityTypes.Message,
             Text = "Hello from Teams",
-            ChannelId = "msteams",
+            ChannelId = new ChannelId("msteams"),
             Recipient = new ChannelAccount { Id = "bot1", Role = RoleTypes.AgenticUser },
             From = new ChannelAccount { Id = "user1" },
             Conversation = new ConversationAccount { Id = "conv1" },
         };
-        var turnContext = new TurnContext(new StubAdapter(), activity);
+        using var turnContext = new TurnContext(new StubAdapter(), activity);
 
         // Act
         await app.OnTurnAsync(turnContext, CancellationToken.None);
@@ -299,7 +299,7 @@ private static async Task AssertSubChannelRoutesToNotificationHandler(
             RouteRank.Last,
             isAgenticOnly: true);
 
-        var turnContext = CreateAgenticNotificationTurnContext(subChannel);
+        using var turnContext = CreateAgenticNotificationTurnContext(subChannel);
 
         // Act
         await app.OnTurnAsync(turnContext, CancellationToken.None);
@@ -356,7 +356,7 @@ private static TurnContext CreateNonAgenticMessageTurnContext()
         {
             Type = ActivityTypes.Message,
             Text = "Hello",
-            ChannelId = "directline",
+            ChannelId = new ChannelId("directline"),
             Recipient = new ChannelAccount { Id = "bot1" },
             From = new ChannelAccount { Id = "user1" },
             Conversation = new ConversationAccount { Id = "conv1" },