feat: support manual construction of Location from LocationLink in navigation requests

Author: observerw

src/lsp_client/capability/request/declaration.py

@@ -81,6 +81,15 @@ async def request_declaration_locations(
                 return [loc]
             case locations if is_locations(locations):
                 return list(locations)
+            case links if is_location_links(links):
+                return [
+                    lsp_type.Location(
+                        uri=link.target_uri, range=link.target_selection_range
+                    )
+                    for link in links
+                ]
+            case None:
+                return None
             case other:
                 logger.warning("Declaration returned with unexpected result: {}", other)
                 return None
@@ -91,6 +100,8 @@ async def request_declaration_links(
         match await self.request_declaration(file_path, position):
             case links if is_location_links(links):
                 return list(links)
+            case None:
+                return None
             case other:
                 logger.warning("Declaration returned with unexpected result: {}", other)
                 return None

src/lsp_client/capability/request/definition.py

@@ -83,14 +83,21 @@ async def request_definition_locations(
         file_path: AnyPath,
         position: Position,
     ) -> Sequence[lsp_type.Location] | None:
-        match await self.request_definition(
-            file_path,
-            position,
-        ):
+        match await self.request_definition(file_path, position):
             case lsp_type.Location() as loc:
                 return [loc]
             case locations if is_locations(locations):
                 return list(locations)
+            case links if is_location_links(links):
+                return [
+                    lsp_type.Location(
+                        uri=link.target_uri,
+                        range=link.target_selection_range,
+                    )
+                    for link in links
+                ]
+            case None:
+                return None
             case other:
                 logger.warning("Definition returned with unexpected result: {}", other)
                 return None
@@ -100,12 +107,11 @@ async def request_definition_links(
         file_path: AnyPath,
         position: Position,
     ) -> Sequence[lsp_type.LocationLink] | None:
-        match await self.request_definition(
-            file_path,
-            position,
-        ):
+        match await self.request_definition(file_path, position):
             case links if is_location_links(links):
                 return list(links)
+            case None:
+                return None
             case other:
                 logger.warning("Definition returned with unexpected result: {}", other)
                 return None

src/lsp_client/capability/request/implementation.py

@@ -81,6 +81,15 @@ async def request_implementation_locations(
                 return [loc]
             case locations if is_locations(locations):
                 return list(locations)
+            case links if is_location_links(links):
+                return [
+                    lsp_type.Location(
+                        uri=link.target_uri, range=link.target_selection_range
+                    )
+                    for link in links
+                ]
+            case None:
+                return None
             case other:
                 logger.warning(
                     "Implementation returned with unexpected result: {}", other
@@ -93,6 +102,8 @@ async def request_implementation_links(
         match await self.request_implementation(file_path, position):
             case links if is_location_links(links):
                 return list(links)
+            case None:
+                return None
             case other:
                 logger.warning(
                     "Implementation returned with unexpected result: {}", other

src/lsp_client/capability/request/type_definition.py

@@ -83,6 +83,15 @@ async def request_type_definition_locations(
                 return [loc]
             case locations if is_locations(locations):
                 return list(locations)
+            case links if is_location_links(links):
+                return [
+                    lsp_type.Location(
+                        uri=link.target_uri, range=link.target_selection_range
+                    )
+                    for link in links
+                ]
+            case None:
+                return None
             case other:
                 logger.warning(
                     "TypeDefinition returned with unexpected result: {}", other
@@ -95,6 +104,8 @@ async def request_type_definition_links(
         match await self.request_type_definition(file_path, position):
             case links if is_location_links(links):
                 return list(links)
+            case None:
+                return None
             case other:
                 logger.warning(
                     "TypeDefinition returned with unexpected result: {}", other