Skip to content

Commit 2eb8e4f

Browse files
committed
Remove confirmation prompts for deleting API providers and tool configurations
1 parent b513cba commit 2eb8e4f

3 files changed

Lines changed: 34 additions & 50 deletions

File tree

packages/vscode/src/commands/open-settings-command/configure-api-providers.ts

Lines changed: 14 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -144,31 +144,23 @@ export const configure_api_providers = async (
144144
}
145145
resolve()
146146
} else if (event.button === delete_button) {
147-
const confirm = await vscode.window.showWarningMessage(
148-
`Are you sure you want to delete "${item.label}"?`,
149-
{ modal: true },
150-
'Delete'
147+
const providers = await providers_manager.get_providers()
148+
const updated_providers = providers.filter(
149+
(p) => p.name != item.provider.name
151150
)
151+
await providers_manager.save_providers(updated_providers)
152152

153-
if (confirm == 'Delete') {
154-
const providers = await providers_manager.get_providers()
155-
const updated_providers = providers.filter(
156-
(p) => p.name != item.provider.name
153+
if (updated_providers.length == 0) {
154+
is_accepted = true
155+
quick_pick.hide()
156+
vscode.window.showInformationMessage(
157+
'All API providers have been removed.'
157158
)
158-
await providers_manager.save_providers(updated_providers)
159-
160-
if (updated_providers.length === 0) {
161-
is_accepted = true
162-
quick_pick.hide()
163-
vscode.window.showInformationMessage(
164-
'All API providers have been removed.'
165-
)
166-
await show_create_provider_quick_pick()
167-
resolve()
168-
} else {
169-
quick_pick.items = await create_provider_items()
170-
quick_pick.show()
171-
}
159+
await show_create_provider_quick_pick()
160+
resolve()
161+
} else {
162+
quick_pick.items = await create_provider_items()
163+
quick_pick.show()
172164
}
173165
} else if (
174166
event.button === move_up_button ||

packages/vscode/src/commands/open-settings-command/open-settings-command.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { configure_api_providers } from './configure-api-providers'
33
import { setup_api_tool_multi_config } from './setup-api-tool-multi-config'
44
import { setup_api_tool } from './setup-api-tool'
55

6-
const LABEL_PROVIDERS = '$(key) API Providers'
6+
const LABEL_PROVIDERS = '$(key) Configure API Providers'
77
const LABEL_CODE_COMPLETIONS = 'Code Completions'
88
const LABEL_EDIT_CONTEXT = 'Edit Context'
99
const LABEL_INTELLIGENT_UPDATE = 'Intelligent Update'

packages/vscode/src/commands/open-settings-command/setup-api-tool-multi-config.ts

Lines changed: 19 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -283,35 +283,27 @@ export const setup_api_tool_multi_config = async (params: {
283283
quick_pick.items = create_config_items()
284284
quick_pick.show()
285285
} else if (event.button === delete_button) {
286-
const confirm = await vscode.window.showWarningMessage(
287-
`Are you sure you want to delete ${item.config.model} (${item.config.provider_name})?`,
288-
{ modal: true },
289-
'Delete'
290-
)
291-
292-
if (confirm == 'Delete') {
293-
current_configs.splice(item.index, 1)
294-
await tool_methods.save_configs(current_configs)
286+
current_configs.splice(item.index, 1)
287+
await tool_methods.save_configs(current_configs)
295288

296-
if (
297-
default_config &&
298-
default_config.provider_type == item.config.provider_type &&
299-
default_config.provider_name == item.config.provider_name &&
300-
default_config.model == item.config.model
301-
) {
302-
default_config = undefined
303-
await tool_methods.set_default_config(null)
304-
}
289+
if (
290+
default_config &&
291+
default_config.provider_type == item.config.provider_type &&
292+
default_config.provider_name == item.config.provider_name &&
293+
default_config.model == item.config.model
294+
) {
295+
default_config = undefined
296+
await tool_methods.set_default_config(null)
297+
}
305298

306-
if (current_configs.length == 0) {
307-
is_accepted = true
308-
quick_pick.hide()
309-
await show_configs_quick_pick()
310-
resolve()
311-
} else {
312-
quick_pick.items = create_config_items()
313-
quick_pick.show()
314-
}
299+
if (current_configs.length == 0) {
300+
is_accepted = true
301+
quick_pick.hide()
302+
await show_configs_quick_pick()
303+
resolve()
304+
} else {
305+
quick_pick.items = create_config_items()
306+
quick_pick.show()
315307
}
316308
} else if (
317309
event.button === move_up_button ||

0 commit comments

Comments
 (0)