refactor(langserver): remove some unneeded casts

Author: d-biehl

packages/language_server/src/robotcode/language_server/robotframework/parts/document_highlight.py

@@ -1,4 +1,4 @@
-from typing import TYPE_CHECKING, Any, List, Optional, cast
+from typing import TYPE_CHECKING, Any, List, Optional
 
 from robotcode.core.concurrent import check_current_task_canceled
 from robotcode.core.language import language_id
@@ -100,12 +100,9 @@ def collect(self, sender: Any, document: TextDocument, position: Position) -> Op
                     ns.import_range
                     if ns.import_source == namespace.source
                     and (position.is_in_range(ns.alias_range, False) or position.is_in_range(ns.import_range, False))
-                    else cast(
-                        Optional[Range],
-                        next(
-                            (r.range for r in ns_refs if position.is_in_range(r.range, False)),
-                            None,
-                        ),
+                    else next(
+                        (r.range for r in ns_refs if position.is_in_range(r.range, False)),
+                        None,
                     )
                 )
 

packages/language_server/src/robotcode/language_server/robotframework/parts/goto.py

@@ -1,10 +1,10 @@
 from __future__ import annotations
 
-from typing import TYPE_CHECKING, Any, List, Optional, Union, cast
+from typing import TYPE_CHECKING, Any, List, Union
 
 from robotcode.core.concurrent import check_current_task_canceled
 from robotcode.core.language import language_id
-from robotcode.core.lsp.types import Location, LocationLink, Position, Range
+from robotcode.core.lsp.types import Location, LocationLink, Position
 from robotcode.core.text_document import TextDocument
 from robotcode.core.uri import Uri
 from robotcode.core.utils.logging import LoggingDescriptor
@@ -55,12 +55,9 @@ def collect(
                 found_range = (
                     variable.name_range
                     if variable.source == namespace.source and position.is_in_range(variable.name_range, False)
-                    else cast(
-                        Optional[Range],
-                        next(
-                            (r.range for r in var_refs if position.is_in_range(r.range)),
-                            None,
-                        ),
+                    else next(
+                        (r.range for r in var_refs if position.is_in_range(r.range)),
+                        None,
                     )
                 )
 
@@ -90,12 +87,9 @@ def collect(
                 found_range = (
                     kw.name_range
                     if kw.source == namespace.source and position.is_in_range(kw.name_range, False)
-                    else cast(
-                        Optional[Range],
-                        next(
-                            (r.range for r in kw_refs if position.is_in_range(r.range, False)),
-                            None,
-                        ),
+                    else next(
+                        (r.range for r in kw_refs if position.is_in_range(r.range, False)),
+                        None,
                     )
                 )
 

packages/language_server/src/robotcode/language_server/robotframework/parts/hover.py

@@ -116,12 +116,9 @@ def _hover_default(self, nodes: List[ast.AST], document: TextDocument, position:
                 found_range = (
                     variable.name_range
                     if variable.source == namespace.source and position.is_in_range(variable.name_range, False)
-                    else cast(
-                        Optional[Range],
-                        next(
-                            (r.range for r in var_refs if position.is_in_range(r.range)),
-                            None,
-                        ),
+                    else next(
+                        (r.range for r in var_refs if position.is_in_range(r.range)),
+                        None,
                     )
                 )
                 value = None
@@ -187,12 +184,9 @@ def _hover_default(self, nodes: List[ast.AST], document: TextDocument, position:
                 found_range = (
                     kw.name_range
                     if kw.source == namespace.source and position.is_in_range(kw.name_range, False)
-                    else cast(
-                        Optional[Range],
-                        next(
-                            (r.range for r in kw_refs if position.is_in_range(r.range, False)),
-                            None,
-                        ),
+                    else next(
+                        (r.range for r in kw_refs if position.is_in_range(r.range, False)),
+                        None,
                     )
                 )
 
@@ -231,12 +225,9 @@ def _hover_default(self, nodes: List[ast.AST], document: TextDocument, position:
                     else (
                         ns.alias_range
                         if ns.import_source == namespace.source and position.is_in_range(ns.alias_range, False)
-                        else cast(
-                            Optional[Range],
-                            next(
-                                (r.range for r in ns_refs if position.is_in_range(r.range, False)),
-                                None,
-                            ),
+                        else next(
+                            (r.range for r in ns_refs if position.is_in_range(r.range, False)),
+                            None,
                         )
                     )
                 )

packages/language_server/src/robotcode/language_server/robotframework/parts/rename.py

@@ -8,7 +8,6 @@
     Tuple,
     TypeVar,
     Union,
-    cast,
 )
 
 from robotcode.core.concurrent import check_current_task_canceled
@@ -166,12 +165,9 @@ def _find_variable_definition_on_pos(
                 found_range = (
                     variable.name_range
                     if variable.source == namespace.source and position.is_in_range(variable.name_range, False)
-                    else cast(
-                        Optional[Range],
-                        next(
-                            (r.range for r in var_refs if position.is_in_range(r.range)),
-                            None,
-                        ),
+                    else next(
+                        (r.range for r in var_refs if position.is_in_range(r.range)),
+                        None,
                     )
                 )
 
@@ -243,12 +239,9 @@ def _find_keyword_definition_on_pos(
                 found_range = (
                     keyword.name_range
                     if keyword.source == namespace.source and position.is_in_range(keyword.name_range, False)
-                    else cast(
-                        Optional[Range],
-                        next(
-                            (r.range for r in kw_refs if position.is_in_range(r.range)),
-                            None,
-                        ),
+                    else next(
+                        (r.range for r in kw_refs if position.is_in_range(r.range)),
+                        None,
                     )
                 )