docs: document chrome_policy on on-demand browsers#377
Conversation
Add a new general /browsers/chrome-policy page covering chrome_policy for both on-demand sessions and reserved pools. Update create-a-browser to show the new options, and trim the pool-specific page to focus on pool-only behavior (discard_all_idle) while linking to the general reference. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Firetiger deploy monitoring skipped This PR didn't match the auto-monitor filter configured on your GitHub connection:
Reason: This PR only modifies documentation files and does not change any kernel API endpoints or Temporal workflows. To monitor this PR anyway, reply with |
There was a problem hiding this comment.
Stale comment
Risk assessment: Very Low.
I assessed the actual diff: it adds
browsers/chrome-policy.mdx, updates related browser docs, and adds one navigation entry indocs.json. This is documentation/sidebar content only, with no production codepaths, infrastructure, auth, billing, permissions, or shared runtime behavior changed. I also found noCODEOWNERSfile indicating a Codeowners approval requirement.The PR is already approved at the current head commit, so I’m not submitting another approval.
Sent by Cursor Automation: Assign PR reviewers
| | `RestoreOnStartupURLs` | `string[]` | URLs to open when the browser starts. Requires `RestoreOnStartup` set to `4` | | ||
| | `BookmarkBarEnabled` | `boolean` | Shows the bookmark bar | | ||
| | `ManagedBookmarks` | `array` | Pre-configured bookmarks. Supports folders via nested `children` arrays | | ||
| | `DownloadBubbleEnabled` | `boolean` | Controls the download bubble UI. Set to `false` to fall back to the classic download shelf | |
There was a problem hiding this comment.
Deprecated Chrome policy documented with incorrect behavior
Medium Severity
DownloadBubbleEnabled is a deprecated Chrome policy (removed after Chrome 118). Its table description claims setting it to false falls back to "the classic download shelf," but this Chromium UI element no longer exists in modern Chrome. This references version-specific Chromium implementation details as product documentation and will mislead users since the policy has no effect. It's also used as a prominent example in code samples across both new pages, which compounds the issue.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit a5e62b0. Configure here.
Absolute URLs were rendering with extra inline spacing inside the underlined link span. Relative paths render cleanly like other api-reference links in the docs.
There was a problem hiding this comment.
Stale comment
Risk assessment: Very Low.
I assessed the current diff at the latest head: it adds
browsers/chrome-policy.mdx, updates related browser documentation, trims duplicated pool-specific documentation, and adds one sidebar entry indocs.json. This is documentation/navigation content only, with no production codepaths, infrastructure, auth, billing, permissions, or shared runtime behavior changed.I found no
CODEOWNERSfile in the repository. The PR is already approved, so I’m not submitting another approval.Sent by Cursor Automation: Assign PR reviewers
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1441b9a. Configure here.
| `chrome_policy` is supported on: | ||
|
|
||
| - [`browsers.create()`](/api-reference/browsers/create-a-browser#body-chrome-policy) — on-demand browser sessions | ||
| - [`browserPools.create()`](/api-reference/browser-pools/create-a-browser-pool#body-chrome-policy) and [`browserPools.update()`](/api-reference/browser-pools/update-a-browser-pool#body-chrome-policy) — reserved browser pools |
There was a problem hiding this comment.
Relative /api-reference/ links will break CI checks
High Severity
All /api-reference/... links in the new browsers/chrome-policy.mdx page (lines 10–11) and the updated browsers/pools/policy-json.mdx (line 6) use relative paths. API reference pages are auto-generated from a remote OpenAPI spec and don't exist locally, so relative links will fail the mint broken-links CI check. These need to be absolute URLs (e.g. https://kernel.sh/docs/api-reference/...). Notably, policy-json.mdx previously had the correct absolute URL and this PR regressed it to a relative one.
Additional Locations (1)
Triggered by learned rule: Use absolute URLs for /api-reference/ links
Reviewed by Cursor Bugbot for commit 1441b9a. Configure here.
The underline on <a> extended past the visible text because the nested <code> element had its own horizontal padding. Zero the padding and background on code inside links so the underline matches the text width.
There was a problem hiding this comment.
Risk assessment: Very Low.
I assessed the latest diff at the current head. The PR adds browsers/chrome-policy.mdx, updates related browser docs, moves duplicated policy reference content into the new page, adds one docs.json navigation entry, and makes a small docs-site CSS adjustment for inline code inside links.
This remains docs/navigation/styling-only for the documentation site. It does not modify production codepaths, infrastructure, auth, billing, permissions, database/schema, generated API specs, or shared runtime behavior. I found no CODEOWNERS file indicating a Codeowners approval requirement.
The PR is already approved, so I’m not submitting another approval. The latest update does not increase the risk level.
Sent by Cursor Automation: Assign PR reviewers




Summary
/browsers/chrome-policypage that coverschrome_policyfor both on-demandbrowsers.create()and reserved pools, with a quick reference of common policy keys./browsers/create-a-browserto show achrome_policy(andstart_url) example./browsers/pools/policy-jsonto focus on the pool-only update flow (discard_all_idle) and link out to the general reference for shared content.browsers/chrome-policyin the Intermediate sidebar group indocs.json.Test plan
browsers/chrome-policypage and confirm code samples render correctly in both Typescript and Python tabscreate-a-browser,pools/policy-json, andchrome-policyto confirm they resolveNote
Low Risk
Low risk documentation and styling-only changes; no runtime or API behavior is modified. Main risk is broken/incorrect links or misleading policy examples in the docs.
Overview
Adds a new
browsers/chrome-policyguide documenting thechrome_policyfield for both on-demandbrowsers.create()sessions and reserved browser pools, including TS/Python examples and a small table of common policy keys.Updates
browsers/create-a-browserto showstart_url+chrome_policyusage, and trimsbrowsers/pools/policy-jsonto focus on the pool update flow (notablydiscard_all_idle) while linking to the new shared reference. Registers the new page indocs.jsonnavigation and tweaksstyle.cssso inlinecodeinside links doesn’t add padding/background (fixing underline width).Reviewed by Cursor Bugbot for commit 4f05af0. Bugbot is set up for automated code reviews on this repo. Configure here.