apunkt/trustgraph
1
0
Fork 0
mirror of https://github.com/trustgraph-ai/trustgraph.git synced 2026-07-02 22:41:01 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

fix(client): use correct put/delete config wire shape

Browse source 
ConfigApi.putConfig and deleteConfig (and the duplicate in FlowsApi) sent
a flat values:[{type,key,value}] array and a keys:{type,key} object —
neither matches the ConfigService schema, which requires keys:[namespace,
...innerKeys] and values:Record<string,unknown>. Every save in the
workbench /mcp-tools page returned `Put requires at least one key
(namespace)`.

putConfig now groups items by type (namespace) and issues one put per
group; deleteConfig sends keys:[type, key].

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
elpresidank 2026-05-01 22:16:28 -05:00
parent f8252ecd54
commit 4c356cd24c
1 changed files with 56 additions and 26 deletions
Download patch file Download diff file Expand all files Collapse all files

82
ts/packages/client/src/socket/trustgraph-socket.ts
View file

@ -1138,28 +1138,43 @@ export class FlowsApi {
} }
/** /**
* Updates configuration values * Updates configuration values. Items are grouped by `type` (the namespace);
* one put request is issued per distinct type.
*/ */
putConfig(values: { type: string; key: string; value: string }[]) { putConfig(items: { type: string; key: string; value: string }[]) {
return this.api.makeRequest<ConfigRequest, ConfigResponse>( const byType = new Map<string, Record<string, unknown>>();
"config", for (const item of items) {
{ let group = byType.get(item.type);
operation: "put", if (!group) {
values: values, group = {};
}, byType.set(item.type, group);
60000, }
); group[item.key] = item.value;
}
return Promise.all(
[...byType.entries()].map(([type, values]) =>
this.api.makeRequest<ConfigRequest, ConfigResponse>(
"config",
{
operation: "put",
keys: [type],
values,
},
60000,
),
),
).then((responses) => responses[responses.length - 1]);
} }
/** /**
* Deletes configuration entries * Deletes a configuration entry
*/ */
deleteConfig(keys: { type: string; key: string }) { deleteConfig(target: { type: string; key: string }) {
return this.api.makeRequest<ConfigRequest, ConfigResponse>( return this.api.makeRequest<ConfigRequest, ConfigResponse>(
"config", "config",
{ {
operation: "delete", operation: "delete",
keys: keys, keys: [target.type, target.key],
}, },
30000, 30000,
); );
@ -2031,28 +2046,43 @@ export class ConfigApi {
} }
/** /**
* Updates configuration values * Updates configuration values. Items are grouped by `type` (the namespace);
* one put request is issued per distinct type.
*/ */
putConfig(values: { type: string; key: string; value: string }[]) { putConfig(items: { type: string; key: string; value: string }[]) {
return this.api.makeRequest<ConfigRequest, ConfigResponse>( const byType = new Map<string, Record<string, unknown>>();
"config", for (const item of items) {
{ let group = byType.get(item.type);
operation: "put", if (!group) {
values: values, group = {};
}, byType.set(item.type, group);
60000, }
); group[item.key] = item.value;
}
return Promise.all(
[...byType.entries()].map(([type, values]) =>
this.api.makeRequest<ConfigRequest, ConfigResponse>(
"config",
{
operation: "put",
keys: [type],
values,
},
60000,
),
),
).then((responses) => responses[responses.length - 1]);
} }
/** /**
* Deletes configuration entries * Deletes a configuration entry
*/ */
deleteConfig(keys: { type: string; key: string }) { deleteConfig(target: { type: string; key: string }) {
return this.api.makeRequest<ConfigRequest, ConfigResponse>( return this.api.makeRequest<ConfigRequest, ConfigResponse>(
"config", "config",
{ {
operation: "delete", operation: "delete",
keys: keys, keys: [target.type, target.key],
}, },
30000, 30000,
); );