Skip to content

CLI: Update hypeman SDK to 8dd4897f39b16acbdbbfe1d72bfcd15a467719c8 and add new commands/flags - #50

Open
kernel-internal[bot] wants to merge 7 commits into
mainfrom
cli-coverage-update
Open

CLI: Update hypeman SDK to 8dd4897f39b16acbdbbfe1d72bfcd15a467719c8 and add new commands/flags#50
kernel-internal[bot] wants to merge 7 commits into
mainfrom
cli-coverage-update

Conversation

@kernel-internal

@kernel-internal kernel-internal Bot commented May 12, 2026

Copy link
Copy Markdown
Contributor

This PR updates the Hypeman Go SDK to 8dd4897f39b16acbdbbfe1d72bfcd15a467719c8 and adds CLI commands/flags for new SDK methods.

SDK Update

  • Updated hypeman-go to 8dd4897f39b16acbdbbfe1d72bfcd15a467719c8 (released as 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.md were enumerated along with every *Params struct field, then compared against every cli.Command and flag defined in pkg/cmd/. The only method with no CLI surface was client.Instances.AutoStandby.Hold. No endpoints are marked x-cli-skip in openapi.yaml.

Note: the command tree was enumerated from source rather than from --help output, because urfave/cli v3.3.2 omits the COMMANDS: section for a command group with a single subcommand. That previously hid auto-standby status and still hides resources reclaim-memory; adding hold incidentally makes the auto-standby group list its subcommands again.

New Commands

  • hypeman auto-standby hold <instance> for client.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. --platform already existed on hypeman run, hypeman image create, and hypeman pull, but was previously sent by rewriting the request body with option.WithJSONSet because the SDK had no typed field. This release adds InstanceNewParams.Platform and ImageNewParams.Platform, so those flags now populate the typed params and the platformRequestOptions workaround is gone.

Response Fields

  • AutoStandbyStatus.HoldUntil is surfaced as HOLD UNTIL in the default auto-standby status and auto-standby hold output.
  • hypeman image list reads the new typed Image.Platform for its PLATFORM column instead of parsing platform back 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 calls Instances.AutoStandby.Hold so clients can block auto-standby (and clear queued standby) before connecting to a possibly idle instance. status and hold now share runAutoStandbyAction for resolve-instance, API call, and output; default table output includes HOLD UNTIL when present.

Existing --platform on run, image create, and pull now sets typed ImageNewParams.Platform / InstanceNewParams.Platform instead of option.WithJSONSet; image list reads Image.Platform for the PLATFORM column via platformOrDash. 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.

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-agent

Copy link
Copy Markdown

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

Any PR that changes the kernel API. Monitor changes to API endpoints (packages/api/cmd/api/) and Temporal workflows (packages/api/lib/temporal) in the kernel repo

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 @firetiger monitor this.

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>
@kernel-internal kernel-internal Bot changed the title CLI: Update Hypeman Go SDK to 664416bf5a15d0376f551cabcce510cdcf67c0e0 CLI: Update Hypeman Go SDK to 3cb2ab28e6ded55636db55b0b560b60044b7ea71 May 18, 2026
kernel-internal Bot and others added 2 commits June 10, 2026 02:35
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>
@kernel-internal kernel-internal Bot changed the title CLI: Update Hypeman Go SDK to 3cb2ab28e6ded55636db55b0b560b60044b7ea71 CLI: Update Hypeman Go SDK to a58cbbbad6e7860d53d9e93751faca80db15064b Jun 10, 2026
kernel-internal Bot and others added 2 commits July 22, 2026 15:42
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>
@kernel-internal kernel-internal Bot changed the title CLI: Update Hypeman Go SDK to a58cbbbad6e7860d53d9e93751faca80db15064b CLI: Update Hypeman Go SDK to d9ebe64be9ec751fc8a32186c95bdbc68fb30d86 Jul 22, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 using high effort 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.

Reviewed by Cursor Bugbot for commit 76e1214. Configure here.

Comment thread go.mod Outdated
…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>
@kernel-internal kernel-internal Bot changed the title CLI: Update Hypeman Go SDK to d9ebe64be9ec751fc8a32186c95bdbc68fb30d86 CLI: Update hypeman SDK to 8dd4897f39b16acbdbbfe1d72bfcd15a467719c8 and add new commands/flags Jul 28, 2026
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.

0 participants