Skip to content

remove all images except the latest two in AnthropicCuaClient#159

Merged
tkattkat merged 4 commits intomainfrom
remove-all-images-except-the-latest-two-in-AnthropicCuaClient
Jul 23, 2025
Merged

remove all images except the latest two in AnthropicCuaClient#159
tkattkat merged 4 commits intomainfrom
remove-all-images-except-the-latest-two-in-AnthropicCuaClient

Conversation

@tkattkat
Copy link
Copy Markdown
Contributor

why
part of STG-586

Currently, we leave all of the images for anthropic cua client within the LLM's context as the task progresses

what changed
We now remove all screenshots aside from the last two when experimental flag is set to true within stagehand config

test plan
tested locally

@miguelg719
Copy link
Copy Markdown
Collaborator

closes #156

@tkattkat tkattkat marked this pull request as ready for review July 22, 2025 23:27
Copy link
Copy Markdown
Collaborator

@miguelg719 miguelg719 left a comment

Choose a reason for hiding this comment

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

Note: for the future, a set might increase performance for large item lists

@tkattkat tkattkat merged commit cd3dc7f into main Jul 23, 2025
12 checks passed
@github-actions github-actions bot mentioned this pull request Jul 22, 2025
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