Skip to content

await next in hono error handler middleware - #2735

Merged
SB-priyankap merged 26 commits into
nextfrom
gingerbenw/hono-async-await
May 6, 2026
Merged

SB-priyankap merged 26 commits into
nextfrom
gingerbenw/hono-async-await

Conversation

@gingerbenw

@gingerbenw gingerbenw commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Goal

Updates the Hono plugin’s error-handling middleware to properly await next() and adds/adjusts integration tests and fixtures to cover async behavior, non-Error throws, and request-body parsing.

Changeset

  • Update @bugsnag/plugin-hono error handler to await next() and handle thrown non-Error values.

Testing

  • Extend Node maze-runner steps/features to allow POSTing with an explicit Content-Type (including JSON).
  • Adjust AWS Lambda Hono feature coverage and upload maze output artifacts on workflow failure.

@github-actions

github-actions Bot commented Apr 22, 2026

Copy link
Copy Markdown

@bugsnag/browser bundle size diff

Minified Minfied + Gzipped
Before 51.76 kB 15.30 kB
After 51.76 kB 15.30 kB
± No change No change

code coverage diff

Ok File Lines Branches Functions Statements
🔴 /home/runner/work/bugsnag-js/bugsnag-js/packages/plugin-hono/src/hono.js 90%
(-4.11%)
75%
(-15%)
100%
(+0%)
85.71%
(-8.57%)

Total:

Lines Branches Functions Statements
84.93%(-0.02%) 77.1%(-0.06%) 85.17%(+0%) 84.17%(-0.06%)

Generated by 🚫 dangerJS against c6503ae

@gingerbenw
gingerbenw force-pushed the gingerbenw/hono-async-await branch from b606807 to 521b9db Compare April 23, 2026 08:45
@gingerbenw
gingerbenw force-pushed the gingerbenw/hono-async-await branch from a175c8d to 2c37bcd Compare April 24, 2026 14:38

This comment was marked as resolved.

Comment thread test/aws-lambda/features/unhandled.feature
Comment thread test/aws-lambda/features/unhandled.feature
Comment thread test/node/features/fixtures/hono/scenarios/app.js
Comment thread .github/workflows/aws-lambda.yml
@gingerbenw
gingerbenw marked this pull request as ready for review April 24, 2026 15:45
@gingerbenw
gingerbenw requested review from a team and hannah-smartbear April 24, 2026 15:46
@gingerbenw gingerbenw changed the title Gingerbenw/hono async await await next in hono error handler middleware Apr 24, 2026
@SB-priyankap
SB-priyankap requested a review from SB-gulshans May 6, 2026 06:14

@SB-gulshans SB-gulshans 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.

LGTM

@SB-priyankap
SB-priyankap merged commit 93c9555 into next May 6, 2026
25 checks passed
@SB-priyankap
SB-priyankap deleted the gingerbenw/hono-async-await branch May 6, 2026 06:35
@SB-priyankap SB-priyankap mentioned this pull request Jul 10, 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.

4 participants