Align managed permission documentation

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: joshspicer

nodejs/README.md

@@ -920,7 +920,7 @@ The handler must return one of the `PermissionDecision` shapes (or `{ kind: "no-
 | `"approve-permanently"`  | Allow this request and persist the approval across sessions (currently used for URL domains) | `domain` (URL domain to approve)                                        |
 | `"reject"`               | Deny the request                                                                             | `feedback?` (optional string surfaced to the agent)                     |
 | `"user-not-available"`   | Deny the request because no user is available to confirm it                                  | —                                                                       |
-| `"no-result"`            | Leave the request unanswered (only valid with protocol v1; rejected by protocol v2 servers)  | —                                                                       |
+| `"no-result"`            | Suppress this SDK client's response so another connected client can answer the pending request | —                                                                     |
 
 ### Resuming Sessions
 

python/README.md

@@ -845,7 +845,8 @@ async def on_permission_request(
 
 The handler returns a ``PermissionRequestResult``, which is an alias for
 ``PermissionDecision | PermissionNoResult`` (the generated wire-level
-union of every decision variant, plus a small sentinel for v1 servers).
+union of every decision variant, plus a sentinel that suppresses this SDK
+client's response).
 Approval decisions are present-tense — they describe the decision to
 apply, not the past-tense outcome reported back on `permission.completed`
 session events.
@@ -855,7 +856,7 @@ session events.
 | `PermissionDecisionApproveOnce()`             | Allow this single request                                                                   |
 | `PermissionDecisionReject(feedback="…")`      | Deny the request (optional feedback string forwarded to the LLM)                            |
 | `PermissionDecisionUserNotAvailable()`        | Deny the request because no user is available to confirm it (the default)                   |
-| `PermissionNoResult()`                        | Leave the request unanswered (only valid with protocol v1; rejected by protocol v2 servers) |
+| `PermissionNoResult()`                        | Suppress this SDK client's response so another connected client can answer the pending request |
 
 Several richer variants (``PermissionDecisionApproveForSession``,
 ``PermissionDecisionApproveForLocation``, ``PermissionDecisionApprovePermanently``,

python/copilot/session.py

@@ -353,7 +353,8 @@ class PermissionNoResult:
 
 # The decision returned by a permission handler. Identical shape to the wire
 # ``PermissionDecision`` discriminated union, plus a :class:`PermissionNoResult`
-# sentinel for v1 servers. Construct via the generated variant classes:
+# sentinel that suppresses this SDK client's response. Construct via the
+# generated variant classes:
 # ``PermissionDecisionApproveOnce()``, ``PermissionDecisionReject(feedback=...)``,
 # etc. The ``kind`` discriminator is baked in as a ``ClassVar`` default by
 # codegen, so callers must not pass it.

rust/src/handler.rs

@@ -280,8 +280,8 @@ pub trait AutoModeSwitchHandler: Send + Sync + 'static {
 
 /// A [`PermissionHandler`] that approves requests when managed settings are disabled.
 ///
-/// Requests that require managed approval remain pending for an explicit human
-/// decision.
+/// When managed settings are enabled, the handler logs an error and returns a
+/// user-not-available decision.
 #[derive(Debug, Clone)]
 pub struct ApproveAllHandler;
 

rust/src/permission.rs

@@ -23,8 +23,7 @@ use crate::types::{PermissionRequestData, RequestId, SessionId};
 /// are disabled.
 ///
 /// When managed settings are enabled, the handler logs an error and returns a
-/// user-not-available decision. Requests that require managed approval remain
-/// pending for an explicit human decision.
+/// user-not-available decision.
 pub fn approve_all() -> Arc<dyn PermissionHandler> {
     Arc::new(PolicyHandler {
         policy: Policy::ApproveAll,