Skip to content

Add browser curl#349

Merged
juecd merged 5 commits intomainfrom
cj/curl
Apr 30, 2026
Merged

Add browser curl#349
juecd merged 5 commits intomainfrom
cj/curl

Conversation

@juecd
Copy link
Copy Markdown
Contributor

@juecd juecd commented Apr 29, 2026

Note

Low Risk
Documentation-only changes: adds a new browsers/curl guide and updates navigation/linking, with no runtime or API behavior changes.

Overview
Adds a new browsers/curl documentation page describing how to issue HTTP requests through a Kernel browser (streaming via browsers.fetch/browsers.request and buffered via browsers.curl), including examples and concurrency/memory cautions.

Updates docs.json navigation to include the new Browser curl page and tweaks skills/overview.mdx to reference Bot Detection more consistently.

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

@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 title 'Add browser curl' does not indicate changes to kernel API endpoints or Temporal workflows; please manually opt in if this affects the monitored paths.

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

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Apr 29, 2026

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

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview Apr 29, 2026, 8:45 PM

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

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

This PR adds a new documentation page (browsers/curl.mdx) and one Mintlify navigation entry in docs.json. The change does not modify application code, infrastructure, auth/permissions, schemas, or shared runtime behavior, so the blast radius is limited to docs rendering/navigation.

No CODEOWNERS file was present in the repo, and the PR had no existing approvals at the time of review.

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 current diff adds a new Mintlify documentation page (browsers/curl.mdx) and a single documentation navigation entry in docs.json. Since the previous approval, the updates are still limited to docs copy/code examples and moving the nav entry within the browser docs section.

I found no application code, infrastructure, auth/permissions, schema, or shared runtime behavior changes, and no CODEOWNERS file is present in the repo. The PR is already approved, so I’m not submitting another approval. No risk increase was identified, so the existing approval can remain in place.

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

I re-evaluated the current diff from main to cj/curl. The PR adds browsers/curl.mdx and one docs.json navigation entry for that new docs page. The latest synchronized commit only changes an internal docs link from /bot-detection/overview to /browsers/bot-detection/overview.

I found no application code, infrastructure, auth/permissions, schema, prompt/instruction, or shared runtime behavior changes. There is also no CODEOWNERS file present in the repo. The PR is already in an approved state, so I’m not submitting another approval, and no risk increase was identified that would require dismissing the existing approval.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

Comment thread browsers/curl.mdx Outdated
Copy link
Copy Markdown
Contributor

@dprevoznik dprevoznik left a comment

Choose a reason for hiding this comment

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

LGTM - left two suggestions to discuss.

Comment thread browsers/curl.mdx Outdated
Comment thread browsers/curl.mdx Outdated
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 re-evaluated the current diff from main to cj/curl at 74f05c09db23d5d1f9d04feb0339190cec007c80. The changes are limited to adding a new Mintlify documentation page (browsers/curl.mdx), adding that page to docs.json navigation, and renaming one documentation link label in skills/overview.mdx.

I found no application code, infrastructure, auth/permissions, schema, prompt/instruction, or shared runtime behavior changes. There is no CODEOWNERS file present in the repo. The PR is already approved, so I’m not submitting another approval; no risk increase was identified that would require dismissing the existing approval.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@juecd juecd merged commit b6ce5c5 into main Apr 30, 2026
4 checks passed
@juecd juecd deleted the cj/curl branch April 30, 2026 20:13
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