wrap page evaluate with timeout

[LawyZheng]

Important

Wraps page evaluations with a timeout using SkyvernFrame.evaluate() and removes deprecated JavaScript-triggered handling.

• Behavior:
  • Wraps page evaluations with a timeout using SkyvernFrame.evaluate() in agent.py, handler.py, scraper.py, and browser_factory.py.
  • Removes handling of javascript_triggered in agent.py and handler.py.
• Functions:
  • Adds SkyvernFrame.evaluate() to handle timeouts for page evaluations.
  • Removes is_javascript_triggered() and related logic.
• Misc:
  • Removes javascript_triggered attribute from ActionResult and related classes in responses.py.

^{This description was created by for ba3fc14. It will automatically update as commits are pushed.}

[ellipsis-dev]

❌ Changes requested. Reviewed everything up to ba3fc14 in 1 minute and 8 seconds

More details

• Looked at 492 lines of code in 7 files
• Skipped 0 files when reviewing.
• Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_fLJusmYOh5YYiJz2

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

asyncio.timeout is not a standard method. Use asyncio.wait_for for timeout functionality.

Suggested change

	async with asyncio.timeout(timeout_ms / 1000):
	async with asyncio.wait_for(frame.evaluate(expression=expression, arg=arg), timeout=timeout_ms / 1000):