refactor(mnp): remove ten operator messages no client sent0.17

node_status, node_settings_set, roster_read, denylist_read, denylist_clear,
node_reload and the signed gek_rotate, member_unpin, transfer_limits,
group_detach leave MNP 6.0; the Node page and the CLI do this work over
loopback. Their ops keep their tests, moved to the ops level.

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

1 files changed, 3 insertions, 4 deletions

diff --git a/packages/meshbay-node/src/meshbay_node/ui/app.py b/packages/meshbay-node/src/meshbay_node/ui/app.py
index f810903..db11a09 100644
--- a/packages/meshbay-node/src/meshbay_node/ui/app.py
+++ b/packages/meshbay-node/src/meshbay_node/ui/app.py
@@ -535,10 +535,9 @@ def create_ui_app(state: dict) -> FastAPI:
 
     @app.put("/api/groups/{group_id}/transfer-limits")
     async def set_transfer_limits(group_id: str, payload: dict):
-        # The same `ops.set_transfer_limits` the signed MNP handler calls. The
-        # op existed with only that one door, and nothing anywhere opened it —
-        # so the per-member cap sat at its default of 2 with no way to change
-        # it, which from outside is indistinguishable from a hardcoded 2.
+        # The only door to `ops.set_transfer_limits` (the CLI uses it). It once
+        # had only a signed MNP message, which nothing anywhere sent — so the
+        # per-member cap sat at its default of 2 with no way to change it.
         return await _op(lambda: ops.set_transfer_limits(
             state, group_id,
             int(payload.get("downloads", 0)), int(payload.get("uploads", 0))))