Remove comments about user data directory handling

pavelfeldman · web-flow · commit d5ef3b29cfbe · 2026-05-26T11:57:32.000-07:00
Removed comments explaining the writable profile data requirement.

Signed-off-by: Pavel Feldman &lt;pavel.feldman@gmail.com&gt;
diff --git a/packages/playwright-core/src/tools/mcp/browserFactory.ts b/packages/playwright-core/src/tools/mcp/browserFactory.ts
@@ -182,9 +182,6 @@ async function createPersistentBrowser(config: FullConfig, clientInfo: ClientInf
 }
 
 async function createUserDataDir(config: FullConfig, clientInfo: ClientInfo) {
-  // Profile data must be writable, so we never derive it from PLAYWRIGHT_BROWSERS_PATH
-  // (which often points at a read-only browser binary cache on NixOS, immutable Docker
-  // mounts, or shared filesystems). Use the platform's user cache directory instead.
   const dir = process.env.PWMCP_PROFILES_DIR_FOR_TEST ?? path.join(defaultCacheDirectory, 'ms-playwright-mcp');
   const browserToken = config.browser.launchOptions?.channel ?? config.browser?.browserName;
   // Hesitant putting hundreds of files into the user's workspace, so using it for hashing instead.

Original file line number	Diff line number	Diff line change
`@@ -182,9 +182,6 @@ async function createPersistentBrowser(config: FullConfig, clientInfo: ClientInf`
`182`	`182`	`}`
`183`	`183`
`184`	`184`	`async function createUserDataDir(config: FullConfig, clientInfo: ClientInfo) {`
`185`		`- // Profile data must be writable, so we never derive it from PLAYWRIGHT_BROWSERS_PATH`
`186`		`- // (which often points at a read-only browser binary cache on NixOS, immutable Docker`
`187`		`- // mounts, or shared filesystems). Use the platform's user cache directory instead.`
`188`	`185`	`const dir = process.env.PWMCP_PROFILES_DIR_FOR_TEST ?? path.join(defaultCacheDirectory, 'ms-playwright-mcp');`
`189`	`186`	`const browserToken = config.browser.launchOptions?.channel ?? config.browser?.browserName;`
`190`	`187`	`// Hesitant putting hundreds of files into the user's workspace, so using it for hashing instead.`