-
Notifications
You must be signed in to change notification settings - Fork 223
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
Open
xtmq
wants to merge
5
commits into
agentclientprotocol:main
Choose a base branch
from
xtmq:evgeniy.stepanov/rfd-get-auth-state
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
9947f35
docs: add RFD for `getAuthState` method
xtmq f52769a
docs: address review feedback for getAuthState RFD
xtmq 1abefd5
docs: formatting for getAuthState RFD
xtmq debb4a8
docs: naming improvements in `auth/status` RFD
xtmq 7094e4d
docs: simplify auth/status response
xtmq File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,295 @@ | ||
| --- | ||
| title: "Agent Authentication State Query" | ||
| --- | ||
|
|
||
| - Author(s): [@xtmq](https://github.com/xtmq) | ||
|
|
||
| > **Note:** This RFD is very preliminary and intended to start a dialog about this feature. The proposed design may change significantly based on feedback and further discussion. | ||
|
|
||
| ## Elevator pitch | ||
|
|
||
| > What are you proposing to change? | ||
|
|
||
| Add a `getAuthState` 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 `getAuthState` 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 getAuthState support | ||
| Agent-->>Client: initialize response<br/>(agentCapabilities.getAuthState) | ||
|
|
||
| Client->>Agent: getAuthState | ||
| Agent-->>Client: getAuthState response<br/>(authenticated, providers) | ||
|
|
||
| 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 `getAuthState` if supported. | ||
| 2. **Auth state query**: The client calls `getAuthState`. 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 `getAuthState` method via a new capability in `agentCapabilities`: | ||
|
|
||
| ```typescript | ||
| interface AgentCapabilities { | ||
| // ... existing fields ... | ||
|
|
||
| /** | ||
| * Auth state query support. | ||
| * If true, the agent supports the getAuthState method. | ||
| */ | ||
| getAuthState?: boolean; | ||
| } | ||
| ``` | ||
|
|
||
| **Initialize Response example:** | ||
| ```json | ||
| { | ||
| "jsonrpc": "2.0", | ||
| "id": 0, | ||
| "result": { | ||
| "protocolVersion": 1, | ||
| "agentInfo": { | ||
| "name": "MyAgent", | ||
| "version": "2.0.0" | ||
| }, | ||
| "agentCapabilities": { | ||
| "getAuthState": true, | ||
|
xtmq marked this conversation as resolved.
Outdated
|
||
| "sessionCapabilities": {} | ||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| ### `getAuthState` method | ||
|
|
||
| A method that can be called after initialization to query the agent's current authentication state. | ||
|
|
||
| ```typescript | ||
| interface AuthProviderState { | ||
|
xtmq marked this conversation as resolved.
Outdated
|
||
| /** | ||
| * Whether the agent has credentials configured for this provider. | ||
| * 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 GetAuthStateRequest { | ||
| /** Extension metadata */ | ||
| _meta?: Record<string, unknown>; | ||
| } | ||
|
|
||
| interface GetAuthStateResponse { | ||
| /** | ||
| * 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-provider breakdown of auth state. | ||
| * Keys are provider identifiers (e.g., "anthropic", "openai"). | ||
| */ | ||
| providers?: Record<string, AuthProviderState>; | ||
|
|
||
| /** Human-readable description of the overall auth state */ | ||
| message?: string; | ||
|
|
||
| /** Extension metadata */ | ||
| _meta?: Record<string, unknown>; | ||
| } | ||
| ``` | ||
|
|
||
| #### JSON Schema Additions | ||
|
|
||
| ```json | ||
| { | ||
| "$defs": { | ||
| "AuthProviderState": { | ||
| "description": "Authentication state for a specific provider.", | ||
| "properties": { | ||
| "authenticated": { | ||
| "type": "boolean", | ||
| "description": "Whether credentials are configured for this provider." | ||
| }, | ||
| "message": { | ||
| "type": ["string", "null"], | ||
| "description": "Human-readable description of the auth state." | ||
| }, | ||
| "_meta": { | ||
| "additionalProperties": true, | ||
| "type": ["object", "null"] | ||
| } | ||
| }, | ||
| "required": ["authenticated"], | ||
| "type": "object" | ||
| }, | ||
| "GetAuthStateResponse": { | ||
| "description": "Response to getAuthState method.", | ||
| "properties": { | ||
| "authenticated": { | ||
| "type": "boolean", | ||
| "description": "Whether the agent has credentials configured." | ||
| }, | ||
| "providers": { | ||
| "type": ["object", "null"], | ||
| "description": "Per-provider auth state breakdown.", | ||
| "additionalProperties": { | ||
| "$ref": "#/$defs/AuthProviderState" | ||
| } | ||
| }, | ||
| "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 | ||
|
|
||
| **getAuthState Request:** | ||
| ```json | ||
| { | ||
| "jsonrpc": "2.0", | ||
| "id": 1, | ||
| "method": "getAuthState", | ||
| "params": {} | ||
| } | ||
| ``` | ||
|
|
||
| **getAuthState Response (authenticated):** | ||
| ```json | ||
| { | ||
| "jsonrpc": "2.0", | ||
| "id": 1, | ||
| "result": { | ||
| "authenticated": true, | ||
| "providers": { | ||
| "anthropic": { | ||
| "authenticated": true, | ||
| "message": "API key configured via local config" | ||
| } | ||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| **getAuthState 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 MUST include `getAuthState` in `agentCapabilities` if it supports the `getAuthState` method. Clients SHOULD check for this capability before calling the method. | ||
|
xtmq marked this conversation as resolved.
Outdated
|
||
|
|
||
| 2. **Timing**: The `getAuthState` 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 SHOULD determine auth state based on locally available information (config files, environment variables, stored tokens). It SHOULD NOT make external API calls to validate credentials. `authenticated: true` means credentials are present, not that they are guaranteed to be valid. | ||
|
xtmq marked this conversation as resolved.
Outdated
|
||
|
|
||
| 4. **No side effects**: Calling `getAuthState` MUST NOT modify any agent state. It is a pure query. | ||
|
|
||
| 5. **Per-provider breakdown**: The `providers` field is optional. Agents that use multiple LLM providers MAY include per-provider state to help clients make fine-grained decisions. | ||
|
|
||
| ## Open questions | ||
|
|
||
| ### Is a per-provider breakdown needed, or is an aggregate state sufficient? | ||
|
|
||
| The current design includes an optional `providers` field with per-provider auth state. However, for many agents a single aggregate `state` may be enough. A per-provider breakdown adds complexity to both the agent implementation and client logic. Should we simplify to just the top-level `state` and `message`? | ||
|
|
||
| ### Should `getAuthState` indicate what kind of auth is expected? | ||
|
|
||
| When the state is `"unauthenticated"`, should the response include hints about what the agent needs? For example: | ||
|
xtmq marked this conversation as resolved.
Outdated
|
||
| - "Needs an API key for Anthropic **or** OpenAI" | ||
|
|
||
| This could help clients build more targeted authorization flows, but it adds complexity. | ||
|
|
||
| ## 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-05: Initial draft — preliminary proposal to start discussion | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.