tracyboehrer

This commit performs a **documentation update** within the **`Microsoft.Agents.Builder`** library, specifically for the `StreamingResponse.cs` component. It enhances code clarity by updating comments that detail the internal process of how text chunks are queued and subsequently transformed into activities for streaming responses. This **maintenance** effort improves developer understanding of the streaming response mechanism without introducing any functional changes to the system.

This commit provides a **documentation update** for the **`Microsoft.Agents.Core` library**, specifically within its `Serialization` component. It **improves the clarity and accuracy of XML documentation comments** for the `ToJsonElement` method in `src/libraries/Core/Microsoft.Agents.Core/Serialization/ProtocolJsonSerializer.cs`. The changes involve correcting a typo and enhancing parameter descriptions, making the API contract easier to understand for developers. This **maintenance** task addresses a pull request finding, ensuring better code readability and maintainability without altering any functional behavior.

This commit implements a **maintenance update** within the **`Microsoft.Agents.Builder` library**, specifically addressing an issue in the `EventRouteBuilder` component. It **corrects the error message string** for an `InvalidOperationException` that can be thrown by the `WithName` method in `src/libraries/Builder/Microsoft.Agents.Builder/App/EventRouteBuilder.cs`. This **bug fix** enhances diagnostic clarity for developers, providing more precise feedback when an invalid operation occurs during event route configuration. The change is localized to error reporting and does not alter functional behavior.

This commit **refines a test case** within the **testing infrastructure** for the `Microsoft.Agents.Builder` module. It **updates the expected exception type** for a specific test method in `src/tests/Microsoft.Agents.Builder.Tests/App/EventRouteBuilderTests.cs`. The test's name was adjusted to accurately reflect that it now anticipates an `InvalidOperationException` instead of an `ArgumentNullException`. This **test refinement** ensures the suite correctly validates the exception handling logic for event route building, improving the accuracy of the `Microsoft.Agents.Builder` component's tests.

This commit performs a **maintenance refactoring** within the **test suite** for the `Microsoft.Agents.Builder` component. Specifically, it **renames a test method** in `src/tests/Microsoft.Agents.Builder.Tests/App/InvokeRouteBuilderTests.cs` to more accurately describe its scenario, which involves duplicate name criteria. This change enhances the **clarity and readability of the tests**, making it easier for developers to understand the intended validation for the `InvokeRouteBuilder` functionality.

This commit performs **test refactoring** within the `Microsoft.Agents.Builder.Tests` project, specifically affecting the `TypeRouteBuilderTests.cs` file. It **renames two test methods** to accurately reflect that they are testing the `WithType` method, rather than `WithSelector`. This **maintenance** change improves the **clarity and maintainability** of the test suite by ensuring test names precisely describe the functionality under examination.

This commit performs a **refactoring** within the **`Microsoft.Agents.Builder` library**, specifically targeting the **streaming response handling**. It changes the access modifier of the `StreamId` property within the `StreamingResponse` class from `public` to `internal`. This modification enhances **encapsulation** by restricting the visibility of `StreamId` to within the assembly, preventing external consumers from directly accessing or modifying this identifier. The change improves the **internal API consistency** and maintainability of the `StreamingResponse` component.

This commit delivers a **bug fix** to the **user authorization** and **token exchange** mechanisms within the `Microsoft.Agents.Builder` library. It resolves an issue where `UserAuthorization.ExchangeToken` was incorrectly reusing cached tokens for subsequent exchanges. The `ExchangeToken` method in `src/libraries/Builder/Microsoft.Agents.Builder/App/UserAuth/UserAuthorization.cs` is now updated to ensure that exchangeable tokens are always re-exchanged, while non-exchangeable tokens are correctly cached, preventing the use of stale credentials. Concurrently, `src/libraries/Builder/Microsoft.Agents.Builder/UserAuth/OBOExchange.cs` was refined to accurately resolve On-Behalf-Of (OBO) exchange providers, including default connections. This ensures more reliable and secure **token exchange** for user authentication.

This commit introduces a **bug fix** within the **`Microsoft.Agents.Builder`** library, specifically addressing a potential `NullReferenceException` in the `GetTenantId` method of the `ActivityExtensions` class. It modifies `src/libraries/Builder/Microsoft.Agents.Builder/ActivityExtensions.cs` by adding a **null-conditional operator** to safely access the `Conversation.TenantId` property. This change enhances the **robustness** of tenant ID retrieval, preventing crashes when the `Conversation` object or its `TenantId` might be null, thereby improving the overall stability of agent interactions.

