Repository navigation
CLI: Update Go SDK to v0.148.0 - #317
kernel-internal[bot] wants to merge 2 commits into
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default 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 d9554ae. Configure here.
| or later may append &return_url=<URL-encoded URL>. invoke it once per advertised | ||
| request and never request another link automatically. | ||
| 1pw_access_request_status: no browser needed; optional timeout_seconds 0-120 (default 10) | ||
| controls how long to keep checking. Kernel checks immediately, then every 2 seconds, |
There was a problem hiding this comment.
Vaults help says Kernel not KERNEL
Low Severity
New 1Password 1pw_access_request_status help in vaults_commands.go and vaults_credentials.go writes Kernel checks immediately. Vaults user-facing copy uses KERNEL for the product name (KERNEL stores it, KERNEL stores them encrypted); Kernel is the form that rule flags.
Additional Locations (1)
Triggered by learned rule: Vaults user-facing copy is lowercase; KERNEL for product
Reviewed by Cursor Bugbot for commit d9554ae. Configure here.


Summary
v0.148.0.Coverage
CLI coverage update for Kernel Go SDK v0.148.0
Coverage review
github.com/kernel/kernel-go-sdkfrom v0.145.0 to v0.148.0.api.md, their parameter structs, and nested parameter fields.x-cli-skip: true:ConfigRegistry.List,ConfigRegistry.Lookup,ConfigRegistry.Resolve,ConfigRegistry.Analyses.Get,ConfigRegistry.Analyses.List, andConfigRegistry.Analyses.Cancel.client.Browsers.LiveView.New.Changes
Scoped browser live views
Added
kernel browsers live-view create <id-or-name>forclient.Browsers.LiveView.New.New flags:
--scope read|controlselects read-only or interactive access and defaults toread.--ttl-secondssets a 1–31,536,000 second lifetime. Omitting it keeps the URL valid until browser deletion or pooled-browser release.--output jsonreturns the raw SDK response.The command validates scope and TTL locally, renders URL/scope/expiration in table output, and warns in help that the URL is a credential, is limited to the live viewer, and cannot be revoked early.
Replay startup and response coverage
The update branch already contained the v0.146/v0.147 replay coverage, retained in this diff:
--start-replaytokernel browsers create, including the pool-acquisition path.--start-replaytokernel browser-pools create,update, andacquire, preserving explicit false values on update/acquire.kernel browser-pools get.Managed-auth browser locale
The update branch also contains managed-auth locale coverage:
kernel auth connections create --localesets the canonical BCP 47 locale used by login, reauthentication, and health-check browser sessions.kernel auth connections update --localereplaces the locale.kernel auth connections update --clear-localeclears the override and returns future sessions to the browser region default.kernel auth connections login, because the SDK/spec explicitly does not support a per-login locale override.1Password vault guidance
Updated
kernel vaults items invoke, credential creation, and connection help to match the revised 1Password schemas:timeout_secondsmeaning, and 429Retry-Afterhandling.1pw_create_access_requestmay be used.expiredreturning the item topending_authorizationandunknownbeing treated as failed.return_urlsupported by 1Password 8.12.46 and later.The new response-only
access_request_referenceremains excluded from the CLI's display-safe vault projection because the API description says to keep it out of logs. The approval URL remains available. Tests now cover this exclusion.Help text synchronized with the API spec
createBrowserLiveViewbrowsers live-view createShort/Long help, including headful-only behavior, viewer-only URL scope, lifetime, pooled release, non-revocability, and credential handling.BrowserLiveViewRequest.scope--scopeusage forreadandcontrol, with thereaddefault.BrowserLiveViewRequest.ttl_seconds--ttl-secondsusage and 1–31,536,000 validation.start_replaybrowsers create --start-replayand replay response rendering.start_replay--start-replayflags with create, replacement, and per-lease semantics.ManagedAuthBrowserConfig.location.localeOnePasswordPollAccessVaultItemOperationRequestandtimeout_secondsOnePasswordAccessApprovalAction.urlreturn_urlbehavior.stateand credentialavailable_operationsexpired,unknown, and re-advertising a new access-request operation after the status-check window.Intentional exclusions
x-cli-skip: true.auth connections loginbecause that request type does not contain the field.access_request_referenceis omitted from display-safe CLI output as noted above.Validation
go mod tidygo build ./...go test ./... -count=1kernel browsers live-view create <browser> --scope control --ttl-seconds 120 --output json.scope: control, a non-empty URL, and an expiration timestamp.The replay and managed-auth locale changes already present on the update branch were covered by their unit and command-wiring tests; they were not re-smoke-tested against live resources in this run.
Validation
Note
Medium Risk
Touches auth connection browser config, pool/browser session creation, and new credential-like live-view URLs; behavior is mostly additive but misconfigured flags could change session locale or replay defaults.
Overview
Bumps kernel-go-sdk to v0.148.0 and wires new API surface into the CLI.
Scoped live views: Adds
kernel browsers live-view createwith--scope(read/control), optional--ttl-seconds, validation, and table/JSON output; help stresses viewer-only URLs and credential handling.Replay: Adds
--start-replayon browser create (including pool lease) and on browser-pool create/update/acquire; surfaces replay metadata in create/acquire/get/update output and optional Replay ID columns in browser and invocation browser lists.Managed auth locale: Adds
--localeon auth connection create/update and--clear-localeon update (with mutual exclusion and empty-value checks); connection detail tables show Browser Locale instead of duplicating region.Docs/tests: Expands 1Password vault/credential invoke help (polling cadence, timeouts,
return_url, expiration); keepsaccess_request_referenceout of display-safe output. Unit tests cover locale flags, replay params, and live-view create.Reviewed by Cursor Bugbot for commit d9554ae. Bugbot is set up for automated code reviews on this repo. Configure here.