aljaz/goose
1
0
Fork 0
mirror of https://github.com/aljazceru/goose.git synced 2025-12-18 22:54:24 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: sanitize names before enabling/disabling extensions in Settings V2 (#2040)

Browse Source
This commit is contained in:
Alex Hancock
2025-04-04 15:25:25 -04:00
committed by GitHub
parent eebb4fa3ee
commit 4aa7758914
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
ui/desktop/src/components/settings_v2/extensions/agent-api.ts
View File

@@ -140,6 +140,8 @@ export async function addToAgent(
extension.cmd = await replaceWithShims(extension.cmd); extension.cmd = await replaceWithShims(extension.cmd);
} }
extension.name = sanitizeName(extension.name);
return await extensionApiCall('/extensions/add', extension, options); return await extensionApiCall('/extensions/add', extension, options);
} catch (error) { } catch (error) {
// Check if this is a 428 error and make the message more descriptive // Check if this is a 428 error and make the message more descriptive
@@ -162,9 +164,13 @@ export async function removeFromAgent(
options: ToastServiceOptions = {} options: ToastServiceOptions = {}
): Promise<Response> { ): Promise<Response> {
try { try {
return await extensionApiCall('/extensions/remove', name, options); return await extensionApiCall('/extensions/remove', sanitizeName(name), options);
} catch (error) { } catch (error) {
console.error(`Failed to remove extension ${name} from agent:`, error); console.error(`Failed to remove extension ${name} from agent:`, error);
throw error; throw error;
} }
} }
function sanitizeName(name: string) {
return name.toLowerCase().replace(/-/g, '').replace(/_/g, '').replace(/\s/g, '');
}