Require the approve property

Tyriar · Tyriar · commit 63c64a66654d · 2025-08-01T06:56:55.000-07:00
Fixes microsoft#259212
diff --git a/src/vs/workbench/contrib/terminalContrib/chatAgentTools/common/terminalChatAgentToolsConfiguration.ts b/src/vs/workbench/contrib/terminalContrib/chatAgentTools/common/terminalChatAgentToolsConfiguration.ts
@@ -77,7 +77,8 @@ export const terminalChatAgentToolsConfiguration: IStringDictionary<IConfigurati
 							],
 							description: localize('autoApprove.matchCommandLine', "Whether to match against the full command line, as opposed to splitting by sub-commands and inline commands."),
 						}
-					}
+					},
+					required: ['approve']
 				},
 				{
 					type: 'null',

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,8 @@ export const terminalChatAgentToolsConfiguration: IStringDictionary<IConfigurati`
`77`	`77`	`],`
`78`	`78`	`description: localize('autoApprove.matchCommandLine', "Whether to match against the full command line, as opposed to splitting by sub-commands and inline commands."),`
`79`	`79`	`}`
`80`		`- }`
	`80`	`+ },`
	`81`	`+ required: ['approve']`
`81`	`82`	`},`
`82`	`83`	`{`
`83`	`84`	`type: 'null',`