Skip to content

feat: add session id to stream events#44

Closed
varin-nair-factory wants to merge 2 commits into
mainfrom
vn/session-id-events
Closed

feat: add session id to stream events#44
varin-nair-factory wants to merge 2 commits into
mainfrom
vn/session-id-events

Conversation

@varin-nair-factory
Copy link
Copy Markdown
Contributor

Description

Adds sessionId to SDK stream events by introducing a shared stream base type and threading the SDK-tracked session id through notification conversion and MessageBridge.

Also updates stream/helper/run/integration tests so converted events and result message fixtures account for the new field.

Related Issue

N/A

Potential Risk & Impact

  • Additive runtime field on stream events.
  • TypeScript consumers constructing stream event objects manually may need to include sessionId.
  • Example script fixes were intentionally excluded from this PR per request.

How Has This Been Tested?

  • npm run lint
  • npm run typecheck
  • npm test
  • npm run typecheck:examples passed locally only with unstaged example-script fixes, which are not included in this PR.

varin-nair-factory and others added 2 commits May 21, 2026 14:32
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
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.

1 participant