Skip to content

Fix event type names for assistant messages#1225

Merged
aaronpowell merged 1 commit intogithub:stagedfrom
moeyui1:staged
Mar 30, 2026
Merged

Fix event type names for assistant messages#1225
aaronpowell merged 1 commit intogithub:stagedfrom
moeyui1:staged

Conversation

@moeyui1
Copy link
Copy Markdown
Contributor

@moeyui1 moeyui1 commented Mar 30, 2026

Fix event type names for assistant messages per https://github.com/github/copilot-sdk/blob/main/nodejs/src/generated/session-events.ts

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • I have read and followed the Guidance for submissions involving paid services.
  • My contribution adds a new instruction, prompt, agent, skill, or workflow file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, agent, skill, or workflow with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.
  • I am targeting the staged branch for this pull request.

Description

Fix the event type names for assistant messages according to the actual definition in SDK: https://github.com/github/copilot-sdk/blob/main/nodejs/src/generated/session-events.ts

Type of Contribution

  • New instruction file.
  • New prompt file.
  • New agent file.
  • New plugin.
  • New skill file.
  • New agentic workflow.
  • Update to existing instruction, prompt, agent, plugin, skill, or workflow.
  • Other (please specify):

Additional Notes


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

Copilot AI review requested due to automatic review settings March 30, 2026 07:59
@moeyui1 moeyui1 requested a review from aaronpowell as a code owner March 30, 2026 07: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

Updates the Node.js Copilot SDK instruction examples to use the correct session event type names for streaming assistant message/reasoning deltas, aligning the docs with the SDK’s generated event definitions.

Changes:

  • Replace assistant.message.delta with assistant.message_delta in streaming switch-case example.
  • Replace assistant.reasoning.delta with assistant.reasoning_delta in streaming switch-case example.
  • Update the “Streaming with Progress” example to check for assistant.message_delta instead of assistant.message.delta.

@aaronpowell aaronpowell merged commit 784f373 into github:staged Mar 30, 2026
12 of 13 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