Skip to content

fix(core): bound web tool failures#33259

Merged
kitlangton merged 3 commits into
devfrom
web-tool-errors
Jun 21, 2026
Merged

fix(core): bound web tool failures#33259
kitlangton merged 3 commits into
devfrom
web-tool-errors

Conversation

@kitlangton

Copy link
Copy Markdown
Contributor

Summary

  • enforce the websearch response limit while consuming the response stream instead of buffering the full body first
  • stop reading and cancel oversized response streams as soon as they cross the limit
  • return HTML-to-Markdown conversion exceptions as model-visible webfetch failures

Test plan

  • bun run test test/tool-websearch.test.ts test/tool-webfetch.test.ts from packages/core (24 passed)
  • bun typecheck from packages/core
  • pre-push bun turbo typecheck (23 packages passed)
  • Prettier and git diff --check

@kitlangton kitlangton merged commit 69f1ec2 into dev Jun 21, 2026
7 of 8 checks passed
@kitlangton kitlangton deleted the web-tool-errors branch June 21, 2026 19:37
Moixia pushed a commit to Moixia/Sonder that referenced this pull request Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant