Skip to content

Remove clipboard reading from chat autocomplete#5078

Merged
markijbema merged 1 commit intomainfrom
mark/remove-clipboard-reading
Jan 19, 2026
Merged

Remove clipboard reading from chat autocomplete#5078
markijbema merged 1 commit intomainfrom
mark/remove-clipboard-reading

Conversation

@markijbema
Copy link
Contributor

Removes the clipboard reading functionality from the chat textarea autocomplete feature.

@changeset-bot
Copy link

changeset-bot bot commented Jan 15, 2026

🦋 Changeset detected

Latest commit: d4cc35d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
kilo-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@markijbema markijbema marked this pull request as ready for review January 15, 2026 17:10
@markijbema markijbema requested a review from jrf0110 January 15, 2026 17:10
@markijbema markijbema enabled auto-merge January 15, 2026 17:10
@kiloconnect
Copy link
Contributor

kiloconnect bot commented Jan 15, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Overview

This PR removes clipboard reading functionality from the chat autocomplete feature. The changes are clean and straightforward:

  1. Removed getClipboardContext() method - The private method that read clipboard content via vscode.env.clipboard.readText() has been removed entirely.

  2. Removed clipboard context from buildPrefix() - The call to getClipboardContext() and the associated clipboard content injection into the context has been removed.

The remaining code in buildPrefix() correctly handles visible code context and user text without the clipboard dependency.

Files Reviewed (2 files)
  • .changeset/remove-clipboard-reading.md - Changeset for the patch
  • src/services/ghost/chat-autocomplete/ChatTextAreaAutocomplete.ts - Removed clipboard reading functionality

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