Fix CI: formatting, missing docstring param, lint warning

- Format http/_server.py (whitespace)
- Add server_version to RpcServer.__init__ docstring
- Use list.extend in conformance validator (PERF401)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: rustyconover

vgi_rpc/access_log_conformance.py

@@ -66,9 +66,9 @@ def validate_access_logs(entries: list[dict[str, object]]) -> list[Violation]:
         status = str(entry.get("status", ""))
 
         # --- Always required ---
-        for field in _ALWAYS_REQUIRED:
-            if field not in entry:
-                violations.append(Violation(i, method, field, "required on all entries"))
+        violations.extend(
+            Violation(i, method, field, "required on all entries") for field in _ALWAYS_REQUIRED if field not in entry
+        )
 
         # --- request_data on dispatch methods ---
         if (method in _REQUEST_DATA_METHODS or _is_catalog_method(method)) and not entry.get("request_data"):

vgi_rpc/http/_server.py

@@ -243,9 +243,7 @@ def _pack_state_token(
     return base64.b64encode(payload + mac)
 
 
-def _unpack_state_token(
-    token: bytes, signing_key: bytes, token_ttl: int = 0
-) -> tuple[bytes, bytes, bytes, str]:
+def _unpack_state_token(token: bytes, signing_key: bytes, token_ttl: int = 0) -> tuple[bytes, bytes, bytes, str]:
     """Unpack and verify a signed state token.
 
     Args:

vgi_rpc/rpc/_server.py

@@ -236,6 +236,7 @@ def __init__(
             implementation: Object implementing all methods from *protocol*.
             external_location: Optional ExternalLocation configuration.
             server_id: Optional server identifier; auto-generated if ``None``.
+            server_version: Version string included in access log entries.
             enable_describe: When ``True``, the server handles ``__describe__``
                 requests returning machine-readable method metadata.
             ipc_validation: Validation level for incoming IPC batches.