proxy page.evaluate()

[seanmcguire12]

why

• sometimes our browser side scripts will get overwritten or removed
• this is problematic, and results in an error like: window._browser_side_function is not a function

what changed

• wrapped .evaluate() in our own .evaluate() function which checks to make sure our browser side functions still exist before attempting to call them. if they do not exist, they are re-injected

test plan

• added a test

[changeset-bot]

🦋 Changeset detected

Latest commit: 7cb8562

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

This PR implements script persistence by wrapping page.evaluate() to ensure browser-side functions remain available, with automatic re-injection if scripts are removed or overwritten.

• Added ensureStagehandScript() in lib/StagehandPage.ts to verify and re-inject scripts before evaluation
• Introduced __stagehandInjected flag in lib/dom/global.d.ts to track script injection state
• Modified page proxy to intercept evaluate-related methods (evaluate, evaluateHandle, $eval, $$eval) for script verification
• Added test case in addInitScript.test.ts to verify automatic re-injection when scripts are deleted
• Implemented guard clause in script injection to prevent duplicate injections

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