Skip to content

CLI: Update SDK to 5e56fc5d99a6a64596f09f85c1530fa51f9d7a7a and add new commands/flags#139

Merged
masnwilliams merged 2 commits intomainfrom
cli-coverage-update
Mar 23, 2026
Merged

CLI: Update SDK to 5e56fc5d99a6a64596f09f85c1530fa51f9d7a7a and add new commands/flags#139
masnwilliams merged 2 commits intomainfrom
cli-coverage-update

Conversation

@kernel-internal
Copy link
Contributor

@kernel-internal kernel-internal bot commented Mar 23, 2026

This PR updates the Go SDK to 5e56fc5d99a6a64596f09f85c1530fa51f9d7a7a and adds CLI commands/flags for new SDK methods.

SDK Update

  • Updated kernel-go-sdk to 5e56fc5d99a6a64596f09f85c1530fa51f9d7a7a

Coverage Analysis

This PR was generated by performing a full enumeration of SDK methods and CLI commands.

New Commands

  • kernel deploy get for client.Deployments.Get()
  • kernel invoke get, kernel invoke update, and kernel invoke delete-browsers for the corresponding invocation SDK methods
  • kernel auth connections update for client.AuthConnections.Update()

New Flags

  • --region for kernel deploy and kernel deploy github
  • --app-version for kernel deploy history
  • --since for async kernel invoke, plus --action, --deployment-id, --offset, --since, and --status for kernel invoke history
  • --query for kernel app list
  • --invocation-id for kernel browsers create and --include-deleted for kernel browsers get
  • --name for kernel credential-providers create and kernel credential-providers update
  • --sign-in-option-id and --sso-provider for kernel auth connections submit, plus the new update flags for kernel auth connections update

Triggered by: kernel/kernel-go-sdk@5e56fc5
Reviewer: @masnwilliams

Made with Cursor


Note

Medium Risk
Moderate risk: expands CLI surface area and request parameter mapping across deployments/invocations/auth connections, which could break workflows if flags or validation are incorrect, but changes are mostly additive and covered by new unit tests.

Overview
Updates kernel-go-sdk to a newer commit and wires newly-available API methods into the CLI.

Adds new CLI capabilities: kernel deploy get, kernel invoke get|update|delete-browsers, and kernel auth connections update, plus richer filtering/options across existing commands (e.g., app list --query, deploy --region + deploy history --app-version, invoke --since and invoke history --action/--deployment-id/--offset/--since/--status).

Extends browser and credential-provider flows with new flags (browsers create --invocation-id, browsers get --include-deleted, and required/provider --name support), tightens/clarifies auth submit modes (now exactly one of field/MFA/sign-in/SSO selector/provider), and improves “Next:” pagination hints by quoting values and using consistent pterm output; includes accompanying tests for new param mappings.

Written by Cursor Bugbot for commit 7bc97e1. This will update automatically on new commits. Configure here.

Expose the new SDK methods and params added since v0.44.0 so the CLI stays aligned with the generated client surface across deploys, invocations, managed auth, browsers, apps, and credential providers.

Tested: go test ./..., go build ./..., kernel deploy get eqo2jzrlf8msevs8n6mbn4bf, kernel deploy history ts-basic --app-version latest, kernel app list --query ts-basic, kernel invoke ts-basic get-page-title --since 10m, kernel invoke get apwmow9vgbs220iwnwhc8lc8, kernel invoke history --app ts-basic --action get-page-title --deployment-id eqo2jzrlf8msevs8n6mbn4bf --status succeeded --since 24h, kernel invoke update bmncbxisjg1nv95hr3fa05dw --status failed --output '{"error":"manual test"}', kernel invoke delete-browsers bmncbxisjg1nv95hr3fa05dw, kernel browsers create --invocation-id qkpb5v6chmvm6bbr7j2fkxxg -t 30, kernel browsers get 2l9r4s41c04qy6ge2v5w38s0 --include-deleted, kernel auth connections update 5d71e33b-a099-4a85-a850-a9d1a5d6456b --health-check-interval 30 --save-credentials, kernel credential-providers update cp_hcpp4bm210p1yxxbw61eg7b4 --name cli-coverage-provider-updated
Made-with: Cursor
@kernel-internal kernel-internal bot requested a review from masnwilliams March 23, 2026 18:05
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

invokeCmd.AddCommand(invocationGetCmd)

invocationUpdateCmd.Flags().String("status", "", "New invocation status: succeeded or failed")
invocationUpdateCmd.Flags().String("output", "", "Updated invocation output rendered as a JSON string")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Flag --output overloaded with conflicting semantics

Medium Severity

The --output flag on invocationUpdateCmd is defined to mean "Updated invocation output rendered as a JSON string" (the invocation's result payload), but every other command in the CLI uses --output / -o for display format (e.g., json for raw API response). In runInvocationUpdate, the output variable is used exclusively as the invocation payload — the function always calls printInvocationDetails() and never supports JSON display format. A user passing --output json (as they would with kernel invoke get, kernel invoke history, etc.) gets a confusing "invalid JSON for --output" error instead of JSON-formatted display output.

Additional Locations (1)
Fix in Cursor Fix in Web

@masnwilliams masnwilliams merged commit 6d54518 into main Mar 23, 2026
2 checks passed
@masnwilliams masnwilliams deleted the cli-coverage-update branch March 23, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant