Have playwright MCP always trace (microsoft#273145)

Have MCP always trace

so that traces can be looked at easily.

Author: TylerLeonhardt

test/mcp/src/application.ts

@@ -254,13 +254,13 @@ export async function getApplication({ recordVideo }: { recordVideo?: boolean }
 		codePath: opts.build,
 		workspacePath: rootPath,
 		logger,
-		logsPath: path.join(logsRootPath, 'suite_unknown'),
-		crashesPath: path.join(crashesRootPath, 'suite_unknown'),
+		logsPath: logsRootPath,
+		crashesPath: crashesRootPath,
 		videosPath: (recordVideo || opts.video) ? videoRootPath : undefined,
 		verbose: opts.verbose,
 		remote: opts.remote,
 		web: opts.web,
-		tracing: opts.tracing,
+		tracing: true,
 		headless: opts.headless,
 		browser: opts.browser,
 		extraArgs: (opts.electronArgs || '').split(' ').map(arg => arg.trim()).filter(arg => !!arg),

test/mcp/src/automationTools/core.ts

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