add timeout for js click

[seanmcguire12]

why

• we have a timeout for playwright click events, we should also have one for the JS click event attempt

what changed

• added a 3.5 second timeout for the JS click event attempt in actHandlerUtils.ts

test plan

• act evals
• combination evals

[changeset-bot]

🦋 Changeset detected

Latest commit: 691b451

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[greptile-apps]

Greptile Summary

This PR adds a crucial timeout of 3,500ms to JavaScript click events in actHandlerUtils.ts. Previously, while Playwright click events had a timeout, the JavaScript fallback click mechanism did not, which could potentially cause indefinite hangs. This change brings consistency between both click mechanisms and improves the overall reliability of the click operations.

The change is particularly relevant as it affects the fallback behavior when Playwright clicks fail, ensuring that the system won't hang indefinitely in such scenarios. The timeout duration (3.5 seconds) matches the existing Playwright click timeout, maintaining consistency across both approaches.

Confidence score: 5/5

1. This PR is extremely safe to merge as it adds a protective measure without changing core functionality
2. The change is simple, well-isolated, and includes a sensible timeout value that matches existing behavior
3. Files needing attention: None - the change is straightforward and well-tested

_{1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile}