switched yieldItems List->Set for quicker "includes" (#302)

neo4j · Dec 3, 2024 · 532ff90 · 532ff90
1 parent 13c81dc
commit 532ff90
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/packages/language-support/src/autocompletion/completionCoreCompletions.ts b/packages/language-support/src/autocompletion/completionCoreCompletions.ts
@@ -478,12 +478,12 @@ export function completionCoreCompletion(
         );
         if (callContext instanceof CallClauseContext) {
           const procedureNameCtx = callContext.procedureName();
-          const existingYieldItems = callContext
-            .procedureResultItem_list()
-            .map((a) => a.getText());
+          const existingYieldItems = new Set(
+            callContext.procedureResultItem_list().map((a) => a.getText()),
+          );
           const name = getMethodName(procedureNameCtx);
           return procedureReturnCompletions(name, dbSchema).filter(
-            (a) => !existingYieldItems.includes(a?.label),
+            (a) => !existingYieldItems.has(a?.label),
           );
         }
       }