prune screenshots from agent cache entry #1400

seanmcguire12 · 2025-12-10T08:05:53Z

why

writing base64 screenshots to disk is unnecessary: screenshots do not get replayed, so there is no sense in writing it to disk

what changed

added a pruneAgentResult() fn which prunes the screenshot entry before it is written to disk

test plan

existing tests & evals should suffice for this one

Summary by cubic

Stop writing base64 screenshots to the agent cache to reduce disk usage and keep cache entries lean. Screenshots aren’t replayed, so pruning them has no impact on behavior.

Refactors
- Added pruneAgentResult to remove screenshot base64 blobs from actions before persisting.
- Prunes only the cached copy; the live AgentResult returned to callers is unchanged.

^{Written for commit 625f982. Summary will update automatically on new commits.}

changeset-bot · 2025-12-10T08:05:58Z

🦋 Changeset detected

Latest commit: 625f982

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@browserbasehq/stagehand	Patch
@browserbasehq/stagehand-evals	Patch
@browserbasehq/stagehand-server	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

cubic-dev-ai

No issues found across 2 files

greptile-apps · 2025-12-10T08:08:26Z

Greptile Overview

Greptile Summary

This PR optimizes agent cache storage by removing base64-encoded screenshot data before writing cache entries to disk.

Added pruneAgentResult() method to AgentCache that clones the result (via cloneForCache) and then removes the base64 property from any screenshot-type actions
Screenshot data is stripped before caching because it's not needed for replay - the cache replay system already skips screenshot actions (see line 559 in the executeAgentReplayStep switch statement)
The implementation safely clones first to avoid mutating the live AgentResult returned to callers
This reduces cache file sizes significantly as base64 screenshots can be quite large

Confidence Score: 5/5

This PR is safe to merge - it's a straightforward optimization with proper defensive coding patterns
The change is well-scoped, uses defensive cloning to avoid mutations, and correctly targets screenshot actions. The logic aligns with the existing cache replay behavior which already skips screenshot actions.
No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
packages/core/lib/v3/cache/AgentCache.ts	5/5	Added `pruneAgentResult()` method to remove base64 screenshot data from agent actions before caching to disk. Correctly clones the result first to avoid mutating the live object, then iterates through actions to delete the `base64` field from screenshot-type actions.
.changeset/cool-otters-follow.md	5/5	Changeset file documenting patch release for not writing base64 encoded screenshots to disk when caching agent actions.

Sequence Diagram

sequenceDiagram
    participant Agent
    participant AgentCache
    participant CacheStorage

    Agent->>AgentCache: store(context, steps, result)
    AgentCache->>AgentCache: pruneAgentResult(result)
    Note over AgentCache: Clone result via cloneForCache()
    Note over AgentCache: Delete base64 from screenshot actions
    AgentCache->>CacheStorage: writeJson(entry)
    Note over CacheStorage: Cache file now excludes<br/>large base64 screenshot data

greptile-apps

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

@miguelg719

