1
65%
main: 65%

Ran 19 Mar 2026 04:02PM UTC

Files 111

Run time 4s

Badge

Embed ▾

Committed 19 Mar 2026 03:46PM UTC coverage: 66.142% (+1.0%) from 65.19%

Job # 23304084914.1

Build Type

push

Committed by

reid-spencer

Commit Message

Fix BAST deserialization bugs for LetStatement and MatchStatement

Two reader-side fixes that caused byte misalignment during BAST
deserialization, manifesting as failures on large models like
reactive-bbq (4428 nodes):

1. LetStatement: Added missing readU8() to consume the
   NODE_PATH_IDENTIFIER tag before readPathIdentifierNode() when
   reading the optional TypeRef added in 1.16.0. The writer uses
   writePathIdentifier() which includes the tag byte, but the
   reader was not consuming it — causing 1-byte drift.

2. MatchStatement: Changed reader to match the writer's byte
   layout where all case headers (loc + pattern + count) are
   written first, followed by all case items sequentially, then
   default items. The reader was incorrectly interleaving each
   case's header with its items.

Also removes pre-populated keywords from StringTable (90 entries
that were always serialized regardless of usage, ~5-85% size
reduction) and adds regression tests for epics and typed let
statements. Removes the reactive-bbq pending marker — all 187
riddl-models now pass round-trip testing.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

6802 of 10284 relevant lines covered (66.14%)

0.66 hits per line

ossuminc / riddl / 23304084914 / 1
65%
main: 65%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 23304084914.1

ossuminc / riddl / 23304084914 / 1 65% main: 65%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 23304084914.1

ossuminc / riddl / 23304084914 / 1
65%
main: 65%

README BADGES
x