[flagsmithsmith]

Contributes to Flagsmith/sdk-spec#13

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: ASSERTIVE
• Plan: Advanced
• Run ID: 7c342a21-742c-4416-acc2-65ffdd7df296

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• Trigger review

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[flagsmithsmith]

Re-applied 0c48af8 as 29cbe7b (test(sdk-spec): Run the experimentation steps against the SDK).

Why CI was red: this PR adds events to the spec tags. All five @events scenarios failed with "The SDK does not implement the step: Create the SDK client with events enabled". The steps file had no wiring for events yet.

What changed: only sdk-spec/Steps.java. The cherry-pick conflicted with the local evaluation client that main added since. I kept both client constructors. The steps wire to SDK features that already exist (#226):

• Create the SDK client with events enabled → FlagsmithConfig.withEnableEvents(true).eventsUri(...)
• the experiment flag, track event and track exposure steps → getExperimentFlag, trackEvent and trackExposureEvent

No library change was needed. v0.7.1's only new spec content is the empty-string tagline flag value, and the SDK already passed those scenarios in the previous run.

Verified locally: I ran the v0.7.1 runner against the spec's compose stack with core or events or local-evaluation. All 9 scenarios passed.

The red CodSpeed Macro Runners check is CodSpeed's own arm64 runner outage, not this change.

[codspeed]

Merging this PR will not alter performance

✅ 15 untouched benchmarks

---

_{Comparing openhands/sdk-spec-v0.7.1 (29cbe7b) with main (e278fd5)}