Repository navigation
D-DT64-AUTH: method-authority census with a two-way ratchet; post-merge record for #1926 - #1927
Merged
Merged
Conversation
Hygiene-only: records the merge of #1926 in PR_ARC_INVENTORY and LATEST_STATE. Per the CLAUDE.md termination clause this commit's own merge needs no further record. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X6y3drwKSE2zSgoexheLFX
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
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. |
Committed table crates/lance-graph-r2il/dt64-authority-census.tsv: LENS rows pin each production behaviour lens's DT64 declaration (F000 polyfill, F100 XLOG); LAB rows list the method identities and dispatch paths DT64 does not see (MethodSignature, tract Registry, xlog Word, FnIndex, FoldDialect, procs). The r2il test sweeps crates/*/src for lens mints both ways, replays the polyfill declaration against its row, and fails on a stale exception; the xlog test checks its own row (r2il does not depend on xlog). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X6y3drwKSE2zSgoexheLFX
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X6y3drwKSE2zSgoexheLFX
AdaWorldAPI
pushed a commit
that referenced
this pull request
Nov 2, 2026
…after its merge point Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X6y3drwKSE2zSgoexheLFX
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
D-DT64-AUTH census (plan
d-dt64-method-authority-census-v1.md). The operator decided that DT64 (0xFFFF::0xF###) is the only method authority. This PR adds the census and a two-way ratchet; it migrates nothing.crates/lance-graph-r2il/dt64-authority-census.tsv. Every row has exactly 10 tab-separated columns.LENSrows pin each production behaviour lens's DT64 declaration:POLYFILL_LENS(F000, rev 1, 17 arities) andXLOG_LENS(F100, rev 1, 3 arities).LABrows list the method identities and dispatch paths that DT64 does not see, each with a reason and a way to remove it:MethodSignature, the tract-harvestRegistry, XLOGWord, theFnIndexopcode,FoldDialectandr2il::procs.lance-graph-r2il/tests/integration/dt64_census.rs(6 tests):crates/*/src/**/*.rsfor lens definitions (compose_classid(HARDCODED…) and matches them against theLENSrows in both directions;BindStore;LABtoken still occurs (C2);lance-graph-xlog/tests/dt64_census.rschecks XLOG's own row. It lives in XLOG because R2IL does not depend on XLOG, and this PR adds no dependency.Also in this PR: the post-merge record for #1926 (merged
ade765d6f) in PR_ARC_INVENTORY and LATEST_STATE.Findings (written up in board entry
2026-11-02-d-dt64-auth-census-shipped):src/never declares the polyfill lens. Only tests and consumers do, and the executor admits words against that declaration. The row therefore pinspolyfill_arities().r2il::procsis reached from production code (lance-graph-nal) without going through admission. It is listed asLAB, to be removed once OGAR hasPROC_DEF/PROC_CALL.Limits:
compose_classid(HARDCODED…. A behaviour lens written as a raw literal is not detected.Test Plan
cargo test -p lance-graph-r2il --test integration dt64_census: 6/6 pass.cargo test -p lance-graph-xlog --test dt64_census: 1/1 passes.-D warningsand fmt are clean on both crates.polyfill.rs: the two-way sweep, the C1 test and the silence test fail.LABtoken:every_lab_exception_still_deviatesand the silence test fail.polyfill_arities()without updating the row: the declaration replay fails.Not to be merged automatically.
🤖 Generated with Claude Code
https://claude.ai/code/session_01X6y3drwKSE2zSgoexheLFX