Merge pull request microsoft#259224 from microsoft/tyriar/258996

Fix /.*/ auto approve case

Author: Tyriar

src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/commandLineAutoApprover.ts

@@ -185,6 +185,11 @@ export class CommandLineAutoApprover extends Disposable {
 				flags = flags.replaceAll('g', '');
 			}
 
+			// Allow .* as users expect this would match everything
+			if (regexPattern === '.*') {
+				return new RegExp(regexPattern);
+			}
+
 			try {
 				const regex = new RegExp(regexPattern, flags || undefined);
 				if (regExpLeadsToEndlessLoop(regex)) {

src/vs/workbench/contrib/terminalContrib/chatAgentTools/test/browser/commandLineAutoApprover.test.ts

@@ -148,6 +148,14 @@ suite('CommandLineAutoApprover', () => {
 	});
 
 	suite('regex patterns', () => {
+		test('should handle /.*/', () => {
+			setAutoApprove({
+				"/.*/": true,
+			});
+
+			ok(isAutoApproved('echo hello'));
+		});
+
 		test('should handle regex patterns in autoApprove', () => {
 			setAutoApprove({
 				"/^echo/": true,