fix(browser_execute): use real helper names in tool description#26
Merged
Alezander9 merged 1 commit intomainfrom Apr 30, 2026
Merged
fix(browser_execute): use real helper names in tool description#26Alezander9 merged 1 commit intomainfrom
Alezander9 merged 1 commit intomainfrom
Conversation
The tool description listed `goto`, `click`, `screenshot` -- none of those
exist. The actual helpers (per harness/src/browser_harness/helpers.py)
are `goto_url`, `click_at_xy`, `capture_screenshot`. Agents followed the
description literally and got NameError on every call.
Also nudge agents toward `new_tab(url)` for first navigation per
SKILL.md, since bare `goto_url` clobbers the user's active tab.
Repro: fresh install, ask any model to "go to wikipedia". Pre-fix:
NameError: name 'goto' is not defined. Post-fix: agent uses
`new_tab("https://wikipedia.org")` directly.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The
browser_executetool description hands the LLM helper names that don't exist.But
harness/src/browser_harness/helpers.pyexportsgoto_url,click_at_xy,capture_screenshot-- notgoto/click/screenshot.Repro on a fresh v0.0.6 install: ask any model to "go to wikipedia". Both attempts fail with
NameError: name 'goto' is not defined. The agent followed the tool description literally.Fix
goto_url,click_at_xy,capture_screenshot).ensure_real_tabto the example list (it's the canonical pre-navigation helper from SKILL.md).new_tab("https://example.com")per SKILL.md ("First navigation is new_tab(url), not goto_url(url)").click(x, y)toclick_at_xy(x, y).Verification
^definhelpers.py.bun run typecheck: 5/5 passed.UPSTREAM.mddivergence needed.Modification zone
Yellow (touches
packages/opencode/src/tool/) but the file was added by browsercode (commit46048c709), not upstream -- so no merge-conflict risk and noEXCEPTIONS.mdentry.