Skip to content

Commit 49c807e

Browse files
author
Andy
authored
Fix duplicate completions bugs (microsoft#23632)
1 parent 96e90dd commit 49c807e

File tree

5 files changed

+44
-63
lines changed

5 files changed

+44
-63
lines changed

‎src/harness/fourslash.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -854,8 +854,7 @@ namespace FourSlash {
854854
const actualByName = ts.createMap<ts.CompletionEntry>();
855855
for (const entry of actualCompletions.entries) {
856856
if (actualByName.has(entry.name)) {
857-
// TODO: GH#23587
858-
if (entry.name !== "undefined" && entry.name !== "require") this.raiseError(`Duplicate completions for ${entry.name}`);
857+
this.raiseError(`Duplicate completions for ${entry.name}`);
859858
}
860859
else {
861860
actualByName.set(entry.name, entry);

‎src/services/completions.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2067,16 +2067,17 @@ namespace ts.Completions {
20672067
const kind = stringToToken(entry.name);
20682068
switch (keywordFilter) {
20692069
case KeywordCompletionFilters.None:
2070-
// "undefined" is a global variable, so don't need a keyword completion for it.
2071-
return kind !== SyntaxKind.UndefinedKeyword;
2070+
// TODO: GH#23631 Includes type keywords because `Array<numb/**/` is treaded as a value location currently.
2071+
return !isContextualKeyword(kind) && !isClassMemberCompletionKeyword(kind) || kind === SyntaxKind.DeclareKeyword || kind === SyntaxKind.ModuleKeyword
2072+
|| isTypeKeyword(kind) && kind !== SyntaxKind.UndefinedKeyword;
20722073
case KeywordCompletionFilters.ClassElementKeywords:
20732074
return isClassMemberCompletionKeyword(kind);
20742075
case KeywordCompletionFilters.InterfaceElementKeywords:
20752076
return isInterfaceOrTypeLiteralCompletionKeyword(kind);
20762077
case KeywordCompletionFilters.ConstructorParameterKeywords:
20772078
return isParameterPropertyModifier(kind);
20782079
case KeywordCompletionFilters.FunctionLikeBodyKeywords:
2079-
return !isClassMemberCompletionKeyword(kind);
2080+
return !isContextualKeyword(kind) && !isClassMemberCompletionKeyword(kind);
20802081
case KeywordCompletionFilters.TypeKeywords:
20812082
return isTypeKeyword(kind);
20822083
default:

‎tests/cases/fourslash/completionEntryForPrimitive.ts

Lines changed: 0 additions & 8 deletions
This file was deleted.

‎tests/cases/fourslash/completionListKeywords.ts

Lines changed: 39 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,42 @@
11
/// <reference path="fourslash.ts"/>
22

3-
/////**/
3+
////
44

5-
goTo.marker();
6-
verify.completionListContains("break");
7-
verify.completionListContains("case");
8-
verify.completionListContains("catch");
9-
verify.completionListContains("class");
10-
verify.completionListContains("constructor");
11-
verify.completionListContains("continue");
12-
verify.completionListContains("debugger");
13-
verify.completionListContains("declare");
14-
verify.completionListContains("default");
15-
verify.completionListContains("delete");
16-
verify.completionListContains("do");
17-
verify.completionListContains("else");
18-
verify.completionListContains("enum");
19-
verify.completionListContains("export");
20-
verify.completionListContains("extends");
21-
verify.completionListContains("false");
22-
verify.completionListContains("finally");
23-
verify.completionListContains("for");
24-
verify.completionListContains("function");
25-
verify.completionListContains("get");
26-
verify.completionListContains("if");
27-
verify.completionListContains("implements");
28-
verify.completionListContains("import");
29-
verify.completionListContains("in");
30-
verify.completionListContains("instanceof");
31-
verify.completionListContains("interface");
32-
verify.completionListContains("module");
33-
verify.completionListContains("new");
34-
verify.completionListContains("private");
35-
verify.completionListContains("public");
36-
verify.completionListContains("return");
37-
verify.completionListContains("set");
38-
verify.completionListContains("static");
39-
verify.completionListContains("super");
40-
verify.completionListContains("switch");
41-
verify.completionListContains("this");
42-
verify.completionListContains("throw");
43-
verify.completionListContains("true");
44-
verify.completionListContains("try");
45-
verify.completionListContains("typeof");
46-
verify.completionListContains("var");
47-
verify.completionListContains("while");
48-
verify.completionListContains("with");
5+
verify.completions({
6+
includes: [
7+
"break",
8+
"case",
9+
"catch",
10+
"class",
11+
"continue",
12+
"debugger",
13+
"declare",
14+
"default",
15+
"delete",
16+
"do",
17+
"else",
18+
"enum",
19+
"export",
20+
"extends",
21+
"false",
22+
"finally",
23+
"for",
24+
"function",
25+
"if",
26+
"instanceof",
27+
"interface",
28+
"module",
29+
"new",
30+
"return",
31+
"super",
32+
"switch",
33+
"this",
34+
"throw",
35+
"true",
36+
"try",
37+
"typeof",
38+
"var",
39+
"while",
40+
"with",
41+
],
42+
});

‎tests/cases/fourslash/completionListPrimitives.ts

Lines changed: 0 additions & 5 deletions
This file was deleted.

0 commit comments

Comments
 (0)