Skip to content

Conversation

@rcholic
Copy link
Contributor

@rcholic rcholic commented Jan 1, 2026

Screenshot Upload Analysis

Date: 2026-01-01
Question: Does the SDK upload screenshots and traces separately? Does it convert base64 to image bytes?

Current Implementation Analysis

1. Screenshot Inclusion in Trace Events

Location: sdk-python/sentience/agent.py (lines 821-830 for async, 181-190 for sync)

Current Behavior:

  • Screenshots are NOT included in trace events
  • When emitting snapshot events, only these fields are included:
    {
        "url": snap.url,
        "element_count": len(snap.elements),
        "timestamp": snap.timestamp,
        "elements": elements_data,
        # screenshot is NOT included!
    }
  • The snap.screenshot field exists in the Snapshot object but is not emitted to the tracer

@rcholic rcholic merged commit 22544d6 into main Jan 2, 2026
3 checks passed
@rcholic rcholic deleted the fix_screenshot2 branch January 2, 2026 01:17
rcholic pushed a commit that referenced this pull request Jan 6, 2026
upload screenshots along with traces
rcholic pushed a commit that referenced this pull request Jan 10, 2026
upload screenshots along with traces
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