Skip to content

Commit 8659fc2

Browse files
benbrandtbenbrandt
authored
fix(unstable): Rename provider method types to singular (#1272)
For set and disable, these only ever operate on one at a time.
1 parent 2a18ab2 commit 8659fc2

9 files changed

Lines changed: 122 additions & 122 deletions

File tree

‎docs/protocol/draft/schema-v2.mdx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -726,7 +726,7 @@ This capability is not part of the spec yet, and may be removed or changed at an
726726

727727
Disables a provider.
728728

729-
#### <span class="font-mono">DisableProvidersRequest</span>
729+
#### <span class="font-mono">DisableProviderRequest</span>
730730

731731
**UNSTABLE**
732732

@@ -750,7 +750,7 @@ See protocol docs: [Extensibility](https://agentclientprotocol.com/protocol/exte
750750
Provider id to disable.
751751
</ResponseField>
752752

753-
#### <span class="font-mono">DisableProvidersResponse</span>
753+
#### <span class="font-mono">DisableProviderResponse</span>
754754

755755
**UNSTABLE**
756756

@@ -834,7 +834,7 @@ This capability is not part of the spec yet, and may be removed or changed at an
834834

835835
Replaces the configuration for a provider.
836836

837-
#### <span class="font-mono">SetProvidersRequest</span>
837+
#### <span class="font-mono">SetProviderRequest</span>
838838

839839
**UNSTABLE**
840840

@@ -870,7 +870,7 @@ May include authorization, routing, or other integration-specific headers.
870870
Provider id to configure.
871871
</ResponseField>
872872

873-
#### <span class="font-mono">SetProvidersResponse</span>
873+
#### <span class="font-mono">SetProviderResponse</span>
874874

875875
**UNSTABLE**
876876

‎docs/protocol/draft/schema.mdx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -726,7 +726,7 @@ This capability is not part of the spec yet, and may be removed or changed at an
726726

727727
Disables a provider.
728728

729-
#### <span class="font-mono">DisableProvidersRequest</span>
729+
#### <span class="font-mono">DisableProviderRequest</span>
730730

731731
**UNSTABLE**
732732

@@ -750,7 +750,7 @@ See protocol docs: [Extensibility](https://agentclientprotocol.com/protocol/exte
750750
Provider id to disable.
751751
</ResponseField>
752752

753-
#### <span class="font-mono">DisableProvidersResponse</span>
753+
#### <span class="font-mono">DisableProviderResponse</span>
754754

755755
**UNSTABLE**
756756

@@ -834,7 +834,7 @@ This capability is not part of the spec yet, and may be removed or changed at an
834834

835835
Replaces the configuration for a provider.
836836

837-
#### <span class="font-mono">SetProvidersRequest</span>
837+
#### <span class="font-mono">SetProviderRequest</span>
838838

839839
**UNSTABLE**
840840

@@ -870,7 +870,7 @@ May include authorization, routing, or other integration-specific headers.
870870
Provider id to configure.
871871
</ResponseField>
872872

873-
#### <span class="font-mono">SetProvidersResponse</span>
873+
#### <span class="font-mono">SetProviderResponse</span>
874874

875875
**UNSTABLE**
876876

‎docs/rfds/custom-llm-endpoint.mdx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ interface ProvidersListResponse {
174174
`providers/set` updates the full configuration for one provider id.
175175

176176
```typescript
177-
interface ProvidersSetRequest {
177+
interface SetProviderRequest {
178178
/** Provider id to configure. */
179179
id: string;
180180

@@ -195,7 +195,7 @@ interface ProvidersSetRequest {
195195
_meta?: Record<string, unknown>;
196196
}
197197

198-
interface ProvidersSetResponse {
198+
interface SetProviderResponse {
199199
/** Extension metadata */
200200
_meta?: Record<string, unknown>;
201201
}
@@ -204,15 +204,15 @@ interface ProvidersSetResponse {
204204
### `providers/disable`
205205

206206
```typescript
207-
interface ProvidersDisableRequest {
207+
interface DisableProviderRequest {
208208
/** Provider id to disable. */
209209
id: string;
210210

211211
/** Extension metadata */
212212
_meta?: Record<string, unknown>;
213213
}
214214

215-
interface ProvidersDisableResponse {
215+
interface DisableProviderResponse {
216216
/** Extension metadata */
217217
_meta?: Record<string, unknown>;
218218
}

‎schema/schema.unstable.json‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -376,18 +376,18 @@
376376
{
377377
"allOf": [
378378
{
379-
"$ref": "#/$defs/SetProvidersResponse"
379+
"$ref": "#/$defs/SetProviderResponse"
380380
}
381381
],
382-
"title": "SetProvidersResponse"
382+
"title": "SetProviderResponse"
383383
},
384384
{
385385
"allOf": [
386386
{
387-
"$ref": "#/$defs/DisableProvidersResponse"
387+
"$ref": "#/$defs/DisableProviderResponse"
388388
}
389389
],
390-
"title": "DisableProvidersResponse"
390+
"title": "DisableProviderResponse"
391391
},
392392
{
393393
"allOf": [
@@ -1234,20 +1234,20 @@
12341234
{
12351235
"allOf": [
12361236
{
1237-
"$ref": "#/$defs/SetProvidersRequest"
1237+
"$ref": "#/$defs/SetProviderRequest"
12381238
}
12391239
],
12401240
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nReplaces the configuration for a provider.",
1241-
"title": "SetProvidersRequest"
1241+
"title": "SetProviderRequest"
12421242
},
12431243
{
12441244
"allOf": [
12451245
{
1246-
"$ref": "#/$defs/DisableProvidersRequest"
1246+
"$ref": "#/$defs/DisableProviderRequest"
12471247
}
12481248
],
12491249
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nDisables a provider.",
1250-
"title": "DisableProvidersRequest"
1250+
"title": "DisableProviderRequest"
12511251
},
12521252
{
12531253
"allOf": [
@@ -2289,7 +2289,7 @@
22892289
"required": ["path", "newText"],
22902290
"type": "object"
22912291
},
2292-
"DisableProvidersRequest": {
2292+
"DisableProviderRequest": {
22932293
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nRequest parameters for `providers/disable`.",
22942294
"properties": {
22952295
"_meta": {
@@ -2307,7 +2307,7 @@
23072307
"x-method": "providers/disable",
23082308
"x-side": "agent"
23092309
},
2310-
"DisableProvidersResponse": {
2310+
"DisableProviderResponse": {
23112311
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nResponse to `providers/disable`.",
23122312
"properties": {
23132313
"_meta": {
@@ -6382,7 +6382,7 @@
63826382
}
63836383
]
63846384
},
6385-
"SetProvidersRequest": {
6385+
"SetProviderRequest": {
63866386
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nRequest parameters for `providers/set`.\n\nReplaces the full configuration for one provider id.",
63876387
"properties": {
63886388
"_meta": {
@@ -6419,7 +6419,7 @@
64196419
"x-method": "providers/set",
64206420
"x-side": "agent"
64216421
},
6422-
"SetProvidersResponse": {
6422+
"SetProviderResponse": {
64236423
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nResponse to `providers/set`.",
64246424
"properties": {
64256425
"_meta": {

‎schema/schema.v2.unstable.json‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -376,18 +376,18 @@
376376
{
377377
"allOf": [
378378
{
379-
"$ref": "#/$defs/SetProvidersResponse"
379+
"$ref": "#/$defs/SetProviderResponse"
380380
}
381381
],
382-
"title": "SetProvidersResponse"
382+
"title": "SetProviderResponse"
383383
},
384384
{
385385
"allOf": [
386386
{
387-
"$ref": "#/$defs/DisableProvidersResponse"
387+
"$ref": "#/$defs/DisableProviderResponse"
388388
}
389389
],
390-
"title": "DisableProvidersResponse"
390+
"title": "DisableProviderResponse"
391391
},
392392
{
393393
"allOf": [
@@ -1234,20 +1234,20 @@
12341234
{
12351235
"allOf": [
12361236
{
1237-
"$ref": "#/$defs/SetProvidersRequest"
1237+
"$ref": "#/$defs/SetProviderRequest"
12381238
}
12391239
],
12401240
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nReplaces the configuration for a provider.",
1241-
"title": "SetProvidersRequest"
1241+
"title": "SetProviderRequest"
12421242
},
12431243
{
12441244
"allOf": [
12451245
{
1246-
"$ref": "#/$defs/DisableProvidersRequest"
1246+
"$ref": "#/$defs/DisableProviderRequest"
12471247
}
12481248
],
12491249
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nDisables a provider.",
1250-
"title": "DisableProvidersRequest"
1250+
"title": "DisableProviderRequest"
12511251
},
12521252
{
12531253
"allOf": [
@@ -2289,7 +2289,7 @@
22892289
"required": ["path", "newText"],
22902290
"type": "object"
22912291
},
2292-
"DisableProvidersRequest": {
2292+
"DisableProviderRequest": {
22932293
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nRequest parameters for `providers/disable`.",
22942294
"properties": {
22952295
"_meta": {
@@ -2307,7 +2307,7 @@
23072307
"x-method": "providers/disable",
23082308
"x-side": "agent"
23092309
},
2310-
"DisableProvidersResponse": {
2310+
"DisableProviderResponse": {
23112311
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nResponse to `providers/disable`.",
23122312
"properties": {
23132313
"_meta": {
@@ -6382,7 +6382,7 @@
63826382
}
63836383
]
63846384
},
6385-
"SetProvidersRequest": {
6385+
"SetProviderRequest": {
63866386
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nRequest parameters for `providers/set`.\n\nReplaces the full configuration for one provider id.",
63876387
"properties": {
63886388
"_meta": {
@@ -6419,7 +6419,7 @@
64196419
"x-method": "providers/set",
64206420
"x-side": "agent"
64216421
},
6422-
"SetProvidersResponse": {
6422+
"SetProviderResponse": {
64236423
"description": "**UNSTABLE**\n\nThis capability is not part of the spec yet, and may be removed or changed at any point.\n\nResponse to `providers/set`.",
64246424
"properties": {
64256425
"_meta": {

‎src/bin/generate.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1170,8 +1170,8 @@ starting with '$/' it is free to ignore the notification."
11701170
"initialize" => self.agent.get("InitializeRequest").unwrap(),
11711171
"authenticate" => self.agent.get("AuthenticateRequest").unwrap(),
11721172
"providers/list" => self.agent.get("ListProvidersRequest").unwrap(),
1173-
"providers/set" => self.agent.get("SetProvidersRequest").unwrap(),
1174-
"providers/disable" => self.agent.get("DisableProvidersRequest").unwrap(),
1173+
"providers/set" => self.agent.get("SetProviderRequest").unwrap(),
1174+
"providers/disable" => self.agent.get("DisableProviderRequest").unwrap(),
11751175
"session/new" => self.agent.get("NewSessionRequest").unwrap(),
11761176
"session/load" => self.agent.get("LoadSessionRequest").unwrap(),
11771177
"session/list" => self.agent.get("ListSessionsRequest").unwrap(),

0 commit comments

Comments
 (0)