show "Select Tools" only for agent mode, swap position with "add context" (microsoft#243474)

Author: jrieken

src/vs/workbench/contrib/chat/browser/actions/chatContextActions.ts

@@ -515,7 +515,8 @@ export class AttachContextAction extends Action2 {
 			{
 				when: ChatContextKeyExprs.inNonUnifiedPanel,
 				id: MenuId.ChatInputAttachmentToolbar,
-				group: 'navigation'
+				group: 'navigation',
+				order: 2
 			}
 		]
 	}) {
@@ -1027,7 +1028,8 @@ registerAction2(class AttachFilesAction extends AttachContextAction {
 			menu: {
 				when: ChatContextKeyExprs.inEditsOrUnified,
 				id: MenuId.ChatInputAttachmentToolbar,
-				group: 'navigation'
+				group: 'navigation',
+				order: 3
 			},
 			icon: Codicon.attach,
 			precondition: ChatContextKeyExprs.inEditsOrUnified,

src/vs/workbench/contrib/mcp/browser/mcpCommands.ts

@@ -184,20 +184,19 @@ export class AttachMCPToolsAction extends Action2 {
 			category: CHAT_CATEGORY,
 			precondition: ContextKeyExpr.and(
 				McpContextKeys.toolsCount.greater(0),
-				ChatContextKeys.chatMode.notEqualsTo(ChatMode.Chat),
-				ChatContextKeys.Editing.hasToolsAgent
+				ChatContextKeys.chatMode.isEqualTo(ChatMode.Agent)
 			),
 			menu: {
 				when: ContextKeyExpr.and(
 					McpContextKeys.toolsCount.greater(0),
-					ChatContextKeys.chatMode.notEqualsTo(ChatMode.Chat),
-					ChatContextKeys.Editing.hasToolsAgent
+					ChatContextKeys.chatMode.isEqualTo(ChatMode.Agent)
 				),
 				id: MenuId.ChatInputAttachmentToolbar,
-				group: 'navigation'
+				group: 'navigation',
+				order: 1
 			},
 			keybinding: {
-				when: ContextKeyExpr.and(ChatContextKeys.inChatInput, ChatContextKeys.chatMode.notEqualsTo(ChatMode.Chat)),
+				when: ContextKeyExpr.and(ChatContextKeys.inChatInput, ChatContextKeys.chatMode.isEqualTo(ChatMode.Agent)),
 				primary: KeyMod.CtrlCmd | KeyMod.Shift | KeyCode.Slash,
 				weight: KeybindingWeight.EditorContrib
 			}