sergehuber opened a new pull request, #773: URL: https://github.com/apache/unomi/pull/773
## Summary Backport PR C from `unomi-3-dev`: condition validation, type resolution, and condition standardization (without tracing — deferred to UNOMI-873 / PR D). - Add `ConditionValidationService`, `TypeResolutionService`, and value-type validators; wire them in the services OSGi blueprint. - Extend `ParserHelper` and `ConditionContextHelper` for effective-condition resolution, parameter validation, and parent-chain handling. - Fix runtime condition evaluation in Karaf: restore `DefinitionsService` wiring on ES/OS query-builder dispatchers, parent fallbacks in evaluators/query builders, and `TypeResolutionService`-based rule indexing in `RulesServiceImpl`. - Standardize baseplugin condition JSON metadata; harden evaluators (null-safe, unified numeric property handling). - Add `unomi:list-invalid-objects` dev shell command and unit tests (ParserHelper, ConditionContextHelper, TypeResolutionService, ConditionValidationService). **Intentionally out of scope for this PR:** tracing/explain (PR D), `plugins/past-event` → `advanced-conditions` module move (PR C2), remaining UNOMI-881 service unit tests that depend on tracing (PR E). ## Test plan - [x] `mvn -pl api,persistence-spi,services,plugins/baseplugin,persistence-elasticsearch/core,persistence-opensearch/core,tools/shell-dev-commands -am test` - [x] `mvn -pl itests test` (integration tests pass locally) - [ ] CI green on this PR - [ ] Optional smoke: invalid rule/segment rejected; `unomi:list-invalid-objects` lists unresolved items in Karaf -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
