Remove dependency on typeguard

pygls/feature_manager.py

@@ -103,12 +103,13 @@ class FeatureManager:
                                     2. name: LanguageServer - add typings
     """
 
-    def __init__(self, server=None):
+    def __init__(self, server=None, converter=None):
         self._builtin_features = {}
         self._feature_options = {}
         self._features = {}
         self._commands = {}
         self.server = server
+        self.converter = converter
 
     def add_builtin_feature(self, feature_name: str, func: Callable) -> None:
         """Registers builtin (predefined) feature."""
@@ -189,7 +190,9 @@ def decorator(f):
 
             if options:
                 options_type = get_method_options_type(feature_name)
-                if options_type and not is_instance(options, options_type):
+                if options_type and not is_instance(
+                    self.converter, options, options_type
+                ):
                     raise TypeError(
                         (
                             f'Options of method "{feature_name}"'

pygls/lsp/__init__.py

@@ -14,6 +14,7 @@
 # See the License for the specific language governing permissions and      #
 # limitations under the License.                                           #
 ############################################################################
+import cattrs
 from typing import Any, Callable, List, Optional, Union
 
 from lsprotocol.types import (
@@ -35,7 +36,6 @@
     SemanticTokensRegistrationOptions,
     ShowDocumentResult,
 )
-from typeguard import check_type
 
 from pygls.exceptions import MethodTypeNotRegisteredError
 
@@ -131,9 +131,9 @@ def get_method_return_type(method_name, lsp_methods_map=METHOD_TO_TYPES):
         raise MethodTypeNotRegisteredError(method_name)
 
 
-def is_instance(o, t):
+def is_instance(cv: cattrs.Converter, o, t):
     try:
-        check_type(o, t)
+        cv.unstructure(o, t)
         return True
     except TypeError:
         return False

pygls/protocol/json_rpc.py

@@ -129,7 +129,7 @@ def __init__(self, server: LanguageServer, converter):
         self._request_futures: Dict[str, Future[Any]] = {}
         self._result_types: Dict[str, Any] = {}
 
-        self.fm = FeatureManager(server)
+        self.fm = FeatureManager(server, converter)
         self.transport: Optional[
             Union[asyncio.WriteTransport, WebSocketTransportAdapter]
         ] = None

pyproject.toml

@@ -15,7 +15,6 @@ readme = "README.md"
 [tool.poetry.dependencies]
 python = ">=3.7.9,<4"
 lsprotocol = "2023.0.0b1"
-typeguard = ">=3.0.0, <5"
 websockets = {version = "^11.0.3", optional = true}
 
 [tool.poetry.extras]

tests/conftest.py

@@ -21,7 +21,7 @@
 import sys
 
 import pytest
-from lsprotocol import types
+from lsprotocol import types, converters
 
 from pygls import uris, IS_PYODIDE, IS_WIN
 from pygls.feature_manager import FeatureManager
@@ -113,7 +113,7 @@ def server_dir():
 @pytest.fixture
 def feature_manager():
     """Return a feature manager"""
-    return FeatureManager()
+    return FeatureManager(None, converters.get_converter())
 
 
 @pytest.fixture

tests/test_feature_manager.py

@@ -30,8 +30,6 @@
     wrap_with_server,
 )
 from lsprotocol import types as lsp
-from typeguard import TypeCheckError
-from typeguard._utils import qualified_name
 
 
 class Temp:
@@ -104,11 +102,8 @@ class Options:
         pass
 
     with pytest.raises(
-        TypeCheckError,
-        match=(
-            f"{qualified_name(Options)} is not an instance of "
-            "lsprotocol.types.CompletionOptions"
-        ),  # noqa
+        AttributeError,
+        match=("'Options' object has no attribute 'trigger_characters'"),  # noqa
     ):
 
         @feature_manager.feature(lsp.TEXT_DOCUMENT_COMPLETION, Options())