black

pylsp/plugins/rope_autoimport.py

@@ -268,7 +268,9 @@ def pylsp_code_actions(
         if "undefined name" not in diagnostic.get("message", "").lower():
             continue
         expr = parso.parse(document.lines[diagnostic["range"]["start"]["line"]])
-        word = expr.get_leaf_for_position((1, diagnostic["range"]["start"]["character"] + 1)).value
+        word = expr.get_leaf_for_position(
+            (1, diagnostic["range"]["start"]["character"] + 1)
+        ).value
         log.debug(f"autoimport: searching for word: {word}")
         rope_config = config.settings(document_path=document.path).get("rope", {})
         autoimport = workspace._rope_autoimport(rope_config, feature="code_actions")

pylsp/workspace.py

@@ -60,6 +60,7 @@ def __init__(self, root_uri, endpoint, config=None):
         self.__rope_config = None
         # We have a sperate AutoImport object for each feature to avoid sqlite errors
         # from accessing the same database from multiple threads.
+        # An upstream fix discussion is here: https://github.com/python-rope/rope/issues/713
         self.__rope_autoimport = (
             {}
         )  # Type: Dict[Literal["completions", "code_actions"], rope.contrib.autoimport.sqlite.AutoImport]

test/plugins/test_autoimport.py

@@ -223,7 +223,10 @@ class sfa:
 
 
 # Tests ruff, flake8 and pylfakes messages
-@pytest.mark.parametrize("message", ["Undefined name `os`", "F821 undefined name 'numpy'", "undefined name 'numpy'"])
+@pytest.mark.parametrize(
+    "message",
+    ["Undefined name `os`", "F821 undefined name 'numpy'", "undefined name 'numpy'"],
+)
 def test_autoimport_code_actions(config, autoimport_workspace, message):
     source = "os"
     autoimport_workspace.put_document(DOC_URI, source=source)