Conversation
|
🧪 Testing To try out this version of the SDK: Expires at: Wed, 06 May 2026 18:48:59 GMT |
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.
| } | ||
| func (r *AuthConnectionNewParams) UnmarshalJSON(data []byte) error { | ||
| return json.Unmarshal(data, &r.ManagedAuthCreateRequest) | ||
| return apijson.UnmarshalRoot(data, r) |
There was a problem hiding this comment.
Untagged fields silently ignored during JSON unmarshaling
High Severity
The UnmarshalJSON methods were changed from json.Unmarshal(data, &r.InnerField) to apijson.UnmarshalRoot(data, r), but the inner fields (e.g. ManagedAuthCreateRequest, CreateCredentialRequest, SubmitFieldsRequest, etc.) have no JSON struct tags. The apijson struct decoder explicitly skips fields without JSON tags, so these primary data fields will never be populated during unmarshaling. The MarshalJSON still serializes the inner field's contents directly, making marshal/unmarshal asymmetric — data serialized by MarshalJSON cannot be recovered by UnmarshalJSON.
Additional Locations (2)
61be6d5 to
540837a
Compare
540837a to
b83cf3c
Compare
b83cf3c to
e5584c3
Compare
|
🤖 Release is at https://github.com/kernel/kernel-go-sdk/releases/tag/v0.46.0 🌻 |


Automated Release PR
0.46.0 (2026-04-06)
Full Changelog: v0.45.0...v0.46.0
Features
Bug Fixes
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
Medium risk because it changes the public
Proxies.Checkmethod signature to require a request body and adjusts JSON unmarshaling for several param wrappers, which could break compilation or subtly change request/response decoding behavior.Overview
Bumps the SDK release to v0.46.0 (manifest,
internal/version.go, README install snippet, changelog, and OpenAPI metadata).Updates the
POST /proxies/{id}/checkclient API to accept a newProxyCheckParamsbody with an optionalurlto test reachability, and updates docs/tests accordingly.Fixes param wrapper unmarshaling by switching several
*Paramstypes (auth connections, credentials, credential providers) fromencoding/json.Unmarshalinto nested fields toapijson.UnmarshalRoot, addressing edge-case decoding failures.Reviewed by Cursor Bugbot for commit e5584c3. Bugbot is set up for automated code reviews on this repo. Configure here.