fix: increase timeouts for long text input

Author: OrKoN

src/tools/input.ts

@@ -159,7 +159,13 @@ async function fillFormElement(
     if (aXNode && aXNode.role === 'combobox') {
       await selectOption(handle, aXNode, value);
     } else {
-      await handle.asLocator().fill(value);
+      // Increase timeout for longer input values.
+      const textLengthIn1kChars = Math.floor(value.length / 1_000);
+      const timeoutPer1kChars = 5_000; // ms
+      const fillTimeout =
+        context.getSelectedPage().getDefaultTimeout() +
+        textLengthIn1kChars * timeoutPer1kChars;
+      await handle.asLocator().setTimeout(fillTimeout).fill(value);
     }
   } finally {
     void handle.dispose();
@@ -183,6 +189,7 @@ 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

@@ -301,6 +301,38 @@ describe('input', () => {
         assert.strictEqual(selectedValue, 'v2');
       });
     });
+
+    it('fills out a textarea with long text', async () => {
+      await withMcpContext(async (response, context) => {
+        const page = context.getSelectedPage();
+        await page.setContent(html`<textarea />`);
+        await page.focus('textarea');
+        await context.createTextSnapshot();
+        await page.setDefaultTimeout(1000);
+        await fill.handler(
+          {
+            params: {
+              uid: '1_1',
+              value: '1'.repeat(3000),
+            },
+          },
+          response,
+          context,
+        );
+        assert.strictEqual(
+          response.responseLines[0],
+          'Successfully filled out the element',
+        );
+        assert.ok(response.includeSnapshot);
+        assert.ok(
+          await page.evaluate(() => {
+            return (
+              document.body.querySelector('textarea')?.value.length === 3_000
+            );
+          }),
+        );
+      });
+    });
   });
 
   describe('drags', () => {