Skip to content

fix(vscode): clarify keep-awake indicator - #13890

Merged
marius-kilocode merged 2 commits into
mainfrom
fix-coffee-icon-enabled-state
Sep 7, 2026
Merged

marius-kilocode merged 2 commits into
mainfrom
fix-coffee-icon-enabled-state

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

What Problem This Solves

Keep Awake used the same outline coffee icon when disabled and enabled, and its tooltip did not explain that it applies while Kilo agents work.

Why This Change Was Made

The enabled state now uses a filled coffee icon. All Agent Manager locale strings describe the computer sleep prevention scope and retain the existing Kilo terminology.

User Impact

Users can identify the enabled Keep Awake state at a glance and understand that it prevents system sleep while Kilo agents work. It does not keep the display on or disable screen locking.

Evidence

Agent Manager Keep Awake enabled with filled coffee icon and clarified tooltip

Focused i18n and caffeination tests pass.

@kilo-code-bot

kilo-code-bot Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (27 files)
  • .changeset/coffee-indicator.md
  • packages/kilo-ui/src/components/icon.tsx
  • packages/kilo-ui/tests/diff-indicators.spec.ts
  • packages/kilo-vscode/tests/fixtures/caffeination-button.tsx
  • packages/kilo-vscode/tests/unit/caffeination-button.test.ts
  • packages/kilo-vscode/webview-ui/agent-manager/CaffeinationButton.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ar.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/br.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/bs.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/da.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/de.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/en.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/es.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fa.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/it.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ja.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ko.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/nl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/no.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/pl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ru.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/th.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/tr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/uk.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zh.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zht.ts
Previous Review Summary (commit a99f63a)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit a99f63a)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (26 files)
  • .changeset/coffee-indicator.md
  • packages/kilo-ui/src/components/icon.tsx
  • packages/kilo-vscode/tests/fixtures/caffeination-button.tsx
  • packages/kilo-vscode/tests/unit/caffeination-button.test.ts
  • packages/kilo-vscode/webview-ui/agent-manager/CaffeinationButton.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ar.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/br.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/bs.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/da.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/de.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/en.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/es.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fa.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/it.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ja.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ko.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/nl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/no.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/pl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ru.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/th.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/tr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/uk.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zh.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zht.ts

Reviewed by grok-4.6 · Input: 30.1K · Output: 4.5K · Cached: 152.7K

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode merged commit ea902b9 into main Sep 7, 2026
34 checks passed
@marius-kilocode
marius-kilocode deleted the fix-coffee-icon-enabled-state branch September 7, 2026 17:21
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