fix: forward web provider through request router

Author: thymikee

src/daemon/request-router.ts

@@ -14,6 +14,7 @@ import {
   type LinuxToolProviderResolver,
   type RequestPlatformProviderScope,
   type RecordingProviderResolver,
+  type WebProviderResolver,
   withRequestPlatformProviderScope,
 } from './request-platform-providers.ts';
 import {
@@ -46,6 +47,7 @@ export type RequestRouterDeps = {
   appleRunnerProvider?: AppleRunnerProviderResolver;
   appleToolProvider?: AppleToolProviderResolver;
   linuxToolProvider?: LinuxToolProviderResolver;
+  webProvider?: WebProviderResolver;
   appLogProvider?: AppLogProviderResolver;
   recordingProvider?: RecordingProviderResolver;
   deviceInventoryProvider?: DeviceInventoryProvider;
@@ -64,6 +66,7 @@ export function createRequestHandler(deps: RequestRouterDeps): DaemonInvokeFn {
     appleRunnerProvider,
     appleToolProvider,
     linuxToolProvider,
+    webProvider,
     appLogProvider,
     recordingProvider,
     deviceInventoryProvider,
@@ -132,6 +135,7 @@ export function createRequestHandler(deps: RequestRouterDeps): DaemonInvokeFn {
                 appleRunnerProvider,
                 appleToolProvider,
                 linuxToolProvider,
+                webProvider,
                 appLogProvider,
                 recordingProvider,
               },

test/integration/provider-scenarios/web-provider.test.ts

@@ -0,0 +1,114 @@
+import assert from 'node:assert/strict';
+import { test } from 'vitest';
+import { WEB_DESKTOP_DEVICE } from '../../../src/__tests__/test-utils/index.ts';
+import type { WebProvider } from '../../../src/platforms/web/provider.ts';
+import { createProviderScenarioHarness } from './harness.ts';
+
+test('web provider is scoped through the request router and dispatch path', async () => {
+  const calls: string[] = [];
+  const webProvider: WebProvider = {
+    async open(target) {
+      calls.push(`open:${target}`);
+    },
+    async close(target) {
+      calls.push(`close:${target ?? ''}`);
+    },
+    async snapshot(options) {
+      calls.push(`snapshot:${options?.scope ?? ''}`);
+      return {
+        nodes: [
+          {
+            index: 0,
+            type: 'section',
+            role: 'main',
+            label: 'main',
+            rect: { x: 0, y: 0, width: 320, height: 240 },
+            depth: 0,
+          },
+          {
+            index: 1,
+            type: 'button',
+            role: 'button',
+            label: 'Launch',
+            rect: { x: 10, y: 20, width: 80, height: 32 },
+            hittable: true,
+            depth: 1,
+            parentIndex: 0,
+          },
+        ],
+      };
+    },
+    async screenshot() {
+      calls.push('screenshot');
+    },
+    async click(x, y) {
+      calls.push(`click:${x}:${y}`);
+    },
+    async fill(x, y, text) {
+      calls.push(`fill:${x}:${y}:${text}`);
+    },
+    async typeText(text) {
+      calls.push(`type:${text}`);
+    },
+    async scroll(direction) {
+      calls.push(`scroll:${direction}`);
+    },
+  };
+
+  const harness = await createProviderScenarioHarness({
+    deviceInventoryProvider: async () => [WEB_DESKTOP_DEVICE],
+    webProvider: ({ device, session }) => {
+      calls.push(`scope:${session?.name ?? 'none'}:${device.id}`);
+      return webProvider;
+    },
+  });
+
+  try {
+    const open = await harness.callCommand(
+      'open',
+      ['https://example.test'],
+      { platform: 'web' },
+      { meta: { requestId: 'req-web-open' } },
+    );
+    assert.equal(open.json.error, undefined);
+
+    const snapshot = await harness.callCommand(
+      'snapshot',
+      [],
+      { platform: 'web', snapshotScope: 'main' },
+      { meta: { requestId: 'req-web-snapshot' } },
+    );
+
+    assert.deepEqual(snapshot.json.result.data.nodes, [
+      {
+        index: 0,
+        type: 'section',
+        role: 'main',
+        label: 'main',
+        rect: { x: 0, y: 0, width: 320, height: 240 },
+        depth: 0,
+        parentIndex: undefined,
+        ref: 'e1',
+      },
+      {
+        index: 1,
+        type: 'button',
+        role: 'button',
+        label: 'Launch',
+        rect: { x: 10, y: 20, width: 80, height: 32 },
+        hittable: true,
+        depth: 1,
+        parentIndex: 0,
+        ref: 'e2',
+      },
+    ]);
+    assert.deepEqual(calls, [
+      'scope:none:agent-browser-chrome',
+      'open:https://example.test',
+      'scope:default:agent-browser-chrome',
+      'snapshot:main',
+    ]);
+  } finally {
+    await harness.close();
+  }
+});