release: 0.19.0#35
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 with only internal/chore changes to a bootstrap script, not a change to kernel API endpoints or Temporal workflows. To monitor this PR anyway, reply with |
|
🧪 Testing To try out this version of the SDK: Expires at: Sun, 31 May 2026 04:57:16 GMT |
b243e2b to
6dac974
Compare
6dac974 to
9f751f0
Compare
9f751f0 to
b9f3593
Compare
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 b9f3593. 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 on DefaultTransport can panic
High Severity
http.DefaultTransport.(*http.Transport) is an unchecked type assertion that will panic at runtime if http.DefaultTransport has been replaced with a custom RoundTripper (commonly done for tracing, instrumentation, or testing). Since defaultHTTPClient() is called from DefaultClientOptions() on every NewClient() invocation, this makes client construction panic-prone. A comma-ok assertion with a fallback (as used by googleapis/google-cloud-go after hitting the same issue) would be safer.
Reviewed by Cursor Bugbot for commit b9f3593. Configure here.
b9f3593 to
6c7cd27
Compare
6c7cd27 to
277c1f0
Compare
277c1f0 to
459e311
Compare


Automated Release PR
0.19.0 (2026-05-01)
Full Changelog: v0.18.0...v0.19.0
Features
Chores
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
Introduces a new default
http.Clientwith a longResponseHeaderTimeoutand adds env-driven custom headers, which can subtly change request behavior across all SDK users if they relied on the previous no-timeout default or static headers.Overview
Updates the SDK release to
0.19.0(manifest,internal/version.go, README pin, changelog, and spec metadata).Changes
DefaultClientOptions()to always use a new defaulthttp.Client(cloned transport +ResponseHeaderTimeout) when callers don’t provide one, and adds support for injecting arbitrary request headers via aHYPEMAN_CUSTOM_HEADERSenv var.Refactors internal encoder/decoder cache keys to store
reflect.Typeas a named field (typ) instead of embedding it, and makes the bootstrap script’sSKIP_BREWcheck robust to unset variables.Reviewed by Cursor Bugbot for commit 459e311. Bugbot is set up for automated code reviews on this repo. Configure here.