Skip to content

Conversation

@ceciliaavila
Copy link
Collaborator

@ceciliaavila ceciliaavila commented Sep 9, 2025

Addresses #384

Description

This PR removes from the public API classes and methods that should be private because they are intended for internal use.

NOTE:
This PR introduces breaking changes, as previously public members are now inaccessible.

Detailed Changes

  • agents-copilotstudio-client
    • Removed the re-export of ExecuteTurnRequest class as it is a wrapper used internally to send the activity in the request's body.
    • Prevented the export of the getCopilotStudioConnectionUrl and getTokenAudience methods as they are used internally to connect with the Copilot Studio agent.
  • agents-hosting
    • Avoided exporting the appRoute and RouteList classes as they are internal.
    • Prevented the export of TranscriptLoggerMiddleware by selectively exporting members from transcript.ts.
    • Prevented the export of HeaderPropagation by selectively exporting members from headerPropagation.ts.
    • Prevented the export of CachedAgentState by selectively exporting members from agentState.ts.
    • Removed unused AgentCallbackHandlerKey constant from turnContext.
    • Removed the export of ClaimsIdentity class and Claim interface as they weren't used.
  • Updated API docs to reflect the changes.

Testing

These images show the copilotstudio-client samples working (webchat and console).
image

And these are some samples using the updated hosting features.
image

export * from './state'
export * from './storage'
export * from './transcript'
export { TranscriptLogger } from './transcript'
Copy link
Contributor

Choose a reason for hiding this comment

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

These can be in a list

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Feedback applied. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants