Fix: Update existing provider instead of creating duplicate
This commit is contained in:
@@ -408,24 +408,54 @@ async function stepProviders(config: OnboardConfig, env: Record<string, string>)
|
|||||||
if (p.isCancel(providerKey)) { p.cancel('Setup cancelled'); process.exit(0); }
|
if (p.isCancel(providerKey)) { p.cancel('Setup cancelled'); process.exit(0); }
|
||||||
|
|
||||||
if (providerKey) {
|
if (providerKey) {
|
||||||
// Create provider via Letta API
|
// Create or update provider via Letta API
|
||||||
const spinner = p.spinner();
|
const spinner = p.spinner();
|
||||||
spinner.start(`Connecting ${provider.displayName}...`);
|
spinner.start(`Connecting ${provider.displayName}...`);
|
||||||
|
|
||||||
try {
|
try {
|
||||||
const response = await fetch('https://api.letta.com/v1/providers', {
|
// First check if provider already exists
|
||||||
method: 'POST',
|
const listResponse = await fetch('https://api.letta.com/v1/providers', {
|
||||||
headers: {
|
headers: {
|
||||||
'Content-Type': 'application/json',
|
'Content-Type': 'application/json',
|
||||||
'Authorization': `Bearer ${apiKey}`,
|
'Authorization': `Bearer ${apiKey}`,
|
||||||
},
|
},
|
||||||
body: JSON.stringify({
|
|
||||||
name: provider.name,
|
|
||||||
provider_type: provider.providerType,
|
|
||||||
api_key: providerKey,
|
|
||||||
}),
|
|
||||||
});
|
});
|
||||||
|
|
||||||
|
let existingProvider: { id: string; name: string } | undefined;
|
||||||
|
if (listResponse.ok) {
|
||||||
|
const providers = await listResponse.json() as Array<{ id: string; name: string }>;
|
||||||
|
existingProvider = providers.find(p => p.name === provider.name);
|
||||||
|
}
|
||||||
|
|
||||||
|
let response: Response;
|
||||||
|
if (existingProvider) {
|
||||||
|
// Update existing provider
|
||||||
|
response = await fetch(`https://api.letta.com/v1/providers/${existingProvider.id}`, {
|
||||||
|
method: 'PATCH',
|
||||||
|
headers: {
|
||||||
|
'Content-Type': 'application/json',
|
||||||
|
'Authorization': `Bearer ${apiKey}`,
|
||||||
|
},
|
||||||
|
body: JSON.stringify({
|
||||||
|
api_key: providerKey,
|
||||||
|
}),
|
||||||
|
});
|
||||||
|
} else {
|
||||||
|
// Create new provider
|
||||||
|
response = await fetch('https://api.letta.com/v1/providers', {
|
||||||
|
method: 'POST',
|
||||||
|
headers: {
|
||||||
|
'Content-Type': 'application/json',
|
||||||
|
'Authorization': `Bearer ${apiKey}`,
|
||||||
|
},
|
||||||
|
body: JSON.stringify({
|
||||||
|
name: provider.name,
|
||||||
|
provider_type: provider.providerType,
|
||||||
|
api_key: providerKey,
|
||||||
|
}),
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
if (response.ok) {
|
if (response.ok) {
|
||||||
spinner.stop(`Connected ${provider.displayName}`);
|
spinner.stop(`Connected ${provider.displayName}`);
|
||||||
config.providers.push({ id: provider.id, name: provider.name, apiKey: providerKey });
|
config.providers.push({ id: provider.id, name: provider.name, apiKey: providerKey });
|
||||||
|
|||||||
Reference in New Issue
Block a user