Fix tests

Author: Tyriar

src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/tools/runInTerminalTool.ts

@@ -899,8 +899,9 @@ export class RunInTerminalTool extends Disposable implements IToolImpl {
 			const subCommandsToSuggest = Array.from(new Set(unapprovedSubCommands.map(command => {
 				const parts = command.trim().split(/\s+/);
 				const baseCommand = parts[0].toLowerCase();
+				const baseSubCommand = parts.length > 1 ? `${parts[0]} ${parts[1]}`.toLowerCase() : '';
 
-				if (commandsWithSubSubCommands.has(baseCommand) && parts.length >= 3) {
+				if (commandsWithSubSubCommands.has(baseSubCommand) && parts.length >= 3) {
 					return `${parts[0]} ${parts[1]} ${parts[2]}`;
 				} else if (commandsWithSubcommands.has(baseCommand) && parts.length >= 2) {
 					return `${parts[0]} ${parts[1]}`;

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

@@ -296,12 +296,12 @@ suite('RunInTerminalTool', () => {
 
 
 			ok(!isSeparator(customActions[0]));
-			strictEqual(customActions[0].label, 'Always Allow Command: npm');
+			strictEqual(customActions[0].label, 'Always Allow Command: npm run build');
 			strictEqual(customActions[0].data.type, 'newRule');
 			ok(Array.isArray(customActions[0].data.rule), 'Expected rule to be an array');
 
 			ok(!isSeparator(customActions[1]));
-			strictEqual(customActions[1].label, 'Always Allow Full Command Line: npm run build');
+			strictEqual(customActions[1].label, 'Always Allow Exact Command Line: npm run build');
 			strictEqual(customActions[1].data.type, 'newRule');
 			ok(!Array.isArray(customActions[1].data.rule), 'Expected rule to be an object');
 
@@ -384,11 +384,11 @@ suite('RunInTerminalTool', () => {
 			strictEqual(customActions.length, 4);
 
 			ok(!isSeparator(customActions[0]));
-			strictEqual(customActions[0].label, 'Always Allow Command: npm install, npm run build');
+			strictEqual(customActions[0].label, 'Always Allow Commands: npm install, npm run build');
 			strictEqual(customActions[0].data.type, 'newRule');
 
 			ok(!isSeparator(customActions[1]));
-			strictEqual(customActions[1].label, 'Always Allow Full Command Line: npm install &&& npm run build');
+			strictEqual(customActions[1].label, 'Always Allow Exact Command Line: npm install &&& npm run build');
 			strictEqual(customActions[1].data.type, 'newRule');
 
 			ok(isSeparator(customActions[2]));
@@ -419,7 +419,7 @@ suite('RunInTerminalTool', () => {
 			strictEqual(customActions[0].data.type, 'newRule');
 
 			ok(!isSeparator(customActions[1]));
-			strictEqual(customActions[1].label, 'Always Allow Full Command Line: foo | head -20');
+			strictEqual(customActions[1].label, 'Always Allow Exact Command Line: foo | head -20');
 			strictEqual(customActions[1].data.type, 'newRule');
 
 			ok(isSeparator(customActions[2]));
@@ -465,7 +465,7 @@ suite('RunInTerminalTool', () => {
 			strictEqual(customActions[0].data.type, 'newRule');
 
 			ok(!isSeparator(customActions[1]));
-			strictEqual(customActions[1].label, 'Always Allow Full Command Line: foo | head -20 &&& bar | tail -10');
+			strictEqual(customActions[1].label, 'Always Allow Exact Command Line: foo | head -20 &&& bar | tail -10');
 			strictEqual(customActions[1].data.type, 'newRule');
 
 			ok(isSeparator(customActions[2]));