refactor: move attribute docstrings out of constructor bodies

Connection's public attributes are now declared in the class body with
their docstrings under the declarations; __init__ only assigns. The
private _inline_methods docstring becomes a comment above the assignment.

Author: maxisbey

src/mcp/server/connection.py

@@ -49,26 +49,39 @@ class Connection(TypedServerRequestMixin):
     remains `None` (no handshake reaches a stateless connection).
     """
 
+    has_standalone_channel: bool
+    session_id: str | None
+
+    client_params: InitializeRequestParams | None
+    """The full `initialize` request params; `None` before initialization."""
+
+    protocol_version: str | None
+
+    initialized: anyio.Event
+    """Set when `notifications/initialized` arrives (matches TS `oninitialized`);
+    the point from which the spec permits server-initiated requests beyond
+    ping/logging. Pre-set on stateless connections."""
+
+    state: dict[str, Any]
+    """Per-connection scratch state; persists across requests on this connection."""
+
+    exit_stack: AsyncExitStack
+    """Per-connection teardown, unwound LIFO (shielded) when the connection
+    closes. Push cleanup from handlers or middleware; exceptions are logged
+    and swallowed."""
+
     def __init__(self, outbound: Outbound, *, has_standalone_channel: bool, session_id: str | None = None) -> None:
         self._outbound = outbound
         self.has_standalone_channel = has_standalone_channel
-        self.session_id: str | None = session_id
-
-        self.client_params: InitializeRequestParams | None = None
-        """The full `initialize` request params; `None` before initialization."""
-        self.protocol_version: str | None = None
-        self.initialized: anyio.Event = anyio.Event()
-        """Set when `notifications/initialized` arrives (matches TS `oninitialized`);
-        the point from which the spec permits server-initiated requests beyond
-        ping/logging. Pre-set on stateless connections."""
-
-        self.state: dict[str, Any] = {}
-        """Per-connection scratch state; persists across requests on this connection."""
-
-        self.exit_stack: AsyncExitStack = AsyncExitStack()
-        """Per-connection teardown, unwound LIFO (shielded) when the connection
-        closes. Push cleanup from handlers or middleware; exceptions are logged
-        and swallowed."""
+        self.session_id = session_id
+
+        self.client_params = None
+        self.protocol_version = None
+        self.initialized = anyio.Event()
+
+        self.state = {}
+
+        self.exit_stack = AsyncExitStack()
 
     @property
     def initialize_accepted(self) -> bool:

src/mcp/shared/jsonrpc_dispatcher.py

@@ -259,15 +259,15 @@ def __init__(
         )
         self._peer_cancel_mode: PeerCancelMode = peer_cancel_mode
         self._raise_handler_exceptions = raise_handler_exceptions
+        # Request methods handled inline in the read loop (awaited before the
+        # next message is dequeued) instead of spawned concurrently. Use for
+        # methods whose side effects must be observable to the next message,
+        # e.g. `initialize`, so a pipelined follow-up sees the initialized state.
+        # Only suitable for handlers that complete quickly, since inline handling
+        # blocks dequeuing; a handler that awaits the peer (`send_raw_request`)
+        # while inline will deadlock because the parked read loop cannot dequeue
+        # the response.
         self._inline_methods = inline_methods
-        """Request methods handled inline in the read loop (awaited before the
-        next message is dequeued) instead of spawned concurrently. Use for
-        methods whose side effects must be observable to the next message,
-        e.g. `initialize`, so a pipelined follow-up sees the initialized state.
-        Only suitable for handlers that complete quickly, since inline handling
-        blocks dequeuing; a handler that awaits the peer (`send_raw_request`)
-        while inline will deadlock because the parked read loop cannot dequeue
-        the response."""
 
         self._next_id = 0
         self._pending: dict[RequestId, _Pending] = {}