Repository navigation
probe(dr-math): guarded theorem/compiled-recipe identities across DR classes [3/5] - #1703
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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. |
Coresearch D-DRMC item X15 (operator-chosen) and X17 for the recipe fixtures. - ProofDisposition (a caller-set "checked" flag) and the caller-supplied proof digest are gone. ProofReference has private fields and one constructor, rechecked(law, ReplayCapsule), which requires the capsule to prove law.statement (Stage B's SemanticKey) and re-runs ReplayCapsule::recheck. A forged nonzero digest used to pass; it can no longer be written (compile_fail doctest), and a real proof of a different statement is ProofOfOtherStatement. - CompiledReference carries Stage B's TargetAbi instead of a bare u32. - Recipe fixtures bind non-zero registers. - dr-math-binding-probe now depends on egglog-r2il-probe (egglog and ogar-loco by git, ndarray through the sibling patch). Limit: Stage B proves exact-integer substitution statements only. No statement language or checker exists for Boolean-DR64 laws, so the tests pair a DR64 shape with E1's statement to exercise the plumbing; that is not a claim that E1 says anything about DR64 (RECIPE_ADMISSION.md). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg
Disabling ReplayCapsule::recheck in ProofReference::rechecked leaves every test green: a capsule can only come from certify in-process. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg
|
Pushed
Two limits, also stated in
Generated by Claude Code |
…t the authority 55f88ef made ReplayCapsule::recheck the proof authority for DR64 recipes and paired a DR64 law shape with Stage B's scalar E1 statement in the tests. That conflated two things: egglog derives and checks proofs; ReplayCapsule only retains and replays one E1 compilation. It also treated E1's scalar substitution language as if it covered relational DR64 laws. Corrected: - RecipeCertificate is opaque (no public constructor, no status field), issued only by a certified-compiler boundary after egglog's proof check, law admission and a lowering check. The caller-set CheckedByExternalAuthority flag and the caller-supplied digest stay removed. - No such boundary exists for Boolean-DR64 laws, so admit_recipe runs every metadata check and then refuses with NoCertifiedCompiler. - The egglog-r2il-probe dependency and the ndarray patch are gone; E1 stays Stage B's own replay artifact. - Law identity no longer carries Stage B's SemanticKey; target ABI is back to this probe's own u32. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G22yT6htkcdyXsihxxXdrg
|
Correction to my previous comment.
How the responsibilities split:
What changed:
The candidate next milestone is recorded as open on the board entry; it is a hypothesis, not built. It is associativity of Boolean relation composition over compatible ClassView domains, plus a specialization-selection gate to be measured against a general relational baseline. Generated by Claude Code |
f999482
into
lab/dr-math-2-population-evidence-20261022
Stage 3 of D-DRMC
New lab-only law/recipe metadata and six tests. It separates (a) a reusable mathematical shape+law revision independent of ClassId/source, (b) an externally claimed proof receipt, (c) the compiled target/ABI artifact, (d) the current witnessed DR64 operand, and (e) independent effect authorization.
Negative tests reject mismatched semantic axes, unverified conjectures, missing proof/code identities, changed law or target ABI, and any proposal/effect conflated with pure mathematics. One positive test admits the SAME law across two different class/method identities while preserving their own bound operands.
Not claimed: proof verification, scientific generalization, runtime compilation or execution.
CheckedByExternalAuthorityis caller asserted and needs real trusted checker attestation before production; merged #1690–#1696 E1 and #1698 owners remain untouched. Tests authored, not run (no Cargo here). Stacked on #1702; no new hot engine or opcodes.