fix: prevent autoimport from importing same package many times

src/autoimport/model.py

@@ -239,11 +239,14 @@ def _move_imports_to_top(self) -> None:
     def _fix_flake_import_errors(self) -> None:
         """Fix python source code to correct missed or unused import statements."""
         error_messages = autoflake.check(self._join_code())
+        fixed_packages = []
 
         for message in error_messages:
             if isinstance(message, (UndefinedName, UndefinedExport)):
                 object_name = message.message_args[0]
-                self._add_package(object_name)
+                if object_name not in fixed_packages:
+                    self._add_package(object_name)
+                    fixed_packages.append(object_name)
             elif isinstance(message, UnusedImport):
                 import_name = message.message_args[0]
                 self._remove_unused_imports(import_name)

tests/unit/test_services.py

@@ -801,3 +801,29 @@ def test_fix_respects_leading_comments_with_new_lines() -> None:
     result = fix_code(source)
 
     assert result == desired_source
+
+
+def test_fix_imports_dependency_only_once() -> None:
+    """
+    Given: Code with a line that uses a package three times.
+    When: Fix code is run.
+    Then: The dependency is imported only once
+    """
+    source = dedent(
+        """\
+        def f(x):
+            return os.getcwd() + os.getcwd() + os.getcwd()
+        """
+    )
+    desired_source = dedent(
+        """\
+        import os
+
+        def f(x):
+            return os.getcwd() + os.getcwd() + os.getcwd()
+        """
+    )
+
+    result = fix_code(source)
+
+    assert result == desired_source