Bug:2009480: [ci full] Evaluate jexl with returned results as json#7156
Merged
yashikakhurana merged 1 commit intomainfrom Jan 13, 2026
Merged
Bug:2009480: [ci full] Evaluate jexl with returned results as json#7156yashikakhurana merged 1 commit intomainfrom
yashikakhurana merged 1 commit intomainfrom
Conversation
07ff660 to
23cd80c
Compare
7 tasks
f9cc0ec to
1656b2f
Compare
1656b2f to
8149371
Compare
8149371 to
ca2e3be
Compare
- Added eval_jexl_detailed() method to NimbusTargetingHelper interface in UDL
- Implemented method in Rust to evaluate JEXL expressions and return detailed JSON results
- Returns {"success": true, "result": <value>} on success or {"success": false, "error": "..."} on error
- Added Android Kotlin implementations in NimbusHelpers.kt
- Created 5 comprehensive tests with common helper function
- Updated CHANGELOG.md with breaking change warning
This is a breaking change that requires consumers to implement the new interface method.
iOS implementation: mozilla-mobile/firefox-ios#31607
freshstrangemusic
approved these changes
Jan 13, 2026
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.
Add
eval_jexl_detailed()method toNimbusTargetingHelperfor CLI testing and debugging. Evaluates JEXL expressions and returns detailed results as JSON.Pull Request checklist
[ci full]to the PR title.ios pr- mozilla-mobile/firefox-ios#31607