Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

# Full acceptance signal for the inlineCompletions #242290

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

marrej
Copy link
Contributor

@marrej marrej commented Feb 28, 2025

Problem statement

The current acceptance always signals only partial edits. Once the full edit is hit the full acceptance command is triggered (even though this was signalled from the partial acceptance), this reduces understanding in to when the partial acceptance signals are used (e.g. users accepting completions via line acceptances at all times).

The patial acceptance handler also can carry more data that can be core calculated, improving the data flow without need for extension overwork.

lastly it allows to paint the full picture with the handlers without the dichotomy of "handlers+commands", and always expects the show->accept->show/reject as next signal. (Note that 242284 tries to add the reject)

Proposal

Extend the current Accept handler to allow the partial accepts to signal through it that they have been invoked.
Add observable value in the didShow to allow it to be always outside ot the Acceptance, allowing for correct interpretation of show->accept->reject/show

note that with #241356 a much better picture could be captured of the whole code completion lifecycle.

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