CLI: Update SDK to cee2050 and add credential-providers commands #82
Conversation
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
|
🔧 CI Fix Available I've pushed a fix for the CI failure. The |
masnwilliams
force-pushed
the
cli-coverage-update
branch
from
d9a77c7 to
e00ff76
Compare
rgarcia
force-pushed
the
cli-coverage-update
branch
from
e00ff76 to
82204fa
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
- Update kernel-go-sdk from v0.26.0 to v0.27.0
- Update BrowsersService.Get interface to include new BrowserGetParams parameter
- Update all browser Get() calls to pass empty BrowserGetParams{}
SDK version: db80dd764a1e279cb1a806e8d87ce037828ea32f
rgarcia
force-pushed
the
cli-coverage-update
branch
from
42ebd6d to
31b8305
Compare
rgarcia
changed the title
…watch commands - Update kernel-go-sdk to v0.26.1-0.20260117213330-c6c0c0d711e9 - Add agents auth commands: create, get, list, delete - Add agents auth invocations commands: create, get, exchange, submit - Add browsers process resize command - Add browsers fs watch commands: start, stop, events
<!-- CURSOR_SUMMARY --> > [!NOTE] > <sup>[Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) is generating a summary for commit 44883ef. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by: Cursor Agent <cursor-agent@onkernel.com>
- Added `kernel agents auth run` command to automate the full authentication process, including profile creation, credential management, and invocation handling. - Introduced support for TOTP codes for two-factor authentication, allowing automatic submission during the auth flow. - Updated `README.md` with detailed usage instructions and examples for the new authentication commands. - Added dependencies for `github.com/pquerna/otp` and `github.com/boombuler/barcode` to support TOTP generation and indirect usage. - Enhanced `go.mod` and `go.sum` to reflect the new dependencies. This update significantly improves the user experience for managing agent authentication, making it more streamlined and efficient.
The generateTOTPCode function unconditionally printed a human-readable message via pterm.Info.Printf when waiting for a fresh TOTP window. This could corrupt JSONL output when --output json was specified in AgentAuthRunCmd.Run. Added a 'quiet' parameter to generateTOTPCode to suppress console output when in JSON mode, preventing non-JSON messages from corrupting the output stream for programmatic consumers. Co-authored-by: mason <mason@onkernel.com>
- Updated kernel-go-sdk to cee2050be3f8136505d41c20c2903dfca2cbc479 - Added new credential-providers command group for managing external credential providers (e.g., 1Password service account integrations) - list: List configured credential providers - get: Get a credential provider by ID - create: Create a new credential provider - update: Update a credential provider's configuration - delete: Delete a credential provider - test: Test a credential provider connection - Fixed browsers.Get() calls to pass new required BrowserGetParams
rgarcia
changed the title
<!-- CURSOR_SUMMARY --> > [!NOTE] > Aligns test fakes with the updated SDK `BrowsersService.Get` signature. > > - Change `FakeBrowsersService.Get` to accept `query kernel.BrowserGetParams` and update its implementation > - Update all tests that stub or invoke `Get` (e.g., `BrowsersView`, `BrowsersGet`, logs/replays/process/fs/computer helpers) to use the new parameter > - No functional behavior changes; assertions and outputs remain the same > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit cf2dd29. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by: Cursor Agent <cursor-agent@onkernel.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
dprevoznik
added a commit
that referenced
this pull request
Jan 21, 2026
#88) This PR updates the Go SDK to cee2050be3f8136505d41c20c2903dfca2cbc479 and adds CLI commands for new SDK methods. ## SDK Update - Updated kernel-go-sdk to cee2050be3f8136505d41c20c2903dfca2cbc479 ## Coverage Analysis This PR was generated by performing a full enumeration of SDK methods and CLI commands. ## New Commands - `kernel credential-providers list` - List configured external credential providers - `kernel credential-providers get <id>` - Get a credential provider by ID - `kernel credential-providers create` - Create a new credential provider (supports 1Password) - `kernel credential-providers update <id>` - Update a credential provider's configuration - `kernel credential-providers delete <id>` - Delete a credential provider - `kernel credential-providers test <id>` - Test a credential provider connection ## Breaking Changes Fixed - Fixed `browsers.Get()` calls to pass new required `BrowserGetParams` parameter Triggered by: kernel/kernel-go-sdk@cee2050 Reviewer: @masnwilliams <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Introduces new CLI surfaces and updates for latest SDK. > > - **Agent Auth CLI**: `kernel agents auth` with `create/get/list/delete`, `invocations {create/get/exchange/submit}`, and end‑to‑end `run` flow (auto field submission, TOTP, optional live view); docs and examples added to `README.md`. > - **Credential Providers CLI**: `kernel credential-providers {list/get/create/update/delete/test}` (supports 1Password), wired into root. > - **Browsers API updates**: adapt to SDK breaking change (`browsers.Get` now requires `BrowserGetParams`); add `process resize` and filesystem watch (`fs watch start/stop/events`) commands; tests updated accordingly. > - **Dependencies**: bump `kernel-go-sdk` to cee2050… and add `pquerna/otp`; regenerate `go.sum`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 0b27df6. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> --------- Co-authored-by: Mason Williams <43387599+masnwilliams@users.noreply.github.com> Co-authored-by: Cursor Agent <cursor-agent@kernel.sh> Co-authored-by: Cursor Agent <cursor-agent@onkernel.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR updates the Go SDK to cee2050be3f8136505d41c20c2903dfca2cbc479 and adds CLI commands for new SDK methods.
SDK Update
Coverage Analysis
This PR was generated by performing a full enumeration of SDK methods and CLI commands.
New Commands
kernel credential-providers list- List configured external credential providerskernel credential-providers get <id>- Get a credential provider by IDkernel credential-providers create- Create a new credential provider (supports 1Password)kernel credential-providers update <id>- Update a credential provider's configurationkernel credential-providers delete <id>- Delete a credential providerkernel credential-providers test <id>- Test a credential provider connectionBreaking Changes Fixed
browsers.Get()calls to pass new requiredBrowserGetParamsparameterTriggered by: kernel/kernel-go-sdk@cee2050
Reviewer: @masnwilliams
Note
Introduces new CLI surfaces and updates for latest SDK.
kernel agents authwithcreate/get/list/delete,invocations {create/get/exchange/submit}, and end‑to‑endrunflow (auto field submission, TOTP, optional live view); docs and examples added toREADME.md.kernel credential-providers {list/get/create/update/delete/test}(supports 1Password), wired into root.browsers.Getnow requiresBrowserGetParams); addprocess resizeand filesystem watch (fs watch start/stop/events) commands; tests updated accordingly.kernel-go-sdkto cee2050… and addpquerna/otp; regeneratego.sum.Written by Cursor Bugbot for commit 0b27df6. This will update automatically on new commits. Configure here.