Expand control.mdx with computer use + playwright execution patterns#375
Conversation
|
Firetiger deploy monitoring skipped This PR didn't match the auto-monitor filter configured on your GitHub connection:
Reason: PR only adds documentation (patterns.mdx guide) with no changes to API endpoints or Temporal workflows in packages/api/. To monitor this PR anyway, reply with |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e9444a4. Configure here.
e9444a4 to
3115bd8
Compare
There was a problem hiding this comment.
Stale comment
Risk assessment: Very Low
I reviewed the synchronized diff directly. The PR adds six lines of documentation guidance to
introduction/control.mdxunder an existing section. There are no runtime code changes, docs config changes, infrastructure changes, auth/permission changes, schema changes, or shared production logic changes.git diff --checkpassed for the requested diff.No
CODEOWNERSfile is present in the repo. The PR already has an approval from the earlier assessment, and this update does not increase risk, so I’m leaving that approval in place without re-approving.Sent by Cursor Automation: Assign PR reviewers
There was a problem hiding this comment.
Stale comment
Risk assessment: Very Low
I assessed the actual diff for
cf6c7bbe2762ba139ecfff086319f2fe061e0c7b...8ba5164cc60cde0f2e3f87e8dc7e0582bc4f49cd. The PR adds six lines of documentation guidance tointroduction/control.mdxand does not modify runtime code, docs configuration, infrastructure, auth/permissions, schemas, or shared production logic.Validation:
git diff --checkpassed for the PR diff. I found noCODEOWNERSfile in the repo. GitHub already reports the PR as approved, and this update does not increase risk, so I’m leaving the existing approval in place without submitting a duplicate approval.Sent by Cursor Automation: Assign PR reviewers
There was a problem hiding this comment.
Stale comment
Risk assessment: Very Low
I assessed the actual diff for
3aa9dd2f778b68be3eafd0638d42e673d6487366...ee13055864bab1111d342e9b96c6f64753cd2f17. The PR only adds six lines of documentation guidance tointroduction/control.mdx; it doesn't modify runtime code, docs configuration, infrastructure, auth/permissions, schemas, or shared production logic.Validation:
git diff --checkpassed for the PR diff. I found noCODEOWNERSfile in the repo. GitHub already reports the PR as approved, so I’m not submitting a duplicate approval.Sent by Cursor Automation: Assign PR reviewers
Adds three named patterns (tool, checkpoint, drop-the-agent) as short bullets under the existing recommended-pattern section. No new page. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
ee13055 to
c879c02
Compare
There was a problem hiding this comment.
Risk assessment: Very Low
I assessed the actual diff for 3aa9dd2f778b68be3eafd0638d42e673d6487366...c879c0274bda1bf4a35585c9ccbb3ecfeb4efde6. The PR adds five lines of documentation guidance to introduction/control.mdx under an existing section. It doesn't modify runtime code, docs configuration, infrastructure, auth/permissions, schemas, or shared production logic.
Validation: git diff --check passed for the PR diff. I found no CODEOWNERS file in the repo. GitHub already reports the PR as approved, and this update does not increase risk, so I’m leaving the existing approval in place without submitting a duplicate approval.
Sent by Cursor Automation: Assign PR reviewers
b142557
into
hypeship/intro-create-control-observe
* add Create, Control, Observe introduction guides Introduce three high-level concept guides under the Introduction group that orient new users around the core surface area: creating browsers, driving them (computer controls vs playwright execution), and observing them (live view, replays, screenshots, logs). * expand standby activity sources to include WebDriver and computer controls * reframe Control intro around three primitives (computer controls, CDP, BiDi) * add lifecycle strip, redirect create-a-browser, add CLI tabs to intro pages * lead with "Computer Use" framing, bridge to Computer Controls API * apply review suggestions: copy tweaks across create, control, observe * promote playwright execution to a peer primitive Add a Playwright Execution tab on Control (position 2, between Computer Use and CDP) and a parallel "Why playwright execution over a direct CDP connection" section. Swap the Create full example to use kernel.browsers.playwright.execute so the recommended path is the one readers see first. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * soften control framing, lead playwright execution with run-from-anywhere - Home page Control card now mentions playwright execution alongside the other three (was drifting from the page itself). - Reframe the Control intro from "four control primitives" to "four ways to drive a session" — playwright execution is a delivery mode on top of CDP, not a separate protocol. - Rewrite the playwright execution tab + bullets around the actual pitch: run any Playwright code from anywhere, no install, no Chromium, no CDP connection to manage. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * restore install snippet on Create, broaden computer-use vs playwright copy - Add the SDK install Info block back to Create (was on the old browsers/create-a-browser page). - Anchor the termination link to the automatic-deletion section. - Rewrite the "Computer use + playwright execution" intro: computer controls don't just lack DOM reads, they lack the whole programmatic surface (page.goto, file uploads, cookies, tab switching). Reach for playwright execution whenever you need any of that. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * Expand control.mdx with computer use + playwright execution patterns (#375) Adds three named patterns (tool, checkpoint, drop-the-agent) as short bullets under the existing recommended-pattern section. No new page. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com> * drop checkpoint/drop-the-agent pattern bullets Two patterns that read more like marketing copy than guidance — the preceding paragraph already says what the combined pattern is for. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> --------- Co-authored-by: ehfeng <279398+ehfeng@users.noreply.github.com> Co-authored-by: masnwilliams <43387599+masnwilliams@users.noreply.github.com> Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>



Summary
Stacks on top of #370. Adds three named patterns as short bullets under the existing "Computer use + playwright execution" section in
introduction/control.mdx:playwright.executefor structured data, the agent stays in charge of navigation.No new page, no extra code. Just names + one-sentence explanations so customers asking "how do I combine these?" have something concrete to point at.
Test plan
mintlify devrenders control.mdx with the new bulletsNote
Low Risk
Documentation-only change that adds usage guidance; no runtime behavior or APIs are modified.
Overview
Expands
introduction/control.mdxwith a short set of recommended patterns for combining computer controls withplaywright.execute, emphasizing when to validate state via Playwright and when to switch from model-driven interaction to fully scripted execution once selectors are stable.Reviewed by Cursor Bugbot for commit c879c02. Bugbot is set up for automated code reviews on this repo. Configure here.