Fix pylint

.github/workflows/pylint.yaml

@@ -37,6 +37,7 @@
             run: |
               mkdir -p .github/linters
               cp pyproject.toml .github/linters
+              pip install .
     
           - name: Register pylint problem matcher
             run: |

pyproject.toml

@@ -40,8 +40,22 @@ slither-lsp = "slither_lsp.__main__:main"
 max-line-length = 120
 
 disable = """
-import-outside-toplevel,
 missing-module-docstring,
-useless-return,
-duplicate-code
+missing-class-docstring,
+missing-function-docstring,
+unnecessary-lambda,
+cyclic-import,
+line-too-long,
+invalid-name,
+fixme,
+too-many-return-statements,
+too-many-ancestors,
+logging-fstring-interpolation,
+logging-not-lazy,
+duplicate-code,
+import-error,
+unsubscriptable-object,
+unnecessary-lambda-assignment,
+too-few-public-methods,
+too-many-instance-attributes
 """

slither_lsp/app/feature_analyses/slither_diagnostics.py

@@ -136,7 +136,7 @@ def clear(self) -> None:
         :return: None
         """
         # Loop through all diagnostic files, publish new diagnostics for each file with no items.
-        for file_uri in self.diagnostics.keys():
+        for file_uri, _ in self.diagnostics:
             self._clear_single(file_uri, False)
 
         # Clear the dictionary

slither_lsp/app/request_handlers/analysis/get_detector_list.py

@@ -1,10 +1,7 @@
-from typing import Any
-
-from pygls.server import LanguageServer
 from slither.__main__ import get_detectors_and_printers, output_detectors_json
 
 
-def get_detector_list(ls: LanguageServer, params: Any) -> Any:
+def get_detector_list():
     """
     Handler which invokes slither to obtain a list of all detectors and some properties that describe them.
     """

slither_lsp/app/request_handlers/analysis/get_version.py

@@ -1,10 +1,7 @@
 from importlib.metadata import version as pkg_version
-from typing import Any
 
-from pygls.server import LanguageServer
 
-
-def get_version(ls: LanguageServer, params: Any) -> Any:
+def get_version():
     """
     Handler which retrieves versions for slither, crytic-compile, and related applications.
     """

slither_lsp/app/request_handlers/call_hierarchy.py

@@ -70,7 +70,7 @@ def on_prepare_call_hierarchy(
                         "offset": offset,
                     },
                 )
-        return [elem for elem in res.values()]
+        return list(res.values())
 
 
 def register_on_get_incoming_calls(ls: "SlitherServer"):

slither_lsp/app/request_handlers/compilation/__init__.py

@@ -1,2 +1 @@
-from .autogenerate_standard_json import autogenerate_standard_json
 from .get_command_line_args import get_command_line_args

slither_lsp/app/request_handlers/compilation/get_command_line_args.py

@@ -1,11 +1,9 @@
 from argparse import ArgumentParser
-from typing import Any
 
 from crytic_compile.cryticparser.cryticparser import init as crytic_parser_init
-from pygls.server import LanguageServer
 
 
-def get_command_line_args(ls: LanguageServer, params: Any) -> Any:
+def get_command_line_args():
     """
     Handler which obtains data regarding all command line arguments available in crytic-compile.
     """

slither_lsp/app/request_handlers/symbols.py

@@ -1,3 +1,5 @@
+# pylint: enable=too-many-branches
+
 from typing import TYPE_CHECKING, List, Optional
 
 import lsprotocol.types as lsp

slither_lsp/app/request_handlers/type_hierarchy.py

@@ -48,15 +48,15 @@ def on_prepare_type_hierarchy(
                 if not isinstance(obj, Contract):
                     continue
                 offset = get_definition(obj, comp).start
-                range = get_object_name_range(obj, comp)
+                range_ = get_object_name_range(obj, comp)
                 if obj.is_interface:
                     kind = lsp.SymbolKind.Interface
                 else:
                     kind = lsp.SymbolKind.Class
                 res.add(
                     TypeItem(
                         name=obj.name,
-                        range=to_range(range),
+                        range=to_range(range_),
                         kind=kind,
                         filename=source.filename.absolute,
                         offset=offset,

slither_lsp/app/request_handlers/types.py

@@ -10,13 +10,13 @@ def to_lsp_pos(pos: Pos) -> lsp.Position:
     return lsp.Position(line=pos[0], character=pos[1])
 
 
-def to_lsp_range(range: Range) -> lsp.Range:
-    return lsp.Range(start=to_lsp_pos(range[0]), end=to_lsp_pos(range[1]))
+def to_lsp_range(range_: Range) -> lsp.Range:
+    return lsp.Range(start=to_lsp_pos(range_[0]), end=to_lsp_pos(range_[1]))
 
 
 def to_pos(pos: lsp.Position) -> Pos:
     return (pos.line, pos.character)
 
 
-def to_range(range: lsp.Range) -> Range:
-    return (to_pos(range.start), to_pos(range.end))
+def to_range(range_: lsp.Range) -> Range:
+    return (to_pos(range_.start), to_pos(range_.end))

slither_lsp/app/slither_server.py

@@ -179,7 +179,7 @@ def queue_compile_workspace(self, uri: str):
         path = uri_to_fs_path(uri)
         workspace_name = split(path)[1]
 
-        def compile():
+        def do_compile():
             detector_classes, _ = get_detectors_and_printers()
             with self.workspace_in_progress[uri]:
                 self.show_message(
@@ -232,7 +232,7 @@ def compile():
                 )
                 self._refresh_detector_output()
 
-        self.analysis_pool.submit(compile)
+        self.analysis_pool.submit(do_compile)
 
     def _on_did_change_workspace_folders(
         self, params: lsp.DidChangeWorkspaceFoldersParams

slither_lsp/app/types/analysis_structures.py

@@ -59,15 +59,15 @@ class SlitherDetectorResultElement:
     """ The type of item this represents (contract, function, etc.) """
 
     @staticmethod
-    def from_dict(dict):
+    def from_dict(dict_):
         return SlitherDetectorResultElement(
-            name=dict["name"],
+            name=dict_["name"],
             source_mapping=(
-                SlitherDetectorResultElementSourceMapping(**dict["source_mapping"])
-                if dict["source_mapping"]
+                SlitherDetectorResultElementSourceMapping(**dict_["source_mapping"])
+                if dict_["source_mapping"]
                 else None
             ),
-            type=dict["type"],
+            type=dict_["type"],
         )
 
 

slither_lsp/app/utils/file_paths.py

@@ -5,7 +5,7 @@
 
 
 def is_solidity_file(path: str) -> bool:
-    filename_base, file_extension = os.path.splitext(path)
+    _, file_extension = os.path.splitext(path)
     return file_extension is not None and file_extension.lower() == ".sol"