fix(migrations): resolve text replacement issue (#59452)

crisbeto · kirjs · commit 14fb8ce4c00f · 2025-01-10T10:47:54.000-05:00
Based on #59353 (comment), fixes a text replacement issue that seems to cause code to be duplicated in some setups. PR Close #59452
diff --git a/packages/core/schematics/ng-generate/cleanup-unused-imports/index.ts b/packages/core/schematics/ng-generate/cleanup-unused-imports/index.ts
@@ -67,7 +67,7 @@ export function migrate(): Rule {
       for (const c of changes) {
         recorder
           .remove(c.data.position, c.data.end - c.data.position)
-          .insertLeft(c.data.position, c.data.toInsert);
+          .insertRight(c.data.position, c.data.toInsert);
       }
       tree.commitUpdate(recorder);
     }

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ export function migrate(): Rule {`
`67`	`67`	`for (const c of changes) {`
`68`	`68`	`recorder`
`69`	`69`	`.remove(c.data.position, c.data.end - c.data.position)`
`70`		`- .insertLeft(c.data.position, c.data.toInsert);`
	`70`	`+ .insertRight(c.data.position, c.data.toInsert);`
`71`	`71`	`}`
`72`	`72`	`tree.commitUpdate(recorder);`
`73`	`73`	`}`