fix(migrations): remove setting that removes comments in CF migration (#53350)

thePunderWoman · dylhunn · commit 0db75ab5b1c8 · 2023-12-04T21:41:41.000-08:00
This setting was added to prevent comment duplication, since the TS AST printer includes prior line comments as part of a given line with no way to really avoid that. However in component imports, it is not safe to remove comments as they could be load bearing for some. PR Close #53350
diff --git a/packages/core/schematics/ng-generate/control-flow-migration/util.ts b/packages/core/schematics/ng-generate/control-flow-migration/util.ts
@@ -78,13 +78,7 @@ function updateImportClause(clause: ts.ImportClause, removeCommonModule: boolean
 
 function updateClassImports(
     propAssignment: ts.PropertyAssignment, removeCommonModule: boolean): string|null {
-  // removeComments is set to true to prevent duplication of comments
-  // when the import declaration is at the top of the file, but right after a comment
-  // without this, the comment gets duplicated when the declaration is updated.
-  // the typescript AST includes that preceding comment as part of the import declaration full text.
-  const printer = ts.createPrinter({
-    removeComments: true,
-  });
+  const printer = ts.createPrinter();
   const importList = propAssignment.initializer as ts.ArrayLiteralExpression;
   const removals = removeCommonModule ? importWithCommonRemovals : importRemovals;
   const elements = importList.elements.filter(el => !removals.includes(el.getText()));
