Skip to content

docs: note computer controls screenshots are faster for webgl#341

Merged
masnwilliams merged 4 commits intomainfrom
hypeship/docs-screenshot-perf
May 1, 2026
Merged

docs: note computer controls screenshots are faster for webgl#341
masnwilliams merged 4 commits intomainfrom
hypeship/docs-screenshot-perf

Conversation

@masnwilliams
Copy link
Copy Markdown
Contributor

@masnwilliams masnwilliams commented Apr 23, 2026

Summary

  • Add a tip under Take screenshots in computer-controls.mdx noting that computer controls captureScreenshot is faster than CDP Page.captureScreenshot on WebGL and animation-heavy pages — relevant for customers driving computer-use loops off screenshots.
  • Add a note on gpu-acceleration.mdx clarifying that GPU speeds up page rendering, not CDP screenshot capture (a common misconception).

Kept the guidance qualitative rather than publishing specific ms numbers — single-page synthetic benches age badly and customers tend to cite absolute figures back.

Test plan

  • Preview renders <Tip> and <Note> callouts
  • Cross-links resolve between the two pages

Note

Low Risk
Low risk documentation-only change that adjusts performance guidance and cross-links between pages; no runtime or API behavior is modified.

Overview
Adds a <Tip> to computer-controls.mdx noting that computer controls captureScreenshot is faster than CDP Page.captureScreenshot on WebGL/animation-heavy pages, and that GPU acceleration provides only a small additional boost for computer-controls screenshots.

Adds a <Note> to gpu-acceleration.mdx clarifying that GPU acceleration improves rendering performance but does not reduce CDP screenshot latency, and links back to the computer controls screenshot guidance.

Reviewed by Cursor Bugbot for commit 9894264. Bugbot is set up for automated code reviews on this repo. Configure here.

Add guidance that computer controls captureScreenshot is faster than
CDP Page.captureScreenshot for animation-heavy and WebGL pages, and
clarify GPU acceleration's scope: it speeds up page rendering, not
CDP screenshot capture.
@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Apr 23, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview Apr 23, 2026, 10:16 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@masnwilliams masnwilliams requested a review from dprevoznik April 30, 2026 14:53
@masnwilliams masnwilliams marked this pull request as ready for review April 30, 2026 14:53
@firetiger-agent
Copy link
Copy Markdown

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

Any PR that changes the kernel API. Monitor changes to API endpoints (packages/api/cmd/api/) and Temporal workflows (packages/api/lib/temporal) in the kernel repo

Reason: This PR only modifies documentation files (computer-controls.mdx and gpu-acceleration.mdx) and does not change any kernel API endpoints or Temporal workflows.

To monitor this PR anyway, reply with @firetiger monitor this.

cursor[bot]
cursor Bot approved these changes Apr 30, 2026
cursor[bot]
cursor Bot approved these changes Apr 30, 2026
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Stale comment

Risk assessment: Very Low.

The current diff only adds short documentation callouts in browsers/computer-controls.mdx and browsers/gpu-acceleration.mdx. There are no codepath, config, infrastructure, schema, prompt, or navigation changes, and the blast radius is limited to rendered docs content. I found no CODEOWNERS file requiring separate owner review.

The PR already has an approval, so I’m not adding another approval. The update does not increase risk, so no approval dismissal is needed.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Risk assessment: Very Low.

The diff only adds short documentation callouts in browsers/computer-controls.mdx and browsers/gpu-acceleration.mdx about screenshot performance and GPU acceleration scope. There are no codepath, config, navigation, infrastructure, schema, prompt, or security-boundary changes, and the blast radius is limited to rendered docs content.

I found no CODEOWNERS file requiring separate owner review. The PR is already approved at the current head, so I’m not adding another approval. The update does not increase risk, so no approval dismissal is needed.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9894264. Configure here.


<Note>
GPU acceleration speeds up page rendering, not screenshot capture. CDP's `Page.captureScreenshot` latency is unaffected by GPU. [Computer controls screenshots](/browsers/computer-controls#take-screenshots) see a small speedup with GPU enabled.
</Note>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Documents Chromium CDP behavior as product spec

Low Severity

The statement "CDP's Page.captureScreenshot latency is unaffected by GPU" documents a Chromium internal behavior as a product-level fact. This violates the rule to keep feature descriptions at the product level rather than describing what Chromium does under the hood — this behavior could change across Chromium versions. The guidance in computer-controls.mdx similarly characterizes CDP screenshot internals. Both notes can convey the same user guidance (prefer computer controls for screenshot-heavy loops) without asserting specifics about CDP's internal capture path.

Additional Locations (1)
Fix in Cursor Fix in Web

Triggered by learned rule: Don't document Chromium internals as product specs

Reviewed by Cursor Bugbot for commit 9894264. Configure here.

@masnwilliams masnwilliams merged commit a299a79 into main May 1, 2026
4 checks passed
@masnwilliams masnwilliams deleted the hypeship/docs-screenshot-perf branch May 1, 2026 17:51
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.

1 participant