cliffano / jazz / 28692078851

100%

main: 100%

Pull #17

github

Fix foreach regression: process escape sequences in scanner echo state

The security fix in d1db791 added backslash escaping in _compileExpr for
ast.Str nodes, which inadvertently broke template text that used \n and
similar escape sequences. Previously, \n in template text was stored as
two chars (backslash + n) and compiled to "\n" in generated JS (a
newline). After the fix, the backslash was double-escaped to \\n,
outputting literal \n instead of a newline.

Fix: add escape sequence processing to _echoState() in scanner.js so
that \n, \r, \t, and \\ in template echo text are converted to their
actual character values. This ensures the compiler's escaping correctly
round-trips them to the proper JS escape sequences in generated code.

403 of 404 branches covered (99.75%)

Branch coverage included in aggregate %.

1 of 22 new or added lines in 1 file covered. (4.55%)

1805 of 1826 relevant lines covered (98.85%)

77.02 hits per line