Isn't lint/complexity/useSimplifiedLogicExpression adding more complexity?

[raky291]

I've noticed that Biome's useSimplifiedLogicExpression suggestion refactors !loading && !user into !(loading || user).

Although both expressions are logically identical, I find the second one more difficult to read in practice. The original version allows us to check the loading and user states as two distinct and sequential conditions.

In contrast, the suggested change requires us to first mentally combine the variables and then negate the entire group, which increases the cognitive load. I believe prioritizing clarity and straightforward readability is more beneficial than reducing the number of operators. What do you think?

[ematipico]

You aren't the first one to make this question (I believe there are some issues or discussions on the topic).

While your argument and others are valid (for our brain is simpler), the rule simply applies the De Morgan's law

[raky291]

Got it, that makes total sense now. Thanks for getting back to me! Maybe we could add a quick note about De Morgan’s law in the rule description? I think it would be a nice touch for clarity.