Skip to content

v2: unhandled handler error kills the polling loop (add a default error boundary) (#1328) - #1329

Merged
yagop merged 6 commits into
masterfrom
issue-1328
Jul 1, 2026
Merged

v2: unhandled handler error kills the polling loop (add a default error boundary) (#1328)#1329
yagop merged 6 commits into
masterfrom
issue-1328

Conversation

@yagop

@yagop yagop commented Jul 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Closes #1328

  • T1 Bot: default log-and-continue error boundary; bot.catch() replaces it, a throw from the boundary still propagates (+ JSDoc + unit tests)
  • T2 webhookCallback: catch a boundary rethrow in the non-fastAck path and return an explicit 500 (+ unit tests)
  • T3 Docs: README error-handling section + regenerate doc/api.md

🤖 Generated with Claude Code

@yagop yagop mentioned this pull request Jul 1, 2026
Closed
4 tasks
@yagop
yagop marked this pull request as ready for review July 1, 2026 21:53
@yagop
yagop merged commit b05e881 into master Jul 1, 2026
7 of 8 checks passed
@yagop

yagop commented Jul 1, 2026

Copy link
Copy Markdown
Owner Author

😺 Michi — merged, all done 🐾

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

v2: unhandled handler error kills the polling loop (add a default error boundary)

1 participant

@yagop