Skip to content

revert(cli): undo Bedrock buffer pin - #12095

Merged
chrarnoldus merged 1 commit into
mainfrom
revert/bedrock-buffer-pin
Jul 10, 2026
Merged

revert(cli): undo Bedrock buffer pin#12095
chrarnoldus merged 1 commit into
mainfrom
revert/bedrock-buffer-pin

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Reverts the dependency and lockfile changes from #11968 because the Bedrock fix has been superseded by #12079.\n\nThe changeset from #11968 is intentionally not included in this revert; it was already consumed by release v7.4.2.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Jul 10, 2026
Comment thread package.json
Comment thread bun.lock
@kilo-code-bot

kilo-code-bot Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 1
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

CRITICAL

File Line Issue
package.json 144 Removing the @smithy/util-buffer-from override lets it resolve back to 4.3.6, the version that caused Bedrock requests to silently return empty output on HTTP 200 (fixed in #11968). No accompanying fix for the root cause was found in this diff or recent history.

WARNING

File Line Issue
bun.lock 1936 Lockfile now resolves @smithy/util-buffer-from@4.3.6, confirming the regression risk described in the package.json comment.
Files Reviewed (2 files)
  • package.json - 1 issue
  • bun.lock - 1 issue

Fix these issues in Kilo Cloud


Reviewed by claude-sonnet-5-20260630 · Input: 38 · Output: 16.7K · Cached: 976.3K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus merged commit 7a5563e into main Jul 10, 2026
27 checks passed
@chrarnoldus
chrarnoldus deleted the revert/bedrock-buffer-pin branch July 10, 2026 11:44
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…-pin

revert(cli): undo Bedrock buffer pin
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