fix variable parsing in act

[seanmcguire12]

why

• when a user wraps their variable name in percentage symbols, those percentage symbols are not currently parsed out, and are therefore included as part of the variable value
• eg, user does something like:

• await stagehand.page.act({
    action: "search for %search_string%",
    variables: {
      search_string: "stagehand",
    },
  });
  

• the resulting search will look like:

what changed

• we now parse out the percentage signs

test plan

• evals

[changeset-bot]

🦋 Changeset detected

Latest commit: 9db9ed1

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

This PR includes changesets to release 1 package

Name	Type
@browserbasehq/stagehand	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]

PR Summary

The PR refines the variable parsing in the act method to correctly strip percentage symbols from variables.

• Updated replacement logic in /lib/handlers/actHandler.ts to handle %variable% patterns precisely.
• Ensures that only exact matches are replaced, preventing unintended inclusion of percentage symbols.
• Aligns implementation with user expectations as documented in the PR body.
• Test plans via evals will verify multiple occurrences and overall behavior.

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}