Skip to content

[codex] R3 gate 77 call pattern lookup authority - #2530

Merged
briansrls merged 3 commits into
mainfrom
session/jolly-otter-830
May 10, 2026
Merged

briansrls merged 3 commits into
mainfrom
session/jolly-otter-830

Conversation

@briansrls

Copy link
Copy Markdown
Contributor

Summary

  • Strengthen the R3 gate Tasks.md actionable items #77 integration receipt so e_p_call_pattern_lookup_authoritative directly exercises per_call_pattern_at as the public lens-facing query.
  • Pin the recursive self-call projection through the per-call evidence row, and pin cross-template SubValueUnknown as a fail-closed None instead of a parallel call-pattern derivation.
  • Promote the R3 program-plan row for gate Tasks.md actionable items #77 to CONSUMER_LANDED with the named test receipt.

Validation

  • cargo test -p v3-compiler --test integration e_p_call_pattern_lookup_authoritative

@briansrls briansrls left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review metadata

  • Provider / model: codex / unknown
  • Commit: 5d66ea1e · Trigger: schedule
  • Thinking: 223s wall

BLOCKING (1)

Root Cause

  • docs/r3-program-plan.md Gate-status receipt was treated as sufficient for expanded hand-written Rust → add exactly one P5 receipt, likely a ROADMAP.md T-PB-B or T-Tests-As-Data-Completeness deferral row for this Rust test.

ROADMAP — Incomplete

  • e_p_call_pattern_lookup_authoritative: The gate may be technically covered by the integration assertion, but the hand-Rust expansion receipt required by P5 is missing from the changed plan text.

⚠️ One blocking P5 receipt issue needs to be fixed before this lands.

Comment thread docs/r3-program-plan.md
| 75 | `pr_anticipation_discipline_ci_active` | CI-discipline | R3 Debt-Paydown (standing) | DECLARED (NEW 2026-05-06) | `scripts/check-pr-sg0-net-shrink-discipline.sh` in CI |
| 76 | `e_p_per_call_descent_evidence_full_coverage` | substrate-shape | T-E-P-Producer-Broadening | **CONSUMER_LANDED** (PR #2147 carrier + PR #2190 consumer; refresh per cluster-analysis audit §1) | per-call DescentEvidence covers all live call sites |
| 77 | `e_p_call_pattern_lookup_authoritative` | substrate-shape | T-E-P-Producer-Broadening | DECLARED (added to §"Acceptance" 2026-05-06) — verify-pending: T-E-P P1 slices 1-7 (PR #2167/#2178/#2182/#2192/#2200/#2207) likely promote to CONSUMER_LANDED; Mgr review needed | CallPattern lookup authoritative |
| 77 | `e_p_call_pattern_lookup_authoritative` | substrate-shape | T-E-P-Producer-Broadening | **CONSUMER_LANDED** — integration `e_p_call_pattern_lookup_authoritative` (`m2_substrate_inhabitance_test.rs`) pins `per_call_pattern_at` as the lens-facing query over the per-call evidence authority | CallPattern lookup authoritative |

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BLOCKING: The changed planning artifact marks the src/v3 Rust integration expansion as CONSUMER_LANDED but does not provide the INVARIANTS.md P5 hand-Rust receipt: deleted scaffold path, SG-0 shrink before/after, or explicit deferral to a concrete ROADMAP row.

@briansrls
briansrls merged commit 88c9c00 into main May 10, 2026
4 checks passed
@briansrls
briansrls deleted the session/jolly-otter-830 branch May 10, 2026 07:52
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