chore: bump SDK deps for EasyJSON removal#380
Draft
aaron-zeisler wants to merge 4 commits into
Draft
Conversation
This was referenced May 19, 2026
Merged
Contributor
Author
|
Just reviewed this pull request: I don't want to include the files in the .cursor/ directory. I'll push a change to remove those files from the PR. |
4787331 to
18487da
Compare
Draft
4 tasks
Updates the three core libraries to their /v4 development pseudo- versions, which ship the v4 majors and remove easyjson: - go-jsonstream: /v3 -> /v4 - go-sdk-common: /v3 -> /v4 - go-server-sdk-evaluation: /v3 -> /v4 Also bumps go-sdk-events/v3 to its new development pseudo-version (stays on /v3 — go-sdk-events itself is not getting a major bump in this cascade) so its transitive easyjson dep is dropped as well. go-server-sdk itself stays on /v7 — only its deps bump. All .go imports of /v3 paths for the three v4 libraries are rewritten to /v4; go-sdk-events imports stay at /v3. Both root go.mod and testservice/go.mod are updated. Squashed from one earlier commit on this branch. Part of the SDK-2113 EasyJSON removal project. Co-authored-by: Cursor <cursoragent@cursor.com>
18487da to
9e2c402
Compare
The easyjson integration has been removed from the SDK. Remove the now-stale documentation section that described how to enable it.
…119/remove-easyjson-build-tag # Conflicts: # client_context_from_config.go # go.mod # ldclient_end_to_end_test.go
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bumps all SDK dependencies to development versions that remove EasyJSON:
go-jsonstream/v3→ feat!: remove easyjson from go-jsonstream go-jsonstream#39go-sdk-common/v3→ feat!: remove easyjson from go-sdk-common go-sdk-common#50go-server-sdk-evaluation/v3→ feat!: remove easyjson from go-server-sdk-evaluation go-server-sdk-evaluation#51go-sdk-events/v3→ fix: bump go-jsonstream and go-sdk-common for EasyJSON removal go-sdk-events#43After this bump,
mailru/easyjsonis completely removed fromgo.mod— it is no longer referenced anywhere in the dependency tree.SDK-2119: EasyJSON Removal — Dependency Chain
Context
Part of SDK-2119. This is a transitive-only change —
go-server-sdkhas no direct EasyJSON code. It just needs its upstream dependencies bumped to versions that no longer carry EasyJSON.Test plan
go build ./...passesgo mod tidycleangrep mailru/easyjson go.mod go.sumreturns emptyvia LD Research 🤖