Skip to content

Conversation

miguelg719
Copy link
Collaborator

why

what changed

test plan

Copy link

linear bot commented Jun 13, 2025

STG-492 Consolidate agent between envs

  • working on bb
  • parametrize api keys
  • agent execute max_steps vs constuctor?

@miguelg719 miguelg719 marked this pull request as ready for review June 13, 2025 06:52
@miguelg719 miguelg719 marked this pull request as draft June 13, 2025 06:52
@miguelg719 miguelg719 marked this pull request as ready for review June 13, 2025 17:31

from .agent import Agent

# from .agent_bb import Agent
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whats this comment for?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to remove good catch

auto_screenshot (Optional[bool]): Whether to automatically capture screenshots after each action. False will let the agent choose when to capture screenshots. Defaults to False.
auto_screenshot (Optional[bool]): Whether to automatically capture screenshots after each action. False will let the agent choose when to capture screenshots. Defaults to True.
wait_between_actions (Optional[int]): Milliseconds to wait between actions.
context (Optional[str]): Additional context for the agent.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we have an example or more description ? by context you mean prompt? ppl could mistake this with browser context api etc...

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just appended the schemas extra params for backwards compatibility, I'd love to add comments to the exposed functions kinda like jsdoc

@miguelg719 miguelg719 merged commit 7068412 into main Jun 13, 2025
6 of 10 checks passed
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.

2 participants