[linear-code]

Local evaluation matched Matches regex segment conditions anywhere in a trait value. Per the Flagsmith docs, patterns must match from the start of the value without having to cover all of it. So prod should match production but not nonproduction. Identities could previously land in segments they don't belong to under local evaluation. The engine now uses Matcher.lookingAt() in place of find(). matches() would have wrongly required a full-string match.

Changes

• Regex segment conditions match only from the start of the trait value, for every alternative (prod|dev matches development but not nondev).
• Patterns that use ^…$ still require a full-value match. Matching stays case-sensitive, and invalid patterns still never match.
• Regression cases taken from the docs' examples.

Closes SDK-25

Review effort: 1/5

[linear-code]

SDK-25

[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: dd15e9d7-c140-4613-936f-80c4cc811de7

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

CodeRabbit Verify

• Run CodeRabbit Verify to test the changes in this pull request.

[linear-code]

@coderabbitai review

[codspeed]

Merging this PR will not alter performance

✅ 15 untouched benchmarks

---

_{Comparing linear/sdk-25-standardise-regex-matching-in-the-java-server-side-sdk-6166 (a339909) with main (82ea4c5)}