Skip to content

Commit c2a7e0a

Browse files
committed
refactor: simplify wait_for description and remove redundant test cases
Updated the description of the wait_for tool to remove unnecessary details about input types. Removed tests that validated string and array inputs for the wait_for schema, as they are no longer applicable.
1 parent c3e57c5 commit c2a7e0a

2 files changed

Lines changed: 1 addition & 10 deletions

File tree

src/tools/snapshot.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ in the DevTools Elements panel (if any).`,
4343

4444
export const waitFor = defineTool({
4545
name: 'wait_for',
46-
description: `Wait for the specified text to appear on the selected page. You can provide a single text value or a list of texts; the tool resolves when any text appears.`,
46+
description: `Wait for the specified text to appear on the selected page.`,
4747
annotations: {
4848
category: ToolCategory.NAVIGATION,
4949
readOnlyHint: true,

tests/tools/snapshot.test.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,6 @@ describe('snapshot', () => {
2020
});
2121
});
2222
describe('browser_wait_for', () => {
23-
it('accepts string or non-empty array text input', () => {
24-
assert.equal(waitFor.schema.text.safeParse('Hello').success, true);
25-
assert.equal(
26-
waitFor.schema.text.safeParse(['Hello', 'World']).success,
27-
true,
28-
);
29-
assert.equal(waitFor.schema.text.safeParse([]).success, false);
30-
});
31-
3223
it('should work', async () => {
3324
await withMcpContext(async (response, context) => {
3425
const page = context.getSelectedPage();

0 commit comments

Comments
 (0)