Add get_browser_telemetry tool for reading archived session telemetry#119
Draft
yummybomb wants to merge 2 commits into
Draft
Add get_browser_telemetry tool for reading archived session telemetry#119yummybomb wants to merge 2 commits into
yummybomb wants to merge 2 commits into
Conversation
- New get_browser_telemetry tool reads archived telemetry events for a session (active or deleted) with category filtering, windowing, and offset paging. Events are compacted (bulky payload fields dropped) and empty results are disambiguated: telemetry_not_enabled vs no_events. - Defaults the read window to the session's creation time instead of the API's 5m default. - Expand telemetry param descriptions on manage_browsers to call out that the default category bundle omits console/network/page. - Add a telemetry section to the debug-browser-session prompt with the event catalog and enable-before-reproduce guidance. - Bump @onkernel/sdk to ^0.74.0 for browsers.telemetry.events; migrate browser pool, extension, and proxy list calls to the paginated response shape the new SDK returns.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
… lists The SDK comma-joins array query params while the API binds category as repeated params, so a multi-category filter matched nothing; send single categories through and filter multi-category requests client-side (the server filters within the page, so semantics are identical). Also wire limit/offset params into the browser pool, extension, and proxy list actions so callers can actually follow the has_more/next_offset the 0.74 SDK migration surfaced, and page the browser-pools resource fully.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
get_browser_telemetrytool — reads archived telemetry events for a browser session viabrowsers.telemetry.events. Works for active and deleted sessions. Supportscategoriesfiltering,since/untilwindows,order(asc/desc), and opaqueoffsetpaging vianext_offset/has_more.{seq, time, category, type, data, truncated}, with bulky payload fields (body,headers,post_data) dropped and surfaced inomitted_fields.sincedefault, so reads of older sessions don't silently come back empty.telemetry_not_enabled(with recreate-and-reproduce guidance) vsno_events.categoryas repeated params, so a multi-value filter sent through the SDK would match nothing. Single categories pass through; semantics are identical because the server filters within the page.manage_browserstelemetry param descriptions now state that telemetry is off unless requested and that the default category bundle (control/connection/system/captcha) omits the debug-critical console/network/page categories.debug-browser-sessionprompt gains a telemetry section: the opt-in gotcha, the read flow (scan forconsole_error,network_loading_failed, non-2xx responses,captcha_*), the event catalog, and a note that events survive session deletion. Existing CLI guidance unchanged.@onkernel/sdkbumped^0.60.0→^0.74.0(required forbrowsers.telemetry.events). The new SDK returns paginated pages frombrowserPools.list,extensions.list, andproxies.list, so those call sites were migrated togetPaginatedItems()with realhas_more/next_offsetvalues, and those tools' list actions gainedlimit/offsetparams so callers can follownext_offset. The browser-pools resource (fixed URI, no params) iterates all pages.Test plan
tsc --noEmitpassesprettier --checkpasses on changed filesnext build— compiles and typechecks, but page-data collection requires OAuth env vars (KERNEL_CLI_PROD_CLIENT_ID) not available locallyget_browser_telemetry, delete the session, read again (also validates the archive write path)No test suite exists in this repo, so the changes were not covered by automated tests.
🤖 Generated with Claude Code