[pre-commit.ci] pre-commit suggestions (#680)

* [pre-commit.ci] pre-commit suggestions

updates:
- [github.com/codespell-project/codespell: v2.4.2 → v2.4.3](codespell-project/codespell@v2.4.2...v2.4.3)
- [github.com/JoC0de/pre-commit-prettier: v3.8.1 → v3.9.6](JoC0de/pre-commit-prettier@v3.8.1...v3.9.6)
- [github.com/PyCQA/docformatter: v1.7.7 → v1.7.8](PyCQA/docformatter@v1.7.7...v1.7.8)
- [github.com/astral-sh/ruff-pre-commit: v0.15.9 → v0.16.1](astral-sh/ruff-pre-commit@v0.15.9...v0.16.1)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* ci: keep docformatter at v1.7.7 and revert its v1.7.8 rewrites

docformatter v1.7.8 regresses in two ways against our existing config
(wrap-summaries/wrap-descriptions = 120, blank = true):

1. It reflows triple-quoted strings that are not docstrings, word-wrapping
   the code samples inside `raise ValueError(...)` messages into unreadable
   prose and emitting a stray line-continuation backslash after the opening
   paren.
2. It inserts a trailing blank line after docstrings that ruff-format 0.16.1
   immediately strips, so the two hooks never converge and every run reports
   "files were modified".

It also split a summary in loggers.py into a 131-char description line,
tripping ruff E501, which ruff cannot autofix.

Pin the rev back to v1.7.7 and restore the four files v1.7.8 rewrote. The
codespell, prettier and ruff bumps in this PR are kept.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Bhimraj Yadav <bhimrajyadav977@gmail.com>

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -23,14 +23,14 @@ repos:
       - id: detect-private-key
 
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.4.2
+    rev: v2.4.3
     hooks:
       - id: codespell
         additional_dependencies: [tomli]
         #args: ["--write-changes"]
 
   - repo: https://github.com/JoC0de/pre-commit-prettier
-    rev: v3.8.1
+    rev: v3.9.6
     hooks:
       - id: prettier
         files: \.(json|yml|yaml|toml)
@@ -46,7 +46,7 @@ repos:
         args: ["--in-place"]
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.15.9
+    rev: v0.16.1
     hooks:
       - id: ruff
         args: ["--fix"]

src/litserve/api.py

@@ -148,7 +148,6 @@ def __init__(
         enable_async: bool = False,
     ):
         """Initialize LitAPI with configuration options."""
-
         if max_batch_size <= 0:
             raise ValueError("max_batch_size must be greater than 0")
 
@@ -381,7 +380,6 @@ def pre_setup(self, spec: Optional[LitSpec] = None):
 
     def set_logger_queue(self, queue: Queue):
         """Set the queue for logging events."""
-
         self._logger_queue = queue
 
     def log(self, key, value):

src/litserve/server.py

@@ -1098,7 +1098,6 @@ def _get_request_queue(self, api_path: str):
 
     def _register_api_endpoints(self, lit_api: LitAPI, request_type, response_type):
         """Register endpoint routes for the FastAPI app."""
-
         self._callback_runner.trigger_event(EventTypes.ON_SERVER_START.value, litserver=self)
 
         # Create handlers

src/litserve/utils.py

@@ -322,7 +322,6 @@ def add_ssl_context_from_env(kwargs: dict[str, Any]) -> dict[str, Any]:
         returns an empty dictionary.
 
     """
-
     if "ssl_keyfile" in kwargs and "ssl_certfile" in kwargs:
         return kwargs
 

tests/parity_fastapi/benchmark.py

@@ -52,7 +52,6 @@ def send_request(args):
 
 def benchmark(num_requests=100, concurrency_level=100, port=8000):
     """Benchmark the ML server."""
-
     # Create a session with appropriate pool size
     session = create_session(pool_connections=min(concurrency_level, 100), pool_maxsize=min(concurrency_level, 100))
 

tests/unit/test_middlewares.py

@@ -92,7 +92,6 @@ def test_middleware_multiple_initialization():
 
 def test_track_requests_middleware_isolation():
     """Test that _prepare_app_run doesn't modify the original app's middleware list."""
-
     lit_api = ls.test_examples.SimpleLitAPI()
     server = ls.LitServer(lit_api, track_requests=True)
 

tests/unit/test_utils.py

@@ -77,7 +77,6 @@ async def test_azip_propagates_stream_errors():
 @pytest.mark.skipif(sys.platform == "win32", reason="This test is for non-Windows platforms only.")
 def test_generate_random_zmq_address_non_windows(tmpdir):
     """Test generate_random_zmq_address on non-Windows platforms."""
-
     temp_dir = str(tmpdir)
     address1 = generate_random_zmq_address(temp_dir=temp_dir)
     address2 = generate_random_zmq_address(temp_dir=temp_dir)