-
Notifications
You must be signed in to change notification settings - Fork 186
docs(rfd): Agent Authentication State Query #658
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,307 @@ | ||
| --- | ||
| title: "Agent Authentication State Query" | ||
| --- | ||
|
|
||
| - Author(s): [@xtmq](https://github.com/xtmq) | ||
|
|
||
| ## Elevator pitch | ||
|
|
||
| > What are you proposing to change? | ||
|
|
||
| Add an `auth/status` method and corresponding capability that allows clients to query the agent's current authentication state. This lets clients determine whether the agent is already configured with valid credentials or requires authorization before creating a session, without relying on the ambiguous error behavior of `session/new`. | ||
|
|
||
| ## Status quo | ||
|
|
||
| > How do things work today and what problems does this cause? Why would we change things? | ||
|
|
||
| Currently, there is no dedicated way for a client to determine whether an agent has valid authentication configured. The typical workaround is: | ||
|
|
||
| 1. Call `initialize` | ||
| 2. Call `session/new` | ||
| 3. If the agent has no credentials, it _may_ return an authorization error | ||
| 4. The client handles the error and initiates an authorization flow | ||
|
|
||
| This approach has significant problems: | ||
|
|
||
| - **Unreliable detection**: The `session/new` method is not required by the specification to check authorization. Some agents validate credentials eagerly, others do so lazily (e.g., on the first LLM call). The client cannot rely on `session/new` to consistently surface auth issues. | ||
| - **Wasted resources**: Creating a session only to discard it on auth failure is wasteful, especially if session creation has side effects (resource allocation, logging, history file creation, etc.). | ||
| - **Poor user experience**: The client cannot proactively guide the user through authorization before session creation. Instead, users encounter errors mid-flow. | ||
|
|
||
| ## Shiny future | ||
|
|
||
| > How will things play out once this feature exists? | ||
|
|
||
| Clients will be able to: | ||
|
|
||
| 1. Discover whether an agent supports auth state queries via capabilities during initialization | ||
| 2. Query the agent's current authentication state immediately after initialization | ||
| 3. Make an informed decision about whether to proceed with session creation, initiate an authorization flow, or configure endpoints (e.g., via `setLlmEndpoints`) | ||
| 4. Provide clear, proactive UX — e.g., showing a "Sign in" prompt before any session is created | ||
| 5. Completely skip authentication process if the agent is already authenticated | ||
|
|
||
| ## Implementation details and plan | ||
|
|
||
| > Tell me more about your implementation. What is your detailed implementation plan? | ||
|
|
||
| ### Intended flow | ||
|
|
||
| The client calls `initialize`, inspects capabilities to confirm `auth/status` support, then queries auth state before deciding how to proceed. | ||
|
|
||
| ```mermaid | ||
| sequenceDiagram | ||
| participant Client | ||
| participant Agent | ||
|
|
||
| Client->>Agent: initialize | ||
| Note right of Agent: Agent reports capabilities,<br/>including auth/status support | ||
| Agent-->>Client: initialize response<br/>(agentCapabilities.auth.status) | ||
|
|
||
| Client->>Agent: auth/status | ||
| Agent-->>Client: auth/status response<br/>(authenticated, authMethods) | ||
|
|
||
| alt Authenticated | ||
| Client->>Agent: session/new | ||
| else Not authenticated | ||
| Note over Client: Client initiates<br/>authorization flow<br/>(e.g., call authenticate, setLlmEndpoints) | ||
| Client->>Agent: authenticate | ||
| Client->>Agent: session/new | ||
| end | ||
| ``` | ||
|
|
||
| 1. **Initialization**: The client calls `initialize`. The agent responds with capabilities, including `auth/status` support via `agentCapabilities.auth`. | ||
| 2. **Auth state query**: The client calls `auth/status`. The agent inspects its local configuration, stored credentials, or environment to determine the current auth state. | ||
| 3. **Client-side decision**: Based on the response, the client either proceeds to session creation or initiates an authorization flow first. | ||
|
|
||
| ### Capability advertisement | ||
|
|
||
| The agent advertises support for the `auth/status` method via the existing `auth` capability in `agentCapabilities`: | ||
|
|
||
| ```typescript | ||
| interface AuthCapabilities { | ||
| // ... existing fields (e.g., terminal) ... | ||
|
|
||
| /** | ||
| * Auth status query support. | ||
| * If true, the agent supports the auth/status method. | ||
| */ | ||
| status?: boolean; | ||
| } | ||
| ``` | ||
|
|
||
| **Initialize Response example:** | ||
|
|
||
| ```json | ||
| { | ||
| "jsonrpc": "2.0", | ||
| "id": 0, | ||
| "result": { | ||
| "protocolVersion": 1, | ||
| "agentInfo": { | ||
| "name": "MyAgent", | ||
| "version": "2.0.0" | ||
| }, | ||
| "agentCapabilities": { | ||
| "auth": { | ||
| "status": true | ||
| }, | ||
| "sessionCapabilities": {} | ||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| ### `auth/status` method | ||
|
|
||
| A method that can be called after initialization to query the agent's current authentication state. | ||
|
|
||
| ```typescript | ||
| interface AuthMethodState { | ||
| /** | ||
| * The ID of the authentication method. | ||
| * Corresponds to an `id` from the `authMethods` array returned during initialization. | ||
| */ | ||
| authMethodId: string; | ||
|
|
||
| /** | ||
| * Whether the agent has credentials configured for this auth method. | ||
| * true means credentials are present (validity is not guaranteed). | ||
| */ | ||
| authenticated: boolean; | ||
|
|
||
| /** Human-readable description of the auth state (e.g., "API key configured via environment") */ | ||
| message?: string; | ||
|
|
||
| /** Extension metadata */ | ||
| _meta?: Record<string, unknown>; | ||
| } | ||
|
|
||
| interface AuthStatusRequest { | ||
| /** Extension metadata */ | ||
| _meta?: Record<string, unknown>; | ||
| } | ||
|
|
||
| interface AuthStatusResponse { | ||
| /** | ||
| * Whether the agent has credentials configured. | ||
| * true means credentials are present (validity is not guaranteed). | ||
| * false means no credentials are configured. | ||
| */ | ||
| authenticated: boolean; | ||
|
|
||
| /** | ||
| * Optional per-auth-method breakdown of auth state. | ||
| * Each entry corresponds to an auth method from the `authMethods` array | ||
| * returned during initialization. | ||
| */ | ||
| authMethods?: AuthMethodState[]; | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah do we need this list is I guess what I am asking?
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The idea -
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. RIght but at least for some of the agents I use, this isn't valid. I could have gotten an API key via a terminal login, which would then mark both the terminal and api auth method as authenticated? |
||
|
|
||
| /** Human-readable description of the overall auth state */ | ||
| message?: string; | ||
|
|
||
| /** Extension metadata */ | ||
| _meta?: Record<string, unknown>; | ||
| } | ||
| ``` | ||
|
|
||
| #### JSON Schema Additions | ||
|
|
||
| ```json | ||
| { | ||
| "$defs": { | ||
| "AuthMethodState": { | ||
| "description": "Authentication state for a specific auth method.", | ||
| "properties": { | ||
| "authMethodId": { | ||
| "type": "string", | ||
| "description": "The ID of the authentication method, matching an entry from the authMethods array returned during initialization." | ||
| }, | ||
| "authenticated": { | ||
| "type": "boolean", | ||
| "description": "Whether credentials are configured for this auth method." | ||
| }, | ||
| "message": { | ||
| "type": ["string", "null"], | ||
| "description": "Human-readable description of the auth state." | ||
| }, | ||
| "_meta": { | ||
| "additionalProperties": true, | ||
| "type": ["object", "null"] | ||
| } | ||
| }, | ||
| "required": ["authMethodId", "authenticated"], | ||
| "type": "object" | ||
| }, | ||
| "AuthStatusResponse": { | ||
| "description": "Response to auth/status method.", | ||
| "properties": { | ||
| "authenticated": { | ||
| "type": "boolean", | ||
| "description": "Whether the agent has credentials configured." | ||
| }, | ||
| "authMethods": { | ||
| "type": ["array", "null"], | ||
| "description": "Per-auth-method state breakdown.", | ||
| "items": { | ||
| "$ref": "#/$defs/AuthMethodState" | ||
| } | ||
| }, | ||
| "message": { | ||
| "type": ["string", "null"], | ||
| "description": "Human-readable description of the overall auth state." | ||
| }, | ||
| "_meta": { | ||
| "additionalProperties": true, | ||
| "type": ["object", "null"] | ||
| } | ||
| }, | ||
| "required": ["authenticated"], | ||
| "type": "object" | ||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| #### Example Exchange | ||
|
|
||
| **auth/status Request:** | ||
|
|
||
| ```json | ||
| { | ||
| "jsonrpc": "2.0", | ||
| "id": 1, | ||
| "method": "auth/status", | ||
| "params": {} | ||
| } | ||
| ``` | ||
|
|
||
| **auth/status Response (authenticated):** | ||
|
|
||
| ```json | ||
| { | ||
| "jsonrpc": "2.0", | ||
| "id": 1, | ||
| "result": { | ||
| "authenticated": true, | ||
| "authMethods": [ | ||
| { | ||
| "authMethodId": "anthropic-api-key", | ||
| "authenticated": true, | ||
| "message": "API key configured via local config" | ||
| } | ||
| ] | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| **auth/status Response (unauthenticated):** | ||
|
|
||
| ```json | ||
| { | ||
| "jsonrpc": "2.0", | ||
| "id": 1, | ||
| "result": { | ||
| "authenticated": false, | ||
| "message": "No credentials configured. Please provide API keys or configure an LLM endpoint." | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| #### Behavior | ||
|
|
||
| 1. **Capability advertisement**: The agent SHOULD include `auth.status` in `agentCapabilities` if it supports the `auth/status` method. Clients MUST check for this capability before calling the method. | ||
|
|
||
| 2. **Timing**: The `auth/status` method MUST be callable after `initialize`. It MAY be called multiple times (e.g., after `authenticate`, to re-check state). | ||
|
|
||
| 3. **Local checks only**: The agent MAY determine auth state based on locally available information (config files, environment variables, stored tokens) or by making external API calls. `authenticated: true` means credentials are present, NOT that they are guaranteed to be valid. | ||
|
|
||
| 4. **No side effects**: Calling `auth/status` MUST NOT modify any agent state. It is a pure query. | ||
|
|
||
| 5. **Per-auth-method breakdown**: The `authMethods` field is optional. Agents that support multiple authentication methods MAY include per-method state to help clients make fine-grained decisions. | ||
|
|
||
| ## Open questions | ||
|
|
||
| ### Is a per-auth-method breakdown needed, or is an aggregate state sufficient? | ||
|
|
||
| The current design includes an optional `authMethods` field with per-auth-method state. However, for many agents a single aggregate `state` may be enough. A per-method breakdown adds complexity to both the agent implementation and client logic. Should we simplify to just the top-level `state` and `message`? | ||
|
|
||
| ## Frequently asked questions | ||
|
|
||
| > What questions have arisen over the course of authoring this document? | ||
|
|
||
| ### Why not rely on `session/new` errors? | ||
|
|
||
| The `session/new` method is designed for session creation, not for auth validation. Per the specification, agents are not required to validate credentials during session creation — some agents defer validation to the first actual LLM call. This means a successful `session/new` does not guarantee the agent is authenticated, and a failed `session/new` may fail for reasons unrelated to authentication. A dedicated method provides a clear, unambiguous signal. | ||
|
|
||
| ### Why not include auth state in the `initialize` response directly? | ||
|
|
||
| The `initialize` response contains capabilities — what the agent _supports_. Auth state is runtime information — what the agent _currently has configured_. Mixing these concerns would make `initialize` less predictable. Additionally, auth state may change after initialization (e.g., after a `setLlmEndpoints` call), and a separate method allows re-querying. | ||
|
|
||
| ### Why not just check for the presence of environment variables on the client side? | ||
|
|
||
| Agents may obtain credentials from many sources: config files, keychains, OAuth tokens, environment variables, or even embedded keys. The client has no visibility into these mechanisms. Only the agent knows whether it has usable credentials configured. | ||
|
|
||
| ## Revision history | ||
|
|
||
| - 2026-03-10: Rename `getAuthState` to `auth/status`, nest capability under `agentCapabilities.auth` | ||
| - 2026-03-07: Address review feedback, replace per-provider breakdown with per-auth-method | ||
| - 2026-03-05: Initial draft — preliminary proposal to start discussion | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm I am not always sure this will map 1:1
For example, a terminal auth may ultimately support more than one login method (they often have a way to choose how you want to authenticate)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, does it matter which one was chosen specifically in terms of ids?
Maybe we can just have the boolean and a message/label to show the user?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it can be useful to be able to distinguish saved api key case from oauth. Don't you think so? But we can put it in _meta later on and now stick to the simpliest solution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is useful... but I just don't think it will map 1:1 to the auth method id is my concern... because several of these agents you can get multiple types of auth from the same auth method
Maybe we need some kind of category of auth? curious to hear your thoughts too @anna239
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's maybe remove auth methods, as it looks like it's controversial and we can live without it for now