Refactor API provider and tool settings to enhance clarity and usability

Author: robertpiosik

…tings-command/configure-api-providers.ts …s/open-settings-command/api-providers.tspackages/vscode/src/commands/open-settings-command/configure-api-providers.ts renamed to packages/vscode/src/commands/open-settings-command/api-providers.ts packages/vscode/src/commands/open-settings-command/configure-api-providers.ts renamed to packages/vscode/src/commands/open-settings-command/api-providers.ts

@@ -11,7 +11,7 @@ const normalize_base_url = (url: string): string => {
   return url.trim().replace(/\/+$/, '')
 }
 
-export const configure_api_providers = async (
+export const api_providers = async (
   context: vscode.ExtensionContext
 ): Promise<void> => {
   const providers_manager = new ApiProvidersManager(context)
@@ -51,7 +51,7 @@ export const configure_api_providers = async (
         label: back_label
       },
       {
-        label: '$(add) Add another API provider...'
+        label: '$(add) Add API provider...'
       },
       {
         label: '',
@@ -106,7 +106,7 @@ export const configure_api_providers = async (
 
     const quick_pick = vscode.window.createQuickPick()
     quick_pick.items = await create_provider_items()
-    quick_pick.title = 'Configure API Providers'
+    quick_pick.title = 'API Providers'
     quick_pick.placeholder = 'Select an API provider to edit or add a new one'
 
     return new Promise<void>((resolve) => {
@@ -117,7 +117,7 @@ export const configure_api_providers = async (
         const selected = quick_pick.selectedItems[0]
         quick_pick.hide()
 
-        if (selected.label == '$(add) Add another API provider...') {
+        if (selected.label == '$(add) Add API provider...') {
           await show_create_provider_quick_pick()
         } else if ('provider' in selected) {
           await edit_provider(selected.provider as Provider)

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

@@ -1,13 +1,13 @@
 import * as vscode from 'vscode'
-import { configure_api_providers } from './configure-api-providers'
+import { api_providers } from './api-providers'
 import { setup_api_tool_multi_config } from './setup-api-tool-multi-config'
 import { setup_api_tool } from './setup-api-tool'
 
-const LABEL_PROVIDERS = '$(key) Configure API Providers'
-const LABEL_CODE_COMPLETIONS = 'Code Completions'
-const LABEL_EDIT_CONTEXT = 'Edit Context'
-const LABEL_INTELLIGENT_UPDATE = 'Intelligent Update'
-const LABEL_COMMIT_MESSAGES = 'Commit Messages'
+const LABEL_PROVIDERS = '$(key) API Providers'
+const LABEL_CODE_COMPLETIONS = '$(tools) Code Completions'
+const LABEL_EDIT_CONTEXT = '$(tools) Edit Context'
+const LABEL_INTELLIGENT_UPDATE = '$(tools) Intelligent Update'
+const LABEL_COMMIT_MESSAGES = '$(tools) Commit Messages'
 
 export const open_settings_command = (context: vscode.ExtensionContext) => {
   return vscode.commands.registerCommand('codeWebChat.settings', async () => {
@@ -25,24 +25,21 @@ export const open_settings_command = (context: vscode.ExtensionContext) => {
           },
           {
             label: LABEL_CODE_COMPLETIONS,
-            description: 'API tool',
             detail: 'Get code at cursor from state-of-the-art reasoning models.'
           },
           {
             label: LABEL_EDIT_CONTEXT,
-            description: 'API tool',
             detail:
               'Create and modify files based on natural language instructions.'
           },
           {
             label: LABEL_INTELLIGENT_UPDATE,
-            description: 'API tool',
             detail: 'Handle "truncated" edit format and fix malformed diffs.'
           },
           {
             label: LABEL_COMMIT_MESSAGES,
-            description: 'API tool',
-            detail: 'Generate meaningful commit messages adhering to your style.'
+            detail:
+              'Generate meaningful commit messages adhering to your style.'
           }
         ],
         {
@@ -58,7 +55,7 @@ export const open_settings_command = (context: vscode.ExtensionContext) => {
 
       switch (selected.label) {
         case LABEL_PROVIDERS:
-          await configure_api_providers(context)
+          await api_providers(context)
           break
         case LABEL_CODE_COMPLETIONS:
           await setup_api_tool_multi_config({

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

@@ -681,19 +681,24 @@ export const setup_api_tool_multi_config = async (params: {
       title: 'Set Temperature',
       prompt: 'Enter a value between 0 and 1',
       value: temperature.toString(),
-      placeHolder: '',
+      placeHolder: 'Leave empty to restore default',
       validateInput: (value) => {
+        if (value === '') return null // Allow empty to restore default
         const num = Number(value)
         if (isNaN(num)) return 'Please enter a valid number'
         if (num < 0 || num > 1) return 'Temperature must be between 0 and 1'
         return null
       }
     })
 
-    if (temperature_input === undefined || temperature_input == '') {
+    if (temperature_input === undefined) {
       return undefined
     }
 
+    if (temperature_input == '') {
+      return DEFAULT_TEMPERATURE[params.tool]
+    }
+
     return Number(temperature_input)
   }
 

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

@@ -288,16 +288,21 @@ export const setup_api_tool = async (params: {
       title: 'Set Temperature',
       prompt: 'Enter a value between 0 and 1',
       value: temperature.toString(),
-      placeHolder: '',
+      placeHolder: 'Leave empty to restore default',
       validateInput: (value) => {
+        if (value === '') return null // Allow empty to restore default
         const num = Number(value)
         if (isNaN(num)) return 'Please enter a valid number'
         if (num < 0 || num > 1) return 'Temperature must be between 0 and 1'
         return null
       }
     })
 
-    if (temperature_input === undefined || temperature_input == '') {
+    if (temperature_input === undefined) {
+      return undefined
+    }
+
+    if (temperature_input == '') {
       return DEFAULT_TEMPERATURE[params.tool]
     }