Revert "Add orjson support (#815)"

This reverts commit 392543e.

README.md

@@ -32,7 +32,3 @@ try:
 except zwave_js_server.client.FailedCommand as err:
     print("Command failed with", err.error_code)
 ```
-
-## Optional dependencies
-
-`zwave-js-server-python` optionally supports `orjson` which can be installed with the `orjson` extra (e.g. `pip install zwave-js-server-python[orjson]`)

pyproject.toml

@@ -21,16 +21,13 @@ classifiers = [
 dependencies = ["aiohttp>3", "pydantic>=1.10.0"]
 dynamic = ["version"]
 
-[project.optional-dependencies]
-orjson = ["orjson>=3.9.9"]
-
-[project.scripts]
-zwave-js-server-python = "zwave_js_server.__main__:main"
-
 [project.urls]
 "Source Code" = "https://github.com/home-assistant-libs/zwave-js-server-python"
 "Bug Reports" = "https://github.com/home-assistant-libs/zwave-js-server-python/issues"
 
+[project.scripts]
+zwave-js-server-python = "zwave_js_server.__main__:main"
+
 [tool.setuptools.dynamic]
 version = { attr = "zwave_js_server.const.__version__" }
 

requirements_dev.txt

@@ -1,5 +1,4 @@
 -r requirements.txt
--r requirements_extra.txt
 -r requirements_lint.txt
 -r requirements_scripts.txt
 -r requirements_test.txt

test/conftest.py

@@ -177,7 +177,7 @@ async def receive():
 
     ws_client.receive.side_effect = receive
 
-    async def close_client(msg, dumps=json.dumps):
+    async def close_client(msg):
         """Close the client."""
         if msg["command"] in ("initialize", "start_listening"):
             return
@@ -328,7 +328,7 @@ def apply_mock_command(
         mock_responses.append((match_command, response, success))
         return ack_commands
 
-    async def set_response(message, dumps=json.dumps):
+    async def set_response(message):
         """Check the message and set the mocked response if a command matches."""
         for match_command, response, success in mock_responses:
             if all(message[key] == value for key, value in match_command.items()):

tox.ini

@@ -1,29 +1,16 @@
 [tox]
-envlist = clean, py311, test_orjson, lint, mypy
+envlist = py311, lint, mypy
 skip_missing_interpreters = True
 
 [gh-actions]
 python =
-  3.11: clean, py311, test_orjson, lint, mypy
-
-[testenv:clean]
-deps = coverage
-skip_install = true
-commands = coverage erase
+  3.11: py311, lint, mypy
 
 [testenv]
 commands =
-  pytest --timeout=30 --cov=zwave_js_server --cov-report=xml --cov-append {posargs}
-deps =
-  -rrequirements.txt
-  -rrequirements_test.txt
-
-[testenv:test_orjson]
-commands =
-  pytest --timeout=30 --cov=zwave_js_server --cov-report=xml --cov-append {posargs}
+  pytest --timeout=30 --cov=zwave_js_server --cov-report=xml {posargs}
 deps =
   -rrequirements.txt
-  -rrequirements_extra.txt
   -rrequirements_test.txt
 
 [testenv:lint]

zwave_js_server/client.py

@@ -6,7 +6,6 @@
 from collections.abc import Callable
 from copy import deepcopy
 from datetime import datetime
-import functools
 import logging
 from operator import itemgetter
 import pprint
@@ -40,11 +39,6 @@
 from .model.log_message import LogMessage
 from .model.version import VersionInfo, VersionInfoDataType
 
-try:
-    import orjson as json
-except ImportError:
-    import json
-
 SIZE_PARSE_JSON_EXECUTOR = 8192
 
 # Message IDs
@@ -464,11 +458,9 @@ async def _receive_json_or_raise(self) -> dict:
 
         try:
             if len(msg.data) > SIZE_PARSE_JSON_EXECUTOR:
-                data: dict = await self._loop.run_in_executor(
-                    None, functools.partial(msg.json, loads=json.loads)
-                )
+                data: dict = await self._loop.run_in_executor(None, msg.json)
             else:
-                data = msg.json(loads=json.loads)
+                data = msg.json()
         except ValueError as err:
             raise InvalidMessage("Received invalid JSON.") from err
 
@@ -565,7 +557,7 @@ async def _send_json_message(self, message: dict[str, Any]) -> None:
                 }
             )
 
-        await self._client.send_json(message, dumps=json.dumps)
+        await self._client.send_json(message)
 
     async def __aenter__(self) -> Client:
         """Connect to the websocket."""