Skip to content

Minor fixes, and npm updates#513

Merged
rido-min merged 7 commits intomainfrom
fix/upd-and-startServer
Aug 12, 2025
Merged

Minor fixes, and npm updates#513
rido-min merged 7 commits intomainfrom
fix/upd-and-startServer

Conversation

@rido-min
Copy link
Copy Markdown
Member

@rido-min rido-min commented Aug 11, 2025

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:

  • Updated various dependencies across package.json files, including @azure/msal-node, @types/node, and typedoc, to newer versions for improved compatibility and security. [1] [2] [3] [4]
  • Added a new development dependency @microsoft/m365agentsplayground to the main package.json.

API and codebase improvements:

  • Changed getMentions and removeMentionText methods from private to public in the Activity class, making them accessible for broader use. [1] [2]
  • Modified the startServer function in startServer.ts to return the Express server instance and improved header propagation handling. [1] [2] [3]

Sample and test enhancements:

  • Enhanced the EmptyAgent sample to handle the 'typing' activity and added a root endpoint to confirm the agent is running. [1] [2]
  • Updated the OAuth flow handler sample to use a more appropriate file path for storing state.

@rido-min
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@rido-min rido-min marked this pull request as ready for review August 12, 2025 08:59
Copilot AI review requested due to automatic review settings August 12, 2025 08:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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, and typedoc to newer versions
  • Exposed getMentions and removeMentionText methods 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

Comment thread samples/basic/emptyAgent.ts
Comment thread packages/agents-hosting-express/src/startServer.ts
@rido-min rido-min marked this pull request as draft August 12, 2025 09:01
@rido-min rido-min marked this pull request as ready for review August 12, 2025 09:53
@rido-min rido-min enabled auto-merge (squash) August 12, 2025 10:02
Comment thread package.json
@rido-min rido-min merged commit 7d36775 into main Aug 12, 2025
7 checks passed
@rido-min rido-min deleted the fix/upd-and-startServer branch August 12, 2025 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants