comunica / traqula / 36433318550

100%

main: 100%

Pull #199

github

Fix ?type VALUES bindings in toAst and canonicalizer output

removeAlgQuadsRecursive took any object with a truthy `type` for an
operation, so a VALUES binding for `?type` got rebuilt, and a `?__proto__`
value in that binding was lost. Operations are now recognized by a string
`type`.

Canonicalizer callbacks still returned the `{ result, recurse }` shape of
sparqlalgebrajs, so patterns and paths came back wrapped. They now return
the new operation and do not iterate into their already handled terms.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

869 of 869 branches covered (100.0%)

Branch coverage included in aggregate %.

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

5840 of 5840 relevant lines covered (100.0%)

342782.13 hits per line