This commit resolves an issue where queries with different component access expressions were incorrectly identified as incompatible when used together in an event handler. The problem was rooted in the logic that combined the component access expressions: it failed to account for the scenario where multiple queries in a handler could be compatible individually but not when combined.
TODO
[ ] add more tests
[ ] clean up docs
[ ] possibly better naming to clarify whether BoolExpr or ComponentAccessExpr.
Thanks for finding this issue! As discussed on Discord, this PR doesn't quite fix the underlying issue. I've implemented a comprehensive fix in #35. Give that a review if you'd like.
This commit resolves an issue where queries with different component access expressions were incorrectly identified as incompatible when used together in an event handler. The problem was rooted in the logic that combined the component access expressions: it failed to account for the scenario where multiple queries in a handler could be compatible individually but not when combined.
TODO
BoolExpr
orComponentAccessExpr
.