CLI: Update Hypeman Go SDK to 12737aa4d289754ed9205d2fb49b17de4689b305#47
CLI: Update Hypeman Go SDK to 12737aa4d289754ed9205d2fb49b17de4689b305#47kernel-internal[bot] wants to merge 4 commits intomainfrom
Conversation
Refresh the CLI to the latest hypeman-go revision after a full SDK/CLI coverage audit confirmed there are no new user-facing commands or flags to add. Made-with: Cursor
…nd add standby compression delay flags Bump the CLI to the latest hypeman-go release and wire through the new standby compression delay settings so the existing run and standby flows stay aligned with the SDK surface. Made-with: Cursor
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 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 08a550e. Configure here.
| compression.Algorithm = parsedAlgorithm | ||
| } | ||
| params.Compression = compression | ||
| request.Compression = compression |
There was a problem hiding this comment.
Missing nested guard sends unintended compression config
Medium Severity
In lifecycle.go, the SnapshotCompressionConfigParam is unconditionally built inside the outer if block. When a user passes only --compression-delay, the compression config is still created with Enabled: true (via the fallback on line 170), even though the user never requested compression. In contrast, run.go correctly uses a nested if to guard the compression config creation, only building it when snapshot-compression-enabled, snapshot-compression-algorithm, or snapshot-compression-level is set. The lifecycle.go code is missing this same nested guard, causing unintended side effects.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 08a550e. Configure here.
Bump the CLI to the latest hypeman-go SDK so it stays aligned with the generated API surface. A full SDK and CLI coverage pass found no new command or flag gaps to add in this update. Made-with: Cursor
kernel-internal
Bot
changed the title
This PR updates the Hypeman Go SDK dependency to the latest version.
SDK Update
Coverage Analysis
A full enumeration of SDK methods and CLI commands was performed. No coverage gaps were found.
Triggered by: kernel/hypeman-go@12737aa
Reviewer: @Sayan-
Note
Medium Risk
Moderate risk due to SDK upgrade and changes to how standby/snapshot compression parameters are serialized into API requests, which could alter runtime behavior for standby and new instance policies.
Overview
Updates
github.com/kernel/hypeman-gofromv0.17.0tov0.18.1-...(plusgo.sum), aligning the CLI with newer SDK request shapes.Extends
hypeman standbywith--compression-delayand wires compression options through a newStandbyInstanceRequestParamnested underInstanceStandbyParams. Extendshypeman runwith--snapshot-compression-delay, mapping toSnapshotPolicyParam.StandbyCompressionDelay, and refactors snapshot policy construction to allow setting delay independent of compression enable/level/algorithm.Reviewed by Cursor Bugbot for commit 1b03594. Bugbot is set up for automated code reviews on this repo. Configure here.