Skip to content

fix(cli): restore Amazon Bedrock output by pinning @smithy/util-buffer-from - #11968

Merged
chrarnoldus merged 1 commit into
mainfrom
fix/bedrock-empty-output
Jul 6, 2026
Merged

fix(cli): restore Amazon Bedrock output by pinning @smithy/util-buffer-from#11968
chrarnoldus merged 1 commit into
mainfrom
fix/bedrock-empty-output

Conversation

@chrarnoldus

Copy link
Copy Markdown
Collaborator

@smithy/util-buffer-from >=4.3.0 became a re-export shim of @smithy/core/serde. Under the browser build condition, @smithy/core's serde exposes fromArrayBuffer as Symbol.for("node-only"), so @smithy/util-utf8's toUtf8 throws inside the Bedrock event-stream decoder, whose catch { break } silently ends the stream. Every Bedrock request completed with empty output on HTTP 200.

Pin to 4.2.2 (last self-contained 4.x) so fromArrayBuffer stays a real function.

…r-from

@smithy/util-buffer-from >=4.3.0 became a re-export shim of @smithy/core/serde.
Under the browser build condition, @smithy/core's serde exposes fromArrayBuffer
as Symbol.for("node-only"), so @smithy/util-utf8's toUtf8 throws inside the
Bedrock event-stream decoder, whose catch { break } silently ends the stream.
Every Bedrock request completed with empty output on HTTP 200.

Pin to 4.2.2 (last self-contained 4.x) so fromArrayBuffer stays a real function.
@chrarnoldus chrarnoldus linked an issue Jul 6, 2026 that may be closed by this pull request
@kilo-code-bot

kilo-code-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Dependency pin (@smithy/util-buffer-from4.2.2) added via the existing overrides block, following the same pattern as other pinned deps in package.json (e.g. lodash, defu, dompurify). bun.lock changes are consistent with regenerating the lockfile after the override, and a user-facing changeset is included.

Files Reviewed (3 files)
  • .changeset/fix-bedrock-empty-output.md
  • bun.lock
  • package.json

Reviewed by claude-sonnet-5-20260630 · Input: 20 · Output: 4.6K · Cached: 386.7K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus enabled auto-merge July 6, 2026 12:28
@chrarnoldus
chrarnoldus merged commit f2639e9 into main Jul 6, 2026
25 checks passed
@chrarnoldus
chrarnoldus deleted the fix/bedrock-empty-output branch July 6, 2026 12:29
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
fix(cli): restore Amazon Bedrock output by pinning @smithy/util-buffer-from
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.

AWS Bedrock models are not working

3 participants