StoneCypher / jssm / 0b444a40187877a41192df44c225b0d95d47cdd8 / 9

100%

master: 100%

push

github

fix(grammar): four bugs flagged in the grammar reference

Four small bugs surfaced by the cataloguing pass in
notes/fsl-grammar-reference.md, all now repaired:

- `OctalDigit` was `[0-1]` (copy-paste from `BinaryDigit`); now `[0-7]`,
  so `0o27` and friends parse correctly.
- `MachineReference` was defined but missing from `MachineAttribute`'s
  alternation, so `machine_reference: foo;` produced a parse error
  despite the AST node being constructed.  Wired up.
- The orphan `Whitespace` rule (no callers) was removed.
- `SdStateLabel "color"` corrected to `SdStateLabel "label"`, so parse
  errors on a malformed state-block `label:` field name the right
  field instead of saying "expected color".

Adds src/ts/tests/grammar_regressions.spec.ts pinning down each fix
with positive, negative, and edge-case tests (19 tests total).

Tooling: a fixparser pass that marks pegjs's generated `error()` and
`expected()` `location` parameter optional, so grammar action blocks
can use the one-arg form without strict-TS errors.  Also adds a
gitignored project-local .npmrc for Windows script-shell setup.

826 of 826 branches covered (100.0%)

Branch coverage included in aggregate %.

7599 of 7599 relevant lines covered (100.0%)

850.21 hits per line