Adopt @onkernel/sdk v0.72.0; drop save_changes from browser pools#117
Adopt @onkernel/sdk v0.72.0; drop save_changes from browser pools#117IlyaasK wants to merge 3 commits into
Conversation
Bumps @onkernel/sdk ^0.60.0 -> ^0.72.0 and adapts to the API/SDK changes in the 12-minor jump: - List endpoints now return paginated results (OffsetPagination) instead of arrays: read `.items` for browser-pools, extensions, and proxies list, and surface the real has_more / next_offset for extensions and proxies (was hardcoded false/null). - Credential `values` is typed string->string; use z.record(z.string(), z.string()) (also required by zod v4's two-arg z.record). - Browser pools no longer accept save_changes on their profile (kernel/kernel #2484): drop the save_profile_changes input from the manage_browser_pools tool and Omit it from PoolConfigParams so the pool path can't send it. Single-session `browsers`/`profiles` tools keep save_profile_changes unchanged. tsc --noEmit passes. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit bb1f1ab. Configure here.
There was a problem hiding this comment.
Additional Suggestion:
manage_credential_providers "list" serializes the entire SDK pagination page object instead of the provider array, because the @onkernel/sdk 0.72 bump changed credentialProviders.list() to return an OffsetPagination page while this tool was not updated like the other list tools.
The SDK now paginates list endpoints; surfacing has_more/next_offset without input controls let callers see there's more but not fetch it. Wire the shared paginationParams (limit/offset) into the manage_extensions and manage_proxies list actions and pass them to list(), matching the apps/profiles/projects tools. Addresses Cursor Bugbot "list lacks pagination inputs". Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
CI runs `bun install --frozen-lockfile`; the SDK bump requires the lockfile entry to match package.json. Regenerated with `bun install` (bun 1.3.14). Only the @onkernel/sdk resolution changes; the `next` spec line is a cosmetic normalization bun applied (16.2.6 -> ^16.2.6, matching package.json's existing ^16.2.6 — resolved version unchanged). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|

What
Bumps
@onkernel/sdk^0.60.0→^0.72.0and adapts the MCP tools to the changes across that 12-minor jump, including the browser-poolsave_changesremoval from kernel/kernel#2484.Changes
list()now returnsOffsetPagination(not arrays). Read.itemsforbrowser-pools,extensions,proxies, and surface the realhas_more/next_offsetfor the extensions/proxies list tools (previously hardcodedfalse/null).valuesisstring → string; usez.record(z.string(), z.string())(also satisfies zod v4's two-argz.record).save_changes(#2484): removed thesave_profile_changesinput frommanage_browser_poolsandOmit'd it fromPoolConfigParams, so the pool path can't send it (the API now silently ignoressave_changeson pools). Single-sessionbrowsers/profilestools keepsave_profile_changesunchanged.Verification
npx tsc --noEmitpasses with zero errors.Notes
zodis a transitive dep that now resolves to v4; thez.record(z.string(), z.string())form is valid in both zod v3 and v4, so the fix is resolution-robust.browser-poolslist now returns the first page; the resource reader / list action preserve their prior (non-paginating) shape. The extensions/proxies tools now report accuratehas_more.🤖 Generated with Claude Code
Note
Medium Risk
SDK jump and pagination changes can truncate large extension/proxy/pool inventories if callers do not paginate; removing pool
save_profile_changesis a behavioral contract change for agents that relied on it.Overview
Upgrades
@onkernel/sdkfrom^0.60.0to^0.72.0(and relaxesnextto^16.2.6) so MCP tools match the newer Kernel client and API shapes.List endpoints no longer return bare arrays:
browserPools,extensions, andproxieslist paths now readpage.items.manage_extensionsandmanage_proxiesaccept sharedlimit/offsetpagination params and return realhas_more/next_offsetinstead of hardcoded false/null. Browser pool list/resource still expose the same response shape but only the first page of pools.Browser pools drop
save_profile_changesfrommanage_browser_poolsand omit it from pool config typing so pools cannot send profile save flags the API no longer supports for pools; single-session browser tools keepsave_profile_changesunchanged.Credentials validation changes
valuestoz.record(z.string(), z.string())for SDK typing and Zod v3/v4 compatibility.Reviewed by Cursor Bugbot for commit 5077eb7. Bugbot is set up for automated code reviews on this repo. Configure here.