Document the platform category and correct control's sensitivity - #474
Document the platform category and correct control's sensitivity#474archandatta wants to merge 1 commit into
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
21c25ec to
778a23a
Compare
778a23a to
32687ed
Compare
There was a problem hiding this comment.
Stale comment
Risk assessment: Very Low
Verdict: No action needed — prior approval remains valid (risk did not increase).
Re-evaluation after synchronize
Previous automation approval was on
778a23a9. Current head is32687ed. Re-assessed the updated diff.Evidence (from diff)
- 1 file changed:
browsers/telemetry/categories.mdx(+15 / −5)- Documentation-only MDX update: revises the
controlcategory description, documents a newplatformcategory, and corrects the data-sensitivity table / guidance around Playwrightcodecapture and CDP command redaction- No application logic, APIs, auth, infra, config, or schema changes
- No CODEOWNERS file present
Risk factors
Factor Assessment Codepaths modified Docs content only Blast radius Readers of telemetry category docs Complexity Low — prose/table updates Infrastructure impact None Operational / security risk None in code; content clarifies sensitivity semantics Decision
Still meets Very Low criteria (documentation-only, small scoped diff, no production logic or shared-system changes). Risk is unchanged from the prior approval, so approval is not revoked and the PR is not re-approved.
Sent by Cursor Automation: Assign PR reviewers
`control` used to be every in-VM API call, and the sensitivity table described the whole default set as session metadata with no page content. Both are about to stop being true: `control` narrows to actions that drive the browser and gains the submitted Playwright source on `api_call`, VM-management calls move to a new opt-in `platform` category, and browser-control commands sent over the CDP proxy arrive as `cdp_command`. The sensitivity table now calls out that a default-on category carries submitted source, with the practical advice that follows from it: pass credentials through variables rather than as literals in the snippet. Callers under a BAA get the same note, since `control` stays available to them when the browser-activity categories don't. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
32687ed to
10591a0
Compare
There was a problem hiding this comment.
Risk assessment: Very Low
Verdict: No action needed — prior approval remains valid (risk did not increase).
Re-evaluation after synchronize
Previous automation approval was on 778a23a9. Current head is 10591a04. Re-assessed the updated diff.
Evidence (from diff)
- 1 file changed:
browsers/telemetry/categories.mdx(+15 / −5) - Documentation-only MDX update: revises the
controlcategory description, documents a newplatformcategory, and corrects the data-sensitivity table / guidance around Playwrightcodecapture and CDP command redaction - No application logic, APIs, auth, infra, config, or schema changes
- No CODEOWNERS file present
Risk factors
| Factor | Assessment |
|---|---|
| Codepaths modified | Docs content only |
| Blast radius | Readers of telemetry category docs |
| Complexity | Low — prose/table updates |
| Infrastructure impact | None |
| Operational / security risk | None in code; content clarifies sensitivity semantics |
Decision
Still meets Very Low criteria (documentation-only, small scoped diff, no production logic or shared-system changes). Risk is unchanged from the prior approval, so approval is not revoked and the PR is not re-approved.
Sent by Cursor Automation: Assign PR reviewers


Summary
Docs half of the browser-telemetry
control/platformsplit (kernel-images#322, kernel-images#323, kernel#3086). Two things on this page stop being true when that ships, one of them in a way that matters for compliance.controlnarrows. It was "computer-control API calls against the session" and in practice every in-VM API endpoint. It is now the actions that drive the browser — computer controls, Playwright execution, screenshots, clipboard — pluscdp_commandfor browser-control commands sent over the CDP proxy.platformis new, opt-in, and carries the VM-management traffic that used to sit incontrol(recording lifecycle, filesystem, process, telemetry and browser configuration). Documented with when you'd actually want it: debugging a profile save or a replay rather than the agent.codelanded. It groupedcontrolwith the categories that carry "session metadata only ... no page content", andapi_callnow carries the source submitted for Playwright execution — in a category that is on by default. The row now says so, notes the 8 KB clip, and states whatcdp_commanddeliberately does not capture (typed text, key names, navigation URLs — only the character count).controlstays available to BAA orgs whennetwork/console/screenshotdon't.The default-set description on the overview page is unchanged and still correct — the default set is still
control,connection,system,captcha.Sequencing
Should merge with or just after the kernel-images release and kernel#3086, not before: until then
platformisn't selectable andcontrolstill carries the platform traffic.Notes
Left formatting alone — this file doesn't currently satisfy prettier at baseline and CI only checks broken links, so running it would bury the change in reflowed tables. No new links added.
Review round
Three accuracy fixes after checking every claim against the generated operation-to-category map and the emitting code:
platformrow omitted three operations it covers: log streaming and both scale-to-zero calls. Added.