Release v1.10.0 · jgstew/pre-commit-bigfix

This release adds two errors to bes-actionscript-validate-script for parameter lines that fail at runtime. Both are auto-fixed.

⚠️ Upgrade notes

bes-actionscript-validate-script can now fail where it used to pass, through the new E527 and E528 errors.
With --auto-fix on (the default when files are given), the hook will now rewrite parameter lines. It quotes unquoted values and moves trailing comments onto their own line. Review the diff after the first run.
To keep a specific line as-is, use the existing actionscript-parameter-ok marker, which now covers both codes. To turn a code off everywhere, use --disable E527 or --disable E528.

E527, a new error: a parameter "<name>" = <value> whose value is not double-quoted. The agent substitutes {...} first and then parses the line, so an unquoted value fails at runtime, even a singular one such as {"a"}. Auto-fixed by wrapping the value in quotes (" in an entity-escaped body). A value with a " outside {...}, or with %22, is reported but not fixed.
E528, a new error: a trailing // comment on a parameter line. Only fixed-syntax lines (if/elseif/else/endif, the prefetch block, override) accept one, so the line fails at runtime even when its value is quoted. Auto-fixed by moving the comment onto its own line directly above. This fix runs before the E527 fix, so an unquoted value with a comment gets both fixes in one run.
Both fixes are checked against BES.xsd before writing, like the other auto-fixes (E526).

Full Changelog: v1.9.3...v1.10.0