This commit performs a **maintenance update** to the `MsalAuth.cs` file within the **Authentication.Msal** library. Despite the commit message, a detailed analysis reveals that the file's content remains **identical to its previous version**, indicating no functional or structural changes were introduced. Consequently, this commit has **no semantic impact** on the application's behavior or the authentication subsystem. It serves as a re-save or re-commit without altering any existing functionality.

This commit **improves documentation** within the **Microsoft.Agents.Builder library** by adding comprehensive XML documentation comments. Specifically, these comments are applied to the `GetAgenticTenantId` method within the `src/libraries/Builder/Microsoft.Agents.Builder/ActivityExtensions.cs` file. This **maintenance** task enhances code clarity and developer understanding for those interacting with the agentic tenant ID retrieval functionality, ultimately making the API easier to consume and maintain.

This commit performs a **documentation update** by correcting a minor typo in the project's `README.md` file. Specifically, it addresses a spelling error, changing 'intenral' to 'internal' within the contributing section of the document. This **maintenance** task ensures the project's **contributor guidelines** are accurate and professional, improving clarity and readability for anyone reviewing the documentation.

This commit performs a minor **documentation update** by **correcting a typo** within the project's `README.md` file. Specifically, it changes 'Micrsoft' to 'Microsoft' in the contributing section. This **maintenance** task ensures the accuracy and professionalism of the project's public-facing information, improving the clarity of the contribution guidelines for potential contributors.

This commit introduces a **refactoring** to the **Microsoft.Agents.Builder** library, specifically within the `StreamingResponse.cs` component. The changes focus on **improving code style and consistency** by updating file formatting and refining a boolean comparison within the `SendActivityAsync` method. This **maintenance** effort enhances the readability and maintainability of the `StreamingResponse` class without altering its functional behavior or introducing any new capabilities.

This commit **updates the XML documentation** for the `EndStreamAsync` method within the `IStreamingResponse` interface, part of the **Microsoft.Agents.Builder library**. Specifically, it provides a more detailed description of the method's return value, enhancing clarity for consumers. This **documentation update** improves the overall understanding and usability of the streaming response mechanism for developers interacting with the `Microsoft.Agents.Builder` components.

This commit introduces a **new capability** by integrating a standardized **feedback loop mechanism** into the **`Microsoft.Agents.Builder` core library**. It adds the `OnFeedbackLoop` method to `AgentApplication` for registering feedback handlers and defines new core types such as `FeedbackData` and `FeedbackLoopHandler`. Concurrently, the **`Microsoft.Agents.Extensions.Teams` extension** undergoes significant **refactoring** to deprecate its previous feedback loop implementation, migrating to and adopting these new core types for consistency. This change enables agents to process user feedback more uniformly across the platform, with a sample demonstrating its usage within a Teams agent.

This commit **fixes a typo** within the **`Microsoft.Agents.Core` library**, specifically in the `src/libraries/Core/Microsoft.Agents.Core/Models/Channels.cs` file. It corrects the spelling from "Microsfot" to "Microsoft" in a comment associated with the `CopilotStudio` constant. This is a **minor documentation update** that enhances the accuracy and readability of internal code comments. The change has no functional impact on the system, serving purely as a **maintenance** improvement.

This commit provides a **documentation update** for the `CopilotStudioSkillConnector` sample. It specifically **removes a hyphen** from the 'Prerequisites' heading within the `src/samples/CopilotStudioSkillConnector/README.md` file. This minor **maintenance** change improves the formatting and readability of the **sample's documentation**, offering a cleaner presentation for users reviewing setup requirements.

This commit performs a **documentation update** for the `CopilotStudioSkillConnector` sample. It specifically **reformats the dev tunnel link** located within the prerequisites section of the `src/samples/CopilotStudioSkillConnector/README.md` file. This **maintenance** change improves the clarity and usability of the setup instructions for developers working with the **Copilot Studio Skill Connector** sample, ensuring they can easily access the necessary resources.

This commit provides a **documentation update** for the `ConnectorUserAuthorization` class within the **Microsoft.Agents.Builder** library. It specifically refines the XML documentation comment for the `storage` parameter in the constructor of `ConnectorUserAuthorization.cs`. This **maintenance** task improves the clarity and accuracy of the API documentation, aiding developers in understanding the purpose and usage of the `storage` parameter when integrating user authorization connectors. The change has no functional impact on the runtime behavior of the system, solely enhancing developer experience.