From 13eb13b9ebab0927484eb3910bd403b4c69b8783 Mon Sep 17 00:00:00 2001 From: Megan Rogge Date: Thu, 3 Jul 2025 11:59:17 -0400 Subject: [PATCH] don't add `inlineCompletionItem` to model unless it's supported (#253886) fix #252367 --- .../suggest/browser/terminalSuggestAddon.ts | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/vs/workbench/contrib/terminalContrib/suggest/browser/terminalSuggestAddon.ts b/src/vs/workbench/contrib/terminalContrib/suggest/browser/terminalSuggestAddon.ts index 1f5365fb007..54457166471 100644 --- a/src/vs/workbench/contrib/terminalContrib/suggest/browser/terminalSuggestAddon.ts +++ b/src/vs/workbench/contrib/terminalContrib/suggest/browser/terminalSuggestAddon.ts @@ -342,11 +342,13 @@ export class SuggestAddon extends Disposable implements ITerminalAddon, ISuggest } const lineContext = new LineContext(normalizedLeadingLineContent, this._cursorIndexDelta); + const items = completions.filter(c => !!c.label).map(c => new TerminalCompletionItem(c)); + if (isInlineCompletionSupported(this.shellType)) { + items.push(this._inlineCompletionItem); + } + const model = new TerminalCompletionModel( - [ - ...completions.filter(c => !!c.label).map(c => new TerminalCompletionItem(c)), - this._inlineCompletionItem, - ], + items, lineContext ); if (token.isCancellationRequested) {