Disable tracing for now due to Playwright MCP behavior (microsoft#273171)

Haven't been able to figure out how to get tracing working without issues... so reverting the change.

Opened an issue here:
microsoft/playwright-mcp#1166

Author: TylerLeonhardt

test/mcp/src/application.ts

@@ -260,7 +260,7 @@ export async function getApplication({ recordVideo }: { recordVideo?: boolean }
 		verbose: opts.verbose,
 		remote: opts.remote,
 		web: opts.web,
-		tracing: true,
+		tracing: opts.tracing,
 		headless: opts.headless,
 		browser: opts.browser,
 		extraArgs: (opts.electronArgs || '').split(' ').map(arg => arg.trim()).filter(arg => !!arg),

test/mcp/src/automationTools/core.ts

@@ -37,7 +37,6 @@ export function applyCoreTools(server: McpServer, appService: ApplicationService
 		'Stop the VS Code application',
 		async () => {
 			const app = await appService.getOrCreateApplication();
-			await app.stopTracing('mcp', true);
 			await app.stop();
 			return {
 				content: [{

test/mcp/src/playwright.ts

@@ -12,8 +12,7 @@ export async function getServer(app?: Application): Promise<Server> {
 	const application = app ?? await getApplication();
 	const connection = await createConnection(
 		{
-			capabilities: ['core', 'pdf', 'vision'],
-			saveTrace: true
+			capabilities: ['core', 'pdf', 'vision']
 		},
 		// eslint-disable-next-line local/code-no-any-casts
 		() => Promise.resolve(application.code.driver.browserContext as any)