Note that if relying only on the shown event + accepts, we might miss out on certain rejections

What rejections would you miss out?

Respond back via the handleItemDidShow with empty insertText if there is lastCompletion, but no currentCompletion.

I think this would be very confusing, as we also pass in the completion item.

Create a new handleItemDidHide callback on the provider, which could also carry additional information about why the completion was hidden (cursor move/esc/new suggestion proposed/dropdown option selected)

Understanding the reason why a completion was hidden might be useful.

@isidorn do you know if Copilot had similar asks?

What rejections would you miss out?

We are relying on some additional heuristics (e.g. waiting for a short time so the user has time to react after a completion has rendered) to account for all usable completions as the didShow might have also been a flicker between 2 different suggestions. The Rejection for us, is basically not accepted usable completion.

Reason why we would miss out: At the moment the didShow will fire when there is a new completion, as well as if the completions change. But it does not signal when a completion has been hidden (e.g. by pressing Esc/changing the CompletionItem in dropdown/not having any more a suggestion provided), so in this case we would incorrectly account for the completion as usable (e.g. we would have to calculate the time to be able to react only between two Completions, which might be wrong in cases when - completion is provided, then hidden, then a new one is provided).

We could potentially calculate it via the items that we provide to the CompletionProvider, but that wouldn't be fully correct, as we can provide multiple items, which do not have to be shown.

Understanding the reason why a completion was hidden might be useful.

+1, that would be very helpful for us and a preferred way, although I understand that it might change multiple places throughout the Stack.

@isidorn do you know if Copilot had similar asks?

I think this would be useful for Copilot, though I haven't thought about it in detail. The new callback seems best for migration.

@hediet I see that there has been rejection added to the inlineCompletion handlers, can we try to bind it to the Show listeners so we can signal to the extensions that the state of a completion has changed? #242284

It would be also helpful if we can add the rejection to the extension as this binding does not currently exist #241328 (This would require updates after the first proposed PR, but didn't want to mash them together)