[matthewelwell]

Local evaluation ignores segment overrides that depend on another flag, so it disagrees with remote evaluation.

Closes SDK-8. Depends on Flagsmith/flagsmith-rust-flag-engine#42; merge once it's released as 0.7.0.

Changes

• Overrides on $.flags.<feature>.enabled never matched; they now apply to identity and environment flags.
• Environment flags ignored every segment; they now get overrides from segments that need no identity (e.g. $.environment.name), as in the Java SDK.
• The SDK spec runs v0.7.0, including its local evaluation scenarios.

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

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: 8a53216a-10f8-4802-bdd0-dda00e905d6d

Walkthrough

Both Cargo manifests now pin flagsmith-flag-engine to the same Git revision. A new environment fixture and parameterised integration test check whether the dependent identity flag receives its segment override when the prerequisite flag is enabled, and retains its default state and value when it is disabled.

Priority: Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: Moderate · previous review 45fdf

The dependency is suitable for interim validation, but this PR is not ready to merge until engine 0.7.0 is available and both manifests use it.

- [ ] 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

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: ASSERTIVE
• Plan: Advanced
• Run ID: 86f3538c-fd98-4876-934a-8f967a33db37

📥 Commits

Reviewing files that changed from the base of the PR and between 1d1a2bb and 45fdfc0.

⛔ Files ignored due to path filters: 1

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing: 4

• Cargo.toml
• sdk-spec/Cargo.toml
• tests/fixtures/mod.rs
• tests/integration_test.rs

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

[coderabbitai]

🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Replace the Git dependency before merge.

The release manifest must use flagsmith-flag-engine = "0.7" when engine 0.7.0 is available on crates.io. This Git revision is valid for interim validation but does not satisfy the stated release contract. Update this entry and the matching sdk-spec/Cargo.toml entry before merge.

[khvn26]

Needs rebasing.