Skip to content

Cover VisualWebArena agent step/parse path with a non-pricy test - #43

Open
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1789678134-vwa-agent-ci-test
Open

devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1789678134-vwa-agent-ci-test

Conversation

@devin-ai-integration

Copy link
Copy Markdown

Summary

The only test for VisualWebArenaAgent was @pytest.mark.pricy and called real gpt-4o-mini, so CI (which runs -m 'not pricy') never executed the agent's prompt assembly or action parsing. Adds a CI-runnable test mirroring tests/agents/test_agent.py: it runs the agent on miniwob.click-test through launch_exp.run_experiments with a cheat LLM, asserting one step and cum_reward == 1.0.

Two details specific to this agent, versus the generic-agent tests:

  • Its prompt is multi-modal, so the cheat LLM flattens message content instead of reading messages[1]["content"] as a string, and it only searches after the last OBSERVATION: marker because the few-shot [REDACTED SECRET]s also contain button rows.
  • The reply must match parser(), i.e. the action inside a fenced code block: ...In summary, the next action I will perform isclick("12") .
  • make_agent() needs action_set_args, which is only set by set_benchmark, so the test calls agent_args.set_benchmark(bgym.DEFAULT_BENCHMARKS["miniwob_tiny_test"](), demo_mode=False).

Parametrized over observation_type in ("axtree", "axtree_som") so the screenshot/set-of-marks branch of get_action is exercised too.

Verification: the MiniWoB server could not be started in this session (cloning Farama-Foundation/miniwob-plusplus is blocked by the session network policy), so the full loop was not run locally; the prompt-build + parse path was verified offline by driving get_action with a synthetic observation for both observation types. CI runs the test with MiniWoB available.

Devin-Org: engineering

Link to Devin session: https://app.devin.ai/sessions/91414c8499de4805b8b9170ce243343f
Open in Devin Desktop: https://app.devin.ai/desktop/session/91414c8499de4805b8b9170ce243343f?variant=devin

@devin-ai-integration

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants