Don't coalesce or remove unused imports with invalid module specifiers

microsoft · Feb 15, 2018 · 0c2babc · 0c2babc
1 parent 7498043
commit 0c2babc
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/services/organizeImports.ts b/src/services/organizeImports.ts
@@ -20,7 +20,9 @@ namespace ts.OrganizeImports {
             compareModuleSpecifiers(group1[0].moduleSpecifier, group2[0].moduleSpecifier));
 
         const newImportDecls = flatMap(sortedImportGroups, importGroup =>
-            coalesceImports(removeUnusedImports(importGroup)));
+            getExternalModuleName(importGroup[0].moduleSpecifier)
+                ? coalesceImports(removeUnusedImports(importGroup))
+                : importGroup);
 
         const changeTracker = textChanges.ChangeTracker.fromContext({ host, formatContext });
 
@@ -60,7 +62,7 @@ namespace ts.OrganizeImports {
      * @param importGroup a list of ImportDeclarations, all with the same module name.
      */
     export function coalesceImports(importGroup: ReadonlyArray<ImportDeclaration>) {
-        if (importGroup.length === 0 || getExternalModuleName(importGroup[0].moduleSpecifier) === undefined) {
+        if (importGroup.length === 0) {
             return importGroup;
         }