release: 0.52.0#98
Conversation
|
Firetiger deploy monitoring skipped This PR didn't match the auto-monitor filter configured on your GitHub connection:
Reason: This is an automated release PR for the Go SDK, not a change to kernel API endpoints or Temporal workflows in the kernel repo. To monitor this PR anyway, reply with |
|
🧪 Testing To try out this version of the SDK: Expires at: Fri, 29 May 2026 14:37:44 GMT |
stainless-app
Bot
force-pushed
the
release-please--branches--main--changes--next
branch
from
c8be7d0 to
4d8d254
Compare
![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 4d8d254. Configure here.
| // and adds a [http.Transport.ResponseHeaderTimeout] so stuck connections | ||
| // fail fast instead of compounding across retries. | ||
| func defaultHTTPClient() *http.Client { | ||
| transport := http.DefaultTransport.(*http.Transport).Clone() |
There was a problem hiding this comment.
Unchecked type assertion may panic on initialization
Medium Severity
The expression http.DefaultTransport.(*http.Transport) is an unchecked type assertion that will panic if http.DefaultTransport has been replaced with a non-*http.Transport implementation (e.g., by test mocking libraries like httpmock or gock, or custom middleware). Since this runs during NewClient() initialization, the panic is unrecoverable. The rest of the codebase (e.g., rawcurl.go) safely uses http.DefaultTransport via the RoundTripper interface without asserting the concrete type. Using the comma-ok form of the assertion would allow a graceful fallback.
Reviewed by Cursor Bugbot for commit 4d8d254. Configure here.
|
🤖 Release is at https://github.com/kernel/kernel-go-sdk/releases/tag/v0.52.0 🌻 |
1 participant
Automated Release PR
0.52.0 (2026-04-29)
Full Changelog: v0.51.0...v0.52.0
Features
This pull request is managed by Stainless's GitHub App.
The semver version number is based on included commit messages. Alternatively, you can manually set the version number in the title of this pull request.
For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request.
🔗 Stainless website
📚 Read the docs
🙋 Reach out for help or questions
Note
Medium Risk
Changes the SDK’s default HTTP client/transport behavior for all consumers, which can alter timeout/failure modes in production. Header injection via env is straightforward but could cause unexpected request differences if misconfigured.
Overview
Release v0.52.0: bumps package/version metadata and updates the README install snippet and changelog.
The client now uses a new default
http.Client(cloned transport with aResponseHeaderTimeout) when callers don’t provide one, andDefaultClientOptions()can additionally apply request headers fromKERNEL_CUSTOM_HEADERS(newline-separatedKey: Valuepairs). A small docstring update clarifiesProfiles.Download()returns a zstd-compressed tar of the user-data directory.Reviewed by Cursor Bugbot for commit 4d8d254. Bugbot is set up for automated code reviews on this repo. Configure here.