Conversation
added 5 commits
August 11, 2025 18:20
Member
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request implements several dependency updates and API improvements to enhance the Microsoft Agents SDK. The changes focus on updating dependencies for security and compatibility, exposing previously private methods for better extensibility, and improving the Express server setup.
- Updated multiple dependencies including
@azure/msal-node,@types/node, andtypedocto newer versions - Exposed
getMentionsandremoveMentionTextmethods in the Activity class from private to public - Enhanced Express server functionality to return the server instance and improved header propagation handling
Reviewed Changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| test-agents/copilotstudio-console/package.json | Updated @azure/msal-node dependency from 3.6.0 to 3.7.0 |
| samples/compat/oauthFlowHandler.ts | Changed OAuth state storage path from '__state' to '../__oauth_state' |
| samples/basic/emptyAgent.ts | Added typing event handler and root endpoint with server instance usage |
| packages/agents-hosting/package.json | Updated @azure/msal-node dependency from 3.6.0 to 3.7.0 |
| packages/agents-hosting-express/src/startServer.ts | Modified function to return Express server instance and improved header propagation |
| packages/agents-activity/src/activity.ts | Changed getMentions and removeMentionText methods from private to public |
| package.json | Updated multiple dev dependencies and added new playground package |
added 2 commits
August 12, 2025 09:07
sw-joelmut
approved these changes
Aug 12, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces several dependency updates, exposes previously private methods for mentions handling, and enhances the Express server setup in the agents-hosting-express package. Additionally, it improves the sample agents by adding new event handlers and refines the OAuth flow sample for better state management.
Dependency updates:
package.jsonfiles, including@azure/msal-node,@types/node, andtypedoc, to newer versions for improved compatibility and security. [1] [2] [3] [4]@microsoft/m365agentsplaygroundto the mainpackage.json.API and codebase improvements:
getMentionsandremoveMentionTextmethods from private to public in theActivityclass, making them accessible for broader use. [1] [2]startServerfunction instartServer.tsto return the Express server instance and improved header propagation handling. [1] [2] [3]Sample and test enhancements:
EmptyAgentsample to handle the 'typing' activity and added a root endpoint to confirm the agent is running. [1] [2]