Evgeniy.Stepanov/RFD providers implementation#899
Open
xtmq wants to merge 3 commits intoagentclientprotocol:mainfrom
Open
Evgeniy.Stepanov/RFD providers implementation#899xtmq wants to merge 3 commits intoagentclientprotocol:mainfrom
xtmq wants to merge 3 commits intoagentclientprotocol:mainfrom
Conversation
anna239
reviewed
Apr 1, 2026
| "description": "Base URL for requests sent through this provider.", | ||
| "type": "string" | ||
| }, | ||
| "headers": { |
Contributor
There was a problem hiding this comment.
Headers are marked as required in RFD, I think they should be optional but better to change rfd
Contributor
Author
There was a problem hiding this comment.
yeah, I explicitly asked the agent to make them optional despite RFD reqires them. I feel it will be much more convenient in use.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Implementation of "Configurable LLM Providers" RFD.
Also contains several QoL improvements, we can drop them from this MR.
title: "Configurable LLM Providers"
Elevator pitch
Add the ability for clients to discover and configure agent LLM providers (identified by
id) via dedicated provider methods:providers/listproviders/setproviders/disableThis allows clients to route LLM requests through their own infrastructure (proxies, gateways, or self-hosted models) without agents needing to know about this configuration in advance.
Status quo
ACP does not currently define a standard method for configuring LLM providers.
In practice, provider configuration is usually done via environment variables or agent-specific config files. That creates several problems:
This particularly affects:
Shiny future
Clients will be able to:
Implementation details and plan
Intended flow
sequenceDiagram participant Client participant Agent Client->>Agent: initialize Agent-->>Client: initialize response (agentCapabilities.providers = {}) Client->>Agent: providers/list Agent-->>Client: providers/list response Client->>Agent: providers/set (id = "main") Agent-->>Client: providers/set response Client->>Agent: providers/disable (optional) Agent-->>Client: providers/disable response Client->>Agent: session/newagentCapabilities.providers.providers/listto discover available providers, their current routing targets (or disabled state), supported protocol types, and whether they are required.providers/setto apply new (required) configuration for a specific provider id.providers/disablewhen a non-required provider should be disabled.Capability advertisement
The agent advertises support with an empty object capability:
If
providersis absent, clients must treat provider methods as unsupported.Types
providers/listproviders/setproviders/setupdates the full configuration for one provider id.providers/disableExample exchange
initialize Response:
{ "jsonrpc": "2.0", "id": 0, "result": { "protocolVersion": 1, "agentInfo": { "name": "MyAgent", "version": "2.0.0" }, "agentCapabilities": { "providers": {}, "sessionCapabilities": {} } } }providers/list Request:
{ "jsonrpc": "2.0", "id": 1, "method": "providers/list", "params": {} }providers/list Response:
{ "jsonrpc": "2.0", "id": 1, "result": { "providers": [ { "id": "main", "supported": ["bedrock", "vertex", "azure", "anthropic"], "required": true, "current": { "apiType": "anthropic", "baseUrl": "http://localhost/anthropic" } }, { "id": "openai", "supported": ["openai"], "required": false, "current": null } ] } }providers/set Request:
{ "jsonrpc": "2.0", "id": 2, "method": "providers/set", "params": { "id": "main", "apiType": "anthropic", "baseUrl": "https://llm-gateway.corp.example.com/anthropic/v1", "headers": { "X-Request-Source": "my-ide" } } }providers/set Response:
{ "jsonrpc": "2.0", "id": 2, "result": {} }providers/disable Request:
{ "jsonrpc": "2.0", "id": 3, "method": "providers/disable", "params": { "id": "openai" } }providers/disable Response:
{ "jsonrpc": "2.0", "id": 3, "result": {} }Behavior
agentCapabilities.providers: {}ininitialize. Clients SHOULD only callproviders/*when this capability is present.initialize. Clients SHOULD configure providers before creating or loading sessions. Agents MAY choose not to apply changes to already running sessions, but SHOULD apply them to sessions created or loaded after the change.providers/listreturns configurable providers, their supported protocol types, current effective routing, andrequiredflag. Providers SHOULD remain discoverable in list afterproviders/disable.providers/disablefor providers whererequired: true.providers/list,current: nullmeans the provider is disabled and MUST NOT be used by the agent for LLM calls.providers/setreplaces the full configuration for the targetid(apiType,baseUrl, fullheaders). Ifidis unknown,apiTypeis unsupported for that provider, or params are malformed, agents SHOULD returninvalid_params.providers/disabledisables the target provider at runtime. A disabled provider MUST appear inproviders/listwithcurrent: null. If target provider hasrequired: true, agents MUST returninvalid_params. Disabling an unknownidSHOULD be treated as success (idempotent behavior).Frequently asked questions
What does
nullmean inproviders/list?current: nullmeans the provider is disabled.When disabled, the agent MUST NOT route LLM calls through that provider until the client enables it again with
providers/set.Why is there a
requiredflag?Some providers are mandatory for agent operation and must not be disabled.
requiredlets clients hide or disable the provider-disable action in UI and avoid callingproviders/disablefor those ids.Why not a single
providers/updatemethod for full list replacement?A full-list update means the client must send complete configuration (including
headers) for all providers every time.If the client wants to change only one provider, it may not know headers for the others. In that case it cannot safely build a correct full-list payload.
Also,
providers/listdoes not return headers, so the client cannot simply "take what the agent returned" and send it back with one edit.Per-provider methods (
setanddisable) avoid this problem and keep updates explicit.Why doesn't
providers/listreturn headers?Header values may contain secrets and should not be echoed by the agent.
providers/listis intentionally limited to non-secret routing information (current.apiType,current.baseUrl).Why are
providers/listandproviders/setpayloads different?providers/setacceptsheaders, including secrets, and is write-oriented.providers/listis read-oriented and returns only non-secret routing summary (current) for UI and capability discovery.Why is this separate from
initializeparams?Clients need capability discovery first, then provider discovery, then configuration. A dedicated method family keeps initialization focused on negotiation and leaves provider mutation to explicit steps.
Why not use
session-configwith aprovidercategory instead?session-configis a possible alternative, and we may revisit it as the spec evolves.We did not choose it as the primary approach in this proposal because provider routing here needs dedicated semantics that are difficult to express with today's session config model:
id, each with its own lifecycleapiType,baseUrl, fullheadersmap) rather than simple scalar valuesproviders/list) and disable (providers/disable) semanticsToday,
session-configvalues are effectively string-oriented and do not define a standard multi-value/structured model for this use case.Revision history
providers/removerenamed toproviders/disable, required providers are non-disableable, and disabled state is represented ascurrent: nullproviders/list,providers/set,providers/remove)LlmProtocolan open string type with well-known values; resolve open questions on identifier standardization and model availabilitysetLlmEndpointsmethod with capability advertisement