Skip to content

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

Open
masnwilliams wants to merge 1 commit into
hypeship/intro-create-control-observefrom
hypeship/intro-patterns
Open

Expand control.mdx with computer use + playwright execution patterns#375
masnwilliams wants to merge 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

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