Skip to content

Update SDK and support us-west placement - #284

Merged
sjmiller609 merged 1 commit into
mainfrom
hypeship/cli-us-west
Oct 7, 2026
Merged

sjmiller609 merged 1 commit into
mainfrom
hypeship/cli-us-west

Conversation

@sjmiller609

@sjmiller609 sjmiller609 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Bump github.com/kernel/kernel-go-sdk from v0.117.0 to v0.120.0 and update its checksums.
  • Add us-west to the shared region validator and existing region flag help. No new commands or unrelated SDK feature exposure.
  • Cover browser/pool create and list forwarding, browser request serialization, and surfacing a disabled-region HTTP 403 without changing the requested region.

Defaults and server-side region authorization are unchanged. Recognizing us-west in the CLI does not enable it for an organization.

Validation

  • GOTOOLCHAIN=go1.25.0 go test ./... passed.
  • Focused browser/pool region and disabled-region tests passed with -race.
  • GOTOOLCHAIN=go1.25.0 go vet ./..., make build, and git diff --check passed.
  • Full go test -race ./... reports pre-existing pterm spinner races in template-creation tests. Reproduced TestCreateCommand's race on unmodified main; these files and dependencies are unchanged.

52 additions / 11 deletions across seven files. No API, feature-flag, or deployment changes.


Note

Low Risk
CLI validation and SDK bump only; region enablement and defaults remain enforced by the API.

Overview
Adds us-west as a supported --region value across browser sessions, browser pools, and auth-connection browser settings by extending the shared region validator and updating related flag help text.

Bumps github.com/kernel/kernel-go-sdk from v0.117.0 to v0.120.0 so create/list calls can forward the new region to the API. Tests cover us-west mapping for browser and pool list/create, plus surfacing a region_not_enabled HTTP 403 when the org has not enabled that region (CLI still accepts the flag; authorization stays server-side).

Reviewed by Cursor Bugbot for commit ae42f5b. Bugbot is set up for automated code reviews on this repo. Configure here.

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedgolang/​github.com/​kernel/​kernel-go-sdk@​v0.117.0 ⏵ v0.120.073 +1100100100100

View full report

@sjmiller609
sjmiller609 marked this pull request as ready for review October 7, 2026 15:29
@sjmiller609
sjmiller609 requested a review from chruffins October 7, 2026 15:29
@sjmiller609
sjmiller609 merged commit 3b5ccb5 into main Oct 7, 2026
8 checks passed
@sjmiller609
sjmiller609 deleted the hypeship/cli-us-west branch October 7, 2026 15: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.

2 participants