[#384] Reorganize exports by named exports - agents-hosting and agents-copilotstudio-client

compat/baseline/agents-copilotstudio-client.api.md

@@ -61,18 +61,6 @@ export interface CopilotStudioWebChatSettings {
     showTyping?: boolean;
 }
 
-// @public
-export class ExecuteTurnRequest {
-    constructor(activity?: Activity);
-    activity?: Activity;
-}
-
-// @public
-export function getCopilotStudioConnectionUrl(settings: ConnectionSettings, conversationId?: string): string;
-
-// @public
-export function getTokenAudience(settings?: ConnectionSettings, cloud?: PowerPlatformCloud, cloudBaseAddress?: string, directConnectUrl?: string): string;
-
 // @public
 export const loadCopilotStudioConnectionSettingsFromEnv: () => ConnectionSettings;
 

packages/agents-copilotstudio-client/src/index.ts

@@ -3,6 +3,4 @@ export * from './connectionSettings.js'
 export * from './copilotStudioClient.js'
 export * from './copilotStudioConnectionSettings.js'
 export * from './copilotStudioWebChat.js'
-export * from './executeTurnRequest.js'
 export * from './powerPlatformCloud.js'
-export * from './powerPlatformEnvironment.js'

packages/agents-hosting/src/app/agentApplication.ts

@@ -593,16 +593,8 @@ export class AgentApplication<TState extends TurnState> {
    * 8. Executes after-turn event handlers
    * 9. Saves turn state
    *
-   * @example
-   * ```typescript
-   * const handled = await app.runInternal(turnContext);
-   * if (!handled) {
-   *   console.log('No handler matched the activity');
-   * }
-   * ```
-   *
    */
-  public async runInternal (turnContext: TurnContext): Promise<boolean> {
+  private async runInternal (turnContext: TurnContext): Promise<boolean> {
     if (turnContext.activity.type === ActivityTypes.Typing) {
       return false
     }

packages/agents-hosting/src/app/index.ts

@@ -1,10 +1,10 @@
 export * from './agentApplication'
 export * from './agentApplicationBuilder'
 export * from './agentApplicationOptions'
-export * from './appRoute'
+export * from './attachmentDownloader'
+export * from './conversationUpdateEvents'
 export { Authorization } from './auth/authorization'
 export * from './routeHandler'
-export * from './routeList'
 export * from './routeRank'
 export * from './routeSelector'
 export * from './attachmentDownloader'

packages/agents-hosting/src/index.ts

@@ -13,7 +13,7 @@ export * from './invoke'
 export * from './oauth'
 export * from './state'
 export * from './storage'
-export * from './transcript'
+export { ConsoleTranscriptLogger, PagedResult, TranscriptInfo, TranscriptLogger, TranscriptStore } from './transcript'
 
 export * from './activityHandler'
 export * from './baseAdapter'
@@ -24,6 +24,6 @@ export * from './statusCodes'
 export * from './turnContext'
 export * from './turnContextStateCollection'
 export * from './storage/storage'
-export * from './headerPropagation'
+export { HeaderPropagationCollection, HeaderPropagationDefinition } from './headerPropagation'
 
 export * from './agent-client'

packages/agents-hosting/src/state/index.ts

@@ -1,4 +1,4 @@
 export * from './userState'
 export * from './conversationState'
 export * from './agentStatePropertyAccesor'
-export * from './agentState'
+export { AgentState, CustomKey } from './agentState'

packages/agents-hosting/src/turnContext.ts

@@ -39,11 +39,6 @@ export type UpdateActivityHandler = (context: TurnContext, activity: Activity, n
  */
 export type DeleteActivityHandler = (context: TurnContext, reference: ConversationReference, next: () => Promise<void>) => Promise<void>
 
-/**
- * Key for the agent callback handler in TurnState collection.
- */
-export const AgentCallbackHandlerKey = 'agentCallbackHandler'
-
 /**
  * Represents the context for a single turn in a conversation between a user and an agent.
  *

packages/agents-hosting/test/hosting/app/agentApplication.test.ts

@@ -58,10 +58,9 @@ describe('Application', () => {
       called = true
     })
     const context = new TurnContext(testAdapter, testActivity)
-    const handled = await app.runInternal(context)
+    await app.run(context)
     await context.sendActivity('test')
     assert.equal(called, true)
-    assert.equal(handled, true)
   })
 
   it('should route to an agentic activity handler', async () => {
@@ -96,10 +95,9 @@ describe('Application', () => {
       called = true
     })
     const context = new TurnContext(testAdapter, testActivity)
-    const handled = await app.runInternal(context)
+    await app.run(context)
     await context.sendActivity(MessageFactory.text('/yo'))
     assert.equal(called, true)
-    assert.equal(handled, true)
   })
 
   it('should route to an agentic message handler with string', async () => {
@@ -135,10 +133,9 @@ describe('Application', () => {
     })
     testActivity.text = 'foo'
     const context = new TurnContext(testAdapter, testActivity)
-    const handled = await app.runInternal(context)
+    await app.run(context)
     await context.sendActivity(testActivity)
     assert.equal(called, false)
-    assert.equal(handled, false)
   })
 
   it('should route to a message handler with string case insensitive', async () => {
@@ -151,10 +148,9 @@ describe('Application', () => {
     })
     testActivity.text = 'FOO'
     const context = new TurnContext(testAdapter, testActivity)
-    const handled = await app.runInternal(context)
+    await app.run(context)
     await context.sendActivity(testActivity)
     assert.equal(called, true)
-    assert.equal(handled, true)
   })
 
   it('should route to a act handler with regex', async () => {
@@ -166,10 +162,9 @@ describe('Application', () => {
       called = true
     })
     const context = new TurnContext(testAdapter, testActivity)
-    const handled = await app.runInternal(context)
+    await app.run(context)
     await context.sendActivity(MessageFactory.text('/yo'))
     assert.equal(called, true)
-    assert.equal(handled, true)
   })
 
   it('should route to a msg handler with regex', async () => {
@@ -181,10 +176,9 @@ describe('Application', () => {
       called = true
     })
     const context = new TurnContext(testAdapter, testActivity)
-    const handled = await app.runInternal(context)
+    await app.run(context)
     await context.sendActivity(MessageFactory.text('/yo'))
     assert.equal(called, true)
-    assert.equal(handled, true)
   })
 
   it('should ignore sencond message', async () => {
@@ -201,10 +195,9 @@ describe('Application', () => {
       timesCalled++
     })
     const context = new TurnContext(testAdapter, testActivity)
-    const handled = await app.runInternal(context)
+    await app.run(context)
     await context.sendActivity('/yo')
     assert.equal(timesCalled, 1)
-    assert.equal(handled, true)
   })
 
   it('should ignore sencond message with act', async () => {
@@ -221,10 +214,9 @@ describe('Application', () => {
       timesCalled++
     })
     const context = new TurnContext(testAdapter, testActivity)
-    const handled = await app.runInternal(context)
+    await app.run(context)
     await context.sendActivity('/yo')
     assert.equal(timesCalled, 1)
-    assert.equal(handled, true)
   })
 
   it('should add TranscriptLoggerMiddleware', async () => {