Blacklist some stdlib modules from auto-import

Author: loic-simon

Lib/_pyrepl/_module_completer.py

@@ -16,6 +16,7 @@
 TYPE_CHECKING = False
 
 if TYPE_CHECKING:
+    from types import ModuleType
     from typing import Any, Iterable, Iterator, Mapping
     from .types import CompletionAction
 
@@ -28,6 +29,14 @@
     "xml.parsers.expat": ["errors", "model"],
 }
 
+AUTO_IMPORT_BLACKLIST = {
+    # Standard library modules/submodules that have import side effects
+    # and must not be automatically imported to complete attributes
+    "antigravity",
+    "this",
+    "idlelib.idle",
+}
+
 
 def make_default_module_completer() -> ModuleCompleter:
     # Inside pyrepl, __package__ is set to None by default
@@ -157,15 +166,7 @@ def _find_attributes(self, path: str, prefix: str) -> tuple[list[str], Completio
         if not imported_module:
             if path in self._failed_imports:  # Do not propose to import again
                 return [], None
-            root = path.split(".")[0]
-            mod_info = next((m for m in self.global_cache if m.name == root),
-                             None)
-            if mod_info and self._is_stdlib_module(mod_info):
-                # Stdlib module: auto-import (no risk of dangerous side-effect)
-                try:
-                    imported_module = importlib.import_module(path)
-                except Exception:
-                    sys.modules.pop(path, None)  # Clean half-imported module
+            imported_module = self._maybe_import_module(path)
         if not imported_module:
             return [], self._get_import_completion_action(path)
         try:
@@ -246,6 +247,21 @@ def global_cache(self) -> list[pkgutil.ModuleInfo]:
             self._global_cache = list(pkgutil.iter_modules())
         return self._global_cache
 
+    def _maybe_import_module(self, fqname: str) -> ModuleType | None:
+        if fqname in AUTO_IMPORT_BLACKLIST or fqname.endswith(".__main__"):
+            # Special-cased modules with known import side-effects
+            return None
+        root = fqname.split(".")[0]
+        mod_info = next((m for m in self.global_cache if m.name == root), None)
+        if not mod_info or not self._is_stdlib_module(mod_info):
+            # Only import stdlib modules (no risk of import side-effects)
+            return None
+        try:
+            return importlib.import_module(fqname)
+        except Exception:
+            sys.modules.pop(fqname, None)  # Clean half-imported module
+            return None
+
     def _get_import_completion_action(self, path: str) -> CompletionAction:
         prompt = ("[ module not imported, press again to import it "
                   "and propose attributes ]")

Lib/test/test_pyrepl/test_pyrepl.py

@@ -1163,6 +1163,9 @@ def test_attribute_completion_module_on_demand(self):
                     ("from pack import b\t\t\t\n", "from pack import ba", {"pack"}),
                     # stdlib modules are automatically imported
                     ("from graphlib import T\t\n", "from graphlib import TopologicalSorter", {"graphlib"}),
+                    # except those with known side-effects
+                    ("from antigravity import g\t\n", "from antigravity import g", set()),
+                    ("from unittest.__main__ import \t\n", "from unittest.__main__ import ", set()),
                 )
                 for code, expected, expected_imports in cases:
                     with self.subTest(code=code), patch.dict(sys.modules):