allow act() to call hover()

[seanmcguire12]

why

• act() is currently unable to call hover()

what changed

• added hover() to SupportedPlaywrightAction enum & UnderstudyMethodHandlerContext interface

test plan

• this is change is pretty benign. existing act evals & unit tests should suffice

[changeset-bot]

🦋 Changeset detected

Latest commit: a6c84e2

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

This PR includes changesets to release 2 packages

Name	Type
@browserbasehq/stagehand	Patch
@browserbasehq/stagehand-evals	Patch

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 Overview

Greptile Summary

Added support for the hover() action in the act() method to enable mouse hover interactions on elements.

The changes integrate the existing Locator.hover() method (packages/core/lib/v3/understudy/locator.ts:315) into the act handler system by:

• Adding HOVER = "hover" to the SupportedPlaywrightAction enum
• Registering hover in METHOD_HANDLER_MAP
• Creating a new hover() handler function that wraps locator.hover() with proper error handling and logging

The implementation follows the established pattern used by other action handlers like selectOption() and doubleClick(), ensuring consistency with the codebase. The underlying Locator.hover() method already has test coverage in packages/core/lib/v3/tests/locator-input-methods.spec.ts.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The changes are minimal, well-structured, and follow existing patterns. The hover() method already exists in the Locator class with test coverage, and this PR simply wires it into the act() handler system. No new logic or edge cases are introduced - only integration code.
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
.changeset/breezy-phones-happen.md	5/5	Standard changeset file for documenting the patch-level change to allow act() to call hover()
packages/core/lib/v3/types/private/handlers.ts	5/5	Added HOVER enum value to SupportedPlaywrightAction, consistent with naming conventions
packages/core/lib/v3/handlers/handlerUtils/actHandlerUtils.ts	5/5	Added hover handler to METHOD_HANDLER_MAP and implemented hover() function following existing pattern with proper error handling and logging

Sequence Diagram

sequenceDiagram
    participant User
    participant ActHandler
    participant performUnderstudyMethod
    participant METHOD_HANDLER_MAP
    participant hover
    participant Locator

    User->>ActHandler: act("hover over element")
    ActHandler->>ActHandler: Build act prompt with SupportedPlaywrightAction.HOVER
    ActHandler->>ActHandler: Get LLM inference with hover method
    ActHandler->>performUnderstudyMethod: Call with method="hover"
    performUnderstudyMethod->>performUnderstudyMethod: Resolve locator for element
    performUnderstudyMethod->>METHOD_HANDLER_MAP: Lookup handler for "hover"
    METHOD_HANDLER_MAP->>hover: Execute hover handler
    hover->>Locator: Call locator.hover()
    Locator->>Locator: Scroll element into view
    Locator->>Locator: Get element box model
    Locator->>Locator: Dispatch mouse move to center
    Locator-->>hover: Success
    hover-->>performUnderstudyMethod: Complete
    performUnderstudyMethod-->>ActHandler: Action completed
    ActHandler-->>User: ActResult

Loading

_{2 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}