Skip to content

fix(vscode): use native sandbox notifications - #11645

Merged
marius-kilocode merged 2 commits into
mainfrom
like-drawer
Jun 25, 2026
Merged

fix(vscode): use native sandbox notifications#11645
marius-kilocode merged 2 commits into
mainfrom
like-drawer

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

The sandbox toggle currently reports successful state changes inside the webview, unlike auto-approve and other VS Code-level mode changes. Its longer explanatory copy also makes a simple enabled or disabled transition harder to scan.

This moves successful sandbox feedback to native VS Code notifications with concise Sandbox enabled and Sandbox disabled messages. Detailed failure feedback remains in the webview, and native notifications are only emitted after a toggle completes successfully.

@marius-kilocode
marius-kilocode enabled auto-merge (squash) June 24, 2026 15:26
@kilo-code-bot

kilo-code-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (6 files)
  • .changeset/notify-sandbox-state.md
  • packages/kilo-vscode/src/KiloProvider.ts
  • packages/kilo-vscode/tests/setup/vscode-mock.ts
  • packages/kilo-vscode/tests/unit/kilo-provider-load-messages.test.ts
  • packages/kilo-vscode/tests/unit/prompt-input-connection-guard.test.ts
  • packages/kilo-vscode/webview-ui/src/components/chat/PromptInput.tsx

Reviewed by gpt-5.4-2026-03-05 · Input: 243.4K · Output: 12.2K · Cached: 462.9K

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode disabled auto-merge June 25, 2026 08:00
@marius-kilocode
marius-kilocode merged commit 9dfd2b0 into main Jun 25, 2026
24 checks passed
@marius-kilocode
marius-kilocode deleted the like-drawer branch June 25, 2026 08:16
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
fix(vscode): use native sandbox notifications
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