Skip to content

fix: harden node adapter lifecycle and performance - #17

Merged
smiggleworth merged 1 commit into
mainfrom
fix/node-correctness-ergonomics
Aug 7, 2026
Merged

fix: harden node adapter lifecycle and performance#17
smiggleworth merged 1 commit into
mainfrom
fix/node-correctness-ergonomics

Conversation

@smiggleworth

Copy link
Copy Markdown
Contributor

Summary

  • harden Node request trust boundaries, response streaming, and server lifecycle behavior
  • make WebSocket and MCP shutdown and cancellation deterministic, including a bounded WebSocket close grace period
  • add native Vitest benchmarks and retain measured request-body and static-asset performance improvements

Validation

  • npm run fmt -- --check
  • npm run check
  • npm audit
  • npm outdated
  • npm run bench:http

Benchmark highlights

  • static asset delivery improved 17-32% in repeated isolated samples
  • consumed POST bodies improved about 8%
  • serve() empty response handling improved about 12%

Copilot AI lite review requested due to automatic review settings August 7, 2026 17:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@smiggleworth
smiggleworth merged commit 9d8d3db into main Aug 7, 2026
3 checks passed
@smiggleworth
smiggleworth deleted the fix/node-correctness-ergonomics branch August 7, 2026 17:03
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.

2 participants