Skip to content

fix(vscode): remember initial prompts in history - #12201

Merged
chrarnoldus merged 1 commit into
mainfrom
session/agent_180c8fe9-118d-4933-9d07-179e1dd84116
Jul 14, 2026
Merged

fix(vscode): remember initial prompts in history#12201
chrarnoldus merged 1 commit into
mainfrom
session/agent_180c8fe9-118d-4933-9d07-179e1dd84116

Conversation

@chrarnoldus

Copy link
Copy Markdown
Collaborator

What changed

Record a dispatched prompt in chat history before the pending draft is promoted to a real session key.

Why

The initial prompt could be sent successfully while session creation changed the active draft key. The cleanup guard then returned before recording that prompt, so ArrowUp only recalled later prompts from the established session.

The existing guard remains around visible draft cleanup, preventing an asynchronous send from clearing another tab's input.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Jul 14, 2026
@marius-kilocode

Copy link
Copy Markdown
Collaborator

Never noticed that

@kilo-code-bot

kilo-code-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • .changeset/remember-initial-prompts.md
  • packages/kilo-vscode/tests/unit/prompt-send-contract.test.ts
  • packages/kilo-vscode/webview-ui/src/components/chat/PromptInput.tsx

Reviewed by gpt-5.6-sol · Input: 73.2K · Output: 4.8K · Cached: 469.2K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus merged commit 7e20e4e into main Jul 14, 2026
23 checks passed
@chrarnoldus
chrarnoldus deleted the session/agent_180c8fe9-118d-4933-9d07-179e1dd84116 branch July 14, 2026 08:12
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…9-118d-4933-9d07-179e1dd84116

fix(vscode): remember initial prompts in history
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