microsoft / RulesEngine / 26523296308

95%

master: 97%

push

github

Fix #576, #519, #696, and add regression guards for #590, #608, #581, #606 (#729)

Code fixes:

(common in custom actions that don't need config). Added null guards for
both the context dict itself and each entry's value.

ErrorMessage template was never interpolated into ExceptionMessage on the
result tree (unlike ExecuteAllRulesAsync which formats correctly). Added
the call.

`$(input1.Inner.Name)` produced the raw JSON of `Inner` (`{"Name":"..."}`)
instead of walking to the leaf. Rewrote UpdateErrorMessage to traverse the
full path via JsonNode and emit the leaf scalar (unquoted) or nested JSON.

Verified-already-fixed (regression tests added):

run was fixed in commit 4ca3cc2 (#592). Test guards against regression.

no longer reproduces — likely fixed by FastExpressionCompiler / Dynamic.Core
version bumps. Test guards against regression.

no longer reproduces, likely resolved by the AutoRegisterInputType work.
Test guards against regression.

mis-resolved on master — likely fixed by Dynamic.Core 1.4.3 → 1.6.7 bump.
Test guards against regression.

Not fixed in this PR:

decision rather than a bug — ActionContext is documented as a static
key/value config dict. Users who want evaluation should use OutputExpression.

All 142 unit tests pass on net6 / net8 / net9 / net10.

Co-authored-by: Yogesh Prajapati <yogeshcprajapati@outlook.com>

451 of 558 branches covered (80.82%)

27 of 29 new or added lines in 2 files covered. (93.1%)

828 of 867 relevant lines covered (95.5%)

544.33 hits per line