Handle deoptimizations while a node is being included.

[lukastaegert]

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes (bugfixes and features will not be merged without tests)
• no

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

List any relevant issue numbers:
Resolves #3324

Description

This fixes an issue with logical and conditional expressions that can arise when an expression that contains a variable reassignment is automatically included without a preceding hasEffects check.

The issue is quite complicated to trigger (see #3324) while the fix was in essence to switch the order of two lines.

[codecov]

Codecov Report

Merging #3327 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3327   +/-   ##
=======================================
  Coverage   93.18%   93.18%           
=======================================
  Files         172      172           
  Lines        6042     6042           
  Branches     1801     1801           
=======================================
  Hits         5630     5630           
  Misses        221      221           
  Partials      191      191

Impacted Files	Coverage Δ
src/ast/nodes/ConditionalExpression.ts	96.15% <ø> (ø)	⬆️
src/ast/nodes/LogicalExpression.ts	98.7% <ø> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35e127c...fc0edd3. Read the comment docs.