fix(bridge): support fileChooserAccept in Bridge mode and WSL environments (#2693)

* fix(bridge): support fileChooserAccept in Bridge mode and WSL environments

- chrome-extension/page.ts: call ensureDebuggerAttached() before
  Page.setInterceptFileChooserDialog in registerFileChooserListener,
  fixing -32000 Not allowed error in Bridge mode

- agent.ts: convert WSL paths in normalizeFilePaths so Windows Chrome
  can read files via DOM.setFileInputFiles:
  /mnt/c/... → C:\...
  /home/... → \\wsl$\<distro>\...

* refactor(core): extract file path normalization

---------

Co-authored-by: yang.li12 <yang.li12@bluefocus.com>
Co-authored-by: ottomao <ottomao@gmail.com>

Author: 3 people

packages/core/src/agent/agent.ts

@@ -52,9 +52,8 @@ import {
   parseYamlScript,
 } from '../yaml/index';
 
-import { existsSync } from 'node:fs';
 import { readFile } from 'node:fs/promises';
-import { basename, resolve } from 'node:path';
+import { basename } from 'node:path';
 import type { AbstractInterface } from '@/device';
 import type { TaskRunner } from '@/task-runner';
 import {
@@ -89,6 +88,7 @@ import {
   commonContextParser,
   createScreenshotBoundUIContext,
   getReportFileName,
+  normalizeFilePaths,
   normalizeScrollType,
   parsePrompt,
 } from './utils';
@@ -1711,25 +1711,9 @@ export class Agent<
     return null;
   }
 
-  private normalizeFilePaths(files: string[]): string[] {
-    if (ifInBrowser) {
-      throw new Error('File chooser is not supported in browser environment');
-    }
-
-    return files.map((file) => {
-      const absolutePath = resolve(file);
-      if (!existsSync(absolutePath)) {
-        throw new Error(
-          `File not found: ${file}. Resolved to: ${absolutePath}. Current working directory: ${process.cwd()}`,
-        );
-      }
-      return absolutePath;
-    });
-  }
-
   private normalizeFileInput(files: string | string[]): string[] {
     const filesArray = Array.isArray(files) ? files : [files];
-    return this.normalizeFilePaths(filesArray);
+    return normalizeFilePaths(filesArray);
   }
 
   /**

packages/core/src/agent/utils.ts

@@ -1,3 +1,5 @@
+import { existsSync } from 'node:fs';
+import { resolve } from 'node:path';
 import { pixelBboxToRect } from '@/ai-model/workflows/inspect/locate-result-rect';
 import type { TMultimodalPrompt, TUserPrompt } from '@/common';
 import type { AbstractInterface } from '@/device';
@@ -27,7 +29,7 @@ import {
 } from '@midscene/shared/img';
 import { getDebug } from '@midscene/shared/logger';
 import { _keyDefinitions } from '@midscene/shared/us-keyboard-layout';
-import { assert, logMsg, uuid } from '@midscene/shared/utils';
+import { assert, ifInBrowser, logMsg, uuid } from '@midscene/shared/utils';
 import dayjs from 'dayjs';
 import type { TaskCache } from './task-cache';
 import { debug as cacheDebug } from './task-cache';
@@ -238,6 +240,51 @@ export function printReportMsg(filepath: string) {
   logMsg(`Midscene - report file updated: ${filepath}`);
 }
 
+type NormalizeFilePathsOptions = {
+  fileExists?: (path: string) => boolean;
+  isInBrowser?: boolean;
+  resolvePath?: (path: string) => string;
+  wslDistroName?: string;
+  cwd?: string;
+};
+
+export function normalizeFilePaths(
+  files: string[],
+  options: NormalizeFilePathsOptions = {},
+): string[] {
+  const {
+    fileExists = existsSync,
+    isInBrowser = ifInBrowser,
+    resolvePath = resolve,
+    wslDistroName = process.env.WSL_DISTRO_NAME,
+    cwd = process.cwd(),
+  } = options;
+
+  if (isInBrowser) {
+    throw new Error('File chooser is not supported in browser environment');
+  }
+
+  return files.map((file) => {
+    const absolutePath = resolvePath(file);
+    if (!fileExists(absolutePath)) {
+      throw new Error(
+        `File not found: ${file}. Resolved to: ${absolutePath}. Current working directory: ${cwd}`,
+      );
+    }
+
+    if (!wslDistroName) {
+      return absolutePath;
+    }
+
+    const wslMount = absolutePath.match(/^\/mnt\/([a-z])\//);
+    if (wslMount) {
+      return `${wslMount[1].toUpperCase()}:\\${absolutePath.slice(7).replace(/\//g, '\\')}`;
+    }
+
+    return `\\\\wsl$\\${wslDistroName}${absolutePath.replace(/\//g, '\\')}`;
+  });
+}
+
 export function isPixelBbox(value: unknown): value is PixelBbox {
   return (
     Array.isArray(value) &&

packages/core/tests/unit-test/file-chooser-accept-path.test.ts

@@ -1,6 +1,7 @@
 import { unlinkSync, writeFileSync } from 'node:fs';
 import { join, relative, resolve } from 'node:path';
 import { Agent } from '@/agent';
+import { normalizeFilePaths } from '@/agent/utils';
 import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 
 const createMockInterface = () =>
@@ -11,6 +12,11 @@ const createMockInterface = () =>
 
 const fixturesDir = join(__dirname, '../fixtures');
 const testFilePath = join(fixturesDir, 'path-test-file.txt');
+const hadWslDistroName = Object.prototype.hasOwnProperty.call(
+  process.env,
+  'WSL_DISTRO_NAME',
+);
+const originalWslDistroName = process.env.WSL_DISTRO_NAME;
 
 // resolve() resolves based on process.cwd(), compute fixture path relative to cwd
 const relativeFromCwd = relative(process.cwd(), testFilePath);
@@ -19,6 +25,7 @@ describe('fileChooserAccept relative path support', () => {
   let agent: Agent;
 
   beforeAll(() => {
+    Reflect.deleteProperty(process.env, 'WSL_DISTRO_NAME');
     writeFileSync(testFilePath, 'path test content');
     agent = new Agent(createMockInterface(), {
       modelConfig: {
@@ -32,6 +39,12 @@ describe('fileChooserAccept relative path support', () => {
     try {
       unlinkSync(testFilePath);
     } catch {}
+
+    if (hadWslDistroName) {
+      process.env.WSL_DISTRO_NAME = originalWslDistroName;
+    } else {
+      Reflect.deleteProperty(process.env, 'WSL_DISTRO_NAME');
+    }
   });
 
   it('should resolve relative path with ./ prefix', () => {
@@ -121,3 +134,53 @@ describe('fileChooserAccept relative path support', () => {
     expect(result.length).toBe(2);
   });
 });
+
+describe('normalizeFilePaths', () => {
+  it('throws in browser environments', () => {
+    expect(() =>
+      normalizeFilePaths(['file.txt'], { isInBrowser: true }),
+    ).toThrow('File chooser is not supported in browser environment');
+  });
+
+  it('resolves relative paths and validates existence', () => {
+    const result = normalizeFilePaths(['./upload.txt'], {
+      fileExists: (filePath) => filePath === '/project/upload.txt',
+      resolvePath: () => '/project/upload.txt',
+      wslDistroName: '',
+    });
+
+    expect(result).toEqual(['/project/upload.txt']);
+  });
+
+  it('throws with the original and resolved paths when a file does not exist', () => {
+    expect(() =>
+      normalizeFilePaths(['./missing.txt'], {
+        cwd: '/project',
+        fileExists: () => false,
+        resolvePath: () => '/project/missing.txt',
+      }),
+    ).toThrow(
+      'File not found: ./missing.txt. Resolved to: /project/missing.txt. Current working directory: /project',
+    );
+  });
+
+  it('converts WSL mounted Windows drive paths for Windows Chrome', () => {
+    const result = normalizeFilePaths(['/mnt/c/Users/me/upload.zip'], {
+      fileExists: () => true,
+      resolvePath: (filePath) => filePath,
+      wslDistroName: 'Ubuntu-22.04',
+    });
+
+    expect(result).toEqual(['C:\\Users\\me\\upload.zip']);
+  });
+
+  it('converts WSL distro-local paths to UNC paths for Windows Chrome', () => {
+    const result = normalizeFilePaths(['/home/me/upload.zip'], {
+      fileExists: () => true,
+      resolvePath: (filePath) => filePath,
+      wslDistroName: 'Ubuntu-22.04',
+    });
+
+    expect(result).toEqual(['\\\\wsl$\\Ubuntu-22.04\\home\\me\\upload.zip']);
+  });
+});

packages/web-integration/src/chrome-extension/page.ts

@@ -404,6 +404,7 @@ export default class ChromeExtensionProxyPage implements AbstractInterface {
     const registrationVersion = ++this.fileChooserRegistrationVersion;
     const tabId = await this.getTabIdOrConnectToCurrentTab();
 
+    await this.ensureDebuggerAttached();
     await this.sendCommandToDebugger('Page.enable', {});
     await this.sendCommandToDebugger('DOM.enable', {});
     await this.sendCommandToDebugger('Page.setInterceptFileChooserDialog', {