Skip to content

Document safe profile writer coordination - #498

Merged
IlyaasK merged 2 commits into
mainfrom
agent/document-profile-writers
Aug 11, 2026
Merged

Document safe profile writer coordination#498
IlyaasK merged 2 commits into
mainfrom
agent/document-profile-writers

Conversation

@IlyaasK

@IlyaasK IlyaasK commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

What

  • explain that profile saves replace the complete stored state rather than merging it
  • document that the browser session ending last wins when multiple sessions save the same profile
  • show how to find active writers with GET /browsers?status=active&query=<profileId> and profile_save_changes
  • require an exact profile.id match because query searches multiple browser fields
  • call out that concurrent workers still need a lock or lease around the check and browser creation

Why

kernel/kernel#3177 exposes profile_save_changes in browser responses. Customers can now identify active sessions that may overwrite a profile, but the Profiles guide did not explain the overwrite semantics or how to use the new field safely.

Customer impact

Customers get a concrete TypeScript, Python, and Go pattern for detecting active profile writers and guidance for preventing two workers from writing the same profile concurrently.

Related

Verification

  • npx --yes mint@latest broken-links
  • git diff --check

The generated API reference is unchanged and continues to update from the deployed OpenAPI specification.


Note

Low Risk
Documentation-only changes with no runtime, auth, or data-path impact.

Overview
Adds a Prevent concurrent profile writes section to the Profiles guide that explains full-profile overwrite semantics (no merge; last session to end wins) and how to detect active writers via GET /browsers?status=active&query=<profileId> plus profile_save_changes, with exact profile.id filtering because query is broad.

Includes TypeScript, Python, and Go examples that list active browsers and fail if another writer exists, plus guidance to use a lock or lease around the check and browser creation when workers run concurrently.

The Notes bullet on parallel profile use is tightened to describe overwrite behavior instead of vague “corruption” wording.

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

@mintlify

mintlify Bot commented Aug 10, 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 Aug 10, 2026, 7:14 PM

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

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Aug 11, 2026 1:27pm

@IlyaasK
IlyaasK marked this pull request as ready for review August 10, 2026 19:23
cursor[bot]
cursor Bot approved these changes Aug 10, 2026
@IlyaasK
IlyaasK requested review from AnnaXWang, ehfeng and sjmiller609 and removed request for AnnaXWang and sjmiller609 August 10, 2026 19:24

@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

Verdict: Risk unchanged after synchronize — prior approval stands (not re-approving).

Evidence (from diff only)

  • 1 file changed vs main: auth/profiles.mdx (+63 / −1)
  • Documentation-only update: new “Prevent concurrent profile writes” section with TypeScript/Python/Go examples for listing active writers via profile_save_changes, plus a Notes bullet clarifying last-write-wins overwrite semantics
  • Latest push is a merge of main into the branch plus a one-line Notes wording tweak (save_changes: true on post-acquire attach); no new production/runtime surface in this PR’s delta vs main
  • No application code, API implementation, config, CI, or infrastructure changes

Risk factors

Factor Assessment
Codepaths / production logic None modified
Blast radius Docs site content only
Complexity Straightforward guide + examples
Infra / security / auth model Unaffected
User-facing surface Documentation only

Re-evaluation after update

Previously approved at 431755f. Re-checked head 16c91fd after synchronize. Risk remains Very Low; no approval dismissal needed. No CODEOWNERS file requiring ownership review. Already approved — not submitting a new approval.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@IlyaasK
IlyaasK merged commit 14b1cd4 into main Aug 11, 2026
6 of 7 checks passed
@IlyaasK
IlyaasK deleted the agent/document-profile-writers branch August 11, 2026 13:32
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