fix: accidental extra typing in the fill tool

Author: OrKoN

src/tools/input.ts

@@ -220,7 +220,6 @@ export const fill = defineTool({
   },
   handler: async (request, response, context) => {
     await context.waitForEventsAfterAction(async () => {
-      await context.getSelectedPage().keyboard.type(request.params.value);
       await fillFormElement(
         request.params.uid,
         request.params.value,

tests/tools/input.test.ts

@@ -22,6 +22,7 @@ import {
 import {parseKey} from '../../src/utils/keyboard.js';
 import {serverHooks} from '../server.js';
 import {html, withMcpContext} from '../utils.js';
+import { McpResponse } from '../../src/McpResponse.js';
 
 describe('input', () => {
   const server = serverHooks();
@@ -382,6 +383,64 @@ describe('input', () => {
         );
       });
     });
+
+    it('reproduction: fill isolation', async () => {
+      await withMcpContext(async (_response, context) => {
+        const page = context.getSelectedPage();
+        await page.setContent(
+          html`<form>
+            <input id="email" value="user@test.com" />
+            <input id="password" type="password" />
+          </form>`,
+        );
+        await context.createTextSnapshot();
+
+        // Fill email
+        const response1 = new McpResponse();
+        await fill.handler(
+          {
+            params: {
+              uid: '1_1', // email input
+              value: 'new@test.com',
+            },
+          },
+          response1,
+          context,
+        );
+        assert.strictEqual(
+          response1.responseLines[0],
+          'Successfully filled out the element',
+        );
+
+        // Fill password
+        const response2 = new McpResponse();
+        await fill.handler(
+          {
+            params: {
+              uid: '1_2', // password input
+              value: 'secret',
+            },
+          },
+          response2,
+          context,
+        );
+        assert.strictEqual(
+          response2.responseLines[0],
+          'Successfully filled out the element',
+        );
+
+        // Verify values
+        const values = await page.evaluate(() => {
+          return {
+            email: (document.getElementById('email') as HTMLInputElement).value,
+            password: (document.getElementById('password') as HTMLInputElement).value,
+          };
+        });
+
+        assert.strictEqual(values.email, 'new@test.com', 'Email should be updated correctly');
+        assert.strictEqual(values.password, 'secret', 'Password should be updated correctly');
+      });
+    });
   });
 
   describe('drags', () => {