Skip to content

Refine SPARQL lowering for required outer filter bindings#64

Merged
flyon merged 1 commit intodevfrom
codex/required-filter-bindings
Apr 6, 2026
Merged

Refine SPARQL lowering for required outer filter bindings#64
flyon merged 1 commit intodevfrom
codex/required-filter-bindings

Conversation

@flyon
Copy link
Copy Markdown
Member

@flyon flyon commented Apr 6, 2026

Summary

  • promote top-level null-rejecting filter bindings to required triples in selectToAlgebra()
  • preserve optional lowering for different-property OR, inline traversal filters, EXISTS, and aggregate HAVING paths
  • add algebra, golden SPARQL, and Fuseki regression coverage plus updated lowering docs and a report

Validation

  • npm test -- --runInBand --runTestsByPath src/tests/sparql-algebra.test.ts src/tests/sparql-select-golden.test.ts src/tests/sparql-fuseki.test.ts
  • npm test -- --runInBand

Notes

  • includes a patch changeset for @_linked/core
  • report: docs/reports/013-required-filter-bindings.md

@flyon flyon merged commit 7be6e9b into dev Apr 6, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant