Repository navigation
Conversation
afb0763 to
58d78fb
Compare
| <span class="sr-only">{language.t("session.messages.scrollToBottom")}</span> | ||
| </Button> | ||
| <Show when={isRetrying()}> | ||
| <Button |
There was a problem hiding this comment.
SUGGESTION: The new copy action gives no user-visible confirmation
Unlike the existing ToolErrorCard copy button (packages/ui/src/components/tool-error-card.tsx), which flips to a transient "Copied" state, this button stays on "Copy error" after activation and a failed clipboard write is only logged to the console. A short-lived confirmation (and/or a visible error surface) would tell the user the copy actually happened.
This is a suggestion only — if the constant label is deliberate to keep the retry row from reflowing, ignore it.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)SUGGESTION
Files Reviewed (4 files)
Fix these issues in Kilo Cloud Reviewed by deepseek-v4.1-flash · Input: 64.3K · Output: 26.1K · Cached: 1.7M Review guidance: REVIEW.md from base branch |

Issue
Fixes #15037
Context
Long provider errors were clipped in the VS Code WebView while a request was retrying. The hidden text contained the reason the request failed and the setting needed to resolve it. The retry row now lets the user copy the full message.
Implementation
Screenshots / Video
Before
provider-error-truncated.mp4
After
provider-error-truncated-after.mp4
How to Test
Manual/local verification
bun run typecheckfrompackages/kilo-vscodepassed.bun run lintfrompackages/kilo-vscodepassed.bunx playwright test tests/session-dock-stability.spec.ts --workers=2frompackages/kilo-vscodepassed: 18 tests.bun run check-kilocode-changefrompackages/kilo-vscodepassed.bun turbo typecheck --filter=!@kilocode/kilo-jetbrainsand passed.Reviewer test steps
The following fixture uses the same HTTP 503 response, error message, and model configuration as the recorded reproduction. It runs locally and requires no provider account.
Set up the local provider fixture
From the Kilo checkout, run this in a macOS/Linux shell with Node.js, Bun, and VS Code installed:
Wait for
provider ready on 18937. The fixture returns the complete error message with HTTP 503 andretry-after: 60. The workspace configuration selectsrepro/error-modelathttp://127.0.0.1:18937/v1using the placeholder keylocal-test.Reply with OK.and wait for the retry row. Confirm the long status is clipped while Copy error and Cancel remain visible.The selected model cannot process this requestthroughAllowed Regions to Global..For a separate deterministic UI check, run
bun run --cwd packages/kilo-vscode storybookfrom the Kilo checkout and open http://localhost:6007/?path=/story/chat--chat-view-session-dock-stability. Click toggle busy, then toggle retry. Verify Copy error with a mouse click and with Tab followed by Enter, pasting after each activation to check the complete message. Click toggle retry again to hide the copy control, then toggle busy to return to idle. The automated coverage runs withbunx playwright test tests/session-dock-stability.spec.ts --workers=2frompackages/kilo-vscode.Blocked checks and substitute verification
Checklist
Get in Touch