refactor: simplify command allow/deny implementation

- Remove redundant allowCommand/denyCommand message types from WebviewMessage
- Remove corresponding handlers from webviewMessageHandler
- Remove unused test file for allowCommand functionality
- Remove unused i18n keys for command_allowed and command_denied
- Simplify to use existing allowedCommands/deniedCommands infrastructure

Author: hannesrudolph

src/core/webview/__tests__/webviewMessageHandler.spec.ts

@@ -771,29 +771,6 @@ export const webviewMessageHandler = async (
 
 			break
 		}
-		case "allowCommand": {
-			// Add a command pattern to the allowed commands list
-			if (message.pattern && typeof message.pattern === "string") {
-				const currentCommands = getGlobalState("allowedCommands") ?? []
-				const validCommands = Array.isArray(currentCommands)
-					? currentCommands.filter((cmd) => typeof cmd === "string" && cmd.trim().length > 0)
-					: []
-
-				// Add the new pattern if it's not already in the list
-				if (!validCommands.includes(message.pattern)) {
-					validCommands.push(message.pattern)
-
-					await updateGlobalState("allowedCommands", validCommands)
-
-					// Show confirmation to the user
-					vscode.window.showInformationMessage(t("common:info.command_allowed", { pattern: message.pattern }))
-
-					// Update the webview state
-					await provider.postStateToWebview()
-				}
-			}
-			break
-		}
 		case "deniedCommands": {
 			// Validate and sanitize the commands array
 			const commands = message.commands ?? []
@@ -805,29 +782,6 @@ export const webviewMessageHandler = async (
 
 			break
 		}
-		case "denyCommand": {
-			// Add a command pattern to the denied commands list
-			if (message.pattern && typeof message.pattern === "string") {
-				const currentCommands = getGlobalState("deniedCommands") ?? []
-				const validCommands = Array.isArray(currentCommands)
-					? currentCommands.filter((cmd) => typeof cmd === "string" && cmd.trim().length > 0)
-					: []
-
-				// Add the new pattern if it's not already in the list
-				if (!validCommands.includes(message.pattern)) {
-					validCommands.push(message.pattern)
-
-					await updateGlobalState("deniedCommands", validCommands)
-
-					// Show confirmation to the user
-					vscode.window.showInformationMessage(t("common:info.command_denied", { pattern: message.pattern }))
-
-					// Update the webview state
-					await provider.postStateToWebview()
-				}
-			}
-			break
-		}
 		case "openCustomModesSettings": {
 			const customModesFilePath = await provider.customModesManager.getCustomModesFilePath()
 

src/core/webview/webviewMessageHandler.ts

@@ -102,9 +102,7 @@
 		"image_copied_to_clipboard": "Image data URI copied to clipboard",
 		"image_saved": "Image saved to {{path}}",
 		"mode_exported": "Mode '{{mode}}' exported successfully",
-		"mode_imported": "Mode imported successfully",
-		"command_allowed": "Command pattern '{{pattern}}' has been added to the allowed commands list",
-		"command_denied": "Command pattern '{{pattern}}' has been added to the denied commands list"
+		"mode_imported": "Mode imported successfully"
 	},
 	"answers": {
 		"yes": "Yes",

src/i18n/locales/en/common.json

@@ -36,9 +36,7 @@ export interface WebviewMessage {
 		| "getListApiConfiguration"
 		| "customInstructions"
 		| "allowedCommands"
-		| "allowCommand"
 		| "deniedCommands"
-		| "denyCommand"
 		| "alwaysAllowReadOnly"
 		| "alwaysAllowReadOnlyOutsideWorkspace"
 		| "alwaysAllowWrite"
@@ -237,7 +235,6 @@ export interface WebviewMessage {
 	visibility?: ShareVisibility // For share visibility
 	hasContent?: boolean // For checkRulesDirectoryResult
 	checkOnly?: boolean // For deleteCustomMode check
-	pattern?: string // For allowCommand
 	codeIndexSettings?: {
 		// Global state settings
 		codebaseIndexEnabled: boolean