Skip to content

Commit 01d1723

Browse files
committed
simlpify check, if not enabled, return nothing
1 parent fdabd74 commit 01d1723

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

src/services/ghost/classic-auto-complete/GhostInlineCompletionProvider.ts

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -230,29 +230,23 @@ export class GhostInlineCompletionProvider implements vscode.InlineCompletionIte
230230
public async provideInlineCompletionItems(
231231
document: vscode.TextDocument,
232232
position: vscode.Position,
233-
context: vscode.InlineCompletionContext,
233+
_context: vscode.InlineCompletionContext,
234234
_token: vscode.CancellationToken,
235235
): Promise<vscode.InlineCompletionItem[] | vscode.InlineCompletionList> {
236-
// Check if auto-trigger is enabled
237-
// Only proceed with LLM call if:
238-
// 1. It's a manual trigger (triggerKind === Invoke), OR
239-
// 2. Auto-trigger is enabled (enableAutoTrigger === true)
240-
const isManualTrigger = context.triggerKind === vscode.InlineCompletionTriggerKind.Invoke
241236
const settings = this.getSettings()
242237
const isAutoTriggerEnabled = settings?.enableAutoTrigger ?? false
243238

244-
if (!isManualTrigger && !isAutoTriggerEnabled) {
245-
// Auto-trigger is disabled and this is not a manual trigger
239+
if (!isAutoTriggerEnabled) {
246240
return []
247241
}
248242

249-
return this.provideInlineCompletionItems_Internal(document, position, context, _token)
243+
return this.provideInlineCompletionItems_Internal(document, position, _context, _token)
250244
}
251245

252246
public async provideInlineCompletionItems_Internal(
253247
document: vscode.TextDocument,
254248
position: vscode.Position,
255-
context: vscode.InlineCompletionContext,
249+
_context: vscode.InlineCompletionContext,
256250
_token: vscode.CancellationToken,
257251
): Promise<vscode.InlineCompletionItem[] | vscode.InlineCompletionList> {
258252
const { prefix, suffix } = extractPrefixSuffix(document, position)

0 commit comments

Comments
 (0)