Skip to content

chore: Use the 2.x OpenFeature server provider - #20

Merged
kinyoklion merged 1 commit into
mainfrom
devin/1793922308-provider-2x
Nov 6, 2026
Merged

kinyoklion merged 1 commit into
mainfrom
devin/1793922308-provider-2x

Conversation

@kinyoklion

@kinyoklion kinyoklion commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

Summary

The example pinned launchdarkly-openfeature-serverprovider to [1.0.0, 2.0.0), so it stopped tracking the provider once 2.0.0 was released and kept demonstrating 1.2.1. This moves the range to [2.0.0, 3.0.0).

The 2.0.0 breaking change (launchdarkly/openfeature-java-server#66) only adds a start wait option and changes initialization timing; new Provider(sdkKey) and setProviderAndWait used by Main are unchanged, so no source changes are needed.

Verified locally: resolves provider 2.0.0 with Java server SDK 7.17.5 and OpenFeature SDK 1.23.0, and the app prints SDK successfully initialized! and the flag evaluation against a real environment.

Link to Devin session: https://app.devin.ai/sessions/e62d0d55d8cc4419a92354d3af135cb9
Open in Devin Desktop: https://app.devin.ai/desktop/session/e62d0d55d8cc4419a92354d3af135cb9?variant=devin

SDK-3285


Note

Overview
Updates the example’s Gradle dependency so launchdarkly-openfeature-serverprovider resolves [2.0.0, 3.0.0) instead of [1.0.0, 2.0.0), keeping the sample on the current 2.x provider after 2.0.0 shipped.

No application code changes—Main still uses new Provider(sdkKey) and setProviderAndWait, which remain compatible with provider 2.x per the upstream breaking-change notes.

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

Co-Authored-By: Ryan Lamb <4955475+kinyoklion@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration
devin-ai-integration Bot marked this pull request as ready for review November 5, 2026 23:50
@devin-ai-integration
devin-ai-integration Bot requested a review from a team as a code owner November 5, 2026 23:50

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Risk: low. Approved. Cursor Bugbot and Cursor Security Agent completed with no findings that need human review, and the change is a one-line example dependency range bump.

Open in Web View Automation 

Sent by Cursor Approval Agent: SDK - Pull Request Router and Approver

@kinyoklion
kinyoklion merged commit ee77bcc into main Nov 6, 2026
7 checks passed
@kinyoklion
kinyoklion deleted the devin/1793922308-provider-2x branch November 6, 2026 19:25
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