Skip to content

docs: pooled browsers don't persist profile changes (save_changes)#423

Merged
IlyaasK merged 1 commit into
mainfrom
docs/pool-profile-save-changes-ignored
Jun 26, 2026
Merged

docs: pooled browsers don't persist profile changes (save_changes)#423
IlyaasK merged 1 commit into
mainfrom
docs/pool-profile-save-changes-ignored

Conversation

@IlyaasK

@IlyaasK IlyaasK commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

What

Documents that browser pools load a profile read-only — pooled browsers never persist changes back to the profile, so save_changes does not apply to pools.

Why

Per Sayan's review note on kernel/kernel#2484: save_changes: true on a pool used to return 400 ("not supported for browser pools") and is now silently ignored. That's friendlier for SDK users, but raw-HTTP callers lose the explicit signal — so the behavior needs a line in the pool docs "so nobody expects pooled browsers to persist changes."

Changes

  • browsers/pools/faq.mdx — new Q&A: "Can pooled browsers save changes back to a profile?" (sits next to the existing profile-update Q&A).
  • auth/profiles.mdx — Notes bullet scoping save_changes to single sessions, with a pool exception.
  • browsers/pools/overview.mdx — read-only-profile caveat under Pool configuration options.

The OpenAPI-generated API reference (api-reference/browser-pools/*) is pulled from the Stainless documented spec and is already updated by #2484 — no manual edit here.

Sequencing

⚠️ Merge after kernel/kernel#2484 deploys — this documents the new (ignored, not rejected) behavior, which goes live with that PR.

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only changes with no runtime, API, or security impact.

Overview
Documents that browser pools load profiles read-only and never write session state back, so save_changes only applies to kernel.browsers.create() single sessions.

Adds a Profiles Notes bullet and short caveats under pool overview (pool configuration) and pool FAQ (new Q&A). All three explain that save_changes on a pool profile is silently ignored (not rejected), and point readers to a one-off browser with save_changes: true when they need to persist profile state.

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

…nges)

save_changes:true on a browser pool used to return 400 and is now a silent
no-op (kernel/kernel#2484). Document that pooled browsers load a profile
read-only and never persist changes back to it — save_changes applies only to
single browser sessions — so users don't expect pooled browsers to save state.

- browsers/pools/faq.mdx: new "Can pooled browsers save changes back to a profile?" Q&A
- auth/profiles.mdx: Notes bullet scoping save_changes to single sessions
- browsers/pools/overview.mdx: read-only profile caveat under pool config options

Per review note on kernel/kernel#2484.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mintlify

mintlify Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview Jun 26, 2026, 6:14 PM

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

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Risk assessment: Very Low.

Evidence from the diff: this PR adds seven lines across three .mdx documentation pages (auth/profiles.mdx, browsers/pools/faq.mdx, and browsers/pools/overview.mdx) clarifying that browser pools load profiles read-only and ignore save_changes. There are no code, config, infrastructure, generated asset, auth, billing, permission, prompt, or navigation changes. No CODEOWNERS file is present in the repo, and there were no existing approvals recorded, so approval is appropriate.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@IlyaasK IlyaasK merged commit 1baeb1e into main Jun 26, 2026
4 checks passed
@IlyaasK IlyaasK deleted the docs/pool-profile-save-changes-ignored branch June 26, 2026 20:41
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