ChatRequest.tools should only include tools that are in the tool picker (#247491)

Otherwise it includes tools that aren't supposed to be enabled for agent mode

Author: roblourens

src/vs/workbench/api/common/extHostChatAgents2.ts

@@ -567,7 +567,9 @@ export class ExtHostChatAgents2 extends Disposable implements ExtHostChatAgentsS
 		}
 		const result = new Map<string, boolean>();
 		for (const tool of this._tools.getTools(extension)) {
-			result.set(tool.name, request.userSelectedTools2[tool.name] ?? false);
+			if (typeof request.userSelectedTools2[tool.name] === 'boolean') {
+				result.set(tool.name, request.userSelectedTools2[tool.name]);
+			}
 		}
 		return result;
 	}

src/vs/workbench/contrib/chat/browser/chatSelectedTools.ts

@@ -142,7 +142,9 @@ export class ChatSelectedTools extends Disposable {
 		const result = new Map<IToolData, boolean>();
 		const enabledTools = new Set(this.tools.get().map(t => t.id));
 		for (const tool of this._allTools.get()) {
-			result.set(tool, enabledTools.has(tool.id));
+			if (tool.supportsToolPicker) {
+				result.set(tool, enabledTools.has(tool.id));
+			}
 		}
 		return result;
 	}