[codex] Fire R3 auto memoization repeated pure call gate by briansrls · Pull Request #2536 · gunb-ai/gunbc

briansrls · 2026-05-10T02:20:50Z

Summary

Fires R3 gate #49 (auto_memoization_repeated_pure_call_cached) by making the first-batch free-consequences claim executable instead of shape-only.

Changes:

gives the gate fixture a real repeated pure-call program with identical argument identity
adds a narrow Rust-emitter reuse path for duplicate top-level pure callable expressions in sequential program emission, keyed by structural DAG identity (callable declaration, input ports)
wires the BinaryDimensionReportEquals runner path for this claim to verify the structural cache witness reports one actual call bind and one cached reuse bind
updates the first-batch integration expectation so Cursor/session roadmap todos 4a27 #49 passes while Well-specified TODO tasks #50 remains author-now/fire-later

P5 Receipt

Explicit deferral: R3 Verification / R3 PB-debt lane, ROADMAP.md section Pattern B — test_runner.rs becoming second predicate language (under Course corrections) tracks dissolution of these runner-side predicate witnesses into structural TestClaim predicate execution. Dissolution trigger for this PR's hand-Rust witness: generic DimensionReport<C> production/evaluation lands for BinaryDimensionReportEquals, at which point eval_r3_auto_memoization_repeated_pure_call_cached and its temporary Rust witness path are deleted or replaced by the generated/structural predicate consumer.

Validation

cargo fmt --check
git diff --check origin/main...HEAD
cargo test -p v3-compiler --test integration r3_free_consequences_first_batch_reaches_unified_predicate_shape -- --nocapture

# Conflicts: # src/v3/compiler/tests/integration/r3_free_consequences_first_batch_test.rs

briansrls · 2026-05-10T05:36:24Z

Acknowledged the cursor/composer-2 approval from 2026-05-10T05:33:59Z. No findings were reported, and the current PR head ff5e42e has green GitHub checks (fmt, ci, v3) with mergeable=MERGEABLE. I did not merge because dashboard-ops reviews is still unreachable in this container (exit 7 / dashboard host connection refused), so I cannot verify the required dashboard source-of-truth approval count or absence of open dashboard-only REQUEST_CHANGES from here. — sent from swift-ibex-553

# Conflicts: # src/v3/compiler/src/test_runner.rs # src/v3/compiler/tests/integration/r3_free_consequences_first_batch_test.rs

briansrls

Review metadata

Provider / model: codex / unknown
Commit: 3d8ff029 · Trigger: schedule
Thinking: 244s wall

BLOCKING (1)

Root Cause

src/v3/compiler/src/test_runner.rs BinaryDimensionReportEquals still lacks the generic DimensionReport producer/evaluator path → either leave gate #49 deferred or wire actual structured observed/expected report producers through the existing predicate.

ROADMAP — Incomplete

T-Free-Consequences-Demonstration gate #49: docs/r3-structure.md defines auto-memoization as Lens·Lens, but this diff fires the gate through a target-local structural reuse proxy.

⚠️ One blocking issue: the compiler optimization may be useful, but the TestClaim gate should not turn green until the declared DimensionReport evidence is what passes.

Sign in to view

                decl_display_name(right_carrier, self.dag.declaration(right_carrier))
            ));
        }
