style: the 98 ruff could not fix, so the linter is a signal again

The pass before this applied ruff's own fixes. These are the ones needing a
decision, and the point of doing them is that `ruff check .` now passes: a
linter reporting 98 known-acceptable findings reports nothing, because the next
real one arrives invisible.

**Lines over 100 (70).** Mostly wrapped where they stood. Two exceptions: the
aligned trailing comments in `protocol.py`'s message table were shortened rather
than wrapped, because wrapping one row of a table breaks the table; and in
`models.py` the column comments moved above their columns for the same reason.

**Imports below the first statement (14).** `csam.py` kept its FastAPI imports
under a section header halfway down the file; two node tests had a constant and
a `pytestmark` wedged between two import blocks. Moved, not suppressed.

**Bindings nothing reads (4).** Three in tests, where the call stays and only the
name goes — `_user(client, "listener")` is there to create the user, not to
return one. The fourth was in `revocation.py` and was not a lint finding at all:
`_connect_and_listen` opened an httpx stream to the WebSocket URL, did `pass`,
and then opened the real connection through the `websockets` library. One
pointless request per connect, left over from before that library was used
directly. Removed, and `httpx` with it.

**`l` as a name (4)**, **semicolons (6)** in the POC spikes, and the rest.

2893 passed, the same count as the two commits before it.

`meshbay_node/revocation.py` is worth a decision separately: 154 lines that
nothing imports, superseded by `hub_client.maintain_ws`'s `on_revocation`. This
commit only stopped it failing the linter.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

1 files changed, 12 insertions, 4 deletions

diff --git a/packages/meshbay-hub/tests/test_hub_api.py b/packages/meshbay-hub/tests/test_hub_api.py
index 9b73701..2afd27b 100644
--- a/packages/meshbay-hub/tests/test_hub_api.py
+++ b/packages/meshbay-hub/tests/test_hub_api.py
@@ -260,7 +260,9 @@ async def test_group_member_add(client):
             "pk_user_ed25519": pk_ed, "pk_user_x25519": pk_x})
 
     alice_token = (await client.post("/v1/users/login",
-                   json={"username": "alice2_test", "password": "alicepass99"})).json()["access_token"]
+                   json={"username": "alice2_test",
+                           "password": "alicepass99"})
+                   ).json()["access_token"]
 
     a_hdrs = {"Authorization": f"Bearer {alice_token}"}
 
@@ -298,9 +300,13 @@ async def test_non_admin_cannot_add_member(client):
             "pk_user_ed25519": pk_ed, "pk_user_x25519": pk_x})
 
     charlie_token = (await client.post("/v1/users/login",
-                     json={"username": "charlie_test", "password": "charliepass"})).json()["access_token"]
+                     json={"username": "charlie_test",
+                           "password": "charliepass"})
+                   ).json()["access_token"]
     dan_token     = (await client.post("/v1/users/login",
-                     json={"username": "dan_test", "password": "danpass1234"})).json()["access_token"]
+                     json={"username": "dan_test",
+                           "password": "danpass1234"})
+                   ).json()["access_token"]
 
     r = await client.post("/v1/groups", json={"name": "charlies-group"},
                           headers={"Authorization": f"Bearer {charlie_token}"})
@@ -338,7 +344,9 @@ async def test_jwt_contains_groups_claim(client):
 
     # Alice creates a group and adds Bob
     alice_token = (await client.post("/v1/users/login",
-                   json={"username": "grp_alice", "password": "alicepass99"})).json()["access_token"]
+                   json={"username": "grp_alice",
+                           "password": "alicepass99"})
+                   ).json()["access_token"]
     r = await client.post("/v1/groups", json={"name": "testgroup"},
                           headers={"Authorization": f"Bearer {alice_token}"})
     group_id = r.json()["group_id"]