Skip to content

fix(core): clarify binary read errors#31031

Closed
opencode-agent[bot] wants to merge 1 commit into
devfrom
fix/v2-read-errors
Closed

fix(core): clarify binary read errors#31031
opencode-agent[bot] wants to merge 1 commit into
devfrom
fix/v2-read-errors

Conversation

@opencode-agent
Copy link
Copy Markdown
Contributor

@opencode-agent opencode-agent Bot commented Jun 5, 2026

Summary

  • preserve specific V2 read failures for oversized and binary files
  • include the affected resource and byte limit in error messages
  • keep generic read failures redacted behind the existing fallback

Test plan

  • bun test test/tool-read.test.ts from packages/core
  • bun typecheck from packages/core
  • repository pre-push bun turbo typecheck

@kitlangton kitlangton marked this pull request as ready for review June 5, 2026 21:56
@kitlangton kitlangton enabled auto-merge (squash) June 5, 2026 21:56
@kitlangton
Copy link
Copy Markdown
Contributor

Superseded by #31038, which incorporates the safe error behavior into the complete binary/media read fix on current .

@kitlangton kitlangton closed this Jun 5, 2026
auto-merge was automatically disabled June 5, 2026 23:16

Pull request was closed

@kitlangton
Copy link
Copy Markdown
Contributor

Correction: #31038 incorporates the safe error behavior into the complete binary/media read fix on current dev.

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