source-academy / py-slang / 30009329634

86%

push

github

Fix CSE machine loop-increment label and tag expression statements (#340)

generateForIncrement's synthetic Assign node (the per-iteration loop
variable update) had synthetic tokens, so serializeControlItem fell back
to the generic "assign" KIND_LABEL instead of showing which variable was
set to what. It now carries a syntheticLabel of "x = <value>".

Fixes #293.

Also tag SimpleExpr control items with `tag: "expressionStatement"` so
the visualizer can render them distinctly. Currently an expression
statement like "1 + 2" is indistinguishable from a plain sub-expression
until a "pop" mysteriously appears on a later step to discard its value.

Fixes #270 (frontend rendering handled separately in source-academy/frontend#4159).

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: Martin Henz <henz@nus.edu.sg>

4395 of 5492 branches covered (80.03%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 2 files covered. (100.0%)

9516 of 10658 relevant lines covered (89.29%)

178611.99 hits per line