CLI: Update hypeman SDK to 8dd4897f39b16acbdbbfe1d72bfcd15a467719c8 and add new commands/flags - #50
CLI: Update hypeman SDK to 8dd4897f39b16acbdbbfe1d72bfcd15a467719c8 and add new commands/flags#50kernel-internal[bot] wants to merge 7 commits into
Conversation
Keep the CLI on the latest SDK revision after confirming the current command surface already covers the public SDK methods. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Firetiger deploy monitoring skipped This PR didn't match the auto-monitor filter configured on your GitHub connection:
Reason: PR updates a Go SDK dependency in the CLI package, not the kernel API endpoints or Temporal workflows specified in the filter. To monitor this PR anyway, reply with |
Bump the hypeman-go dependency to the latest SDK release and refresh module sums after a full CLI coverage audit found no additional command or flag gaps. Co-authored-by: Cursor <cursoragent@cursor.com>
Bumps the github.com/kernel/hypeman-go dependency to the latest commit. A full enumeration of SDK methods and CLI commands found no coverage gaps. Co-authored-by: Cursor <cursoragent@cursor.com>
Bump github.com/kernel/hypeman-go to the latest generated version. A full enumeration of SDK methods (api.md + param structs) against existing CLI commands found no coverage gaps; all SDK methods and param fields already have corresponding CLI commands and flags. Co-authored-by: Cursor <cursoragent@cursor.com>
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 76e1214. Configure here.
…hold Bump github.com/kernel/hypeman-go to v0.21.0 (commit 8dd4897) and close the coverage gaps that version introduces. Add `hypeman auto-standby hold <instance>` for the new client.Instances.AutoStandby.Hold method, so callers can prevent a candidate-idle instance from entering standby before connecting to it. Both auto-standby subcommands now share one response renderer, which also surfaces the new hold_until field as HOLD UNTIL. Replace the raw-JSON platform workarounds with the now-typed SDK fields: ImageNewParams.Platform and InstanceNewParams.Platform are set directly instead of via option.WithJSONSet, and `image list` reads Image.Platform rather than parsing it back out of the response body. Co-authored-by: Cursor <cursoragent@cursor.com>

This PR updates the Hypeman Go SDK to 8dd4897f39b16acbdbbfe1d72bfcd15a467719c8 and adds CLI commands/flags for new SDK methods.
SDK Update
v0.21.0, which is what the module proxy resolves that commit to)Coverage Analysis
This PR was generated by performing a full enumeration of SDK methods and CLI commands.
All 53 endpoints in
api.mdwere enumerated along with every*Paramsstruct field, then compared against everycli.Commandand flag defined inpkg/cmd/. The only method with no CLI surface wasclient.Instances.AutoStandby.Hold. No endpoints are markedx-cli-skipinopenapi.yaml.Note: the command tree was enumerated from source rather than from
--helpoutput, because urfave/cli v3.3.2 omits theCOMMANDS:section for a command group with a single subcommand. That previously hidauto-standby statusand still hidesresources reclaim-memory; addingholdincidentally makes theauto-standbygroup list its subcommands again.New Commands
hypeman auto-standby hold <instance>forclient.Instances.AutoStandby.Hold()Places a hold that keeps the auto-standby controller from putting an instance into standby, and cancels any queued standby attempt. Intended to be called before opening a connection to a candidate-idle instance.
New Flags
No new flags were required.
--platformalready existed onhypeman run,hypeman image create, andhypeman pull, but was previously sent by rewriting the request body withoption.WithJSONSetbecause the SDK had no typed field. This release addsInstanceNewParams.PlatformandImageNewParams.Platform, so those flags now populate the typed params and theplatformRequestOptionsworkaround is gone.Response Fields
AutoStandbyStatus.HoldUntilis surfaced asHOLD UNTILin the defaultauto-standby statusandauto-standby holdoutput.hypeman image listreads the new typedImage.Platformfor itsPLATFORMcolumn instead of parsingplatformback out of the raw response body.Triggered by: kernel/hypeman-go@8dd4897
Reviewer: @hiroTamada
Note
Medium Risk
Touches instance and image create/pull paths and adds a new auto-standby control API; changes are mostly SDK-aligned refactors plus operational hold behavior before connections.
Overview
Bumps hypeman-go to v0.21.0 and wires new SDK surface into the CLI.
Adds
hypeman auto-standby hold <instance>, which callsInstances.AutoStandby.Holdso clients can block auto-standby (and clear queued standby) before connecting to a possibly idle instance.statusandholdnow sharerunAutoStandbyActionfor resolve-instance, API call, and output; default table output includesHOLD UNTILwhen present.Existing
--platformonrun,image create, andpullnow sets typedImageNewParams.Platform/InstanceNewParams.Platforminstead ofoption.WithJSONSet;image listreadsImage.Platformfor the PLATFORM column viaplatformOrDash. Help text notes platform defaults to the host when omitted.Reviewed by Cursor Bugbot for commit 2aac63f. Bugbot is set up for automated code reviews on this repo. Configure here.