Fix lint/format issues and run CI lint before tests

- Fix ruff format in test_http.py
- Use ternary for prefix auto-detection (SIM108)
- Add needs: lint to test job so lint failures fail fast

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

Author: rustyconover

.github/workflows/ci.yml

@@ -18,6 +18,7 @@ env:
 jobs:
   test:
     name: Python ${{ matrix.python-version }} on ${{ matrix.os }}
+    needs: lint
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false

tests/test_http.py

@@ -78,9 +78,7 @@
 def client(request: pytest.FixtureRequest) -> Iterator[_SyncTestClient]:
     """Create a sync Falcon test client with proper cleanup, tested with both empty and /vgi prefix."""
     prefix: str = request.param
-    c = make_sync_client(
-        RpcServer(RpcFixtureService, RpcFixtureServiceImpl()), signing_key=b"test-key", prefix=prefix
-    )
+    c = make_sync_client(RpcServer(RpcFixtureService, RpcFixtureServiceImpl()), signing_key=b"test-key", prefix=prefix)
     yield c
     c.close()
 

vgi_rpc/http/_client.py

@@ -416,10 +416,7 @@ def http_connect[P](
         client = httpx.Client(base_url=base_url, follow_redirects=True)
 
     # Auto-detect prefix from _SyncTestClient when not explicitly provided
-    if prefix is None:
-        url_prefix = getattr(client, "prefix", "")
-    else:
-        url_prefix = prefix
+    url_prefix = getattr(client, "prefix", "") if prefix is None else prefix
     try:
         yield cast(
             P,