implement renaming of keywords

Author: d-biehl

robotcode/jsonrpc2/protocol.py

@@ -525,10 +525,10 @@ def send_request(
         method: str,
         params: Optional[Any] = None,
         return_type_or_converter: Optional[Type[_TResult]] = None,
-    ) -> asyncio.Future[_TResult]:
+    ) -> asyncio.Future[Optional[_TResult]]:
 
         with self._sended_request_lock:
-            result: asyncio.Future[_TResult] = create_sub_future()
+            result: asyncio.Future[Optional[_TResult]] = create_sub_future()
             self._sended_request_count += 1
             id = self._sended_request_count
 
@@ -544,7 +544,7 @@ async def send_request_async(
         method: str,
         params: Optional[Any] = None,
         return_type: Optional[Type[_TResult]] = None,
-    ) -> _TResult:
+    ) -> Optional[_TResult]:
 
         return await self.send_request(method, params, return_type)
 
@@ -571,7 +571,9 @@ def handle_response(self, message: JsonRPCResponse) -> None:
 
         try:
             if not entry.future.done():
-                res = from_dict(message.result, entry.result_type)
+                res = None
+                if message.result is not None:
+                    res = from_dict(message.result, entry.result_type)
                 if entry.future._loop == asyncio.get_running_loop():
                     entry.future.set_result(res)
                 else:

robotcode/language_server/common/parts/rename.py

@@ -3,7 +3,7 @@
 from asyncio import CancelledError
 from typing import TYPE_CHECKING, Any, List, Optional
 
-from ....jsonrpc2.protocol import rpc_method
+from ....jsonrpc2.protocol import JsonRPCErrorException, rpc_method
 from ....utils.async_tools import async_tasking_event
 from ....utils.logging import LoggingDescriptor
 from ..decorators import language_id_filter
@@ -26,6 +26,10 @@
 from .protocol_part import LanguageServerProtocolPart
 
 
+class CantRenameException(Exception):
+    pass
+
+
 class RenameProtocolPart(LanguageServerProtocolPart, HasExtendCapabilities):
 
     _logger = LoggingDescriptor()
@@ -118,6 +122,9 @@ async def _text_document_prepare_rename(
             self, document, position, callback_filter=language_id_filter(document)
         ):
             if isinstance(result, BaseException):
+                if isinstance(result, CantRenameException):
+                    raise JsonRPCErrorException(100, str(result))
+
                 if not isinstance(result, CancelledError):
                     self._logger.exception(result, exc_info=result)
             else:

robotcode/language_server/common/parts/window.py

@@ -34,12 +34,13 @@ def show_log_message(self, message: str, type: MessageType = MessageType.INFO) -
 
     async def show_message_request(
         self, message: str, actions: List[str] = [], type: MessageType = MessageType.INFO
-    ) -> MessageActionItem:
-        return await self.parent.send_request_async(
+    ) -> Optional[str]:
+        r = await self.parent.send_request_async(
             "window/showMessageRequest",
             ShowMessageRequestParams(type=type, message=message, actions=[MessageActionItem(title=a) for a in actions]),
             MessageActionItem,
         )
+        return r.title if r is not None else None
 
     async def show_document(
         self,
@@ -48,13 +49,12 @@ async def show_document(
         take_focus: Optional[bool] = None,
         selection: Optional[Range] = None,
     ) -> bool:
-        return (
-            await self.parent.send_request_async(
-                "window/showDocument",
-                ShowDocumentParams(uri=uri, external=external, take_focus=take_focus, selection=selection),
-                ShowDocumentResult,
-            )
-        ).success
+        r = await self.parent.send_request_async(
+            "window/showDocument",
+            ShowDocumentParams(uri=uri, external=external, take_focus=take_focus, selection=selection),
+            ShowDocumentResult,
+        )
+        return r.success if r is not None else False
 
     __progress_tokens: Dict[ProgressToken, bool] = {}
 

robotcode/language_server/common/parts/workspace.py

@@ -303,20 +303,20 @@ async def get_configuration(
             and self.parent.client_capabilities.workspace
             and self.parent.client_capabilities.workspace.configuration
         ):
-            return (
-                await self.parent.send_request_async(
-                    "workspace/configuration",
-                    ConfigurationParams(
-                        items=[
-                            ConfigurationItem(
-                                scope_uri=str(scope_uri) if isinstance(scope_uri, Uri) else scope_uri,
-                                section=str(section),
-                            )
-                        ]
-                    ),
-                    list,
-                )
-            )[0]
+            r = await self.parent.send_request_async(
+                "workspace/configuration",
+                ConfigurationParams(
+                    items=[
+                        ConfigurationItem(
+                            scope_uri=str(scope_uri) if isinstance(scope_uri, Uri) else scope_uri,
+                            section=str(section),
+                        )
+                    ]
+                ),
+                list,
+            )
+
+            return r[0] if r is not None else None
 
         result = self.settings
         for sub_key in str(section).split("."):

robotcode/language_server/robotframework/diagnostics/library_doc.py

@@ -314,6 +314,14 @@ def __str__(self) -> str:
     def is_deprecated(self) -> bool:
         return self.deprecated or DEPRECATED_PATTERN.match(self.doc) is not None
 
+    @property
+    def is_resource_keyword(self) -> bool:
+        return self.libtype == "RESOURCE"
+
+    @property
+    def is_library_keyword(self) -> bool:
+        return self.libtype == "LIBRARY"
+
     @property
     def deprecated_message(self) -> str:
         if (m := DEPRECATED_PATTERN.match(self.doc)) is not None: