Skip to content

Conversation

@meganrogge
Copy link
Contributor

part of #274403

Copilot AI review requested due to automatic review settings February 3, 2026 22:50
@meganrogge meganrogge self-assigned this Feb 3, 2026
@meganrogge meganrogge added this to the February 2026 milestone Feb 3, 2026
Copy link
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 migrates task execution and output monitoring components from using string-based sessionId to URI-based sessionResource for tracking chat sessions, as part of the broader effort outlined in issue #274403.

Changes:

  • Updated IExecution interface to use sessionResource: URI | undefined instead of sessionId: string | undefined
  • Modified OutputMonitor methods to accept and use sessionResource parameter
  • Removed unnecessary LocalChatSessionUri import as the code now uses URIs directly

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
types.ts Changed IExecution interface property from sessionId to sessionResource with proper URI type
outputMonitor.ts Updated method signatures and implementation to use sessionResource, removed LocalChatSessionUri import, now uses URIs directly with _chatService.getSession()
taskHelpers.ts Updated execution object creation to assign sessionResource from invocationContext.sessionResource

rzhao271
rzhao271 previously approved these changes Feb 3, 2026
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