This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## @browserbasehq/[email protected] ### Patch Changes - [#1388](#1388) [`605ed6b`](605ed6b) Thanks [@miguelg719](https://github.com/miguelg719)! - Fix multiple click event dispatches on CDP and Anthropic CUA handling (double clicks) - [#1400](#1400) [`34e7e5b`](34e7e5b) Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - don't write base64 encoded screenshots to disk when caching agent actions - [#1345](#1345) [`943d2d7`](943d2d7) Thanks [@tkattkat](https://github.com/tkattkat)! - Add support for aborting / stopping an agent run & continuing an agent run using messages from prior runs - [#1334](#1334) [`0e95cd2`](0e95cd2) Thanks [@tkattkat](https://github.com/tkattkat)! - Add support for google vertex provider - [#1410](#1410) [`d4237e4`](d4237e4) Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - fix: include extract in stagehand.history() - [#1315](#1315) [`86975e7`](86975e7) Thanks [@tkattkat](https://github.com/tkattkat)! - Add streaming support to agent through stream:true in the agent config - [#1304](#1304) [`d5e119b`](d5e119b) Thanks [@miguelg719](https://github.com/miguelg719)! - Add support for Microsoft's Fara-7B - [#1346](#1346) [`4e051b2`](4e051b2) Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - fix: don't attach to targets twice - [#1327](#1327) [`6b5a3c9`](6b5a3c9) Thanks [@miguelg719](https://github.com/miguelg719)! - Informed error parsing from api - [#1335](#1335) [`bb85ad9`](bb85ad9) Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - add support for page.addInitScript() - [#1331](#1331) [`88d28cc`](88d28cc) Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - fix: page.evaluate() now works with scripts injected via context.addInitScript() - [#1316](#1316) [`45bcef0`](45bcef0) Thanks [@tkattkat](https://github.com/tkattkat)! - Add support for callbacks in stagehand agent - [#1374](#1374) [`6aa9d45`](6aa9d45) Thanks [@miguelg719](https://github.com/miguelg719)! - Fix key action mapping in Anthropic CUA - [#1330](#1330) [`d382084`](d382084) Thanks [@seanmcguire12](https://github.com/seanmcguire12)! - fix: make act, extract, and observe respect user defined timeout param - [#1336](#1336) [`1df08cc`](1df08cc) Thanks [@tkattkat](https://github.com/tkattkat)! - Patch agent on api - [#1358](#1358) [`2b56600`](2b56600) Thanks [@tkattkat](https://github.com/tkattkat)! - Add support for 4.5 opus in cua agent ## @browserbasehq/[email protected] ### Patch Changes - [#1364](#1364) [`ca0630e`](ca0630e) Thanks [@tkattkat](https://github.com/tkattkat)! - Update model handling in agent evals cli - Updated dependencies \[[`605ed6b`](605ed6b), [`34e7e5b`](34e7e5b), [`943d2d7`](943d2d7), [`0e95cd2`](0e95cd2), [`d4237e4`](d4237e4), [`86975e7`](86975e7), [`d5e119b`](d5e119b), [`4e051b2`](4e051b2), [`6b5a3c9`](6b5a3c9), [`bb85ad9`](bb85ad9), [`88d28cc`](88d28cc), [`45bcef0`](45bcef0), [`6aa9d45`](6aa9d45), [`d382084`](d382084), [`1df08cc`](1df08cc), [`2b56600`](2b56600)]: - @browserbasehq/[email protected] ## @browserbasehq/[email protected] ### Patch Changes - Updated dependencies \[[`605ed6b`](605ed6b), [`34e7e5b`](34e7e5b), [`943d2d7`](943d2d7), [`0e95cd2`](0e95cd2), [`d4237e4`](d4237e4), [`86975e7`](86975e7), [`d5e119b`](d5e119b), [`4e051b2`](4e051b2), [`6b5a3c9`](6b5a3c9), [`bb85ad9`](bb85ad9), [`88d28cc`](88d28cc), [`45bcef0`](45bcef0), [`6aa9d45`](6aa9d45), [`d382084`](d382084), [`1df08cc`](1df08cc), [`2b56600`](2b56600)]: - @browserbasehq/[email protected] Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

@miguelg719

prune screenshots from agent cache entry

625f982

cubic-dev-ai bot reviewed Dec 10, 2025

View reviewed changes

greptile-apps bot reviewed Dec 10, 2025

View reviewed changes

tkattkat approved these changes Dec 10, 2025

View reviewed changes

seanmcguire12 merged commit 34e7e5b into main Dec 10, 2025
31 of 32 checks passed

This was referenced Dec 10, 2025

Version Packages #1305

Merged

Version Packages CloudEngineHub/stagehand#1

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

prune screenshots from agent cache entry #1400

prune screenshots from agent cache entry #1400

Uh oh!

seanmcguire12 commented Dec 10, 2025 •

edited by cubic-dev-ai bot

Loading

Uh oh!

changeset-bot bot commented Dec 10, 2025

Uh oh!

cubic-dev-ai bot left a comment

Uh oh!

greptile-apps bot commented Dec 10, 2025

Uh oh!

greptile-apps bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

prune screenshots from agent cache entry #1400

prune screenshots from agent cache entry #1400

Uh oh!

Conversation

seanmcguire12 commented Dec 10, 2025 • edited by cubic-dev-ai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

why

what changed

test plan

Summary by cubic

Uh oh!

changeset-bot bot commented Dec 10, 2025

🦋 Changeset detected

Uh oh!

cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot commented Dec 10, 2025

Greptile Overview

Greptile Summary

Confidence Score: 5/5

Important Files Changed

Sequence Diagram

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

seanmcguire12 commented Dec 10, 2025 •

edited by cubic-dev-ai bot

Loading