CLI: Update Hypeman Go SDK to 392f55111d188fad255676e90c0c55d7c4b35b73 - #64
Open
kernel-internal[bot] wants to merge 6 commits into
Open
CLI: Update Hypeman Go SDK to 392f55111d188fad255676e90c0c55d7c4b35b73#64kernel-internal[bot] wants to merge 6 commits into
kernel-internal[bot] wants to merge 6 commits into
Conversation
The SDK now accepts Docker-style registry credentials on ImageNewParams, so a private-registry image can be pulled without the server holding credentials for that registry. Expose them as --username/--password/--registry-token on `hypeman image create` and `hypeman pull`. `hypeman push create` already had the same three flags for the outbound push credentials, so both call sites now share one flag set and one builder. A full enumeration of the 61 SDK methods in api.md and their param struct fields against the CLI command tree found no other coverage gaps. Co-authored-by: Cursor <cursoragent@cursor.com>
Bump github.com/kernel/hypeman-go to 7f21c67d750f6dd66c6b6af04e88c710841f2daf, which adds the GET /capabilities resource. Expose it as `hypeman capabilities` so users can discover which runtimes and features a host actually supports instead of hard-coding hypervisor knowledge. Co-authored-by: Cursor <cursoragent@cursor.com>
kernel-internal
Bot
changed the title
Co-authored-by: Cursor <cursoragent@cursor.com> # Conflicts: # pkg/cmd/pushcmd.go
kernel-internal
Bot
changed the title
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
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 b91ebcf. Configure here.
|
|
||
| if format == "auto" || format == "" { | ||
| return showCapabilities(os.Stdout, res) | ||
| } |
There was a problem hiding this comment.
Transform ignored in default format
Medium Severity
handleCapabilities always prints the full showCapabilities table when --format is the default auto, and never applies --transform. The advertised hypeman capabilities --transform runtimes path therefore still dumps the whole summary instead of falling through to ShowJSON.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit b91ebcf. Configure here.
Bumps github.com/kernel/hypeman-go to ea5a7f8d0d069b51e10189a8d0196f6155aaa4a8. A full enumeration of api.md against the pkg/cmd command tree found every SDK method already reachable from the CLI, but the SDK bump adds instance TTL / absolute expiration to InstanceNewParams and InstanceUpdateParams, which had no CLI surface. Expose them as --ttl and --expires-at on `hypeman run` and a new `hypeman update expiration` subcommand, rejecting the mutually exclusive combination and malformed values before the API round trip. Co-authored-by: Cursor <cursoragent@cursor.com>
kernel-internal
Bot
changed the title
Pin the CLI to the tagged v0.26.0 release of github.com/kernel/hypeman-go instead of the pseudo-version for the unreleased ea5a7f8d commit. A full enumeration of api.md methods and their param structs against the CLI command tree found no coverage gaps: all 62 SDK methods and every param field already map to an existing command or flag. Co-authored-by: Cursor <cursoragent@cursor.com>
kernel-internal
Bot
changed the title
Resolves the go.sum conflict by regenerating it against the current module graph, and bumps github.com/kernel/hypeman-go to v0.26.1 (392f55111d188fad255676e90c0c55d7c4b35b73). A full enumeration of api.md methods and their param structs against pkg/cmd found no coverage gaps, so no new commands or flags are needed. Co-authored-by: Cursor <cursoragent@cursor.com>
kernel-internal
Bot
changed the title
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
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 Hypeman Go SDK dependency to the latest version.
SDK Update
v0.26.1)Coverage Analysis
A full enumeration of SDK methods and CLI commands was performed. No coverage gaps were found.
api.mdresolve to a CLI call site.client.Instances.Logsandclient.Builds.Eventsare server-sent-event endpoints and are consumed through their streaming variants (LogsStreaminginhypeman logs,EventsStreaminginhypeman build).AutoStandbyPolicyParam,HealthCheckParam,RestartPolicyParam,SetSnapshotScheduleRequestParam,CreatePushRequestParam, andMemoryReclaimRequestParam) are reachable from an existing CLI flag. TheInstanceNewParamsCredential.Inject/.Hostssub-fields are supplied as part of the JSON payload accepted byhypeman run --credentials-json.openapi.yamlcontains nox-cli-skipmarkers, so no endpoints were excluded.This branch also merges
main(resolving ago.sumconflict by regenerating it against the current module graph).go build ./...,go vet ./..., andgo test ./...all pass.Triggered by: kernel/hypeman-go@392f551
Reviewer: @stainless-app[bot]
Note
Medium Risk
Touches instance lifecycle (expiration) and passes registry secrets on pull/push/create; changes are mostly additive with client-side validation before API calls.
Overview
Bumps hypeman-go to v0.26.1 and exposes several SDK capabilities in the CLI.
Adds
hypeman capabilities, which calls the Capabilities API and prints host/server/runtime/network/image feature info (default table, or--format json/--transform runtimes). README documents the new command.Instance expiration is available via
--ttl/--expires-atonhypeman runandhypeman update expiration, with shared parsing that rejects both flags together and validates Go durations vs RFC3339 (0sclears expiration).Registry credentials (
--username,--password,--registry-token) are centralized inregistrycredentials.goand wired intohypeman pull,hypeman image create, and push (push refactored to use the same helper). Credentials are only sent when at least one value is set.Unit tests cover capabilities formatting, expiration parsing, and credential building.
Reviewed by Cursor Bugbot for commit d9957ff. Bugbot is set up for automated code reviews on this repo. Configure here.