Skip to content

feat: surface subscribeToStream polling errors - #426

Merged
Jaydbrown merged 1 commit into
conduit-protocol:mainfrom
BrightHeart23:main
Aug 26, 2026
Merged

feat: surface subscribeToStream polling errors#426
Jaydbrown merged 1 commit into
conduit-protocol:mainfrom
BrightHeart23:main

Conversation

@BrightHeart23

Copy link
Copy Markdown

Summary

  • invoke the optional StreamEventHandlers.onError callback when event polling fails
  • normalize non-Error rejection values while retaining the existing console warning
  • keep polling if the consumer's error handler throws

Validation

  • npm test -- --run src/tests/events-subscribe.test.ts (11 tests passed)
  • npm run typecheck
  • npm run lint -- --no-fix

Closes #200

@Jaydbrown

Copy link
Copy Markdown
Contributor

Thanks for the contribution here — squash-merging this now. Any follow-ups we'll track in a fresh issue. 🚀

@Jaydbrown
Jaydbrown merged commit c086d4b into conduit-protocol:main Aug 26, 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.

Enhancement: add an onError handler for subscribeToStream polling failures

2 participants