remove auto approve commands from command window

Author: navedmerchant

webview-ui/src/components/chat/CommandExecution.tsx

@@ -42,6 +42,7 @@ interface CommandExecutionProps {
 export const CommandExecution = ({ executionId, text, icon, title, isDenied = false }: CommandExecutionProps) => {
 	const {
 		terminalShellIntegrationDisabled = false,
+		destructiveCommandGuardEnabled = false,
 		allowedCommands = [],
 		deniedCommands = [],
 		setAllowedCommands,
@@ -246,7 +247,7 @@ export const CommandExecution = ({ executionId, text, icon, title, isDenied = fa
 					<CodeBlock source={command} language="shell" />
 					<OutputContainer isExpanded={isExpanded} output={output} />
 				</div>
-				{command && command.trim() && !isDenied && (
+				{command && command.trim() && !destructiveCommandGuardEnabled && !isDenied && (
 					<CommandPatternSelector
 						patterns={commandPatterns}
 						allowedCommands={allowedCommands}

webview-ui/src/components/chat/__tests__/CommandExecution.spec.tsx

@@ -44,6 +44,7 @@ vi.mock("../CommandPatternSelector", () => ({
 // Mock ExtensionStateContext
 const mockExtensionState = {
 	terminalShellIntegrationDisabled: false,
+	destructiveCommandGuardEnabled: false,
 	allowedCommands: ["npm"],
 	deniedCommands: ["rm"],
 	setAllowedCommands: vi.fn(),
@@ -110,6 +111,22 @@ describe("CommandExecution", () => {
 		expect(selector).toHaveTextContent("npm install express")
 	})
 
+	it("should hide the command pattern selector while destructive command guard is enabled", () => {
+		const state = {
+			...mockExtensionState,
+			destructiveCommandGuardEnabled: true,
+		}
+
+		render(
+			<ExtensionStateContext.Provider value={state as any}>
+				<CommandExecution executionId="test-1" text="npm install express" />
+			</ExtensionStateContext.Provider>,
+		)
+
+		expect(screen.getByTestId("code-block")).toHaveTextContent("npm install express")
+		expect(screen.queryByTestId("command-pattern-selector")).not.toBeInTheDocument()
+	})
+
 	it("should hide the command pattern selector for a denied command", () => {
 		render(
 			<ExtensionStateWrapper>