Skip to content

OpenCode v1.3.7#8810

Merged
catrielmuller merged 13 commits into
mainfrom
catrielmuller/kilo-opencode-v1.3.7
Apr 12, 2026
Merged

OpenCode v1.3.7#8810
catrielmuller merged 13 commits into
mainfrom
catrielmuller/kilo-opencode-v1.3.7

Conversation

@catrielmuller

Copy link
Copy Markdown
Contributor

Core

  • Added first-class PowerShell support on Windows
  • Plugin installs now preserve JSONC comments in configuration files
  • Fixed OPENCODE_DISABLE_CLAUDE_CODE_PROMPT not being respected for project-level CLAUDE.md

TUI

  • Improved variant modal behavior to be less intrusive
  • Added theme colors for dialog textarea placeholders

@catrielmuller catrielmuller merged commit 1fbb9d2 into main Apr 12, 2026
16 of 17 checks passed
@catrielmuller catrielmuller deleted the catrielmuller/kilo-opencode-v1.3.7 branch April 12, 2026 17:41
})

const append = (chunk: Buffer) => {
output += chunk.toString()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

WARNING: UTF-8 stream decoding regression

This switches the bash tool back to chunk.toString() even though this path previously used StringDecoder to preserve multibyte UTF-8 characters split across stream chunks. If a code point arrives across two data events, the tool output will now be corrupted with replacement characters.

Comment thread script/changelog.ts
)
.then((x) => x.data?.parts?.find((y) => y.type === "text")?.text ?? message)
return result.trim()
for (const commit of commits) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

WARNING: Changelog prompt contains pasted script logic

The template literal that starts on the previous line now includes a for (const commit of commits) block instead of continuing with the commit prompt, so this function no longer returns a valid one-line summary. As written, script/changelog.ts ends up referencing undefined names like commits, seen, and list, which will break changelog generation.

@kilo-code-bot

kilo-code-bot Bot commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 2
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

WARNING

File Line Issue
packages/opencode/src/tool/bash.ts 340 UTF-8 output can be corrupted when multibyte characters are split across stream chunks.
script/changelog.ts 193 The changelog prompt template contains pasted script logic and now breaks changelog generation.
Other Observations (not in diff)

No additional high-confidence issues found outside the diff hunks.

Files Reviewed (48 files)
  • .gitignore - 0 issues
  • .opencode/command/changelog.md - 0 issues
  • bun.lock - 0 issues
  • flake.nix - 0 issues
  • package.json - 0 issues
  • packages/app/src/i18n/de.ts - 0 issues
  • packages/app/src/i18n/fr.ts - 0 issues
  • packages/app/src/i18n/th.ts - 0 issues
  • packages/app/src/i18n/tr.ts - 0 issues
  • packages/app/src/i18n/zht.ts - 0 issues
  • packages/app/src/pages/layout/sidebar-items.tsx - 0 issues
  • packages/extensions/zed/extension.toml - 0 issues
  • packages/opencode/package.json - 0 issues
  • packages/opencode/specs/tui-plugins.md - 0 issues
  • packages/opencode/src/cli/cmd/run.ts - 0 issues
  • packages/opencode/src/cli/cmd/tui/app.tsx - 0 issues
  • packages/opencode/src/cli/cmd/tui/component/dialog-model.tsx - 0 issues
  • packages/opencode/src/cli/cmd/tui/component/dialog-variant.tsx - 0 issues
  • packages/opencode/src/cli/cmd/tui/context/local.tsx - 0 issues
  • packages/opencode/src/cli/cmd/tui/routes/session/question.tsx - 0 issues
  • packages/opencode/src/cli/cmd/tui/ui/dialog-export-options.tsx - 0 issues
  • packages/opencode/src/cli/cmd/tui/ui/dialog-prompt.tsx - 0 issues
  • packages/opencode/src/cli/cmd/tui/ui/dialog-select.tsx - 0 issues
  • packages/opencode/src/config/migrate-tui-config.ts - 0 issues
  • packages/opencode/src/plugin/install.ts - 0 issues
  • packages/opencode/src/pty/index.ts - 0 issues
  • packages/opencode/src/session/compaction.ts - 0 issues
  • packages/opencode/src/session/instruction.ts - 0 issues
  • packages/opencode/src/session/llm.ts - 0 issues
  • packages/opencode/src/session/prompt.ts - 0 issues
  • packages/opencode/src/shell/shell.ts - 0 issues
  • packages/opencode/src/tool/bash.ts - 1 issue
  • packages/opencode/src/tool/bash.txt - 0 issues
  • packages/opencode/src/tool/external-directory.ts - 0 issues
  • packages/opencode/src/tool/read.ts - 0 issues
  • packages/opencode/src/util/filesystem.ts - 0 issues
  • packages/opencode/test/config/config.test.ts - 0 issues
  • packages/opencode/test/plugin/install.test.ts - 0 issues
  • packages/opencode/test/pty/pty-shell.test.ts - 0 issues
  • packages/opencode/test/shell/shell.test.ts - 0 issues
  • packages/opencode/test/tool/bash.test.ts - 0 issues
  • packages/opencode/test/tool/external-directory.test.ts - 0 issues
  • packages/opencode/test/tool/read.test.ts - 0 issues
  • packages/opencode/test/util/filesystem.test.ts - 0 issues
  • packages/sdk/js/src/client.ts - 0 issues
  • script/changelog.ts - 1 issue
  • script/version.ts - 0 issues
  • turbo.json - 0 issues

Reviewed by gpt-5.4-20260305 · 2,266,304 tokens

jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 18, 2026
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.

8 participants