Merge pull request #165 from CodeshiftCommunity/fix-import-spec-inserter

reference handling insertImportSpec

Author: danieldelcore

.changeset/lazy-crabs-vanish.md

@@ -0,0 +1,5 @@
+---
+'@codeshift/utils': patch
+---
+
+InsertImportSpecifier now retains the reference to the original import declaration

packages/utils/src/imports.ts

@@ -122,18 +122,9 @@ export function insertImportSpecifier(
   importSpecifier: ImportSpecifier | ImportDefaultSpecifier,
   sourcePath: string,
 ) {
-  getImportDeclaration(j, source, sourcePath).replaceWith(declaration =>
-    j.importDeclaration(
-      [
-        // we are appending to the existing specifiers
-        // We are doing a filter hear because sometimes specifiers can be removed
-        // but they hang around in the declaration
-        ...(declaration.value.specifiers || []).filter(
-          item => item.type === 'ImportSpecifier' && item.imported != null,
-        ),
-        importSpecifier,
-      ],
-      j.literal(sourcePath),
-    ),
-  );
+  const importDeclaration = getImportDeclaration(j, source, sourcePath);
+
+  if (!importDeclaration) return;
+
+  importDeclaration.get().value.specifiers.push(importSpecifier);
 }