Addressed PR comments

Armando Aguirre · Armando Aguirre · commit f17603d3187b · 2018-04-18T12:58:40.000-07:00
diff --git a/src/compiler/core.ts b/src/compiler/core.ts
@@ -2677,7 +2677,7 @@ namespace ts {
 
         const extensions = [
             ...needJsExtensions ? allSupportedExtensions : supportedTypeScriptExtensions,
-            ...extraFileExtensions.filter(x => x.scriptKind === ScriptKind.Deferred || needJsExtensions && isJavaScriptLike(x.scriptKind)).map(x => x.extension)
+            ...mapDefined(extraFileExtensions, x => x.scriptKind === ScriptKind.Deferred || needJsExtensions && isJavaScriptLike(x.scriptKind) ? x.extension : undefined)
         ];
 
         return deduplicate(extensions, equateStringsCaseSensitive, compareStringsCaseSensitive);
diff --git a/src/compiler/types.ts b/src/compiler/types.ts
@@ -4066,6 +4066,8 @@ namespace ts {
         Prototype,
     }
 
+    export type JsFileExtensionInfo = FileExtensionInfo;
+
     export interface FileExtensionInfo {
         extension: string;
         isMixedContent: boolean;
diff --git a/tests/baselines/reference/api/tsserverlibrary.d.ts b/tests/baselines/reference/api/tsserverlibrary.d.ts
@@ -2266,6 +2266,7 @@ declare namespace ts {
         AlwaysStrict = 64,
         PriorityImpliesCombination = 28
     }
+    type JsFileExtensionInfo = FileExtensionInfo;
     interface FileExtensionInfo {
         extension: string;
         isMixedContent: boolean;
diff --git a/tests/baselines/reference/api/typescript.d.ts b/tests/baselines/reference/api/typescript.d.ts
@@ -2266,6 +2266,7 @@ declare namespace ts {
         AlwaysStrict = 64,
         PriorityImpliesCombination = 28
     }
+    type JsFileExtensionInfo = FileExtensionInfo;
     interface FileExtensionInfo {
         extension: string;
         isMixedContent: boolean;

Original file line number	Diff line number	Diff line change
`@@ -4066,6 +4066,8 @@ namespace ts {`
`4066`	`4066`	`Prototype,`
`4067`	`4067`	`}`
`4068`	`4068`
	`4069`	`+ export type JsFileExtensionInfo = FileExtensionInfo;`
	`4070`	`+`
`4069`	`4071`	`export interface FileExtensionInfo {`
`4070`	`4072`	`extension: string;`
`4071`	`4073`	`isMixedContent: boolean;`
Original file line number	Diff line number	Diff line change
`@@ -2266,6 +2266,7 @@ declare namespace ts {`
`2266`	`2266`	`AlwaysStrict = 64,`
`2267`	`2267`	`PriorityImpliesCombination = 28`
`2268`	`2268`	`}`
	`2269`	`+ type JsFileExtensionInfo = FileExtensionInfo;`
`2269`	`2270`	`interface FileExtensionInfo {`
`2270`	`2271`	`extension: string;`
`2271`	`2272`	`isMixedContent: boolean;`