comunica / traqula / 33636991362

100%

main: 100%

push

github

Fix toAst hoisting EXTEND out of GRAPH (task.md entry 1)

registerProjection (core.ts) no longer treats GRAPH as transparent
to projection scope, so a BIND inside GRAPH ?g {...} renders as an
inline BIND instead of getting hoisted into the outer SELECT list.

That surfaced a second, previously-masked bug: removeAlgQuadsRecursive
(quads.ts), which reconstructs an explicit Graph node from "quads
mode" .graph-tagged BGPs, deferred picking where to place that Graph
node too far up the tree - through EXTEND, GROUP, and ORDER_BY - all
the way to the nearest PROJECT/SERVICE boundary. Combined with the
registerProjection fix, this misplaced solution modifiers (aggregate
aliases, GROUP BY, subquery SELECTs) as if they were inside the
GRAPH, and demoted genuine SELECT expressions to inline BINDs.

Fixed by stopping the deferral at GROUP/ORDER_BY (always external
state) and at EXTEND specifically when in projection scope, computed
by threading a projectionScope flag through the recursion that
mirrors registerProjection's own c.project logic (an EXTEND's
treatment depends on context: a genuine SELECT alias vs. a BIND under
CONSTRUCT or inside a FILTER's EXISTS). FILTER and the multi-branch
combinators still defer deliberately, so sibling patterns sharing one
graph still merge into a single GRAPH block.

Also corrected task.md's own entry 1, which claimed no lib/ changes
were needed and pointed at a pullUpExtends/agent-task.md that don't
exist in this repo.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SyvR9PGgW192nJmaEpTkNq

794 of 794 branches covered (100.0%)

Branch coverage included in aggregate %.

14 of 14 new or added lines in 3 files covered. (100.0%)

5597 of 5597 relevant lines covered (100.0%)

295186.98 hits per line