Skip to content

Expose validated BindingSpace trie cursor contracts - #94

Closed
MesTTo wants to merge 28 commits into
trueagi-io:mainfrom
MesTTo:pr/mork-binding-trie-cursor-contract
Closed

Expose validated BindingSpace trie cursor contracts#94
MesTTo wants to merge 28 commits into
trueagi-io:mainfrom
MesTTo:pr/mork-binding-trie-cursor-contract

Conversation

@MesTTo

@MesTTo MesTTo commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Stack dependency: review after #90.

Adds an explicit cursor contract for replaying the selected BindingSpace trie traversal without materializing the join result.

The stack is split into three commits:

  • expose per-factor cursor contexts through the selected-plan report
  • validate exact ordered factor domains in trace summaries and replay diffs
  • report monotone seek skips from lower-bound cursor movement

The formal LowerBound proof from the local development branch is intentionally excluded and will be proposed separately with the proof suite.

Validation:

  • RUSTFLAGS='-C target-cpu=native -Awarnings' cargo +nightly-2026-06-01 test -p mork --lib binding_space (16 passed)
  • RUSTFLAGS='-C target-cpu=native -Awarnings' cargo +nightly-2026-06-01 test -p mork --lib binding_plan (15 passed)
  • RUSTFLAGS='-C target-cpu=native -Awarnings' cargo +nightly-2026-06-01 test -p mork --lib
  • jscpd --reporters ai --format rust kernel/src/binding_plan.rs kernel/src/binding_space.rs
  • git diff --check

@MesTTo

MesTTo commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

Closing with the rest of the binding_space line: the contract-and-validation shape is exactly what new zipper types should ship with, but here it validates the sidecar's bespoke cursor rather than a PathMap zipper. The rebuild is property tests for a real zipper type against existing PathMap ops, as a small standalone PR.

@MesTTo MesTTo closed this Jul 4, 2026
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