Skip to content

Fix Copilot SDK session event handler compile issue#2056

Merged
aaronpowell merged 1 commit into
github:stagedfrom
jongalloway:fix-copilot-sdk-session-event
Jun 19, 2026
Merged

Fix Copilot SDK session event handler compile issue#2056
aaronpowell merged 1 commit into
github:stagedfrom
jongalloway:fix-copilot-sdk-session-event

Conversation

@jongalloway

Copy link
Copy Markdown
Contributor

Summary

  • Updates the Copilot SDK PR visualization recipe to subscribe with session.On<SessionEvent>(...).
  • Fixes a compile issue caused by the SDK breaking change that now requires the event type for this handler.
  • Supersedes Revert Copilot SDK model/session event update #2054, which was opened with an incorrect revert-focused title/base for this fix.

Testing

  • Not run. This recipe invokes live Copilot/GitHub tooling, so I kept validation to the focused one-line diff.

Copilot AI review requested due to automatic review settings June 18, 2026 23:47
@jongalloway jongalloway requested a review from aaronpowell as a code owner June 18, 2026 23:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the .NET Copilot SDK PR visualization runnable recipe to match a breaking change in the SDK’s session event subscription API, restoring compile-time compatibility for the example.

Changes:

  • Updates the PR visualization recipe’s session event handler subscription to explicitly specify the event type (session.On<SessionEvent>(...)).
Show a summary per file
File Description
cookbook/copilot-sdk/dotnet/recipe/pr-visualization.cs Adjusts the session event subscription to use the SDK’s typed On<TEvent> handler form to address the compile issue.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@aaronpowell aaronpowell merged commit 05dd83e into github:staged Jun 19, 2026
11 checks passed
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