From ebe0bd76439e8f6112aee9f2e60a135104501c8f Mon Sep 17 00:00:00 2001 From: Daniel Imms <2193314+Tyriar@users.noreply.github.com> Date: Fri, 2 Jan 2026 02:47:46 -0800 Subject: [PATCH] Explain flag detection in comment --- .../chatAgentTools/browser/runInTerminalHelpers.ts | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/runInTerminalHelpers.ts b/src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/runInTerminalHelpers.ts index d5f9d0d76389..983f3472780c 100644 --- a/src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/runInTerminalHelpers.ts +++ b/src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/runInTerminalHelpers.ts @@ -118,6 +118,10 @@ export function generateAutoApproveActions(commandLine: string, subCommands: str // For each unapproved sub-command (within the overall command line), decide whether to // suggest new rules for the command, a sub-command, a sub-command of a sub-command or to // not suggest at all. + // + // This includes support for detecting flags between the commands, so `mvn -DskipIT test a` + // would suggest `mvn -DskipIT test` as that's more useful than only suggesting the exact + // command line. const subCommandsToSuggest = Array.from(new Set(coalesce(unapprovedSubCommands.map(command => { const parts = command.trim().split(/\s+/); const baseCommand = parts[0].toLowerCase();