Skip to content

Expand control.mdx with computer use + playwright execution patterns#375

Merged
masnwilliams merged 1 commit into
hypeship/intro-create-control-observefrom
hypeship/intro-patterns
May 23, 2026
Merged

Expand control.mdx with computer use + playwright execution patterns#375
masnwilliams merged 1 commit into
hypeship/intro-create-control-observefrom
hypeship/intro-patterns

Conversation

@masnwilliams
Copy link
Copy Markdown
Contributor

@masnwilliams masnwilliams commented May 22, 2026

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:

  • Expose it as a tool — model calls playwright.execute for structured data, the agent stays in charge of navigation.
  • Checkpoint between steps — assert cookies / DOM state after a CUA step before continuing.
  • Drop the agent once selectors are stable — replay at scale without a model in the loop.

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 dev renders control.mdx with the new bullets
  • No broken links

Note

Low Risk
Documentation-only change that adds usage guidance; no runtime behavior or APIs are modified.

Overview
Expands introduction/control.mdx with a short set of recommended patterns for combining computer controls with playwright.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.

@masnwilliams masnwilliams marked this pull request as ready for review May 22, 2026 17:06
@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: 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 @firetiger monitor this.

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 e9444a4. Configure here.

Comment thread introduction/patterns.mdx Outdated
cursor[bot]
cursor Bot approved these changes May 22, 2026
@masnwilliams masnwilliams force-pushed the hypeship/intro-patterns branch from e9444a4 to 3115bd8 Compare May 22, 2026 17:32
@masnwilliams masnwilliams changed the title Add patterns guide for computer use + playwright execution Expand control.mdx with computer use + playwright execution patterns May 22, 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

I reviewed the synchronized diff directly. The PR adds six lines of documentation guidance to introduction/control.mdx under 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 --check passed for the requested diff.

No CODEOWNERS file 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.

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.

Stale comment

Risk assessment: Very Low

I assessed the actual diff for cf6c7bbe2762ba139ecfff086319f2fe061e0c7b...8ba5164cc60cde0f2e3f87e8dc7e0582bc4f49cd. The PR adds six lines of documentation guidance to introduction/control.mdx and does not 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.

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.

Stale comment

Risk assessment: Very Low

I assessed the actual diff for 3aa9dd2f778b68be3eafd0638d42e673d6487366...ee13055864bab1111d342e9b96c6f64753cd2f17. The PR only adds six lines of documentation guidance to introduction/control.mdx; 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, so I’m not submitting a duplicate approval.

Open in Web View Automation 

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>
@masnwilliams masnwilliams force-pushed the hypeship/intro-patterns branch from ee13055 to c879c02 Compare May 23, 2026 02:35
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

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.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@masnwilliams masnwilliams merged commit b142557 into hypeship/intro-create-control-observe May 23, 2026
4 checks passed
@masnwilliams masnwilliams deleted the hypeship/intro-patterns branch May 23, 2026 17:51
masnwilliams added a commit that referenced this pull request May 23, 2026
* 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>
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