Repository navigation
lab(dir-sim): hybrid ordinal chain v2 — no consumer lookup, DR frame witness, zero hot allocations - #1964
Conversation
…h DR frames The hybrid ordinal-chain lab no longer scans Exchange Online at bind time: it reads the sole Entra->EXO row from ogar_dir_core::Output::entra_mailbox. The lab-local value Stamp is replaced by lance_graph_dt64::dr::FrameStore: a bound coordinate carries the owner-issued Frame and every read checks it, so a foreign store or a re-seal at equal revision numbers is refused. New falsifiers: every row against the independent oracle, foreign frame and re-seal, a unique match turned ambiguous after a source change, missing backsync, and zero heap allocations on the hot read and fold path. CI now runs clippy on the lab as well. Requires the paired OGAR change (Output::entra_mailbox). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMMJzKZrYnckXRC8x2U
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMMJzKZrYnckXRC8x2U
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
|
Exchange dir-sim fidelity is red with This is the paired-change dependency this PR states, not a defect in this PR. The job checks out OGAR A re-run cannot go green before #371 merges, so I have not re-run the job. Once #371 is on OGAR Generated by Claude Code |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMMJzKZrYnckXRC8x2U
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMMJzKZrYnckXRC8x2U
Summary
Follow-up to merged #1962. Requires AdaWorldAPI/OGAR#371 (
Output::entra_mailbox). This PR's CI checks out OGARmain, so it stays red until #371 merges.Stampis gone. A bound coordinate now carries alance_graph_dt64::dr::Frame, issued by aFrameStore(revisions: source, relation_map, coordinates), and every read callsFrameStore::check. A frame cannot be built from numbers, so a foreign store or a re-seal at equal revisions is refused; the old value stamp treated those as the same coordinate. The identity cross-check stays as a second line of defence, for an owner that forgets to advance.SourceChanged) and rebinding refuses instead of choosing the first mailbox;Measured (release, two-row fixture; reported, not gated)
SumI32over a borrowed laneDisable runs (each committed first, then broken, then restored)
entra_mailbox == NO_ROWrefusal aloneNot proved here
RouteVersion).FrameStoreandBindStorerun on separate clocks in this lab.Generated by Claude Code