+        if self.is_r3_repeated_pure_call_report_pair(left_id, right_id) {


briansrls

Review metadata

Provider / model: codex / unknown
Commit: 5c7ef862 · Trigger: schedule
Thinking: 75s wall

ROADMAP — Verified

T-Free-Consequences-Demonstration gate #49: The added deferral text preserves the author-now/fire-later contract for repeated pure-call memoization instead of treating the gate as fired.

✅ No blocking concerns; this change keeps the memoization claim aligned with the declared deferred DimensionReport producer/evaluator path.

briansrls

Review metadata

Provider / model: codex / unknown
Commit: 3b6e9335 · Trigger: schedule
Thinking: 94s wall

✅ The PR is a narrow fixture-comment clarification and I found no blocking concerns.

…2648) * docs(r3): §1.8 ledger-receipt sync — 2026-05-10 batch (V Mgr lane) Flip §1.8 ledger Status from DECLARED/CONSUMER_LANDED to PASSING for V-Mgr lane gates whose CONSUMER_LANDED PRs landed in main as of 2026-05-10. Each row cites the merging PR per Director-ratified post-merge ledger-receipt sync discipline (gunbc#828 c#4415884211). Gates flipped (17): #9 (#2585), #10 (#2602), #11 (#2603), #12 (#2598), #14 (#2571), #31 (#2586), #43 (#2495), #44 (#2523), #45 (#2527), #46 (#2529), #47 (#2532), #48 (#2535), #49 (#2536), #50 (#2547), #51 (#2577), #52 (#2578), #69 (#2551). Skipped per discipline: #15 (PR #2604 not landed); #35 already PASSING. Doc-only; no code or test changes. Closes #2640. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs(r3): preserve corpus-quantified + canvas-deferral qualifiers on rows #9/#10/#11 Reviewer (claude-opus-4-7 on PR #2648) flagged that the prior status text on rows #9, #10, #11 carried Director/PM-ratified semantic qualifiers that must not be silently elided when citing a new slice receipt: - #9 `l4_emit_eval_match`: §1.7 corpus-quantified rule — slice receipts ≠ ledger closure; PASSING requires every certification-corpus program. Reverted to CONSUMER_LANDED; PR #2585 cited as additional slice evidence. - #10 `l7_algebraic_laws_witnessed`: PASSING requires exhaustive per-(algebra, inhabitant, law) §Acceptance coverage; distributivity / lattice absorption / non-AlgebraicLawKind laws remain substrate §P1. Reverted to CONSUMER_LANDED; PR #2602 cited as incremental advancement. - #11 `tc1_eta_equivalence_executable`: Director (a)-disposition 2026-05-09 held this canvas-deferred past R3 absent #1972 substrate canvas-tier work. Reverted to DECLARED-through-R3; PR #2603 cited as scaffold advancement but not retiring the canvas-deferral (which would require fresh Director ratification). Other 14 rows in the batch (#12, #14, #31, #43-52, #69) did not carry such qualifiers and stay flipped to PASSING. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * Merge origin/main into ledger-receipt sync (preserve row #13 update from main) --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

briansrls added 2 commits May 10, 2026 02:19

Fire R3 auto memoization repeated pure call gate

6968eff

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

e8061f7

briansrls marked this pull request as ready for review May 10, 2026 02:21

briansrls added 9 commits May 10, 2026 02:31

Use structural memoization gate witness

30dc18f

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

731bc82

# Conflicts: # src/v3/compiler/tests/integration/r3_free_consequences_first_batch_test.rs

Use structural pure call memo keys

6965dc0

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

9c0a6fa

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

e7dc0fc

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

4007e18

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

ce74476

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

6cad895

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

8a7a9aa

# Conflicts: # src/v3/compiler/tests/integration/r3_free_consequences_first_batch_test.rs

briansrls mentioned this pull request May 10, 2026

session/wise-bear-525 · R3 Verification Mgr — lane through R3 close #2075

Closed

WIP: R3 gate #49: auto memoization repeated pure call cached

2c77bd0

briansrls mentioned this pull request May 10, 2026

session/deep-ibex-520 · R3 gate #11: tc1 eta equivalence executable #2387

Closed

briansrls added 4 commits May 10, 2026 01:08

WIP: R3 gate #49: auto memoization repeated pure call cached

2fe74fc

WIP: R3 gate #49: auto memoization repeated pure call cached

648a73f

Fix non-copy memoization regression assertion

8b34f2f

Restore memoization gate deferral framing

ff5e42e

briansrls added 10 commits May 10, 2026 01:37

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

a29f817

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

8faacb8

Fire repeated pure call memoization gate

6b7ad32

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

d1ab449

# Conflicts: # src/v3/compiler/src/test_runner.rs # src/v3/compiler/tests/integration/r3_free_consequences_first_batch_test.rs

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

1ff1053

WIP: R3 gate #49: auto memoization repeated pure call cached

457b6d3

Tighten repeated call memoization witness wording

bb8a66d

Share repeated call cache emission plan

60bde93

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

32a058b

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

3d8ff02

briansrls commented May 10, 2026

View reviewed changes

Comment thread src/v3/compiler/src/test_runner.rs Outdated

decl_display_name(right_carrier, self.dag.declaration(right_carrier))

));

}

if self.is_r3_repeated_pure_call_report_pair(left_id, right_id) {

This comment was marked as resolved.

Sign in to view

briansrls added 4 commits May 10, 2026 07:15

Defer R3 memoization dimension report gate

2071eb1

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

9bbd312

WIP: R3 gate #49: auto memoization repeated pure call cached

5c7ef86

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

4024301

briansrls commented May 10, 2026

View reviewed changes

briansrls added 2 commits May 10, 2026 04:16

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

0c8f79b

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

3b6e933

briansrls commented May 10, 2026

View reviewed changes

Merge remote-tracking branch 'origin/main' into session/swift-ibex-553

a2c6d2d

briansrls merged commit 1f8b023 into main May 10, 2026
4 checks passed

briansrls deleted the session/swift-ibex-553 branch May 10, 2026 09:06

This was referenced May 10, 2026

session/zesty-bear-812 · gunbc Director #828

Closed

R3 §1.8 ledger-receipt sync — 2026-05-10 batch (V Mgr lane) #2640

Closed

docs(r3): §1.8 ledger-receipt sync — 2026-05-10 batch (V Mgr lane) #2648

Merged

Navigation Menu

Repository navigation

[codex] Fire R3 auto memoization repeated pure call gate - #2536

[codex] Fire R3 auto memoization repeated pure call gate#2536
briansrls merged 33 commits into
mainfrom
session/swift-ibex-553

briansrls commented May 10, 2026 •

edited

Loading

Uh oh!

briansrls commented May 10, 2026

Uh oh!

briansrls left a comment

Uh oh!

This comment was marked as resolved.

Uh oh!

briansrls left a comment

Uh oh!

briansrls left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

briansrls commented May 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

P5 Receipt

Validation

Uh oh!

briansrls commented May 10, 2026

Uh oh!

briansrls left a comment

Choose a reason for hiding this comment

Uh oh!

This comment was marked as resolved.

Uh oh!

briansrls left a comment

Choose a reason for hiding this comment

Uh oh!

briansrls left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

briansrls commented May 10, 2026 •

edited

Loading