Repository navigation
R3 Phases A+B: BigInt host repr + u128 coverage (isize/usize deferred) - #1914
Conversation
…ame) Renames CardinalityPayload::new_unchecked to new_unchecked_bypassing_idempotence so every call-site that does not route through Dag::alloc_cardinality_decl is, by name, declaring it is not applying the cardinality_idempotent_target rule. Per Director routing #828 #issuecomment-4383647114 (Path B over Path A); brief: docs/briefs/r2-impossible-bugs-nested-optional-codegen-bypass-closure-worker.md (#1778, warm-dove-810). - Discipline-visible doc comment names the load-bearing invariant. - 205 call-sites updated across 6 files (4 hand-written non-test sites + 4 test sites + ~50 generated sites in 3 bootstrap_*_generated.rs files refreshed via the regen_bootstrap_emit format string). - Mechanical refresh; zero semantic change. Existing cardinality_idempotent / cardinality_idempotence tests pass; cargo fmt clean. - r2-closure-ledger row updated: nested-optional flatten now cites #1803 as the codegen-bypass closure receipt. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The prior commit's mass-rename of CardinalityPayload::new_unchecked → new_unchecked_bypassing_idempotence was applied with sed, preserving multi-line formatting; fresh regen_bootstrap emits on a single line. The regen_bootstrap --verify CI gate caught the divergence. This commit runs regen_bootstrap (no --verify) to align the committed snapshots with the format string. No semantic change. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…probe Authorized single retrigger per R3 Mgr at #1761 #issuecomment-4384216296. Prior run (job 74523060739) was cancelled at 30m hard timeout mid-rustc release recompile of v3-compiler for self_host_fixed_point; ci/fmt/v3 green. Probing whether warmer cache / different runner state lands within budget; if same failure recurs, escalate. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…e gap Address blocking review on PR #1819 (codex@9ccfe165): 1. Scale was a bare coproduct; Grounding consumers and downstream algebra would have to re-encode the SI exponent table at each call site. Add `fn scale_exponent(s: Scale) -> Int` as the canonical typed exponent authority (single-source-of-truth invariant; total over all Scale inhabitants). 2. `Measure<Q, S>` admitted any type-reference for Q/S at the parser level — denotationally Q must inhabit Quantity and S must inhabit Scale, but no parametric where-clause syntax exists in the substrate. Document the CONSTRAINED-INHABITANCE GAP scaffold matching the pattern at `dsl/std/algebra.dag` (`GroupCompletion<M>` / `FieldOfFractions<R>`), with a named dissolution trigger (parametric where-clause syntax lands → tighten to `<Q> : Quantity, <S> : Scale`). 3. Refresh the COMPOSITION block and citation discipline: brief paths live in the parent worktree (cross-program review reaches them via the gunbc#828 ratification thread). Inline rule-text quotes for Practice 4 / `feedback_construction_over_ratchets` / `feedback_reason_not_label` per `brief-authoring-checklist.md`. Update the conceptual composition lines to reflect option (c) RATIFIED (no `Refined<>` wrapper at this layer). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Add `cross_target_coverage_carrier_test.rs` to EXPECTED_HAND_AUTHORED_TEST in `sg0_census_test.rs` (sorted ASCII-ascending; receipt cites Director Option 2 RATIFIED at gunbc#828 #issuecomment-4377533390). - Refresh `parse_corpus_manifest.txt` via the `refresh_handwritten_parse_snapshot_manifest` host (picks up the new `src/v3/std/cross_target_coverage.dag` entry + the `extdeps_bootstrap_fixtures.dag` / `bootstrap_authority.dag` row changes). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Address blocking review on PR #1842 (codex@2eb2632a): ShapeATarget was stamped 🟢 TERMINAL with an inline checkpoint comment but lacked the four-pattern dissolution receipt that `feedback_coproduct_dissolution` requires for the GREEN classification. Pattern follows the DB-18 / DB-9 / DB-16 receipt format at `docs/design-db18-workflow-effect-carrier.md` §"Dissolution receipt". Pattern 1 (fact-placement): n/a — all variants attach to the same typed slot at the same DAG location. Pattern 2 (variant-is-data): n/a — tag identity governs typed dispatch into per-target method-template-contract authorities; collapsing to `target_name: String` would re-introduce string-name dispatch (Option 2 §4.C=(i) ratified specifically to prevent that). Pattern 3 (algebraic-form): n/a — no algebraic operation over {Rust, Python, Go}; the set is Director-ratified per Shape A portability. Pattern 4 (dimensional): n/a — targets do not factor into orthogonal coordinates; per-target primitives + method-template contracts + emit are irreducible per-target profiles. Stamp confirmed: 🟢 TERMINAL. Bootstrap regen + parse-corpus-manifest refresh follow. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… (Phase B-2) R3 Substrate Rust-primitive-full-coverage Phase B step 2 (Director Path A RATIFIED at gunbc#1739 #issuecomment-4392731264; Option (ii) confirmed at gunbc#1761 #issuecomment-4393145631). Changes: - `src/v3/spec/rust.dag` import: add `UInt128` to `std.integer` import line (sibling to `Int128`). - `data rust_u128: TypeRealization`: u128 carrier realization. Symmetric to `rust_i128`; consumes the `UInt128` kernel integer (`Semiring<Word128>`) per `dsl/std/integer.dag:61`. - `data rust_integer_inhabit_u128_bound_unspecified: TargetIntegerTypeInhabitance`: inhabitance row for u128 mirroring the i128 `BoundUnspecified` shape above. Static interval bound facts on u128 (`StaticBoundFact(IntInterval)`) lower through the BigInt-backed `IntervalInt::ExactInterval` once narrowing consumers exercise them — Phase A's host-repr widening is what made this representable. - Bootstrap snapshots regenerated. The existing 5 `TargetIntegerTypeInhabitance` rows (u32 / i32 / int64 / i128 / i32_at_program_bound) are untouched per Mgr's confirmed Option (ii) dispatch path. NOT in this commit: - isize / usize rows: blocked on missing `IntSize` / `UIntSize` kernel integer types in `dsl/std/integer.dag` — `kernel_integer:` field on `TargetIntegerTypeInhabitance` requires a std integer type, but the std lacks platform-sized integer carriers. Surfacing as scope question to Substrate Mgr (#1739) before authoring; std-side IntSize/UIntSize introduction is substrate-fact-introduction, not in this slice's bundled scope. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…on follow-on (Director ratification at #4393248961) Per Director Q1+Q2+Q3+Q4 RATIFICATION at gunbc#1739 #issuecomment-4393248961 (2026-05-07): pre-author follow-on brief for the substrate cascade surfaced by valiant-ibex-312 STOP at gunbc#1761 #issuecomment-4393222862. Scope: - Q1 naming: IntPlatform / UIntPlatform per feedback_reason_not_label - Q2 algebra: Compose<Int, MachineWidth<Platform>> + Compose<UInt, MachineWidth<Platform>>; Platform is substrate token - Q3 substrate-concept layer (target-agnostic per feedback_target_agnostic_ir) - Q4 worker pin: valiant-ibex-312 post-PR-#1914 close (freshest substrate context) Brief covers Platform substrate token + IntPlatform/UIntPlatform declarations + spec/rust.dag isize/usize row population + co-receipt advancing PR #1914's rust_primitive_full_coverage gate PARTIAL → CONSUMER_LANDED. Closure gate: int_platform_uint_platform_substrate_landed. Dispatchable post-PR-#1914 close per pre-authored-brief-queue discipline. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Review metadata
Findings
Verdict APPROVE_WITH_COMMENTS — The substantive change is coherent: |
|
Review metadata
Findings: none blocking.
Verdict: APPROVE — Phase A (host repr widening to |
Address non-blocking doc-drift finding on PR #1914 (cursor@b6989ec7): the block comment above `EXPECTED_INTEGER_ROWS` still said "9 rows" and referenced the resolved "Slice B2" deferral marker, but the const + the pilot list now carry 10 rows including u128. Refresh the comment to reflect live state per INVARIANTS.md "Documentation Describes Live State" spirit. Also update the docstring above to say "decimal BigInt" instead of "i128" (host repr widened in Phase A). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* fix(briefs): retire INVARIANTS.md line-number cites in r3-426 brief
Per codex review on PR #1782 (sha c2512cde): the new briefs
introduced fresh `INVARIANTS.md:<line>` citations — the same
stale-line-anchor shape the :425 sweep brief itself names as needing
dissolution. Replaces four line-number references in
r3-426-sfp-subdoc-rename-and-author-worker.md (lines 16, 28, 58, 85)
with prose-anchored locators citing the section anchor + bullet text.
The other PB-lane briefs codex flagged (r3-pb-runtime-equivalence-
corpus-seed-audit.md, r3-pb-t-fixedpoint-worker.md) belong to
neat-bear-351's lane and are not in this PR's scope; will route
separately.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): correct L6 type count + r3-435 dual-path retirement receipt
Per openai-pro review on PR #1782 (sha c2512cde):
1. L6 brief said acceptance ratchet asserts "five types" exist but
declares six (ShapeATarget, FormAxis, BehaviorAxis,
MethodTemplateContractKey, EmissionCell, EmissionPathProjection).
Updated both occurrences to enumerate all six explicitly.
2. r3-435 brief authorized both Path A (typed-AST reader, deletes
IntegrationRsScan) and Path B (scanner widening with Char state),
but retirement receipt language only covered Path B. Updated
ROADMAP/ledger retirement and authority-audit step 5 to cover
both authorized dissolution paths.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): L6 second 'five types' site + T-E-P lookup-gate authority
Per cursor review on PR #1782 (sha 2072b986):
1. L6 brief acceptance bullet at :131 still said "five types" while
the parenthetical and the Slice step at :117 said six. The prior
fix only caught one of two sites; corrected the second.
2. T-E-P brief `e_p_call_pattern_lookup_authoritative` acceptance
named only `lower_call_pattern` as "the only route from call
site to LoweringTarget" — but Phase 2 explicitly requires
lenses route through `per_call_pattern_at` (the L-7 single-
authority lookup), NOT `lower_call_pattern` (compiler-internal).
Acceptance text rewritten to name `per_call_pattern_at` as the
consumer-facing gate and clarify `lower_call_pattern`'s
internal-only scope.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): strengthen L6 ratchet from count-parity to per-row bijection
Per openai-pro REQUEST_CHANGES on PR #1782 (sha 2072b986): the L6
acceptance ratchet asserted only row-count parity between
emission_path_projections and the union of MethodTemplateContract
rows. Count parity permits duplicate projection keys and missing
source rows — the bridge from list-non-empty proxy to per-row
projection would not be mechanically proven. The whole point of
introducing MethodTemplateContractKey { target, dag_method } is to
provide 1:1 row identity.
Rewrites the ratchet to assert per-row bijection: every
MethodTemplateContract source row has exactly one matching
EmissionPathProjection row by key; every projection key resolves
to exactly one source row; duplicate projection keys fail closed.
(The other openai-pro BLOCKING finding — T-E-P
e_p_call_pattern_lookup_authoritative naming the wrong authority
— was already fixed at 6282a0704; the review citation was against
the pre-fix sha 2072b986.)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): split L6 ratchet into slice-active vs deferred-activation gates
Per BLOCKING inline review on PR #1782 at d92459423: the prior
bijection ratchet was unsatisfiable as authored. The
*_method_template_contracts lists at HEAD (rust/python/go) are
non-empty; shipping emission_path_projections: [] alongside a
strict bijection requirement means the bijection cannot pass when
the slice lands — source rows would be uncovered. Prior text
incorrectly said "vacuously true on the empty source set" — the
SOURCE set isn't empty, only the projection set is.
Splits the ratchet into two gate categories:
- Slice-active gates: types/data shape exist + empty-state
predicate (emission_path_projections == []).
- Deferred-activation gate: per-row key bijection, authored as a
test scaffold by this slice but #[ignore]'d (or feature-gated
on len > 0) until Grounding's follow-up populates rows. On
activation, asserts the bijection over the populated set.
This honors P1 doc faithfulness: the slice's acceptance contract
is satisfiable as authored, and the bijection becomes the
load-bearing gate the row-population PR satisfies.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): require L6 bootstrap enrollment to honor P2 facts-flow-forward
Per BLOCKING inline review on PR #1782 at L6:34: the slice authored
src/v3/std/cross_target_coverage.dag but never required enrolling
it in the bootstrap/load authority. Without enrollment the file
is dead-letter on disk and EmissionPathProjection can be absent
from the downstream Dag — silent P2 facts-flow-forward violation.
Adds a new Slice step 2 requiring a cross_target_coverage field
on BootstrapFixtures in src/v3/std/extdeps_bootstrap_fixtures.dag
mirroring the *_method_template_contracts enrollment pattern, plus
a loader-visibility ratchet. Renumbers subsequent Slice steps.
Adds matching Acceptance bullet.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* WIP: R3 Substrate
* WIP: R3 Substrate
* docs(briefs): normalize Practice 4 citation in S3 brief per codex review
Codex review on PR #1782 (sha 2614217e) flagged stale citation
"Practice 4 (Step 4 of the type-introduction checklist)" in
S3 brief — no such checklist step exists in the pinned rubric.
Live rule per docs/modeling-discipline.md:131-134 is the "What to
check" checkpoint-comment requirement under Practice 4 (coproduct
dissolution). Updated the S3 reference to cite that line directly
and drop the stale "Step 4 of type-introduction checklist" wording.
S2/S8/S9 already use bare "Practice 4" without the stale steps wording.
S1 (committed earlier today) does not contain the stale phrase.
S6 (pre-existing brief flagged by reviewer as carrying the same
pattern) is out of this branch's scope; flag for separate paydown.
* docs(briefs): sweep stale Practice 4 citations corpus-wide
Codex review on PR #1782 flagged the stale "Step 4 of the
type-introduction checklist" wording — no such checklist exists in
docs/modeling-discipline.md. The live rule is the "What to check"
checkpoint-comment requirement under Practice 4 (coproduct
dissolution) at docs/modeling-discipline.md:131.
Sweep across docs/ found 2 additional briefs carrying the stale
phrasing (S3 fixed previously at 9db003d40):
- docs/briefs/r3-l6-emission-path-projection-substrate-worker.md
- docs/briefs/r3-x1b-s1-transform-dispatch-substrate-worker.md
Both updated to cite the live rule directly. Verified zero
remaining "type-introduction checklist" or "Step 4 of the"
references in docs/ tree.
* docs(briefs): add Practice 4 citation discipline note to authoring checklist
Per parent inbox status flag at gunbc#828 #issuecomment-4385346221:
add a Citation discipline section to brief-authoring-checklist.md
naming the correct shape — Practice 4 (coproduct dissolution) +
"What to check" rule at docs/modeling-discipline.md:131 — and
calling out the stale "Step 4 of the type-introduction checklist"
phrasing as a failure mode. References the corpus sweep at
2dc92f34f.
Prevents future briefs that copy old templates from re-introducing
the stale citation pattern.
* docs(r3): cascade work post-Director ratification — S1 worker, R4 routing, Unit/Quantity canvas
Per Director ratification at gunbc#828 inbox response 2026-05-06
(zesty-bear-812):
1. S1 Q1+Q2 RATIFIED → author S1 gap-test representative worker brief
(`r3-substrate-s1-gap-test-representative-worker.md`). Dispatchable
on T-E-P Phase 1 + E6-G0d landing; closes ledger row #61.
2. S2 Q2 RATIFIED (option-(b) T-LBP narrowing) + S3 + S8 deferrals →
R4 carve-out routing ledger (`docs/r4-carve-out-routing.md`).
Captures C1 (parallelism lens), C2 (effect_enum lens), C3
(register narrowing), C4 (deferred MachineConstraint axes),
C5 (rounding-mode product extension), C6 (Unit/Quantity).
Cascade messages to Verification / Grounding / Debt-Paydown Mgrs.
3. S9 Phase-3 dimensional refinement boundary → Unit/Quantity carrier
canvas (`r3-substrate-s9-unit-quantity-carrier-canvas.md`).
Surfaces Q-Unit-1..5 for Director ratification: carrier name,
2-axis vs 3-axis Phase-1, Duration/Seconds collapse, Refined
composition order, Practice 4 classification.
Updates §1.8 ledger receipts for carved items; R3 closure receipts
treat carved items as explicitly-deferred (not drift).
* docs(briefs): citation-discipline fix-forward — anchors/quotes, not line numbers
Director redirect at gunbc#828 #issuecomment-... noted the citation
paydown's replacement form (`docs/modeling-discipline.md:131`) is the
same drift class as the original fabricated-step error. Bare line
numbers shift silently on any doc edit above the cited line.
Generalized fix:
- brief-authoring-checklist.md "Citation discipline" section
rewritten to prohibit bare line-numbers across ALL cross-doc
references (modeling-discipline.md, r3-program-plan.md,
r3-structure.md, INVARIANTS.md, MODELING.md, etc.). Use section
anchors (#section-name) or rule-text quotes.
- 3 briefs that swapped in :131 (S3, S6, X1B) re-converted:
- docs/modeling-discipline.md#4-coproduct-dissolution anchor
- "What to check" rule quoted inline (self-contained)
Brief-authoring-checklist failure-mode section names both vectors
(fabricated step + bare line numbers) and references both sweep
commits in audit trail.
Broader cross-doc citation sweep across briefs I authored on this
branch (frontmatter authority-docs lists, body prose) is recommended
per Director note — flagging as separate paydown task; this commit
closes the immediate recurrence vector.
* docs(briefs): fix unfaithful Int=AbelianGroup<Nat> + sweep stale 'Step 4' refs
Per openai-pro review on PR #1782 sha 0ecd4818 (gunbc#1782 #...):
BLOCKING fix (P1 modeling faithfulness):
- S9 + S3 directed workers to model `Int = AbelianGroup<Nat>`,
but Nat is not an Abelian-group carrier (additive inverses
not representable). Reframed:
- `Int` is the additive Abelian group on integers (carrier Z);
`Nat` is a commutative monoid under addition.
- Algebraic relationship is **explicit group-completion**:
`Int ≡ GroupCompletion<CommutativeMonoid<Nat>>` — NOT
`AbelianGroup<Nat>` parameterization.
- Worker either models `Int` directly as `AbelianGroup`
(terminal Abelian-group instance) OR introduces explicit
`GroupCompletion<M>` substrate carrier; decision lands in
Phase-1 PR per P1 procedure.
- S3 cross-reference + Phase-4 algebra-axis preservation note
reframed accordingly.
NON-BLOCKING in-PR cleanup (Practice 4 citation discipline):
- 6 briefs still carrying stale 'Practice 4 Step 4' wording
(r3-coproduct-1/2/3, r3-l6, r3-t-e-p, r3-x1b) updated to use
the section anchor (`docs/modeling-discipline.md#4-coproduct-dissolution`)
+ inline rule-text quote ("What to check") form per the
citation discipline established in `b4cae5299`.
Closes both findings from openai-pro review.
* docs(briefs): Q-Unit-1..5 RATIFIED — apply Scale.Unit→One rename + ratification banner
Director ratified Q-Unit-1..5 at gunbc#828 #issuecomment-... 2026-05-06
(zesty-bear-812). All 5 questions ratified; one rename required:
- Scale enum's `Unit` value collides with outer `Unit<Q, S>` carrier
name (`Unit<Time, Unit>` is shadowing-confusing). Rename to `One`.
- Duration ≡ Unit<Time, One> (collapses with Seconds per Q-Unit-3).
- Milliseconds ≡ Unit<Time, Milli>.
Canvas updated:
- Top-level RATIFICATION banner records all 5 ratifications + rename
- Scale enum: `Unit` → `One` (with inline rename receipt)
- Sketch product expansions use `Unit<Time, One>` / `Unit<Time, Milli>`
- Phase-3 reframe table products updated
- Q-Unit-3 prose collapse reference updated
Subsequent worker brief authoring (Unit/Quantity carrier landing
post-prerequisite) inherits the renamed shape.
* docs(briefs): Unit/Quantity carrier worker brief — Q-Unit-1..5 RATIFIED
Worker brief consumes the ratified Unit/Quantity canvas
(`5f22fd06e` ratification banner). Phase-1 lands `Unit<Q, S>` +
`Quantity` + `Scale` declarations at the ratified shape (Scale's
`One` value per Q-Unit-1 rename). Phase-2 reframes S9 Phase-3
dimensional refinements (Duration / Seconds / Milliseconds) to
outer-Refined / inner-Unit composition per Q-Unit-4 RATIFIED.
EpochMs deferred to R4 C6 (Aspect-axis follow-up).
Practice 4 marks pre-determined per Q-Unit-5: `Unit<Q, S>` 🟢
PRIMITIVE; `Quantity` + `Scale` 🟡 SCAFFOLD with "consumed by ≥1
emission rule" dissolution trigger.
Cross-program handoff to Grounding Mgr (#1745) for dimensional
emission consumption documented.
Worker pin TBD — dispatchable when idle pool refreshes.
* docs(briefs): S9 Phase-1 reframed — Int landed at #1466; UInt-only migration
Per proud-lynx-311 dispatch ack at gunbc#1739 #issuecomment-...
2026-05-06: re-grep at HEAD revealed substrate state diverges
materially from S9 brief's "Option A vs B decide" framing.
Reality check at HEAD:
- `dsl/std/integer.dag:55` already carries
`type Int = AbelianGroup<GroupCompletion<Nat>>` per Slice 3 PR
#1466 / commit `4ba0a6d04`. Single-authority Q6 audit doc at
`docs/audit/t-numeric-construction-group-completion-6q.md`
ratified this shape and explicitly rejected the compact form.
- `dsl/std/integer.dag:56` still on legacy `type UInt = UInt64`
alias; UInt migration is the actual Phase-1 worker scope.
Brief Phase-1 step 1 reframed: Int landing acknowledged; worker
confirms HEAD match. Phase-1 step 2 reframed: UInt migration
target named (`type UInt = CommutativeMonoid<Nat>`); fixed-width
rows preserved per existing legacy-rows policy.
Historical Option A vs B framing kept as note (superseded).
Confirmed S9 dispatch ack from worker; UInt-only Phase-1 PR is
in-scope. Emission entries deferred per S3 (valiant-ant-72
pre-landing); Phase-2 deferred per S8 (quiet-boar-160 pre-landing).
* docs(briefs): fix two BLOCKING citation drift items per codex review
Codex review on PR #1782 sha 5f22fd06 flagged two authority-receipt
gaps:
1. brief-authoring-checklist.md:118 — `<this-commit>` placeholder in
the failure-mode section was unverifiable post-merge. Replaced
with `b4cae5299` (the actual anchor/quote conversion commit).
2. r3-378-http-path-helper-fail-open-worker.md:84 — `INVARIANTS.md
C-8 fail-closed` cited without anchor or rule-text quote, in
violation of the citation discipline added in this PR.
Converted to `INVARIANTS.md#p3-fail-closed` (P3 heading anchor)
+ inline rule-text quote naming P3 + C-8.
PR's own discipline now consistently applied across the diff.
* docs(briefs): sweep bare .md:NNN citations in Substrate-Mgr-authored briefs
Per Director recommendation in citation-discipline thread (gunbc#828
follow-on): bare line-number references drift silently. Swept 19
remaining instances across 5 briefs I authored on this branch:
- docs/briefs/r3-substrate-s1-q-class-2-chain-break-gap-test-canvas.md
- docs/briefs/r3-substrate-s1-gap-test-representative-worker.md
- docs/briefs/r3-substrate-s2-t-lbp-scope-calibration-canvas.md
- docs/briefs/r3-substrate-s3-machine-constraint-carrier-worker.md
Conversion: bare `file.md:NNN` → `file.md §"Section name"` form.
Stable; survives doc edits; human-readable. Section names per
HEAD heading audit.
Verification: `grep -nE '.md:[0-9]+' docs/briefs/r3-substrate-*.md
docs/r4-carve-out-routing.md` returns zero results in
Substrate-Mgr-authored scope.
Broader corpus scope (~232 references across non-Substrate briefs)
remains as separate paydown task.
* docs(briefs): fix BLOCKING + NON-BLOCKING items per openai-pro review on a462f007
Three findings addressed; 2 BLOCKING + 1 NON-BLOCKING:
BLOCKING — R3/R4 dual authority for Unit/Quantity carrier:
R4 carve-out C6 narrowed from "Unit/Quantity carrier landing"
(deferred all of it) to **"Aspect-axis (PointKind) follow-on
for EpochMs and instant/rate-shaped refinements only"**. The
2-axis Measure<Q,S> carrier landing IS in R3 per Q-Unit-1..5
RATIFIED; only the Aspect axis (PointKind = Magnitude | Instant
| Rate) carves to R4. Cascade-message C6 reference + R4 program-
plan input list updated to match. Cross-Mgr message clarifies
Grounding consumes 2-axis dimensional refinements in R3.
BLOCKING — Locked design partially applied (Scale.Unit→One
rename in canvas alternate-sketch sections):
- Canvas line 105-108 alternate sketch ("Open question: Instant
vs Duration axis") still used pre-rename `Unit` Scale value;
added pre-ratification disclaimer marking the section as
deferred to R4 C6 + noting `Unit` is pre-rename
- Canvas line 163 Q-Unit-3 collapse note: `Scale = Unit` → `Scale = One`
(BLOCKING — bare line-numbers in S1 canvas frontmatter): NOT
APPLICABLE — already fixed at sweep commit 127287a66 prior to
review; reviewer was on sha a462f007 pre-sweep.
NON-BLOCKING — placeholder receipts:
- brief-authoring-checklist.md `#issuecomment-...` → concrete ID
4385393971 (Director's anchor-discipline reply)
- r3-substrate-unit-quantity-carrier-worker.md (3 occurrences)
+ canvas RATIFICATION banner: `#issuecomment-...` → concrete ID
4385412256 (Q-Unit ratification reply)
- `<this-commit>` placeholder in checklist already fixed at 5be1c99f2
Verification: zero `#issuecomment-...` or `<this-commit>` placeholders
in docs/briefs/ + docs/r4-carve-out-routing.md.
* docs(briefs): codex BLOCKING review fixes — Practice 4 source-level + policy scope
Three BLOCKING findings on PR #1782 (codex sha b913e2f4):
1. **Citation-discipline policy scope**: addressed by scoping
the rule prospectively to briefs landing after the policy
commit (`b4cae5299`). Pre-existing corpus references are
flagged as paydown but NOT individually blocking; coordinated
single-PR sweep is the right paydown shape. Substrate-Mgr scope
already swept at `127287a66`.
2. **R4 C6 reconciliation**: already fixed at `6e5a5481a` (R4 C6
narrowed to Aspect-axis follow-on for EpochMs only; 2-axis
carrier landing IS in R3). Codex sha b913e2f4 was pre-fix.
3. **Practice 4 source-level vs PR-body conflation**: corrected.
Acceptance bullet for Unit/Quantity worker brief now requires
inline checkpoint comments on the LIVE declarations in
`dsl/std/units.dag` (load-bearing artifact per
`docs/modeling-discipline.md#4-coproduct-dissolution` "What to
check"). PR-body summary is supplementary, not substitute.
Closes the review's reconciliation gates.
* docs(briefs): absorb Q-MachineConstraint + Q-Unit-1-Recanvas ratifications
Two major Director ratifications absorbed:
1. Q-MachineConstraint-Carrier RATIFIED at gunbc#828
#issuecomment-4385530115 (Brian directive 2026-05-06: "universal
substrate, ratify defaults"; PR #1817 queued). 6 sub-decisions:
- MachineWidth<bits> only for R3; defer RegisterClass / EndianMode
/ Alignment / signedness post-R3
- Parametric Compose<Algebra, MachineConstraint> type-level
interaction shape (lookup-maps REJECTED — AlgebraMachineProduct
record-keyed table form retracted)
- Int<64> ≡ Compose<AbelianGroup, MachineWidth<64>> (algebra-side
options A and B preserve interaction semantics)
- Real<64> ≡ Compose<ApproximateField<Rational>, MachineWidth<64>>
(algebra approx + machine approx as independent composing axes)
- ≥3 pairs is minimum, not target
- UNIVERSAL substrate: every target carries machine-constraint
facts; targets without native machine-width semantics handle
omission at Grounding-level discharge (target-conditioned
lowering NOT target-conditioned substrate)
Reframed: S3 brief Phase-1, S3 Phase-3/4 emission entries,
S8 Phase-1/2/3 (ApproximateField<Real> → ApproximateField<Rational>;
AlgebraMachineProduct → Compose<...>), S9 emission entries.
2. Q-Unit-1-Recanvas RATIFIED Measure<Q, S> at gunbc#828
#issuecomment-4385539791. valiant-ibex-312's substrate-state
grep caught Unit<Q, S> collision with kernel Unit at
dsl/std/types.dag:176; Director ratified Measure<Q, S> as
outer carrier name (conventional in scientific computing;
no dsl/std/ collision).
Reframed: Unit/Quantity worker brief throughout (Unit<Q,S> →
Measure<Q,S>; dsl/std/units.dag → dsl/std/measure.dag); canvas
spec sections + sketch products (Refined<Measure<Time, ...>...>);
R4 carve-out C6 with informal/formal name disambiguation note.
Both rename + reshape are now consistent across S3 / S8 / S9 /
Unit-Q briefs + canvas + R4 routing. Workers (valiant-ant-72 on S3,
quiet-boar-160 on S8, proud-lynx-311 on S9, valiant-ibex-312 on
Unit-Q) all dispatch against the ratified shape.
* docs(briefs): S11 Slice C worker brief — ROADMAP :425 retirement post-#1801
Cascade-clearance trigger fired post-#1801 (Slice B) merge
2026-05-06. S11 brief authored per B1 RATIFIED prose+regen
bundling discipline.
Phase 1: per-file edits across the post-A+B residual list (~10
files per smart-ram-167's Slice B post-rebase enumeration);
anchor + rule-text-quote pattern per current citation discipline;
bootstrap regen in same PR (no split).
Phase 2: ROADMAP :425 row PARTIAL → Retired with 3-slice receipt;
Q-Slice-C-Retirement-Receipt resolution.
Worker pin smart-ram-167 (#1759) — Slice B precedent owner.
Dispatch packet sent separately to #1759.
* docs(briefs): Q-Refined-Phantom-Composition (c) RATIFIED — defer literal Refined form
Director ratified option (c) at gunbc#828 #issuecomment-... 2026-05-06:
drop `non_negative` predicate at Measure (phantom) layer; rely on
`Quantity = Time` tagging for non-negative-magnitude semantics.
Q-Unit-4 outer-Refined / inner-Measure composition is preserved as
**conceptual model**; the literal `Refined<Measure<...>, predicate>`
form is **deferred to value-typed-integration follow-up** when
substrate gains Refined<> as first-class type expression and
dimensional refinements get value-typed integration.
Updates:
- Canvas RATIFICATION banner Q-Unit-4 row reframed to "conceptual
model; literal form deferred"
- Canvas Phase-3 reframe table: Duration / Milliseconds drop the
Refined<...> wrapper; rely on Quantity tag
- Worker brief Phase-2 prose + reframe table + Acceptance bullet
all reflect deferred-literal-form posture
valiant-ibex-312 unblocked to proceed with Phase-2 (Grounding
handoff doc) per the (c)-shaped Acceptance.
* docs(briefs): S12 F2 + F8 doc-sharpening worker brief — post-#1820 cascade
#1820 (Slice C) merged 2026-05-06; ROADMAP :425 row Retired closes
the prose-reference paydown program. S12 cascade-clearance trigger
fires per B6 RATIFIED single bundled Mgr-tier doc-sharpening PR.
Scope: F2 (.v3 filename-suffix grammar dispatch) + F8 (bootstrap
load-order/exclusion authority) — single bundled PR per B6.
Cross-program coordination with PB Mgr (#1742) per design schedule
§3 P5 (comment-only OR co-author; no duplicate PR).
Worker pin smart-ram-167 (#1759) — Slice C precedent owner;
pattern-familiar with prose+regen bundling discipline. Dispatch
packet sent separately to #1759.
* docs(briefs): S9 brief — UInt landed via #1818; algebra label corrected to CommutativeSemiring
Per proud-lynx-311 merge report at gunbc#1746 #issuecomment-...
2026-05-06: PR #1818 landed `type UInt = Nat` after codex BLOCKING
review on initial `CommutativeMonoid<Nat>` form (P2 facts-flow-forward
violation — wrap projects out Nat's multiplicative monoid + identity 1).
Final landed form preserves full `CommutativeSemiring<Magnitude>`
surface — Nat IS a commutative semiring (addition + multiplication
+ 0 + 1). Algebra-side label for UInt is CommutativeSemiring, NOT
CommutativeMonoid (corrects my earlier dispatch labeling).
S9 Phase-1 step 2 marked LANDED with merge ref. Phase-1 step 3
emission entries reframed:
- Int<N> ≡ Compose<AbelianGroup, MachineWidth<N>>
- UInt<N> ≡ Compose<CommutativeSemiring, MachineWidth<N>>
Worker discipline + reviewer caught a substrate-state error in my
brief framing; brief now matches landed reality.
* docs(briefs): fix contradictory (c) guidance in Unit/Quantity worker brief
Codex BLOCKING review on PR #1825 sha 1bce0418 caught a real
contradiction: brief Phase-2 prose at lines 115-118 stated
"predicates do NOT attach at the Measure (phantom) layer" per
Q-Refined-Phantom-Composition (c) RATIFIED, but immediately
following paragraph at lines 130-132 still said "Refined predicate
non_negative applies at the Measure-typed level" — left over from
the original Q-Unit-4 framing.
Updated lines 130-137 to consistently apply (c) shape: predicates
deferred to value-typed-integration follow-up; dimensional semantics
ride on Quantity tag at Measure layer; worker authors 3 in-scope
refinements WITHOUT predicate at this layer.
P2 boundary discipline restored: brief now gives single-authority
guidance for Phase-2 substrate landing.
* docs(briefs): name predicate authority + concretize ratification ID — codex BLOCKING
Codex BLOCKING review on PR #1825 sha 32541a56:
1. (BLOCKING) Q-Refined-Phantom-Composition (c) ratification deferred
phantom-carrier predicates without naming the value-level predicate
authority. Updated canvas Q-Unit-4 RATIFICATION row to explicitly
name: 'predicates re-attach at value-typed Int/UInt level via
where-sugar (e.g., range(min: 0) per current dsl/std/types.dag
precedent)' when value-typed integration lands.
2. (BLOCKING) Worker brief option-c reframe didn't sweep the audit
receipt §5 (Carrier dissolves the bridge). Updated to consistently
apply (c) shape — dimensional semantics ride on Quantity tag at
Measure layer; value-level predicate authority deferred; until
then, range/gt_zero constraints attach at value-typed Int/UInt
via where-sugar per dsl/std/types.dag convention.
3. (NON-BLOCKING) Canvas had #issuecomment-... placeholder for the
(c) ratification message ID. Replaced with concrete
#issuecomment-4385687075 per dashboard relay header.
Brief now names a single value-level authority (Int/UInt where-sugar)
for value-range facts; phantom-layer Quantity tag carries only
dimensional semantics. P2 boundary discipline maintained.
* docs(briefs): S12 RETIRED per Director option (2) ratification
Director ratified option (2) at gunbc#828 #issuecomment-4385873...
2026-05-06 post smart-ram-167's substrate-state STOP-AND-ESCALATE:
- F2 absorbed by 2026-05-04 routing + #1662 closure
(`Result/DivError span.file-keyed` retired)
- F8 topical concerns distributed across live rows
(Class 5 Gap 3 `bootstrap.rs::std_fixtures()`; Character-level
load-set / bootstrap; Filename-sentinel-bridges) with own owners
- Doc-discipline goals absorbed by per-Mgr citation paydown
routing at gunbc#828 #issuecomment-4385583247
Frontmatter updated to RETIRED with absorption receipt. Worker
(smart-ram-167) pin freed.
Director-side acknowledgment of Substrate Mgr's option (2) recommendation:
"Substrate Mgr's option (2) recommendation is the right move
structurally. Most Mgrs default to 'synthesize work to fill the brief'
when canvas-vs-HEAD mismatch surfaces. Recommending retirement when
the work is genuinely absorbed is harder discipline."
* docs(briefs): S7 PR-F (BoundDeclaration consumer + ReferenceModel<T>) worker brief
Per cascade-clearance trigger fired post-#1782 merge 2026-05-06.
loyal-wolf-828 (#1764) acknowledged S7 dispatch with substrate-state
pre-flight grep (gunbc#1764 #issuecomment-4385872...) and surfaced
await-brief posture. This brief satisfies that prerequisite.
Worker pre-flight inventory absorbed:
- BoundDeclaration substrate landed (substrate.dag StaticBound +
PlatformDependent); partial BoundDeclarationView consumer in
fold.rs (ScratchIntExamples scope; doc-explicit partial)
- ReferenceModel<T> NOT yet in dsl/std/ (substrate-fact-introduction
required; design-emission-model.md §Q2 framing)
Phase-1: BoundDeclaration consumer broadening to full
design-emission-model.md surface (u128 / isize / usize / walker
arms / pilot mirror).
Phase-2: ReferenceModel<T> P1 substrate-fact-introduction with
DFS-of-concept-DAG + Practice 4 in-source checkpoint comment +
named consumer demand (Grounding G1).
Phase-3: cross-program handoff to Grounding Mgr (#1745) for
G1 (T-Ground-Rust Phase 1).
Closure predicate: unblocks Grounding T-Ground-Rust Phase 1.
Worker pin loyal-wolf-828 holds; dispatch fires immediately
on this brief landing.
* docs(briefs): S9 Slice 2.5 — predicate-registry + PB-1 shim retirement (Path 3 RATIFIED)
Director ratified Path 3 at gunbc#828 #issuecomment-4390333451
post proud-lynx-311's pre-flight DFS at gunbc#1746 #issuecomment-4390253616.
Brief reframes Slice 2.5 from S-sized (primitive + 1 consumer) to
L-sized (predicate registry + PB-1 shim retirement). 5 deliverables:
1. gt_zero primitive predicate
2. Predicate registry infrastructure (resolver-side substrate-fact-introduction)
3. PB-1 shim retirement (span.file=='dsl/std/types.dag' special case dissolves)
4. Migration of existing types.dag predicates (range, pattern, etc.)
5. PositiveInt = Refined<Nat, gt_zero> consumer update
Per Director hard scope bars: no parallel predicate-resolution path;
no per-call-site special-casing; PB-1 shim retirement non-negotiable
in same slice. Single PR per Path 3 RATIFIED.
Discipline anchors: feedback_construction_over_ratchets (PB-1 shim
is bridge); feedback_dissolve_bridges (registry is structural fix);
feedback_no_textual_enforcement_bridges (file-path special-case is
anti-pattern shape); P1 substrate-fact-introduction procedure.
Worker pin proud-lynx-311 — Director pre-cleared on pre-flight DFS
context. PR #1840 (partial Slice 2 — NonNegativeInt = Nat) MERGED
at 16:54Z; Slice 2.5 dispatchable immediately on this brief landing.
* docs(briefs): Slice 2.5 audit receipt — Mgr independent grep parallel-verifies worker DFS
Codex BLOCKING review on PR #1845 sha 2d230949 flagged: external PR
receipt absorbed as live substrate state without re-grepping local
std declarations at brief authoring time. Authority receipt §1
cited proud-lynx-311's pre-flight DFS as authority but didn't
record Mgr-side independent verification.
Updated audit receipt §1 + §4 to record Mgr's independent grep at
HEAD as parallel verification:
- gt_zero zero matches in dsl/std/ + src/v3/ (verified)
- NonNegativeInt = Nat at dsl/std/integer.dag:133 (#1840 land)
- PositiveInt = Int where range(min: 1) at dsl/std/types.dag:255
- PB-1 shim doc-block confirms bridge-text framing
- Predicate registry confirmed not present
Worker DFS remains the dispatch-time discipline (substrate-state
grep at dispatch); Mgr's independent grep is parallel verification,
not substitute. Per Director's pattern note at gunbc#828
#issuecomment-4390199218 + #issuecomment-4390333451: dispatch-packet
authoring discipline includes Mgr-side substrate-state-grep, not
just relay of worker findings.
* docs(briefs): Slice 2.5 brief — Int extensibility scope-bound; lower.rs cites anchored
openai-pro REQUEST_CHANGES on PR #1845 sha 2d230949 flagged 2 issues:
BLOCKING — P1 modeling faithfulness / substrate scope leak:
`gt_zero` Int-side extensibility was left to 'worker judgment if
generalization cheap'. Per feedback_construction_over_ratchets:
don't speculatively extend predicate-carrier compatibility ahead
of named consumer demand. Reframed to **EXPLICITLY OUT-OF-SCOPE**
for Slice 2.5; Nat-only registration. Future Int-side consumer
surfaces as separate substrate-fact-introduction (P1 procedure)
brief with own consumer-demand receipt.
NON-BLOCKING — citation discipline self-violation:
3 instances of bare `lower.rs:821-835` line-range citations
contradict the brief's own 'no bare :NNN' requirement at line 211.
Converted to function-name anchored form:
`src/v3/compiler/src/lower.rs` (`lower_type_alias_refinements_phase` doc-block).
Both findings absorbed; no parallel scope ambiguity remains;
brief now self-consistent on citation discipline.
* WIP: R3 Substrate
* docs(briefs): Slice 2.5 brief — audit-receipt cites converted to declaration-name anchors
cursor APPROVE_WITH_COMMENTS on PR #1845 sha ebf6ac12 noted citation
self-inconsistency: audit-receipt section §1 + §4 used bare
`dsl/std/integer.dag:133` and `dsl/std/types.dag:255` line-number
pins despite the brief's own no-bare-:NNN rule.
Converted to declaration-name-anchored form:
- 'declaration named NonNegativeInt' (with reproducible grep command)
- 'declaration named PositiveInt' (with reproducible grep command)
- 'lower_type_alias_refinements_phase doc-block' for PB-1 location
Brief now fully self-consistent on its own citation discipline.
Verified zero remaining .dag:NNN / .rs:NNN / .md:NNN bare cites.
* docs(briefs): Slice 2.5 brief — Path (a) RATIFIED carrier+argument contract verification
Director Path (a) RATIFIED at gunbc#828 #issuecomment-4390760353
(triage) + #issuecomment-4390794121 (convergence ack):
predicate-body lowering with carrier+argument contract
verification at lower-time IN SAME SLICE. Placeholder semantics
REMOVED from registry path.
Codex REQUEST_CHANGES surfaced that registered-predicate-without-
contract = silent acceptance per feedback_fail_closed_discipline.
Brief reframe absorbs Path (a):
Hard scope bar #5 added:
- Same-slice carrier+argument-contract verification
- Mismatched carrier OR malformed argument → Diagnostic::ResolveError
- No 'named but not checked' path remains
Scope expansion section added (L → XL):
- Predicate-body lowering for all 7 registered predicates
- Per-predicate arg-shape validation
- Multi-session work acknowledged
- Carrier-check at 788d6acb4 retained; predicate-body builds on top
Acceptance bullets updated: gt_zero with arg-shape validation +
all 7 predicates with carrier+argument contract verification.
Director-pinned brief-authoring discipline: dissolution trigger
named in a brief authored as substrate-fact-introduction must
specify same-slice acceptance, not deferred follow-up. Otherwise
the brief encodes 'land bridge + defer dissolution' anti-pattern,
the same shape feedback_construction_over_ratchets warns against
at implementation layer applied at brief authoring layer.
This is the 11th miss this session — but the pattern surfacing it
(direct sweep + Codex REQUEST_CHANGES + Director ratification)
produces structural fixes that prevent the next miss class. Brief
revision is the structural recovery.
* docs(briefs): Slice 2.5 brief — Path 2 RATIFIED (Gap 1 discharge + Q-Regex carve-out)
Director Path 2 RATIFIED at gunbc#828 #issuecomment-4391985613
post Rung 3 STOP at gunbc#1746 #issuecomment-4391946213.
Path 2 narrowing of Path (a):
- gt_zero + range body synthesis already landed (5f3c40c3a)
- Gap 1 discharge mechanism: extends scalar-literal predicate
evaluation at lower-time. L-class; lands SAME SLICE per
same-slice-dissolution discipline
- non_empty + brand body synthesis: lands via Gap 1 discharge
- pattern / format / content: fail-closed-with-named-dep at
user-code authoring layer (Diagnostic::ResolveError naming
Q-Regex-Primitive). NOT placeholder; structural rejection
- PB-1 shim FULLY RETIRED per hard scope bar #3
- Q-Regex-Primitive carved as follow-on substrate-fact-introduction
(XL substrate; not pre-authored per feedback_construction_over_ratchets)
Acceptance updated:
- 4 of 7 body-synthesized in same slice
- 3 of 7 fail-closed-with-named-dep (no placeholder semantics)
- Gap 1 discharge mechanism landed (no partial implementation)
Worker disposition: proud-lynx-311 continues Rung 3 with Gap 1
implementation + non_empty/brand body synthesis + pattern/format/
content fail-closed-with-named-dep diagnostic.
* docs(briefs): S9 Phase-1 Step 3 emission entries — Tier-1 brief authoring (1/5)
First Tier-1 brief landing post Director auto-nudge on assignment
#1858. Authored against Q-MachineConstraint sub-decisions ratified
shape now landed at #1856:
- Int<N> ≡ Compose<AbelianGroup, MachineWidth<N>> → Rust i32/i64/i128
- UInt<N> ≡ Compose<CommutativeSemiring, MachineWidth<N>> → Rust u32/u64/u128
- 6 concrete instantiations (≥3 minimum per sub-decision 5)
- Cross-program handoff to Grounding Mgr (#1745) for G2 emission consumption
- numeric_construction_demonstration Acceptance bullet (Int<32> round-trip);
Real<64> half deferred to S9 Phase 2 / S8 Float cascade
Brief authored with substrate-state-grep discipline + same-slice-
dissolution discipline (no deferred trigger; demonstration in same
slice or explicitly carved). 5-question authority audit completed
in brief body.
Worker pin proud-lynx-311 (S9 holds); brief queues post-Slice-2.5
completion as natural follow-on.
Tier-1 brief queue progress:
- 1/5 authored (this brief)
- 4 remaining: S9 Phase-2 Float coordination; T-LBP complexity
cementing; T-LBP cost cementing; S3 Phase-2 parser-grammar
* docs(briefs): S3 Phase-2 parser-grammar surface — Tier-1 brief 2/5
Authored per Director freed-pool pressure at gunbc#828
#issuecomment-4392095857 + Tier-1 brief-queue commitment.
Brief covers parser-grammar surface for Compose<Algebra, MachineConstraint>
interaction syntax post-#1856 (Phase-1 carrier slice landed):
- Phase 2.1: Parser surface per ratified Q-MachineConstraint-Grammar-Shape
- Phase 2.2: Bootstrap demonstrator with >=3 algebra*machine-axis pairs
- Phase 2.3: Class 1 5-criteria Pass receipt (criteria 1-4; criterion 5
v2-oracle parity may carve)
- Phase 2.4: numeric_construction_demonstration co-receipt with S9 Phase-1
Step 3 emission entries brief
Director ratification HOLDS on grammar bikeshed:
- Candidate A: Int @ Width<32> (annotation form)
- Candidate B: Int with MachineWidth<32> (with-form)
- Candidate C: Int<32> (positional generic — fits existing convention)
- Candidate D: Compose<AbelianGroup, MachineWidth<32>> (no parser change;
type aliases provide convenience names)
Worker pin valiant-ant-72 holds (S3 Phase-1 precedent owner); freed-pool
until Director ratifies grammar shape.
Tier-1 brief queue progress: 2/5 authored (this brief + S9 Phase-1 Step 3).
Remaining: S9 Phase-2 Float coordination; T-LBP complexity cementing;
T-LBP cost cementing.
* docs(briefs): S3 Phase-2 brief — fix internal contradiction on Class 1 closure scope
Codex BLOCKING review on PR #1889 sha ec609b4a flagged real
contradiction:
- Provenance section claimed '5-criteria Pass closed' (line 229)
- Phase 2.3 said 'criterion 5 may queue separately' (line 154)
- STOP-AND-ESCALATE bullet 1 ratified 4-of-5 carve-out (line 181)
Updated Provenance to consistently state criteria 1-4 close in this
slice; criterion 5 (v2-oracle parity) cementing test may carve to
separate slice. Single 4-of-5 framing throughout. INVARIANTS.md
'Documentation Describes Live State' satisfied.
Per feedback_construction_over_ratchets: don't claim closure that
isn't structurally complete. 4-of-5 + named carve is the honest
shape; 5/5 only on cementing-test land.
* briefs: cite Q-MachineConstraint sub-decision 6 supersession of T-V2-Retirement gate
Address Codex BLOCKING #1 on PR #1889: S3 Phase-2 + S9 Phase-1 Step 3
worker briefs consumed Q-MachineConstraint ratification without
reconciling the historical T-V2-Retirement-landing-first cascade
gate. Add an explicit precondition-framing section to each brief
citing Director ratification at gunbc#828 #issuecomment-4385530115
(Q-MachineConstraint sub-decision 6 UNIVERSAL substrate posture)
as the implicit supersession; PR #1856 landing without T-V2-Retirement
first confirms supersession in practice. STOP-AND-ESCALATE if
supersession contested at dispatch.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* slice-2.5 brief: mark pattern/format/content Path 2 acceptance pending Director ratification
Address Codex BLOCKING #2 on PR #1889: Path 2 deliverable + acceptance
language asserted fail-closed-with-named-dep semantics for
pattern/format/content, but Director rejected that shape (Option 4)
at gunbc#828 #issuecomment-4392.... Options A (consumer migration),
B (Q-Regex bundled into slice), C (typed fail-closed exception with
named trigger) await ratification per migration-cost catalog at
#issuecomment-4392116215. Brief now flags those three predicates +
PB-1 shim retirement coupling as PENDING DIRECTOR RATIFICATION; worker
STOP-AND-ESCALATE if dispatched pre-ratification.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* slice-2.5 brief: reconcile PB-1 retirement shape with Options A/B/C gating
Address Codex BLOCKING #4 on PR #1889: prior text claimed "PB-1
FULLY RETIRED" while preceding bullet said retirement gates on
chosen path — internal contradiction. Reframe to gate the
pattern/format/content half of shim retirement on Options A/B/C
ratification while preserving Director hard scope bar #3 binding
on the gt_zero/range/non_empty/brand half (cleanly retired via
Gap 1). No path leaves PB-1 alive as parallel-authority shim on
slice landing.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: tier-1 3/5 — S9 Phase-2 Float emission coordination worker brief
Tier-1 brief queue progress (3/5): S9 Phase-2 Float<N> emission
entries (Float<32>/Float<64> via Compose<ApproximateField<Real>,
MachineWidth<N>>) + Real<64> round-trip demonstration completing
§1.8 #67 Acceptance bullet. Coordination/synthesis brief — gates on
both S8 (ApproximateField<F> + Real base carrier) and S9 Phase-1
Step 3 (Int<N>/UInt<N> emission-entry pattern) landing before
dispatch. Inherits T-V2-Retirement supersession framing from
Phase-1 Step 3. STOP-AND-ESCALATE on missing preconditions or
unratified Q-ApproximateField-Axiom-Set.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: ledger advancement honesty (PRODUCER_LANDED not CONSUMER_LANDED) + restore exact issuecomment IDs
Address openai-pro BLOCKING + NON-BLOCKING findings on PR #1889:
BLOCKING (boundary discipline / same-PR consumer): S9 Phase-1 Step 3
+ Phase-2 Float briefs claimed DECLARED → CONSUMER_LANDED upon merge
while only requiring cross-program handoff receipt to Grounding G2.
For target-primitive substrate entries, handoff is not landed-consumer
proof. Reframed both briefs to advance rows DECLARED → PRODUCER_LANDED;
Grounding G2 follow-on PR (per-pair lowering rules + emitted-Rust-
primitive verification) advances to CONSUMER_LANDED. Default
split-PR producer-then-consumer per bundled-scope discipline ratified
at gunbc#1739 #issuecomment-4392225548 (parallel infrastructure
DISALLOWED in same PR).
NON-BLOCKING (citation hygiene): replaced ellipsized
"#issuecomment-4392..." stubs in slice-2.5 brief with the exact
Director Option-4-rejection comment ID #issuecomment-4392081719 so
the ratification claim is mechanically checkable.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* slice-2.5 brief: absorb Director Option A revised RATIFICATION (final shape)
Director ratified Option A revised at gunbc#1739 #issuecomment-4392329709
(2026-05-06). Final brief shape:
- 4 predicates body-synthesize via Gap 1 (gt_zero / range / non_empty
/ brand)
- pattern / format / content STRUCTURALLY ABSENT from registry (not
pending-with-placeholder); user-code where-clause produces
Diagnostic::ResolveError naming Q-Regex-Primitive as separate
substrate-fact-introduction follow-on
- ~12 types.dag refined-type declarations migrate to drop
pattern/format/content where-clauses with inline-comment receipts
pointing to Q-Regex restoration path; 1 extdep (sts_endpoint)
adapts as needed
- PB-1 shim FULLY RETIRED — no exception, no preserved branch
- Q-Regex-Primitive as separate brief / worker pin / Director
ratification cycle when concretely needed (Option B XXL bundling
REJECTED)
Phase ordering rewritten to absorb migration as explicit phase
between predicate enrollment and shim retirement. proud-lynx-311
re-dispatches against revised brief; PR #1846 absorbs additional
scope per Director directive.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* s3-phase-2 brief: absorb Q-MC sub-decision 3 ratification (grammar shape was already answered)
Director observation at gunbc#1739 #issuecomment-4392382517: Q-MC
sub-decision 3 (Brian directive 2026-05-06 at gunbc#828
#issuecomment-4385530115) already ratified the grammar shape —
`Int<N>` user surface desugars to `Compose<Int, MachineWidth<N>>`
substrate parametrically. Earlier brief HOLD framing was redundant.
Brief revised:
- Replace bikeshed section with Q-MC sub-decision 3 quotation +
surface/substrate framing (Candidate C user-facing; Candidate D
substrate elaboration; A/B rejected per Director assessment)
- Phase 2.1 parser surface scoped to numeric-literal-position
recognition + parametric desugar
- Front-matter status: dispatchable per ratification (not HOLD)
- Provenance: ratification-state-grep discipline pin folded into
standing dispatch-checklist
valiant-ant-72 dispatch fires immediately per Director directive.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: remove truncated issuecomment-4392... stub from S9 Phase-1 Step 3 provenance
Address cursor NON-BLOCKING finding on PR #1889 sha 94292e48: line 144
truncated issuecomment ID. The originally-intended Director auto-nudge
comment doesn't map cleanly to a single ID, so replace with the
standing assignment reference (gunbc#1858) which is unambiguous.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: tier-1 4/5 + 5/5 — T-LBP complexity + cost lens cementing-test worker briefs
Tier-1 brief queue completes (4/5 + 5/5):
4/5 — T-LBP complexity-lens cementing test (§1.8 rows #79 + #87
first entry). Consumer-side cementing test against frozen v2-oracle
snapshot; producer-side Lens<Complexity> instance is upstream
precondition. Per Q-Lens-Behavioral-Parity-R3-Closeability option
(b) RATIFIED at gunbc#828 #issuecomment-4385329180 — T-LBP narrowed
to complexity + cost lenses only.
5/5 — T-LBP cost-lens cementing test + cost_lens_demonstration
(§1.8 rows #80 + #70 + #87 second entry). Absorbs row #70 demo
naturally — same source corpus + lens run; demonstration adds
observable-cost-bound assertions on top of parity check. ≥2
algebra-instances composed + ≥1 recursive call required per row #70.
Both briefs gate on:
- T-LBP per-lens substrate-producer brief landing (Lens<C> instance)
- Frozen v2-oracle snapshot capture (separate brief, pre-v2-retirement)
- DO NOT capture-against-live-oracle (violates
v2_oracle_no_remaining_test_consumers)
- DO NOT bundle producer/snapshot/algebra-instance authoring per
Director bundled-scope ratification at gunbc#1739
#issuecomment-4392225548
Tier-1 5/5 complete. Worker pins assigned at dispatch time once
preconditions land.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* canvas: emission-provenance substrate shape disposition (supersedes PR #1902 PROPOSAL)
Substrate Mgr canvas authored post-codex BLOCKING + PM disposition
handoff at gunbc#846 #issuecomment-4392510255.
Two real substrate-shape questions stacked:
Q1 (category mismatch): Lens<C>.read is per-Behavior; emission
provenance is per-emitted-line. 4 options surfaced — (a) per-Behavior
Lens-compatible / (b) per-line instrumentation NOT a lens / (c)
withdraw + re-canvas / (d) Mgr-surface both substrates distinct.
Mgr recommends (d) if visualization is load-bearing; (a) suffices
if per-Behavior aggregate works for the slide. Brian-channel
sub-question surfaced.
Q2 (fold-rule enumerability): T-Rule-Enumeration substrate-fact-
introduction is prerequisite for every Q1 path (rust_target.rs uses
inline &str template names; rules not enumerable). Mgr recommends
parallel-dispatch with Q1 deliberation — satisfies Brian's ASAP
framing while shape ratifies.
Q3 (§1.8 ledger retarget): #89 already taken by T-LAS; new cluster
location follows Q1 disposition.
Director ratification ask + Brian-channel sub-question surfaced.
PR #1902 close-superseded post-ratification.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: reconcile Compose<...> slot-1 spellings to algebraic-concept names per Q-MC sub-decision 3
Address cursor NON-BLOCKING on PR #1889 sha 2b49b30f: S3 Phase-2
brief mixed Compose<AbelianGroup, ...> with Compose<Int, ...>; S9
Phase-1 Step 3 used Compose<AbelianGroup, ...> + Compose<CommutativeSemiring,
...>; S9 Phase-2 used Compose<ApproximateField<Real>, ...>. All wrong
per Q-MC sub-decision 3 critical correction at gunbc#828
#issuecomment-4385530115:
"prior phrasing Compose<AbelianGroup, MachineWidth<64>> was wrong
— AbelianGroup<T> is a witness shape over carrier T, not a carrier
constructor; bare AbelianGroup in slot-1 composes witness rather
than concept."
Slot-1 is the fully-applied algebraic-concept name:
- Int<N> = Compose<Int, MachineWidth<N>> (Int = AbelianGroup<GroupCompletion<Nat>>)
- UInt<N> = Compose<UInt, MachineWidth<N>> (UInt = CommutativeSemiring<Nat>)
- Real<N> = Compose<Real, MachineWidth<N>> (Real = ApproximateField<Rational>)
Phase-2 Float brief also retargets user-facing surface from Float<N>
to Real<N> (Float is target-language name for the Rust primitive;
Real is the algebraic-concept name per Q-MC sub-decision 3 example).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* s9-p1-s3 brief: scope Deliverable 4 to producer-side verification (defer demonstration to Grounding G2)
Address Codex BLOCKING (line 82 inline + bundled review): brief
claimed end-to-end Int<32> → Rust i32 round-trip in Acceptance
while Deliverable 3 said lowering rules belong to Grounding G2
follow-on. Internal contradiction.
Reframe per Director bundled-scope discipline (gunbc#1739
#issuecomment-4392225548 — parallel infrastructure DISALLOWED
same-PR):
- Deliverable 4: producer-side substrate-axis verification only
(type-check + carrier acceptance + bootstrap snapshot/manifest
hold). NO end-to-end Rust emission in this PR.
- §1.8 #67 numeric_construction_demonstration receipts split:
producer half here (entries exist), consumer half on Grounding
G2 PR (full Int<32> → Rust i32 round-trip).
- Same-PR cross-program co-author available if Substrate Mgr
coordinates with Grounding Mgr at dispatch; default is split.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: pre-author T-Rule-Enumeration substrate-fact-introduction worker brief
Pre-authored per pre-authored-brief-queue discipline; dispatch fires
on Director Q2 ratification of emission-provenance canvas at
gunbc#828 #issuecomment-4392519713.
Brief covers LangSpec emission-rule names as enumerable substrate
(rust_target.rs uses inline &str template names today; 0 hits on
RuleName/FoldRule/EmitRule across src/v3/). Two carrier shape options
(α sum-type / β named-string list) with worker DFS at dispatch
deciding shape; Practice 4 checkpoint discipline + ledger-row receipt.
Prerequisite to every Q1 path (a/b/d) — Lens<EmissionProvenance>,
per-line instrumentation, and the both-substrates option all consume
the rule-name set. Path (c) leaves substrate-unblocked-but-unconsumed.
Mgr authoring authority per PM concur at gunbc#846
#issuecomment-4392510255 / #issuecomment-4392543633 (substrate-fact-
introduction is Mgr-tier, not PM tactical authoring).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* t-rule-enumeration: align gate name + status to Director ratification at #4392562911
* emission-provenance lens brief: revise per Director Q1 (a) RATIFICATION
Director Q1 (a) per-Behavior Lens<C>-compatible RATIFIED at gunbc#1739
#issuecomment-4392562911 (2026-05-06); (b) instrumentation REJECTED;
(d) parallel-substrates REJECTED.
Brief revised:
- Carrier shape: Lens<List<EmissionProvenance>> per (a)
- EmissionOrigin = SubstrateDeclMirror(SourceSpan) | FoldRuleAutoEmit(EmissionRule)
typed sum (closed; no third silent class; structural fail-closed)
- All 6 Lens<C> field bindings concrete (read/sequential/branch/iterate/validate)
- T-Rule-Enumeration named as PRECONDITION (must land first)
- §1.8 retarget under T-CostLens-Composition cluster per Director Q3
- Brian's slide visualization served by (a) projection (per-Behavior
list flattens to per-line view at visualization layer; no parallel
substrate needed)
Supersedes PM-authored proposal at PR #1902 (merged at 54419badf).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: bundled Rust-primitive-full-coverage substrate-prerequisite worker brief (Director Path A)
Author bundled substrate-prerequisite brief per Director Path A
RATIFICATION at gunbc#1739 #issuecomment-4392731264 (2026-05-06).
Three structurally-coupled gaps surfaced via Grounding loyal-stag-699
STOP at gunbc#1907 #issuecomment-4392585090 land as one bundled slice
per "necessary structural fix" exception (bundled-scope discipline):
1. IntervalInt::ExactInterval host-repr widening (α BigInt-based
recommended; β typed-variants if α infeasible)
2. RustPrimitive structural BoundDeclaration field (replaces static
range_min/max strings; consumes existing BoundDeclaration carrier
StaticBound + PlatformDependent variants)
3. spec/rust.dag PlatformDependent row population for isize/usize +
u128 row addition consuming widened ExactInterval
Closure gate: rust_primitive_full_coverage (Director-ratified name).
Worker pin: valiant-ibex-312 (freed-pool; substrate-authoring fresh;
numeric-domain-adjacent post-#1842).
Unblocks Grounding G2 Phase 2 full-coverage dispatch. G2 Phase 1
narrowed dispatch (i8-i64 + u8-u64) runs in parallel on existing
substrate; cleared at gunbc#1745 #issuecomment-4392795954.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: align Practice 4 classifications to GREEN/YELLOW rubric + add GREEN ledger receipts
Address codex BLOCKING on PR #1910 sha f36e1205: briefs used
PRIMITIVE/SCAFFOLD terminology that doesn't match docs/modeling-
discipline.md rubric. Per modeling-discipline.md lines 97/103/108:
the three classifications are 🟢 GREEN (terminal) / 🟡 YELLOW
(scaffold) / 🔴 RED (dissolvable-now); line 132 requires a ledger
entry if GREEN.
Replaced PRIMITIVE → GREEN, SCAFFOLD → YELLOW across all three
emission-provenance / Rust-primitive-full-coverage briefs. Added
explicit §1.8 ledger entry receipt for the GREEN classification on
EmissionOrigin (sibling row emission_origin_classification_green to
parent emission_provenance_lens_landed gate).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: Practice 4 terminal receipt for EmissionOrigin + bound for β scaffold
Address openai-pro BLOCKING (sha f36e1205, ran post-rubric-rename):
1. EmissionOrigin GREEN terminal classification needs documented
attempted-dissolutions per modeling-discipline.md "no richer
source exists" framing. Added 3 attempted dissolutions:
- collapse to optional-pair (rejected by codex Finding 1 prior)
- factor common richer source (orthogonal payload types prevent)
- third arm for unattributable lines (rejected per Slice 2.5
Option 4 placeholder anti-pattern)
2. T-Rule-Enumeration β (YELLOW) scaffold needs bound per three-part
bridge rule. Added: allowed call sites scoped to emission code only;
max lifetime gated on rule_name_typo_fail_closed_landed graduation;
new emission-introspection consumers MUST author against closed-sum
α form, not β.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* WIP: R3 Substrate
* briefs: address codex BLOCKING bundle (sha 2ed1046e) — record-form provenance + emit_model carrier alignment + T-Rule-Enumeration retirement flag
Three findings addressed:
1. Lens brief (Finding #1 — provenance shape conflates source vs rule
attribution): replaced typed-sum EmissionOrigin with record
EmissionProvenance { rule: EmissionRule [mandatory], source_span:
Option<SourceSpan> [optional] }. Co-inhabitance is real (e.g.,
#[derive(Debug)] line has both rule=derive_for_disj AND span
attribution to Foo decl). Rule-mandatory enforces fail-closed
structurally; Practice 4 N/A (record, not coproduct).
2. T-Rule-Enumeration brief (Finding #2 — rule authority lives in
spec/rust.dag + v3.std.emit_model, not call-site strings): brief
status flipped to FLAGGED FOR RETIREMENT. Same finding as
smart-ram-167 STOP at gunbc#1759 #issuecomment-4392696623; Reading
C surfaced to Director at gunbc#828 #issuecomment-4392780871.
3. Rust-primitive-full-coverage brief (Finding #3 — BoundDeclaration
not wired into TargetIntegerInhabitanceBound row surface):
reframed Deliverable 2 to target the actual carrier
`v3.std.emit_model::TargetIntegerInhabitanceBound` (used by
`TargetIntegerTypeInhabitance` rows in src/v3/spec/rust.dag lines
169-207). Two paths surfaced: Option (i) wire BoundDeclaration into
TargetIntegerInhabitanceBound; Option (ii) populate via existing
variants if PlatformDependent already exists on the live carrier.
Worker DFS at dispatch picks. Deliverable 3 retargeted to
TargetIntegerTypeInhabitance rows (the actual surface).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* rust-primitive-full-coverage: codex inline confirms TargetIntegerInhabitanceBound has no PlatformDependent variant — Option (ii) infeasible; carrier refactor REQUIRED
Codex BLOCKING inline at PR #1910 line 65 (sha 98507c432) verified at
HEAD: TargetIntegerInhabitanceBound = BoundUnspecified | StaticBoundFact(IntInterval).
No PlatformDependent variant. Earlier brief framing of Option (ii)
(populate via existing variants) was infeasible.
Brief reframed: carrier refactor is required (not optional). Two
structural shapes for the refactor — (i.a) embed BoundDeclaration
(Mgr recommendation; single-authority discipline) vs (i.b) extend
variant set with PlatformDependent. Existing 5 rows migrate either
way; bootstrap snapshot semantic-equivalence non-negotiable.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* emission-provenance lane: T-Rule-Enumeration RETIRED + Lens brief revised per Director Reading C
Director Reading C RATIFIED at gunbc#1739 #issuecomment-4392797954
(2026-05-07):
- Q1: T-Rule-Enumeration retires (substrate gap was phantom; ~37
*SyntaxBinding/*OpsBinding field paths already provide enumeration
per smart-ram-167 substrate-state-grep at gunbc#1759 #issuecomment-4392696623)
- Q2: Lens<EmissionProvenance> dispatches now (no precondition gate)
- Q3: EmissionRule = String of field-path (per feedback_reason_not_label)
Actions:
- Delete docs/briefs/r3-substrate-t-rule-enumeration-worker.md
- Lens brief: replace T-Rule-Enumeration precondition gate with
Reading C absorption section; EmissionRule = field-path String;
cementing test verifies field-path round-trip; STOP triggers
retargeted to *SyntaxBinding field-set divergence
Worker pin: smart-ram-167 (Mgr discretion per Director ratification;
fresh context on field-path enumeration). valiant-ibex-312 stays on
Rust-primitive-full-coverage / T-Interval-Representation.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* emission-provenance lens brief: absorb codex BLOCKING bundle (sha 4229cd09 post-merge findings)
Two findings from codex bundled review on now-merged PR #1910 sha
4229cd09 land as brief revisions for the in-flight smart-ram-167
authoring cycle (PR #1928):
1. Field-path identity threading: render_named_template call sites
receive template values extracted from binding structs; field-path
may not be carried alongside the value at emission time. New STOP
trigger flags this as substrate-cascade if threading isn't
structurally possible (typed template-with-rule-path or
DeclarationRef carrier needed).
2. Read domain coverage gap: Lens<C>.read is per-Behavior; emitted
output also includes declarations + program scaffold. Acceptance
scope explicitly narrowed to Behavior-attributable lines only;
declarations + scaffold provenance out-of-scope (separate substrate
cascade if surfaced). Acceptance bullet retargeted to
"Behavior-attributable span-Some + Behavior-attributable span-None"
rather than full-emission coverage.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* rust-primitive-full-coverage: revert to Option (ii) — Mgr ratification-state-grep miss caught by valiant-ibex-312
valiant-ibex-312 ratification-state-grep at gunbc#1761
#issuecomment-4393074602 surfaced that origin/main HEAD has
TargetIntegerInhabitanceBound with 3 variants including
PlatformDependentFact (mirrors BoundDeclaration::PlatformDependent).
Earlier brief framing at 4229cd09d was against pre-PlatformDependentFact
state per codex BLOCKING reading; main moved since.
Option (ii) IS feasible: isize/usize rows populate via existing
PlatformDependentFact variant directly; u128 row populates StaticBoundFact
with BigInt-widened ExactInterval (Deliverable 1 unchanged). No
carrier refactor needed; 5 existing rows untouched.
Second Mgr-tier ratification-state-grep miss today (first was symbol-only
grep on T-Rule-Enumeration; now stale-codex-vs-current-HEAD on
TargetIntegerInhabitanceBound). Worker-tier discipline caught it before
sinking time into invasive Option (i.a) refactor.
Single-authority concern (parallel-named variants on
TargetIntegerInhabitanceBound vs BoundDeclaration) noted as future
substrate-fact-introduction; not blocking this slice — explicit-mirror
is the existing convention.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* rust-primitive-full-coverage: align Acceptance + Phase ordering + STOP triggers to Option (ii) (resolve codex internal-contradiction finding)
codex BLOCKING on PR #1929 sha 14f68b63 caught real internal contradiction: Context section flipped to Option (ii) feasible at 57ae4349b but Acceptance + Phase ordering still referenced BoundDeclaration migration / RustPrimitive bound field refactor from earlier Option (i.a) framing.
All sections now consistent with Option (ii):
- Phase 4 = no-op (TargetIntegerInhabitanceBound carrier untouched)
- Phase 5 = spec/rust.dag rows for isize/usize via existing PlatformDependentFact variant
- Phase 6 = u128 row via StaticBoundFact(IntInterval) + ratchet update
- Acceptance retargeted to row-population shape (no carrier refactor; existing 5 TargetIntegerTypeInhabitance rows untouched)
- STOP triggers retargeted to TargetIntegerInhabitanceBound variant-set sufficiency at HEAD
- Why-bundled rationale rewritten to Option (ii) shape (Phase A widening unblocks u128 StaticBoundFact; isize/usize use existing PlatformDependentFact)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* lens brief: narrow Deliverable 3 verification step 4 to Behavior-attributable lines (resolve codex #2 consistency)
codex BLOCKING #2 on PR #1929 sha 812602b4 caught: Acceptance + STOP triggers narrowed to Behavior-attributable lines but Deliverable 3 cementing-test step 4 still said 'every emitted line'. Now consistent — step 4 explicitly scopes to Behavior-attributable subset; declarations + program scaffold OUT-OF-SCOPE per same Finding #2 framing.
(codex BLOCKING #1 — claim that PlatformDependentFact NOT on main — is wrong; verified at origin/main HEAD: TargetIntegerInhabitanceBound has 3 variants including PlatformDependentFact. Rebut on PR.)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: pre-author IntPlatform/UIntPlatform substrate-fact-introduction follow-on (Director ratification at #4393248961)
Per Director Q1+Q2+Q3+Q4 RATIFICATION at gunbc#1739 #issuecomment-4393248961 (2026-05-07): pre-author follow-on brief for the substrate cascade surfaced by valiant-ibex-312 STOP at gunbc#1761 #issuecomment-4393222862.
Scope:
- Q1 naming: IntPlatform / UIntPlatform per feedback_reason_not_label
- Q2 algebra: Compose<Int, MachineWidth<Platform>> + Compose<UInt, MachineWidth<Platform>>; Platform is substrate token
- Q3 substrate-concept layer (target-agnostic per feedback_target_agnostic_ir)
- Q4 worker pin: valiant-ibex-312 post-PR-#1914 close (freshest substrate context)
Brief covers Platform substrate token + IntPlatform/UIntPlatform declarations + spec/rust.dag isize/usize row population + co-receipt advancing PR #1914's rust_primitive_full_coverage gate PARTIAL → CONSUMER_LANDED.
Closure gate: int_platform_uint_platform_substrate_landed.
Dispatchable post-PR-#1914 close per pre-authored-brief-queue discipline.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
briansrls
left a comment
There was a problem hiding this comment.
Review metadata
- Provider / model:
codex/unknown - Commit:
f994e1f3· Trigger:schedule - Thinking:
215s wall
Non-blocking — Improvements (fix in-PR if easy, else defer to roadmap)
dsl/extdeps/languages/rust/primitives.dagThe added u128 row resolves Slice B2, but the file-level pilot-scope/deferral text still says u128 is deferred; update that authority text in the R3 Phase B cleanup lane.
✅ No blocking concerns; the PR keeps range facts single-authority and extends Rust primitive coverage structurally.
|
Review metadata
FindingsNone. Nothing in this diff clearly violates INVARIANTS.md, docs/modeling-discipline.md, CODING.md, or TESTING.md.
The large VerdictAPPROVE — The change is tightly scoped to R3 Phase A+B (BigInt-backed exact intervals + |
…state Address non-blocking finding on PR #1914 (codex@f994e1f3): file-level pilot-scope text still listed u128 as deferred even though the row is present in this PR. Refresh: - PILOT SCOPE list now includes u128. - T-Int128 SLICE B1/B2 block annotates the resolution: B2 deferral closed by Phase A `BigInt` host repr widening (commit `e7ba022c6`); u128 row landed in Phase B-1 (commit `59511503e`). - New FOLLOW-ON DEFERRAL block names the next cascade explicitly: isize/usize wait on substrate-side IntSize/UIntSize kernel integer introduction (Director-level scope per Mgr disposition Path 1 at gunbc#1761 #issuecomment-4393235461). Bootstrap snapshots + parse manifest regenerated to absorb the comment edit through the bootstrap span byte ranges. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Review metadata
Reviewed the hand-authored slice of the diff ( Findings: None. Nothing in the changed lines clearly violates INVARIANTS, Verdict: APPROVE — Scoped R3 Phase A+B as described: host |
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* WIP: R3 Substrate
* fix(briefs): retire INVARIANTS.md line-number cites in r3-426 brief
Per codex review on PR #1782 (sha c2512cde): the new briefs
introduced fresh `INVARIANTS.md:<line>` citations — the same
stale-line-anchor shape the :425 sweep brief itself names as needing
dissolution. Replaces four line-number references in
r3-426-sfp-subdoc-rename-and-author-worker.md (lines 16, 28, 58, 85)
with prose-anchored locators citing the section anchor + bullet text.
The other PB-lane briefs codex flagged (r3-pb-runtime-equivalence-
corpus-seed-audit.md, r3-pb-t-fixedpoint-worker.md) belong to
neat-bear-351's lane and are not in this PR's scope; will route
separately.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): correct L6 type count + r3-435 dual-path retirement receipt
Per openai-pro review on PR #1782 (sha c2512cde):
1. L6 brief said acceptance ratchet asserts "five types" exist but
declares six (ShapeATarget, FormAxis, BehaviorAxis,
MethodTemplateContractKey, EmissionCell, EmissionPathProjection).
Updated both occurrences to enumerate all six explicitly.
2. r3-435 brief authorized both Path A (typed-AST reader, deletes
IntegrationRsScan) and Path B (scanner widening with Char state),
but retirement receipt language only covered Path B. Updated
ROADMAP/ledger retirement and authority-audit step 5 to cover
both authorized dissolution paths.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): L6 second 'five types' site + T-E-P lookup-gate authority
Per cursor review on PR #1782 (sha 2072b986):
1. L6 brief acceptance bullet at :131 still said "five types" while
the parenthetical and the Slice step at :117 said six. The prior
fix only caught one of two sites; corrected the second.
2. T-E-P brief `e_p_call_pattern_lookup_authoritative` acceptance
named only `lower_call_pattern` as "the only route from call
site to LoweringTarget" — but Phase 2 explicitly requires
lenses route through `per_call_pattern_at` (the L-7 single-
authority lookup), NOT `lower_call_pattern` (compiler-internal).
Acceptance text rewritten to name `per_call_pattern_at` as the
consumer-facing gate and clarify `lower_call_pattern`'s
internal-only scope.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): strengthen L6 ratchet from count-parity to per-row bijection
Per openai-pro REQUEST_CHANGES on PR #1782 (sha 2072b986): the L6
acceptance ratchet asserted only row-count parity between
emission_path_projections and the union of MethodTemplateContract
rows. Count parity permits duplicate projection keys and missing
source rows — the bridge from list-non-empty proxy to per-row
projection would not be mechanically proven. The whole point of
introducing MethodTemplateContractKey { target, dag_method } is to
provide 1:1 row identity.
Rewrites the ratchet to assert per-row bijection: every
MethodTemplateContract source row has exactly one matching
EmissionPathProjection row by key; every projection key resolves
to exactly one source row; duplicate projection keys fail closed.
(The other openai-pro BLOCKING finding — T-E-P
e_p_call_pattern_lookup_authoritative naming the wrong authority
— was already fixed at 6282a0704; the review citation was against
the pre-fix sha 2072b986.)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): split L6 ratchet into slice-active vs deferred-activation gates
Per BLOCKING inline review on PR #1782 at d92459423: the prior
bijection ratchet was unsatisfiable as authored. The
*_method_template_contracts lists at HEAD (rust/python/go) are
non-empty; shipping emission_path_projections: [] alongside a
strict bijection requirement means the bijection cannot pass when
the slice lands — source rows would be uncovered. Prior text
incorrectly said "vacuously true on the empty source set" — the
SOURCE set isn't empty, only the projection set is.
Splits the ratchet into two gate categories:
- Slice-active gates: types/data shape exist + empty-state
predicate (emission_path_projections == []).
- Deferred-activation gate: per-row key bijection, authored as a
test scaffold by this slice but #[ignore]'d (or feature-gated
on len > 0) until Grounding's follow-up populates rows. On
activation, asserts the bijection over the populated set.
This honors P1 doc faithfulness: the slice's acceptance contract
is satisfiable as authored, and the bijection becomes the
load-bearing gate the row-population PR satisfies.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(briefs): require L6 bootstrap enrollment to honor P2 facts-flow-forward
Per BLOCKING inline review on PR #1782 at L6:34: the slice authored
src/v3/std/cross_target_coverage.dag but never required enrolling
it in the bootstrap/load authority. Without enrollment the file
is dead-letter on disk and EmissionPathProjection can be absent
from the downstream Dag — silent P2 facts-flow-forward violation.
Adds a new Slice step 2 requiring a cross_target_coverage field
on BootstrapFixtures in src/v3/std/extdeps_bootstrap_fixtures.dag
mirroring the *_method_template_contracts enrollment pattern, plus
a loader-visibility ratchet. Renumbers subsequent Slice steps.
Adds matching Acceptance bullet.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* WIP: R3 Substrate
* WIP: R3 Substrate
* docs(briefs): normalize Practice 4 citation in S3 brief per codex review
Codex review on PR #1782 (sha 2614217e) flagged stale citation
"Practice 4 (Step 4 of the type-introduction checklist)" in
S3 brief — no such checklist step exists in the pinned rubric.
Live rule per docs/modeling-discipline.md:131-134 is the "What to
check" checkpoint-comment requirement under Practice 4 (coproduct
dissolution). Updated the S3 reference to cite that line directly
and drop the stale "Step 4 of type-introduction checklist" wording.
S2/S8/S9 already use bare "Practice 4" without the stale steps wording.
S1 (committed earlier today) does not contain the stale phrase.
S6 (pre-existing brief flagged by reviewer as carrying the same
pattern) is out of this branch's scope; flag for separate paydown.
* docs(briefs): sweep stale Practice 4 citations corpus-wide
Codex review on PR #1782 flagged the stale "Step 4 of the
type-introduction checklist" wording — no such checklist exists in
docs/modeling-discipline.md. The live rule is the "What to check"
checkpoint-comment requirement under Practice 4 (coproduct
dissolution) at docs/modeling-discipline.md:131.
Sweep across docs/ found 2 additional briefs carrying the stale
phrasing (S3 fixed previously at 9db003d40):
- docs/briefs/r3-l6-emission-path-projection-substrate-worker.md
- docs/briefs/r3-x1b-s1-transform-dispatch-substrate-worker.md
Both updated to cite the live rule directly. Verified zero
remaining "type-introduction checklist" or "Step 4 of the"
references in docs/ tree.
* docs(briefs): add Practice 4 citation discipline note to authoring checklist
Per parent inbox status flag at gunbc#828 #issuecomment-4385346221:
add a Citation discipline section to brief-authoring-checklist.md
naming the correct shape — Practice 4 (coproduct dissolution) +
"What to check" rule at docs/modeling-discipline.md:131 — and
calling out the stale "Step 4 of the type-introduction checklist"
phrasing as a failure mode. References the corpus sweep at
2dc92f34f.
Prevents future briefs that copy old templates from re-introducing
the stale citation pattern.
* docs(r3): cascade work post-Director ratification — S1 worker, R4 routing, Unit/Quantity canvas
Per Director ratification at gunbc#828 inbox response 2026-05-06
(zesty-bear-812):
1. S1 Q1+Q2 RATIFIED → author S1 gap-test representative worker brief
(`r3-substrate-s1-gap-test-representative-worker.md`). Dispatchable
on T-E-P Phase 1 + E6-G0d landing; closes ledger row #61.
2. S2 Q2 RATIFIED (option-(b) T-LBP narrowing) + S3 + S8 deferrals →
R4 carve-out routing ledger (`docs/r4-carve-out-routing.md`).
Captures C1 (parallelism lens), C2 (effect_enum lens), C3
(register narrowing), C4 (deferred MachineConstraint axes),
C5 (rounding-mode product extension), C6 (Unit/Quantity).
Cascade messages to Verification / Grounding / Debt-Paydown Mgrs.
3. S9 Phase-3 dimensional refinement boundary → Unit/Quantity carrier
canvas (`r3-substrate-s9-unit-quantity-carrier-canvas.md`).
Surfaces Q-Unit-1..5 for Director ratification: carrier name,
2-axis vs 3-axis Phase-1, Duration/Seconds collapse, Refined
composition order, Practice 4 classification.
Updates §1.8 ledger receipts for carved items; R3 closure receipts
treat carved items as explicitly-deferred (not drift).
* docs(briefs): citation-discipline fix-forward — anchors/quotes, not line numbers
Director redirect at gunbc#828 #issuecomment-... noted the citation
paydown's replacement form (`docs/modeling-discipline.md:131`) is the
same drift class as the original fabricated-step error. Bare line
numbers shift silently on any doc edit above the cited line.
Generalized fix:
- brief-authoring-checklist.md "Citation discipline" section
rewritten to prohibit bare line-numbers across ALL cross-doc
references (modeling-discipline.md, r3-program-plan.md,
r3-structure.md, INVARIANTS.md, MODELING.md, etc.). Use section
anchors (#section-name) or rule-text quotes.
- 3 briefs that swapped in :131 (S3, S6, X1B) re-converted:
- docs/modeling-discipline.md#4-coproduct-dissolution anchor
- "What to check" rule quoted inline (self-contained)
Brief-authoring-checklist failure-mode section names both vectors
(fabricated step + bare line numbers) and references both sweep
commits in audit trail.
Broader cross-doc citation sweep across briefs I authored on this
branch (frontmatter authority-docs lists, body prose) is recommended
per Director note — flagging as separate paydown task; this commit
closes the immediate recurrence vector.
* docs(briefs): fix unfaithful Int=AbelianGroup<Nat> + sweep stale 'Step 4' refs
Per openai-pro review on PR #1782 sha 0ecd4818 (gunbc#1782 #...):
BLOCKING fix (P1 modeling faithfulness):
- S9 + S3 directed workers to model `Int = AbelianGroup<Nat>`,
but Nat is not an Abelian-group carrier (additive inverses
not representable). Reframed:
- `Int` is the additive Abelian group on integers (carrier Z);
`Nat` is a commutative monoid under addition.
- Algebraic relationship is **explicit group-completion**:
`Int ≡ GroupCompletion<CommutativeMonoid<Nat>>` — NOT
`AbelianGroup<Nat>` parameterization.
- Worker either models `Int` directly as `AbelianGroup`
(terminal Abelian-group instance) OR introduces explicit
`GroupCompletion<M>` substrate carrier; decision lands in
Phase-1 PR per P1 procedure.
- S3 cross-reference + Phase-4 algebra-axis preservation note
reframed accordingly.
NON-BLOCKING in-PR cleanup (Practice 4 citation discipline):
- 6 briefs still carrying stale 'Practice 4 Step 4' wording
(r3-coproduct-1/2/3, r3-l6, r3-t-e-p, r3-x1b) updated to use
the section anchor (`docs/modeling-discipline.md#4-coproduct-dissolution`)
+ inline rule-text quote ("What to check") form per the
citation discipline established in `b4cae5299`.
Closes both findings from openai-pro review.
* docs(briefs): Q-Unit-1..5 RATIFIED — apply Scale.Unit→One rename + ratification banner
Director ratified Q-Unit-1..5 at gunbc#828 #issuecomment-... 2026-05-06
(zesty-bear-812). All 5 questions ratified; one rename required:
- Scale enum's `Unit` value collides with outer `Unit<Q, S>` carrier
name (`Unit<Time, Unit>` is shadowing-confusing). Rename to `One`.
- Duration ≡ Unit<Time, One> (collapses with Seconds per Q-Unit-3).
- Milliseconds ≡ Unit<Time, Milli>.
Canvas updated:
- Top-level RATIFICATION banner records all 5 ratifications + rename
- Scale enum: `Unit` → `One` (with inline rename receipt)
- Sketch product expansions use `Unit<Time, One>` / `Unit<Time, Milli>`
- Phase-3 reframe table products updated
- Q-Unit-3 prose collapse reference updated
Subsequent worker brief authoring (Unit/Quantity carrier landing
post-prerequisite) inherits the renamed shape.
* docs(briefs): Unit/Quantity carrier worker brief — Q-Unit-1..5 RATIFIED
Worker brief consumes the ratified Unit/Quantity canvas
(`5f22fd06e` ratification banner). Phase-1 lands `Unit<Q, S>` +
`Quantity` + `Scale` declarations at the ratified shape (Scale's
`One` value per Q-Unit-1 rename). Phase-2 reframes S9 Phase-3
dimensional refinements (Duration / Seconds / Milliseconds) to
outer-Refined / inner-Unit composition per Q-Unit-4 RATIFIED.
EpochMs deferred to R4 C6 (Aspect-axis follow-up).
Practice 4 marks pre-determined per Q-Unit-5: `Unit<Q, S>` 🟢
PRIMITIVE; `Quantity` + `Scale` 🟡 SCAFFOLD with "consumed by ≥1
emission rule" dissolution trigger.
Cross-program handoff to Grounding Mgr (#1745) for dimensional
emission consumption documented.
Worker pin TBD — dispatchable when idle pool refreshes.
* docs(briefs): S9 Phase-1 reframed — Int landed at #1466; UInt-only migration
Per proud-lynx-311 dispatch ack at gunbc#1739 #issuecomment-...
2026-05-06: re-grep at HEAD revealed substrate state diverges
materially from S9 brief's "Option A vs B decide" framing.
Reality check at HEAD:
- `dsl/std/integer.dag:55` already carries
`type Int = AbelianGroup<GroupCompletion<Nat>>` per Slice 3 PR
#1466 / commit `4ba0a6d04`. Single-authority Q6 audit doc at
`docs/audit/t-numeric-construction-group-completion-6q.md`
ratified this shape and explicitly rejected the compact form.
- `dsl/std/integer.dag:56` still on legacy `type UInt = UInt64`
alias; UInt migration is the actual Phase-1 worker scope.
Brief Phase-1 step 1 reframed: Int landing acknowledged; worker
confirms HEAD match. Phase-1 step 2 reframed: UInt migration
target named (`type UInt = CommutativeMonoid<Nat>`); fixed-width
rows preserved per existing legacy-rows policy.
Historical Option A vs B framing kept as note (superseded).
Confirmed S9 dispatch ack from worker; UInt-only Phase-1 PR is
in-scope. Emission entries deferred per S3 (valiant-ant-72
pre-landing); Phase-2 deferred per S8 (quiet-boar-160 pre-landing).
* docs(briefs): fix two BLOCKING citation drift items per codex review
Codex review on PR #1782 sha 5f22fd06 flagged two authority-receipt
gaps:
1. brief-authoring-checklist.md:118 — `<this-commit>` placeholder in
the failure-mode section was unverifiable post-merge. Replaced
with `b4cae5299` (the actual anchor/quote conversion commit).
2. r3-378-http-path-helper-fail-open-worker.md:84 — `INVARIANTS.md
C-8 fail-closed` cited without anchor or rule-text quote, in
violation of the citation discipline added in this PR.
Converted to `INVARIANTS.md#p3-fail-closed` (P3 heading anchor)
+ inline rule-text quote naming P3 + C-8.
PR's own discipline now consistently applied across the diff.
* docs(briefs): sweep bare .md:NNN citations in Substrate-Mgr-authored briefs
Per Director recommendation in citation-discipline thread (gunbc#828
follow-on): bare line-number references drift silently. Swept 19
remaining instances across 5 briefs I authored on this branch:
- docs/briefs/r3-substrate-s1-q-class-2-chain-break-gap-test-canvas.md
- docs/briefs/r3-substrate-s1-gap-test-representative-worker.md
- docs/briefs/r3-substrate-s2-t-lbp-scope-calibration-canvas.md
- docs/briefs/r3-substrate-s3-machine-constraint-carrier-worker.md
Conversion: bare `file.md:NNN` → `file.md §"Section name"` form.
Stable; survives doc edits; human-readable. Section names per
HEAD heading audit.
Verification: `grep -nE '.md:[0-9]+' docs/briefs/r3-substrate-*.md
docs/r4-carve-out-routing.md` returns zero results in
Substrate-Mgr-authored scope.
Broader corpus scope (~232 references across non-Substrate briefs)
remains as separate paydown task.
* docs(briefs): fix BLOCKING + NON-BLOCKING items per openai-pro review on a462f007
Three findings addressed; 2 BLOCKING + 1 NON-BLOCKING:
BLOCKING — R3/R4 dual authority for Unit/Quantity carrier:
R4 carve-out C6 narrowed from "Unit/Quantity carrier landing"
(deferred all of it) to **"Aspect-axis (PointKind) follow-on
for EpochMs and instant/rate-shaped refinements only"**. The
2-axis Measure<Q,S> carrier landing IS in R3 per Q-Unit-1..5
RATIFIED; only the Aspect axis (PointKind = Magnitude | Instant
| Rate) carves to R4. Cascade-message C6 reference + R4 program-
plan input list updated to match. Cross-Mgr message clarifies
Grounding consumes 2-axis dimensional refinements in R3.
BLOCKING — Locked design partially applied (Scale.Unit→One
rename in canvas alternate-sketch sections):
- Canvas line 105-108 alternate sketch ("Open question: Instant
vs Duration axis") still used pre-rename `Unit` Scale value;
added pre-ratification disclaimer marking the section as
deferred to R4 C6 + noting `Unit` is pre-rename
- Canvas line 163 Q-Unit-3 collapse note: `Scale = Unit` → `Scale = One`
(BLOCKING — bare line-numbers in S1 canvas frontmatter): NOT
APPLICABLE — already fixed at sweep commit 127287a66 prior to
review; reviewer was on sha a462f007 pre-sweep.
NON-BLOCKING — placeholder receipts:
- brief-authoring-checklist.md `#issuecomment-...` → concrete ID
4385393971 (Director's anchor-discipline reply)
- r3-substrate-unit-quantity-carrier-worker.md (3 occurrences)
+ canvas RATIFICATION banner: `#issuecomment-...` → concrete ID
4385412256 (Q-Unit ratification reply)
- `<this-commit>` placeholder in checklist already fixed at 5be1c99f2
Verification: zero `#issuecomment-...` or `<this-commit>` placeholders
in docs/briefs/ + docs/r4-carve-out-routing.md.
* docs(briefs): codex BLOCKING review fixes — Practice 4 source-level + policy scope
Three BLOCKING findings on PR #1782 (codex sha b913e2f4):
1. **Citation-discipline policy scope**: addressed by scoping
the rule prospectively to briefs landing after the policy
commit (`b4cae5299`). Pre-existing corpus references are
flagged as paydown but NOT individually blocking; coordinated
single-PR sweep is the right paydown shape. Substrate-Mgr scope
already swept at `127287a66`.
2. **R4 C6 reconciliation**: already fixed at `6e5a5481a` (R4 C6
narrowed to Aspect-axis follow-on for EpochMs only; 2-axis
carrier landing IS in R3). Codex sha b913e2f4 was pre-fix.
3. **Practice 4 source-level vs PR-body conflation**: corrected.
Acceptance bullet for Unit/Quantity worker brief now requires
inline checkpoint comments on the LIVE declarations in
`dsl/std/units.dag` (load-bearing artifact per
`docs/modeling-discipline.md#4-coproduct-dissolution` "What to
check"). PR-body summary is supplementary, not substitute.
Closes the review's reconciliation gates.
* docs(briefs): absorb Q-MachineConstraint + Q-Unit-1-Recanvas ratifications
Two major Director ratifications absorbed:
1. Q-MachineConstraint-Carrier RATIFIED at gunbc#828
#issuecomment-4385530115 (Brian directive 2026-05-06: "universal
substrate, ratify defaults"; PR #1817 queued). 6 sub-decisions:
- MachineWidth<bits> only for R3; defer RegisterClass / EndianMode
/ Alignment / signedness post-R3
- Parametric Compose<Algebra, MachineConstraint> type-level
interaction shape (lookup-maps REJECTED — AlgebraMachineProduct
record-keyed table form retracted)
- Int<64> ≡ Compose<AbelianGroup, MachineWidth<64>> (algebra-side
options A and B preserve interaction semantics)
- Real<64> ≡ Compose<ApproximateField<Rational>, MachineWidth<64>>
(algebra approx + machine approx as independent composing axes)
- ≥3 pairs is minimum, not target
- UNIVERSAL substrate: every target carries machine-constraint
facts; targets without native machine-width semantics handle
omission at Grounding-level discharge (target-conditioned
lowering NOT target-conditioned substrate)
Reframed: S3 brief Phase-1, S3 Phase-3/4 emission entries,
S8 Phase-1/2/3 (ApproximateField<Real> → ApproximateField<Rational>;
AlgebraMachineProduct → Compose<...>), S9 emission entries.
2. Q-Unit-1-Recanvas RATIFIED Measure<Q, S> at gunbc#828
#issuecomment-4385539791. valiant-ibex-312's substrate-state
grep caught Unit<Q, S> collision with kernel Unit at
dsl/std/types.dag:176; Director ratified Measure<Q, S> as
outer carrier name (conventional in scientific computing;
no dsl/std/ collision).
Reframed: Unit/Quantity worker brief throughout (Unit<Q,S> →
Measure<Q,S>; dsl/std/units.dag → dsl/std/measure.dag); canvas
spec sections + sketch products (Refined<Measure<Time, ...>...>);
R4 carve-out C6 with informal/formal name disambiguation note.
Both rename + reshape are now consistent across S3 / S8 / S9 /
Unit-Q briefs + canvas + R4 routing. Workers (valiant-ant-72 on S3,
quiet-boar-160 on S8, proud-lynx-311 on S9, valiant-ibex-312 on
Unit-Q) all dispatch against the ratified shape.
* docs(briefs): S11 Slice C worker brief — ROADMAP :425 retirement post-#1801
Cascade-clearance trigger fired post-#1801 (Slice B) merge
2026-05-06. S11 brief authored per B1 RATIFIED prose+regen
bundling discipline.
Phase 1: per-file edits across the post-A+B residual list (~10
files per smart-ram-167's Slice B post-rebase enumeration);
anchor + rule-text-quote pattern per current citation discipline;
bootstrap regen in same PR (no split).
Phase 2: ROADMAP :425 row PARTIAL → Retired with 3-slice receipt;
Q-Slice-C-Retirement-Receipt resolution.
Worker pin smart-ram-167 (#1759) — Slice B precedent owner.
Dispatch packet sent separately to #1759.
* docs(briefs): Q-Refined-Phantom-Composition (c) RATIFIED — defer literal Refined form
Director ratified option (c) at gunbc#828 #issuecomment-... 2026-05-06:
drop `non_negative` predicate at Measure (phantom) layer; rely on
`Quantity = Time` tagging for non-negative-magnitude semantics.
Q-Unit-4 outer-Refined / inner-Measure composition is preserved as
**conceptual model**; the literal `Refined<Measure<...>, predicate>`
form is **deferred to value-typed-integration follow-up** when
substrate gains Refined<> as first-class type expression and
dimensional refinements get value-typed integration.
Updates:
- Canvas RATIFICATION banner Q-Unit-4 row reframed to "conceptual
model; literal form deferred"
- Canvas Phase-3 reframe table: Duration / Milliseconds drop the
Refined<...> wrapper; rely on Quantity tag
- Worker brief Phase-2 prose + reframe table + Acceptance bullet
all reflect deferred-literal-form posture
valiant-ibex-312 unblocked to proceed with Phase-2 (Grounding
handoff doc) per the (c)-shaped Acceptance.
* docs(briefs): S12 F2 + F8 doc-sharpening worker brief — post-#1820 cascade
#1820 (Slice C) merged 2026-05-06; ROADMAP :425 row Retired closes
the prose-reference paydown program. S12 cascade-clearance trigger
fires per B6 RATIFIED single bundled Mgr-tier doc-sharpening PR.
Scope: F2 (.v3 filename-suffix grammar dispatch) + F8 (bootstrap
load-order/exclusion authority) — single bundled PR per B6.
Cross-program coordination with PB Mgr (#1742) per design schedule
§3 P5 (comment-only OR co-author; no duplicate PR).
Worker pin smart-ram-167 (#1759) — Slice C precedent owner;
pattern-familiar with prose+regen bundling discipline. Dispatch
packet sent separately to #1759.
* docs(briefs): S9 brief — UInt landed via #1818; algebra label corrected to CommutativeSemiring
Per proud-lynx-311 merge report at gunbc#1746 #issuecomment-...
2026-05-06: PR #1818 landed `type UInt = Nat` after codex BLOCKING
review on initial `CommutativeMonoid<Nat>` form (P2 facts-flow-forward
violation — wrap projects out Nat's multiplicative monoid + identity 1).
Final landed form preserves full `CommutativeSemiring<Magnitude>`
surface — Nat IS a commutative semiring (addition + multiplication
+ 0 + 1). Algebra-side label for UInt is CommutativeSemiring, NOT
CommutativeMonoid (corrects my earlier dispatch labeling).
S9 Phase-1 step 2 marked LANDED with merge ref. Phase-1 step 3
emission entries reframed:
- Int<N> ≡ Compose<AbelianGroup, MachineWidth<N>>
- UInt<N> ≡ Compose<CommutativeSemiring, MachineWidth<N>>
Worker discipline + reviewer caught a substrate-state error in my
brief framing; brief now matches landed reality.
* docs(briefs): fix contradictory (c) guidance in Unit/Quantity worker brief
Codex BLOCKING review on PR #1825 sha 1bce0418 caught a real
contradiction: brief Phase-2 prose at lines 115-118 stated
"predicates do NOT attach at the Measure (phantom) layer" per
Q-Refined-Phantom-Composition (c) RATIFIED, but immediately
following paragraph at lines 130-132 still said "Refined predicate
non_negative applies at the Measure-typed level" — left over from
the original Q-Unit-4 framing.
Updated lines 130-137 to consistently apply (c) shape: predicates
deferred to value-typed-integration follow-up; dimensional semantics
ride on Quantity tag at Measure layer; worker authors 3 in-scope
refinements WITHOUT predicate at this layer.
P2 boundary discipline restored: brief now gives single-authority
guidance for Phase-2 substrate landing.
* docs(briefs): name predicate authority + concretize ratification ID — codex BLOCKING
Codex BLOCKING review on PR #1825 sha 32541a56:
1. (BLOCKING) Q-Refined-Phantom-Composition (c) ratification deferred
phantom-carrier predicates without naming the value-level predicate
authority. Updated canvas Q-Unit-4 RATIFICATION row to explicitly
name: 'predicates re-attach at value-typed Int/UInt level via
where-sugar (e.g., range(min: 0) per current dsl/std/types.dag
precedent)' when value-typed integration lands.
2. (BLOCKING) Worker brief option-c reframe didn't sweep the audit
receipt §5 (Carrier dissolves the bridge). Updated to consistently
apply (c) shape — dimensional semantics ride on Quantity tag at
Measure layer; value-level predicate authority deferred; until
then, range/gt_zero constraints attach at value-typed Int/UInt
via where-sugar per dsl/std/types.dag convention.
3. (NON-BLOCKING) Canvas had #issuecomment-... placeholder for the
(c) ratification message ID. Replaced with concrete
#issuecomment-4385687075 per dashboard relay header.
Brief now names a single value-level authority (Int/UInt where-sugar)
for value-range facts; phantom-layer Quantity tag carries only
dimensional semantics. P2 boundary discipline maintained.
* docs(briefs): S12 RETIRED per Director option (2) ratification
Director ratified option (2) at gunbc#828 #issuecomment-4385873...
2026-05-06 post smart-ram-167's substrate-state STOP-AND-ESCALATE:
- F2 absorbed by 2026-05-04 routing + #1662 closure
(`Result/DivError span.file-keyed` retired)
- F8 topical concerns distributed across live rows
(Class 5 Gap 3 `bootstrap.rs::std_fixtures()`; Character-level
load-set / bootstrap; Filename-sentinel-bridges) with own owners
- Doc-discipline goals absorbed by per-Mgr citation paydown
routing at gunbc#828 #issuecomment-4385583247
Frontmatter updated to RETIRED with absorption receipt. Worker
(smart-ram-167) pin freed.
Director-side acknowledgment of Substrate Mgr's option (2) recommendation:
"Substrate Mgr's option (2) recommendation is the right move
structurally. Most Mgrs default to 'synthesize work to fill the brief'
when canvas-vs-HEAD mismatch surfaces. Recommending retirement when
the work is genuinely absorbed is harder discipline."
* docs(briefs): S7 PR-F (BoundDeclaration consumer + ReferenceModel<T>) worker brief
Per cascade-clearance trigger fired post-#1782 merge 2026-05-06.
loyal-wolf-828 (#1764) acknowledged S7 dispatch with substrate-state
pre-flight grep (gunbc#1764 #issuecomment-4385872...) and surfaced
await-brief posture. This brief satisfies that prerequisite.
Worker pre-flight inventory absorbed:
- BoundDeclaration substrate landed (substrate.dag StaticBound +
PlatformDependent); partial BoundDeclarationView consumer in
fold.rs (ScratchIntExamples scope; doc-explicit partial)
- ReferenceModel<T> NOT yet in dsl/std/ (substrate-fact-introduction
required; design-emission-model.md §Q2 framing)
Phase-1: BoundDeclaration consumer broadening to full
design-emission-model.md surface (u128 / isize / usize / walker
arms / pilot mirror).
Phase-2: ReferenceModel<T> P1 substrate-fact-introduction with
DFS-of-concept-DAG + Practice 4 in-source checkpoint comment +
named consumer demand (Grounding G1).
Phase-3: cross-program handoff to Grounding Mgr (#1745) for
G1 (T-Ground-Rust Phase 1).
Closure predicate: unblocks Grounding T-Ground-Rust Phase 1.
Worker pin loyal-wolf-828 holds; dispatch fires immediately
on this brief landing.
* docs(briefs): S9 Slice 2.5 — predicate-registry + PB-1 shim retirement (Path 3 RATIFIED)
Director ratified Path 3 at gunbc#828 #issuecomment-4390333451
post proud-lynx-311's pre-flight DFS at gunbc#1746 #issuecomment-4390253616.
Brief reframes Slice 2.5 from S-sized (primitive + 1 consumer) to
L-sized (predicate registry + PB-1 shim retirement). 5 deliverables:
1. gt_zero primitive predicate
2. Predicate registry infrastructure (resolver-side substrate-fact-introduction)
3. PB-1 shim retirement (span.file=='dsl/std/types.dag' special case dissolves)
4. Migration of existing types.dag predicates (range, pattern, etc.)
5. PositiveInt = Refined<Nat, gt_zero> consumer update
Per Director hard scope bars: no parallel predicate-resolution path;
no per-call-site special-casing; PB-1 shim retirement non-negotiable
in same slice. Single PR per Path 3 RATIFIED.
Discipline anchors: feedback_construction_over_ratchets (PB-1 shim
is bridge); feedback_dissolve_bridges (registry is structural fix);
feedback_no_textual_enforcement_bridges (file-path special-case is
anti-pattern shape); P1 substrate-fact-introduction procedure.
Worker pin proud-lynx-311 — Director pre-cleared on pre-flight DFS
context. PR #1840 (partial Slice 2 — NonNegativeInt = Nat) MERGED
at 16:54Z; Slice 2.5 dispatchable immediately on this brief landing.
* docs(briefs): Slice 2.5 audit receipt — Mgr independent grep parallel-verifies worker DFS
Codex BLOCKING review on PR #1845 sha 2d230949 flagged: external PR
receipt absorbed as live substrate state without re-grepping local
std declarations at brief authoring time. Authority receipt §1
cited proud-lynx-311's pre-flight DFS as authority but didn't
record Mgr-side independent verification.
Updated audit receipt §1 + §4 to record Mgr's independent grep at
HEAD as parallel verification:
- gt_zero zero matches in dsl/std/ + src/v3/ (verified)
- NonNegativeInt = Nat at dsl/std/integer.dag:133 (#1840 land)
- PositiveInt = Int where range(min: 1) at dsl/std/types.dag:255
- PB-1 shim doc-block confirms bridge-text framing
- Predicate registry confirmed not present
Worker DFS remains the dispatch-time discipline (substrate-state
grep at dispatch); Mgr's independent grep is parallel verification,
not substitute. Per Director's pattern note at gunbc#828
#issuecomment-4390199218 + #issuecomment-4390333451: dispatch-packet
authoring discipline includes Mgr-side substrate-state-grep, not
just relay of worker findings.
* docs(briefs): Slice 2.5 brief — Int extensibility scope-bound; lower.rs cites anchored
openai-pro REQUEST_CHANGES on PR #1845 sha 2d230949 flagged 2 issues:
BLOCKING — P1 modeling faithfulness / substrate scope leak:
`gt_zero` Int-side extensibility was left to 'worker judgment if
generalization cheap'. Per feedback_construction_over_ratchets:
don't speculatively extend predicate-carrier compatibility ahead
of named consumer demand. Reframed to **EXPLICITLY OUT-OF-SCOPE**
for Slice 2.5; Nat-only registration. Future Int-side consumer
surfaces as separate substrate-fact-introduction (P1 procedure)
brief with own consumer-demand receipt.
NON-BLOCKING — citation discipline self-violation:
3 instances of bare `lower.rs:821-835` line-range citations
contradict the brief's own 'no bare :NNN' requirement at line 211.
Converted to function-name anchored form:
`src/v3/compiler/src/lower.rs` (`lower_type_alias_refinements_phase` doc-block).
Both findings absorbed; no parallel scope ambiguity remains;
brief now self-consistent on citation discipline.
* WIP: R3 Substrate
* docs(briefs): Slice 2.5 brief — audit-receipt cites converted to declaration-name anchors
cursor APPROVE_WITH_COMMENTS on PR #1845 sha ebf6ac12 noted citation
self-inconsistency: audit-receipt section §1 + §4 used bare
`dsl/std/integer.dag:133` and `dsl/std/types.dag:255` line-number
pins despite the brief's own no-bare-:NNN rule.
Converted to declaration-name-anchored form:
- 'declaration named NonNegativeInt' (with reproducible grep command)
- 'declaration named PositiveInt' (with reproducible grep command)
- 'lower_type_alias_refinements_phase doc-block' for PB-1 location
Brief now fully self-consistent on its own citation discipline.
Verified zero remaining .dag:NNN / .rs:NNN / .md:NNN bare cites.
* docs(briefs): Slice 2.5 brief — Path (a) RATIFIED carrier+argument contract verification
Director Path (a) RATIFIED at gunbc#828 #issuecomment-4390760353
(triage) + #issuecomment-4390794121 (convergence ack):
predicate-body lowering with carrier+argument contract
verification at lower-time IN SAME SLICE. Placeholder semantics
REMOVED from registry path.
Codex REQUEST_CHANGES surfaced that registered-predicate-without-
contract = silent acceptance per feedback_fail_closed_discipline.
Brief reframe absorbs Path (a):
Hard scope bar #5 added:
- Same-slice carrier+argument-contract verification
- Mismatched carrier OR malformed argument → Diagnostic::ResolveError
- No 'named but not checked' path remains
Scope expansion section added (L → XL):
- Predicate-body lowering for all 7 registered predicates
- Per-predicate arg-shape validation
- Multi-session work acknowledged
- Carrier-check at 788d6acb4 retained; predicate-body builds on top
Acceptance bullets updated: gt_zero with arg-shape validation +
all 7 predicates with carrier+argument contract verification.
Director-pinned brief-authoring discipline: dissolution trigger
named in a brief authored as substrate-fact-introduction must
specify same-slice acceptance, not deferred follow-up. Otherwise
the brief encodes 'land bridge + defer dissolution' anti-pattern,
the same shape feedback_construction_over_ratchets warns against
at implementation layer applied at brief authoring layer.
This is the 11th miss this session — but the pattern surfacing it
(direct sweep + Codex REQUEST_CHANGES + Director ratification)
produces structural fixes that prevent the next miss class. Brief
revision is the structural recovery.
* docs(briefs): Slice 2.5 brief — Path 2 RATIFIED (Gap 1 discharge + Q-Regex carve-out)
Director Path 2 RATIFIED at gunbc#828 #issuecomment-4391985613
post Rung 3 STOP at gunbc#1746 #issuecomment-4391946213.
Path 2 narrowing of Path (a):
- gt_zero + range body synthesis already landed (5f3c40c3a)
- Gap 1 discharge mechanism: extends scalar-literal predicate
evaluation at lower-time. L-class; lands SAME SLICE per
same-slice-dissolution discipline
- non_empty + brand body synthesis: lands via Gap 1 discharge
- pattern / format / content: fail-closed-with-named-dep at
user-code authoring layer (Diagnostic::ResolveError naming
Q-Regex-Primitive). NOT placeholder; structural rejection
- PB-1 shim FULLY RETIRED per hard scope bar #3
- Q-Regex-Primitive carved as follow-on substrate-fact-introduction
(XL substrate; not pre-authored per feedback_construction_over_ratchets)
Acceptance updated:
- 4 of 7 body-synthesized in same slice
- 3 of 7 fail-closed-with-named-dep (no placeholder semantics)
- Gap 1 discharge mechanism landed (no partial implementation)
Worker disposition: proud-lynx-311 continues Rung 3 with Gap 1
implementation + non_empty/brand body synthesis + pattern/format/
content fail-closed-with-named-dep diagnostic.
* docs(briefs): S9 Phase-1 Step 3 emission entries — Tier-1 brief authoring (1/5)
First Tier-1 brief landing post Director auto-nudge on assignment
#1858. Authored against Q-MachineConstraint sub-decisions ratified
shape now landed at #1856:
- Int<N> ≡ Compose<AbelianGroup, MachineWidth<N>> → Rust i32/i64/i128
- UInt<N> ≡ Compose<CommutativeSemiring, MachineWidth<N>> → Rust u32/u64/u128
- 6 concrete instantiations (≥3 minimum per sub-decision 5)
- Cross-program handoff to Grounding Mgr (#1745) for G2 emission consumption
- numeric_construction_demonstration Acceptance bullet (Int<32> round-trip);
Real<64> half deferred to S9 Phase 2 / S8 Float cascade
Brief authored with substrate-state-grep discipline + same-slice-
dissolution discipline (no deferred trigger; demonstration in same
slice or explicitly carved). 5-question authority audit completed
in brief body.
Worker pin proud-lynx-311 (S9 holds); brief queues post-Slice-2.5
completion as natural follow-on.
Tier-1 brief queue progress:
- 1/5 authored (this brief)
- 4 remaining: S9 Phase-2 Float coordination; T-LBP complexity
cementing; T-LBP cost cementing; S3 Phase-2 parser-grammar
* docs(briefs): S3 Phase-2 parser-grammar surface — Tier-1 brief 2/5
Authored per Director freed-pool pressure at gunbc#828
#issuecomment-4392095857 + Tier-1 brief-queue commitment.
Brief covers parser-grammar surface for Compose<Algebra, MachineConstraint>
interaction syntax post-#1856 (Phase-1 carrier slice landed):
- Phase 2.1: Parser surface per ratified Q-MachineConstraint-Grammar-Shape
- Phase 2.2: Bootstrap demonstrator with >=3 algebra*machine-axis pairs
- Phase 2.3: Class 1 5-criteria Pass receipt (criteria 1-4; criterion 5
v2-oracle parity may carve)
- Phase 2.4: numeric_construction_demonstration co-receipt with S9 Phase-1
Step 3 emission entries brief
Director ratification HOLDS on grammar bikeshed:
- Candidate A: Int @ Width<32> (annotation form)
- Candidate B: Int with MachineWidth<32> (with-form)
- Candidate C: Int<32> (positional generic — fits existing convention)
- Candidate D: Compose<AbelianGroup, MachineWidth<32>> (no parser change;
type aliases provide convenience names)
Worker pin valiant-ant-72 holds (S3 Phase-1 precedent owner); freed-pool
until Director ratifies grammar shape.
Tier-1 brief queue progress: 2/5 authored (this brief + S9 Phase-1 Step 3).
Remaining: S9 Phase-2 Float coordination; T-LBP complexity cementing;
T-LBP cost cementing.
* docs(briefs): S3 Phase-2 brief — fix internal contradiction on Class 1 closure scope
Codex BLOCKING review on PR #1889 sha ec609b4a flagged real
contradiction:
- Provenance section claimed '5-criteria Pass closed' (line 229)
- Phase 2.3 said 'criterion 5 may queue separately' (line 154)
- STOP-AND-ESCALATE bullet 1 ratified 4-of-5 carve-out (line 181)
Updated Provenance to consistently state criteria 1-4 close in this
slice; criterion 5 (v2-oracle parity) cementing test may carve to
separate slice. Single 4-of-5 framing throughout. INVARIANTS.md
'Documentation Describes Live State' satisfied.
Per feedback_construction_over_ratchets: don't claim closure that
isn't structurally complete. 4-of-5 + named carve is the honest
shape; 5/5 only on cementing-test land.
* briefs: cite Q-MachineConstraint sub-decision 6 supersession of T-V2-Retirement gate
Address Codex BLOCKING #1 on PR #1889: S3 Phase-2 + S9 Phase-1 Step 3
worker briefs consumed Q-MachineConstraint ratification without
reconciling the historical T-V2-Retirement-landing-first cascade
gate. Add an explicit precondition-framing section to each brief
citing Director ratification at gunbc#828 #issuecomment-4385530115
(Q-MachineConstraint sub-decision 6 UNIVERSAL substrate posture)
as the implicit supersession; PR #1856 landing without T-V2-Retirement
first confirms supersession in practice. STOP-AND-ESCALATE if
supersession contested at dispatch.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* slice-2.5 brief: mark pattern/format/content Path 2 acceptance pending Director ratification
Address Codex BLOCKING #2 on PR #1889: Path 2 deliverable + acceptance
language asserted fail-closed-with-named-dep semantics for
pattern/format/content, but Director rejected that shape (Option 4)
at gunbc#828 #issuecomment-4392.... Options A (consumer migration),
B (Q-Regex bundled into slice), C (typed fail-closed exception with
named trigger) await ratification per migration-cost catalog at
#issuecomment-4392116215. Brief now flags those three predicates +
PB-1 shim retirement coupling as PENDING DIRECTOR RATIFICATION; worker
STOP-AND-ESCALATE if dispatched pre-ratification.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* slice-2.5 brief: reconcile PB-1 retirement shape with Options A/B/C gating
Address Codex BLOCKING #4 on PR #1889: prior text claimed "PB-1
FULLY RETIRED" while preceding bullet said retirement gates on
chosen path — internal contradiction. Reframe to gate the
pattern/format/content half of shim retirement on Options A/B/C
ratification while preserving Director hard scope bar #3 binding
on the gt_zero/range/non_empty/brand half (cleanly retired via
Gap 1). No path leaves PB-1 alive as parallel-authority shim on
slice landing.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: tier-1 3/5 — S9 Phase-2 Float emission coordination worker brief
Tier-1 brief queue progress (3/5): S9 Phase-2 Float<N> emission
entries (Float<32>/Float<64> via Compose<ApproximateField<Real>,
MachineWidth<N>>) + Real<64> round-trip demonstration completing
§1.8 #67 Acceptance bullet. Coordination/synthesis brief — gates on
both S8 (ApproximateField<F> + Real base carrier) and S9 Phase-1
Step 3 (Int<N>/UInt<N> emission-entry pattern) landing before
dispatch. Inherits T-V2-Retirement supersession framing from
Phase-1 Step 3. STOP-AND-ESCALATE on missing preconditions or
unratified Q-ApproximateField-Axiom-Set.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: ledger advancement honesty (PRODUCER_LANDED not CONSUMER_LANDED) + restore exact issuecomment IDs
Address openai-pro BLOCKING + NON-BLOCKING findings on PR #1889:
BLOCKING (boundary discipline / same-PR consumer): S9 Phase-1 Step 3
+ Phase-2 Float briefs claimed DECLARED → CONSUMER_LANDED upon merge
while only requiring cross-program handoff receipt to Grounding G2.
For target-primitive substrate entries, handoff is not landed-consumer
proof. Reframed both briefs to advance rows DECLARED → PRODUCER_LANDED;
Grounding G2 follow-on PR (per-pair lowering rules + emitted-Rust-
primitive verification) advances to CONSUMER_LANDED. Default
split-PR producer-then-consumer per bundled-scope discipline ratified
at gunbc#1739 #issuecomment-4392225548 (parallel infrastructure
DISALLOWED in same PR).
NON-BLOCKING (citation hygiene): replaced ellipsized
"#issuecomment-4392..." stubs in slice-2.5 brief with the exact
Director Option-4-rejection comment ID #issuecomment-4392081719 so
the ratification claim is mechanically checkable.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* slice-2.5 brief: absorb Director Option A revised RATIFICATION (final shape)
Director ratified Option A revised at gunbc#1739 #issuecomment-4392329709
(2026-05-06). Final brief shape:
- 4 predicates body-synthesize via Gap 1 (gt_zero / range / non_empty
/ brand)
- pattern / format / content STRUCTURALLY ABSENT from registry (not
pending-with-placeholder); user-code where-clause produces
Diagnostic::ResolveError naming Q-Regex-Primitive as separate
substrate-fact-introduction follow-on
- ~12 types.dag refined-type declarations migrate to drop
pattern/format/content where-clauses with inline-comment receipts
pointing to Q-Regex restoration path; 1 extdep (sts_endpoint)
adapts as needed
- PB-1 shim FULLY RETIRED — no exception, no preserved branch
- Q-Regex-Primitive as separate brief / worker pin / Director
ratification cycle when concretely needed (Option B XXL bundling
REJECTED)
Phase ordering rewritten to absorb migration as explicit phase
between predicate enrollment and shim retirement. proud-lynx-311
re-dispatches against revised brief; PR #1846 absorbs additional
scope per Director directive.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* s3-phase-2 brief: absorb Q-MC sub-decision 3 ratification (grammar shape was already answered)
Director observation at gunbc#1739 #issuecomment-4392382517: Q-MC
sub-decision 3 (Brian directive 2026-05-06 at gunbc#828
#issuecomment-4385530115) already ratified the grammar shape —
`Int<N>` user surface desugars to `Compose<Int, MachineWidth<N>>`
substrate parametrically. Earlier brief HOLD framing was redundant.
Brief revised:
- Replace bikeshed section with Q-MC sub-decision 3 quotation +
surface/substrate framing (Candidate C user-facing; Candidate D
substrate elaboration; A/B rejected per Director assessment)
- Phase 2.1 parser surface scoped to numeric-literal-position
recognition + parametric desugar
- Front-matter status: dispatchable per ratification (not HOLD)
- Provenance: ratification-state-grep discipline pin folded into
standing dispatch-checklist
valiant-ant-72 dispatch fires immediately per Director directive.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: remove truncated issuecomment-4392... stub from S9 Phase-1 Step 3 provenance
Address cursor NON-BLOCKING finding on PR #1889 sha 94292e48: line 144
truncated issuecomment ID. The originally-intended Director auto-nudge
comment doesn't map cleanly to a single ID, so replace with the
standing assignment reference (gunbc#1858) which is unambiguous.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: tier-1 4/5 + 5/5 — T-LBP complexity + cost lens cementing-test worker briefs
Tier-1 brief queue completes (4/5 + 5/5):
4/5 — T-LBP complexity-lens cementing test (§1.8 rows #79 + #87
first entry). Consumer-side cementing test against frozen v2-oracle
snapshot; producer-side Lens<Complexity> instance is upstream
precondition. Per Q-Lens-Behavioral-Parity-R3-Closeability option
(b) RATIFIED at gunbc#828 #issuecomment-4385329180 — T-LBP narrowed
to complexity + cost lenses only.
5/5 — T-LBP cost-lens cementing test + cost_lens_demonstration
(§1.8 rows #80 + #70 + #87 second entry). Absorbs row #70 demo
naturally — same source corpus + lens run; demonstration adds
observable-cost-bound assertions on top of parity check. ≥2
algebra-instances composed + ≥1 recursive call required per row #70.
Both briefs gate on:
- T-LBP per-lens substrate-producer brief landing (Lens<C> instance)
- Frozen v2-oracle snapshot capture (separate brief, pre-v2-retirement)
- DO NOT capture-against-live-oracle (violates
v2_oracle_no_remaining_test_consumers)
- DO NOT bundle producer/snapshot/algebra-instance authoring per
Director bundled-scope ratification at gunbc#1739
#issuecomment-4392225548
Tier-1 5/5 complete. Worker pins assigned at dispatch time once
preconditions land.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* canvas: emission-provenance substrate shape disposition (supersedes PR #1902 PROPOSAL)
Substrate Mgr canvas authored post-codex BLOCKING + PM disposition
handoff at gunbc#846 #issuecomment-4392510255.
Two real substrate-shape questions stacked:
Q1 (category mismatch): Lens<C>.read is per-Behavior; emission
provenance is per-emitted-line. 4 options surfaced — (a) per-Behavior
Lens-compatible / (b) per-line instrumentation NOT a lens / (c)
withdraw + re-canvas / (d) Mgr-surface both substrates distinct.
Mgr recommends (d) if visualization is load-bearing; (a) suffices
if per-Behavior aggregate works for the slide. Brian-channel
sub-question surfaced.
Q2 (fold-rule enumerability): T-Rule-Enumeration substrate-fact-
introduction is prerequisite for every Q1 path (rust_target.rs uses
inline &str template names; rules not enumerable). Mgr recommends
parallel-dispatch with Q1 deliberation — satisfies Brian's ASAP
framing while shape ratifies.
Q3 (§1.8 ledger retarget): #89 already taken by T-LAS; new cluster
location follows Q1 disposition.
Director ratification ask + Brian-channel sub-question surfaced.
PR #1902 close-superseded post-ratification.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: reconcile Compose<...> slot-1 spellings to algebraic-concept names per Q-MC sub-decision 3
Address cursor NON-BLOCKING on PR #1889 sha 2b49b30f: S3 Phase-2
brief mixed Compose<AbelianGroup, ...> with Compose<Int, ...>; S9
Phase-1 Step 3 used Compose<AbelianGroup, ...> + Compose<CommutativeSemiring,
...>; S9 Phase-2 used Compose<ApproximateField<Real>, ...>. All wrong
per Q-MC sub-decision 3 critical correction at gunbc#828
#issuecomment-4385530115:
"prior phrasing Compose<AbelianGroup, MachineWidth<64>> was wrong
— AbelianGroup<T> is a witness shape over carrier T, not a carrier
constructor; bare AbelianGroup in slot-1 composes witness rather
than concept."
Slot-1 is the fully-applied algebraic-concept name:
- Int<N> = Compose<Int, MachineWidth<N>> (Int = AbelianGroup<GroupCompletion<Nat>>)
- UInt<N> = Compose<UInt, MachineWidth<N>> (UInt = CommutativeSemiring<Nat>)
- Real<N> = Compose<Real, MachineWidth<N>> (Real = ApproximateField<Rational>)
Phase-2 Float brief also retargets user-facing surface from Float<N>
to Real<N> (Float is target-language name for the Rust primitive;
Real is the algebraic-concept name per Q-MC sub-decision 3 example).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* s9-p1-s3 brief: scope Deliverable 4 to producer-side verification (defer demonstration to Grounding G2)
Address Codex BLOCKING (line 82 inline + bundled review): brief
claimed end-to-end Int<32> → Rust i32 round-trip in Acceptance
while Deliverable 3 said lowering rules belong to Grounding G2
follow-on. Internal contradiction.
Reframe per Director bundled-scope discipline (gunbc#1739
#issuecomment-4392225548 — parallel infrastructure DISALLOWED
same-PR):
- Deliverable 4: producer-side substrate-axis verification only
(type-check + carrier acceptance + bootstrap snapshot/manifest
hold). NO end-to-end Rust emission in this PR.
- §1.8 #67 numeric_construction_demonstration receipts split:
producer half here (entries exist), consumer half on Grounding
G2 PR (full Int<32> → Rust i32 round-trip).
- Same-PR cross-program co-author available if Substrate Mgr
coordinates with Grounding Mgr at dispatch; default is split.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: pre-author T-Rule-Enumeration substrate-fact-introduction worker brief
Pre-authored per pre-authored-brief-queue discipline; dispatch fires
on Director Q2 ratification of emission-provenance canvas at
gunbc#828 #issuecomment-4392519713.
Brief covers LangSpec emission-rule names as enumerable substrate
(rust_target.rs uses inline &str template names today; 0 hits on
RuleName/FoldRule/EmitRule across src/v3/). Two carrier shape options
(α sum-type / β named-string list) with worker DFS at dispatch
deciding shape; Practice 4 checkpoint discipline + ledger-row receipt.
Prerequisite to every Q1 path (a/b/d) — Lens<EmissionProvenance>,
per-line instrumentation, and the both-substrates option all consume
the rule-name set. Path (c) leaves substrate-unblocked-but-unconsumed.
Mgr authoring authority per PM concur at gunbc#846
#issuecomment-4392510255 / #issuecomment-4392543633 (substrate-fact-
introduction is Mgr-tier, not PM tactical authoring).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* t-rule-enumeration: align gate name + status to Director ratification at #4392562911
* emission-provenance lens brief: revise per Director Q1 (a) RATIFICATION
Director Q1 (a) per-Behavior Lens<C>-compatible RATIFIED at gunbc#1739
#issuecomment-4392562911 (2026-05-06); (b) instrumentation REJECTED;
(d) parallel-substrates REJECTED.
Brief revised:
- Carrier shape: Lens<List<EmissionProvenance>> per (a)
- EmissionOrigin = SubstrateDeclMirror(SourceSpan) | FoldRuleAutoEmit(EmissionRule)
typed sum (closed; no third silent class; structural fail-closed)
- All 6 Lens<C> field bindings concrete (read/sequential/branch/iterate/validate)
- T-Rule-Enumeration named as PRECONDITION (must land first)
- §1.8 retarget under T-CostLens-Composition cluster per Director Q3
- Brian's slide visualization served by (a) projection (per-Behavior
list flattens to per-line view at visualization layer; no parallel
substrate needed)
Supersedes PM-authored proposal at PR #1902 (merged at 54419badf).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: bundled Rust-primitive-full-coverage substrate-prerequisite worker brief (Director Path A)
Author bundled substrate-prerequisite brief per Director Path A
RATIFICATION at gunbc#1739 #issuecomment-4392731264 (2026-05-06).
Three structurally-coupled gaps surfaced via Grounding loyal-stag-699
STOP at gunbc#1907 #issuecomment-4392585090 land as one bundled slice
per "necessary structural fix" exception (bundled-scope discipline):
1. IntervalInt::ExactInterval host-repr widening (α BigInt-based
recommended; β typed-variants if α infeasible)
2. RustPrimitive structural BoundDeclaration field (replaces static
range_min/max strings; consumes existing BoundDeclaration carrier
StaticBound + PlatformDependent variants)
3. spec/rust.dag PlatformDependent row population for isize/usize +
u128 row addition consuming widened ExactInterval
Closure gate: rust_primitive_full_coverage (Director-ratified name).
Worker pin: valiant-ibex-312 (freed-pool; substrate-authoring fresh;
numeric-domain-adjacent post-#1842).
Unblocks Grounding G2 Phase 2 full-coverage dispatch. G2 Phase 1
narrowed dispatch (i8-i64 + u8-u64) runs in parallel on existing
substrate; cleared at gunbc#1745 #issuecomment-4392795954.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: align Practice 4 classifications to GREEN/YELLOW rubric + add GREEN ledger receipts
Address codex BLOCKING on PR #1910 sha f36e1205: briefs used
PRIMITIVE/SCAFFOLD terminology that doesn't match docs/modeling-
discipline.md rubric. Per modeling-discipline.md lines 97/103/108:
the three classifications are 🟢 GREEN (terminal) / 🟡 YELLOW
(scaffold) / 🔴 RED (dissolvable-now); line 132 requires a ledger
entry if GREEN.
Replaced PRIMITIVE → GREEN, SCAFFOLD → YELLOW across all three
emission-provenance / Rust-primitive-full-coverage briefs. Added
explicit §1.8 ledger entry receipt for the GREEN classification on
EmissionOrigin (sibling row emission_origin_classification_green to
parent emission_provenance_lens_landed gate).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: Practice 4 terminal receipt for EmissionOrigin + bound for β scaffold
Address openai-pro BLOCKING (sha f36e1205, ran post-rubric-rename):
1. EmissionOrigin GREEN terminal classification needs documented
attempted-dissolutions per modeling-discipline.md "no richer
source exists" framing. Added 3 attempted dissolutions:
- collapse to optional-pair (rejected by codex Finding 1 prior)
- factor common richer source (orthogonal payload types prevent)
- third arm for unattributable lines (rejected per Slice 2.5
Option 4 placeholder anti-pattern)
2. T-Rule-Enumeration β (YELLOW) scaffold needs bound per three-part
bridge rule. Added: allowed call sites scoped to emission code only;
max lifetime gated on rule_name_typo_fail_closed_landed graduation;
new emission-introspection consumers MUST author against closed-sum
α form, not β.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* WIP: R3 Substrate
* briefs: address codex BLOCKING bundle (sha 2ed1046e) — record-form provenance + emit_model carrier alignment + T-Rule-Enumeration retirement flag
Three findings addressed:
1. Lens brief (Finding #1 — provenance shape conflates source vs rule
attribution): replaced typed-sum EmissionOrigin with record
EmissionProvenance { rule: EmissionRule [mandatory], source_span:
Option<SourceSpan> [optional] }. Co-inhabitance is real (e.g.,
#[derive(Debug)] line has both rule=derive_for_disj AND span
attribution to Foo decl). Rule-mandatory enforces fail-closed
structurally; Practice 4 N/A (record, not coproduct).
2. T-Rule-Enumeration brief (Finding #2 — rule authority lives in
spec/rust.dag + v3.std.emit_model, not call-site strings): brief
status flipped to FLAGGED FOR RETIREMENT. Same finding as
smart-ram-167 STOP at gunbc#1759 #issuecomment-4392696623; Reading
C surfaced to Director at gunbc#828 #issuecomment-4392780871.
3. Rust-primitive-full-coverage brief (Finding #3 — BoundDeclaration
not wired into TargetIntegerInhabitanceBound row surface):
reframed Deliverable 2 to target the actual carrier
`v3.std.emit_model::TargetIntegerInhabitanceBound` (used by
`TargetIntegerTypeInhabitance` rows in src/v3/spec/rust.dag lines
169-207). Two paths surfaced: Option (i) wire BoundDeclaration into
TargetIntegerInhabitanceBound; Option (ii) populate via existing
variants if PlatformDependent already exists on the live carrier.
Worker DFS at dispatch picks. Deliverable 3 retargeted to
TargetIntegerTypeInhabitance rows (the actual surface).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* rust-primitive-full-coverage: codex inline confirms TargetIntegerInhabitanceBound has no PlatformDependent variant — Option (ii) infeasible; carrier refactor REQUIRED
Codex BLOCKING inline at PR #1910 line 65 (sha 98507c432) verified at
HEAD: TargetIntegerInhabitanceBound = BoundUnspecified | StaticBoundFact(IntInterval).
No PlatformDependent variant. Earlier brief framing of Option (ii)
(populate via existing variants) was infeasible.
Brief reframed: carrier refactor is required (not optional). Two
structural shapes for the refactor — (i.a) embed BoundDeclaration
(Mgr recommendation; single-authority discipline) vs (i.b) extend
variant set with PlatformDependent. Existing 5 rows migrate either
way; bootstrap snapshot semantic-equivalence non-negotiable.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* emission-provenance lane: T-Rule-Enumeration RETIRED + Lens brief revised per Director Reading C
Director Reading C RATIFIED at gunbc#1739 #issuecomment-4392797954
(2026-05-07):
- Q1: T-Rule-Enumeration retires (substrate gap was phantom; ~37
*SyntaxBinding/*OpsBinding field paths already provide enumeration
per smart-ram-167 substrate-state-grep at gunbc#1759 #issuecomment-4392696623)
- Q2: Lens<EmissionProvenance> dispatches now (no precondition gate)
- Q3: EmissionRule = String of field-path (per feedback_reason_not_label)
Actions:
- Delete docs/briefs/r3-substrate-t-rule-enumeration-worker.md
- Lens brief: replace T-Rule-Enumeration precondition gate with
Reading C absorption section; EmissionRule = field-path String;
cementing test verifies field-path round-trip; STOP triggers
retargeted to *SyntaxBinding field-set divergence
Worker pin: smart-ram-167 (Mgr discretion per Director ratification;
fresh context on field-path enumeration). valiant-ibex-312 stays on
Rust-primitive-full-coverage / T-Interval-Representation.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* emission-provenance lens brief: absorb codex BLOCKING bundle (sha 4229cd09 post-merge findings)
Two findings from codex bundled review on now-merged PR #1910 sha
4229cd09 land as brief revisions for the in-flight smart-ram-167
authoring cycle (PR #1928):
1. Field-path identity threading: render_named_template call sites
receive template values extracted from binding structs; field-path
may not be carried alongside the value at emission time. New STOP
trigger flags this as substrate-cascade if threading isn't
structurally possible (typed template-with-rule-path or
DeclarationRef carrier needed).
2. Read domain coverage gap: Lens<C>.read is per-Behavior; emitted
output also includes declarations + program scaffold. Acceptance
scope explicitly narrowed to Behavior-attributable lines only;
declarations + scaffold provenance out-of-scope (separate substrate
cascade if surfaced). Acceptance bullet retargeted to
"Behavior-attributable span-Some + Behavior-attributable span-None"
rather than full-emission coverage.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* rust-primitive-full-coverage: revert to Option (ii) — Mgr ratification-state-grep miss caught by valiant-ibex-312
valiant-ibex-312 ratification-state-grep at gunbc#1761
#issuecomment-4393074602 surfaced that origin/main HEAD has
TargetIntegerInhabitanceBound with 3 variants including
PlatformDependentFact (mirrors BoundDeclaration::PlatformDependent).
Earlier brief framing at 4229cd09d was against pre-PlatformDependentFact
state per codex BLOCKING reading; main moved since.
Option (ii) IS feasible: isize/usize rows populate via existing
PlatformDependentFact variant directly; u128 row populates StaticBoundFact
with BigInt-widened ExactInterval (Deliverable 1 unchanged). No
carrier refactor needed; 5 existing rows untouched.
Second Mgr-tier ratification-state-grep miss today (first was symbol-only
grep on T-Rule-Enumeration; now stale-codex-vs-current-HEAD on
TargetIntegerInhabitanceBound). Worker-tier discipline caught it before
sinking time into invasive Option (i.a) refactor.
Single-authority concern (parallel-named variants on
TargetIntegerInhabitanceBound vs BoundDeclaration) noted as future
substrate-fact-introduction; not blocking this slice — explicit-mirror
is the existing convention.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* rust-primitive-full-coverage: align Acceptance + Phase ordering + STOP triggers to Option (ii) (resolve codex internal-contradiction finding)
codex BLOCKING on PR #1929 sha 14f68b63 caught real internal contradiction: Context section flipped to Option (ii) feasible at 57ae4349b but Acceptance + Phase ordering still referenced BoundDeclaration migration / RustPrimitive bound field refactor from earlier Option (i.a) framing.
All sections now consistent with Option (ii):
- Phase 4 = no-op (TargetIntegerInhabitanceBound carrier untouched)
- Phase 5 = spec/rust.dag rows for isize/usize via existing PlatformDependentFact variant
- Phase 6 = u128 row via StaticBoundFact(IntInterval) + ratchet update
- Acceptance retargeted to row-population shape (no carrier refactor; existing 5 TargetIntegerTypeInhabitance rows untouched)
- STOP triggers retargeted to TargetIntegerInhabitanceBound variant-set sufficiency at HEAD
- Why-bundled rationale rewritten to Option (ii) shape (Phase A widening unblocks u128 StaticBoundFact; isize/usize use existing PlatformDependentFact)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* lens brief: narrow Deliverable 3 verification step 4 to Behavior-attributable lines (resolve codex #2 consistency)
codex BLOCKING #2 on PR #1929 sha 812602b4 caught: Acceptance + STOP triggers narrowed to Behavior-attributable lines but Deliverable 3 cementing-test step 4 still said 'every emitted line'. Now consistent — step 4 explicitly scopes to Behavior-attributable subset; declarations + program scaffold OUT-OF-SCOPE per same Finding #2 framing.
(codex BLOCKING #1 — claim that PlatformDependentFact NOT on main — is wrong; verified at origin/main HEAD: TargetIntegerInhabitanceBound has 3 variants including PlatformDependentFact. Rebut on PR.)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* briefs: pre-author IntPlatform/UIntPlatform substrate-fact-introduction follow-on (Director ratification at #4393248961)
Per Director Q1+Q2+Q3+Q4 RATIFICATION at gunbc#1739 #issuecomment-4393248961 (2026-05-07): pre-author follow-on brief for the substrate cascade surfaced by valiant-ibex-312 STOP at gunbc#1761 #issuecomment-4393222862.
Scope:
- Q1 naming: IntPlatform / UIntPlatform per feedback_reason_not_label
- Q2 algebra: Compose<Int, MachineWidth<Platform>> + Compose<UInt, MachineWidth<Platform>>; Platform is substrate token
- Q3 substrate-concept layer (target-agnostic per feedback_target_agnostic_ir)
- Q4 worker pin: valiant-ibex-312 post-PR-#1914 close (freshest substrate context)
Brief covers Platform substrate token + IntPlatform/UIntPlatform declarations + spec/rust.dag isize/usize row population + co-receipt advancing PR #1914's rust_primitive_full_coverage gate PARTIAL → CONSUMER_LANDED.
Closure gate: int_platform_uint_platform_substrate_landed.
Dispatchable post-PR-#1914 close per pre-authored-brief-queue discipline.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* intplatform brief: rename inner token Platform → PointerWidth per Director ack at…
* WIP: valiant-ibex-312 * fix(v3): close nested-optional cardinality codegen bypass (Path B rename) Renames CardinalityPayload::new_unchecked to new_unchecked_bypassing_idempotence so every call-site that does not route through Dag::alloc_cardinality_decl is, by name, declaring it is not applying the cardinality_idempotent_target rule. Per Director routing #828 #issuecomment-4383647114 (Path B over Path A); brief: docs/briefs/r2-impossible-bugs-nested-optional-codegen-bypass-closure-worker.md (#1778, warm-dove-810). - Discipline-visible doc comment names the load-bearing invariant. - 205 call-sites updated across 6 files (4 hand-written non-test sites + 4 test sites + ~50 generated sites in 3 bootstrap_*_generated.rs files refreshed via the regen_bootstrap_emit format string). - Mechanical refresh; zero semantic change. Existing cardinality_idempotent / cardinality_idempotence tests pass; cargo fmt clean. - r2-closure-ledger row updated: nested-optional flatten now cites #1803 as the codegen-bypass closure receipt. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * fix(v3): refresh bootstrap generated snapshots via regen_bootstrap The prior commit's mass-rename of CardinalityPayload::new_unchecked → new_unchecked_bypassing_idempotence was applied with sed, preserving multi-line formatting; fresh regen_bootstrap emits on a single line. The regen_bootstrap --verify CI gate caught the divergence. This commit runs regen_bootstrap (no --verify) to align the committed snapshots with the format string. No semantic change. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * ci: empty-commit retrigger for #1803 self_host_ratchet runner-budget probe Authorized single retrigger per R3 Mgr at #1761 #issuecomment-4384216296. Prior run (job 74523060739) was cancelled at 30m hard timeout mid-rustc release recompile of v3-compiler for self_host_fixed_point; ci/fmt/v3 green. Probing whether warmer cache / different runner state lands within budget; if same failure recurs, escalate. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * WIP: valiant-ibex-312 * fix(measure): typed scale_exponent authority + constrained-inhabitance gap Address blocking review on PR #1819 (codex@9ccfe165): 1. Scale was a bare coproduct; Grounding consumers and downstream algebra would have to re-encode the SI exponent table at each call site. Add `fn scale_exponent(s: Scale) -> Int` as the canonical typed exponent authority (single-source-of-truth invariant; total over all Scale inhabitants). 2. `Measure<Q, S>` admitted any type-reference for Q/S at the parser level — denotationally Q must inhabit Quantity and S must inhabit Scale, but no parametric where-clause syntax exists in the substrate. Document the CONSTRAINED-INHABITANCE GAP scaffold matching the pattern at `dsl/std/algebra.dag` (`GroupCompletion<M>` / `FieldOfFractions<R>`), with a named dissolution trigger (parametric where-clause syntax lands → tighten to `<Q> : Quantity, <S> : Scale`). 3. Refresh the COMPOSITION block and citation discipline: brief paths live in the parent worktree (cross-program review reaches them via the gunbc#828 ratification thread). Inline rule-text quotes for Practice 4 / `feedback_construction_over_ratchets` / `feedback_reason_not_label` per `brief-authoring-checklist.md`. Update the conceptual composition lines to reflect option (c) RATIFIED (no `Refined<>` wrapper at this layer). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * WIP: valiant-ibex-312 * WIP: valiant-ibex-312 * test(r3-l6): wire cross_target_coverage carrier ratchet through SG-0 - Add `cross_target_coverage_carrier_test.rs` to EXPECTED_HAND_AUTHORED_TEST in `sg0_census_test.rs` (sorted ASCII-ascending; receipt cites Director Option 2 RATIFIED at gunbc#828 #issuecomment-4377533390). - Refresh `parse_corpus_manifest.txt` via the `refresh_handwritten_parse_snapshot_manifest` host (picks up the new `src/v3/std/cross_target_coverage.dag` entry + the `extdeps_bootstrap_fixtures.dag` / `bootstrap_authority.dag` row changes). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * fix(r3-l6): four-pattern dissolution receipt for ShapeATarget 🟢 TERMINAL Address blocking review on PR #1842 (codex@2eb2632a): ShapeATarget was stamped 🟢 TERMINAL with an inline checkpoint comment but lacked the four-pattern dissolution receipt that `feedback_coproduct_dissolution` requires for the GREEN classification. Pattern follows the DB-18 / DB-9 / DB-16 receipt format at `docs/design-db18-workflow-effect-carrier.md` §"Dissolution receipt". Pattern 1 (fact-placement): n/a — all variants attach to the same typed slot at the same DAG location. Pattern 2 (variant-is-data): n/a — tag identity governs typed dispatch into per-target method-template-contract authorities; collapsing to `target_name: String` would re-introduce string-name dispatch (Option 2 §4.C=(i) ratified specifically to prevent that). Pattern 3 (algebraic-form): n/a — no algebraic operation over {Rust, Python, Go}; the set is Director-ratified per Shape A portability. Pattern 4 (dimensional): n/a — targets do not factor into orthogonal coordinates; per-target primitives + method-template contracts + emit are irreducible per-target profiles. Stamp confirmed: 🟢 TERMINAL. Bootstrap regen + parse-corpus-manifest refresh follow. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * ci: re-trigger after PR body SG-0 hand-path delta line restoration CI run 25447982331 failed the SG-0 PR-body discipline gate because the prior PR body got wiped by a var-substitution edit; the body now carries `SG-0 hand-path delta: +1` and `SG-0 pairing: (c)`. Empty commit forces a fresh run that re-reads the corrected body. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * WIP: valiant-ibex-312 * test(r3-l6): tighten ratchet to assert exact variant labels + record fields Address blocking review on PR #1842 (openai-pro@da1ada86): The previous ratchet only checked Disj variant *counts* (3/6/5) and declaration presence for the three record carriers — would silently pass if labels drifted while preserving count, or if fields renamed. Tightened per the openai-pro reviewer's gap analysis: - `shape_a_target_disj_variants_match_ratified_labels`: exact label sequence [Rust, Python, Go] (declaration order). - `form_axis_disj_variants_match_type_connective_discriminants`: exact label sequence mirroring `v3_compiler::dag::TypeConnective` discriminants in substrate-declaration order. - `behavior_axis_disj_variants_match_behavior_discriminants`: exact label sequence mirroring `v3_compiler::dag::Behavior` discriminants in L1 model order. - `method_template_contract_key_record_fields_match_ratified_shape`: exact `(label, type-name)` pairs `[(target, ShapeATarget), (dag_method, MethodRef)]`. - `emission_cell_record_fields_match_ratified_shape`: exact pairs `[(connective, FormAxis), (behavior, BehaviorAxis)]`. - `emission_path_projection_record_fields_match_ratified_shape`: `row_identity: MethodTemplateContractKey` + `cells` resolves through Cardinality/Instantiation wrapper around `EmissionCell`. - `emission_path_projections_data_is_empty_list`: extended to verify the data declaration's connective references `EmissionPathProjection`'s DeclarationId (so an empty `List<Foo>` doesn't silently pass if the element type drifts). Helpers `disj_variant_labels` + `conj_field_label_and_type_names` centralize the typed-substrate read; no string scanning of source. Now 8 tests (was 5), all green. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs(sg0-census): correct PR-number cite (this is #1842, precedent #1819) Address non-blocking exploratory observation on PR #1842 (cursor@bcbb8565): the SG-0 census comment for `cross_target_coverage_carrier_test.rs` cited "#1819 follow-on" but #1819 was the Measure-carrier precedent; this slice lands in #1842. Future archaeology now points at the right ticket. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * feat(v3): widen IntervalInt::ExactInterval host repr to BigInt (R3 Phase A) R3 Substrate Rust-primitive-full-coverage Deliverable 1 (Director Path A RATIFIED at gunbc#1739 #issuecomment-4392731264). Bundled brief at `docs/briefs/r3-substrate-rust-primitive-full-coverage-worker.md`. `u128::MAX` exceeds `i128` representable range; the previous narrow `i128` host repr forced an explicit `u128` row deferral in `dsl/extdeps/languages/rust/primitives.dag` (substrate gap #1 surfaced by Grounding loyal-stag-699 STOP at gunbc#1907). Widen the internal host repr to arbitrary-precision `BigInt` (`num-bigint = "0.4"`, single stable workspace dep) so any integer-width primitive is representable structurally without per-width host-repr variant explosion. Changes: - Add `num-bigint` workspace dep + `v3-compiler` consumption. - `IntervalInt::ExactInterval { min: i128, max: i128 }` → `{ min: BigInt, max: BigInt }`. - `contains_i64`: lift the i64 input to `BigInt` for comparison. - Range-fact parsing: `min_s.parse::<i128>()` → `min_s.parse::<BigInt>()`; diagnostic copy updated ("must parse as a decimal integer", not "must parse to i128"). - Public API surface (`IntervalInt`, `IntegerRangeLookup`, `integer_range_for_decl`, `magnitude_out_of_range_for_interval`, `int_literal_fits_expected_type`) unchanged — consumers in `infer.rs`, `lower.rs`, `int_literal_cardinality_test.rs` are source-compatible. - Existing 9 ratcheted i8-i64+u8-u64 rows continue to parse identically (semantic equivalence; verified by `cargo test -p v3-compiler --lib int_literal_ranges` passing all 6 module tests, including the `int128_range_lookup_accepts_all_i64_literals` ratchet that exercises i128-row reads through the BigInt-backed bound). Practice 4 N/A on this Phase: α path is type-substitution (BigInt for i128) on a non-coproduct field; no new variants added. Phase B (RustPrimitive bound-field refactor + 9-row migration) and Phase C (u128/isize/usize rows + spec/rust.dag PlatformDependent population + ratchet update 9→12) follow in subsequent commits within this same PR per bundled brief discipline. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs(v3): refresh module-level doc to match BigInt host repr Address non-blocking exploratory note on PR #1914 (cursor@3e6c8219): module-level `//!` doc still said "host `i128` comparison" — outside the prior diff but trivially aligned with the new `BigInt` host repr now that Phase A has landed. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * WIP: valiant-ibex-312 * feat(v3): add u128 IntegerPrimitive row to rust_pilot_primitives (R3 Phase B-1) R3 Substrate Rust-primitive-full-coverage Phase B step 1 (Director Path A RATIFIED at gunbc#1739 #issuecomment-4392731264; Option (ii) confirmed at gunbc#1761 #issuecomment-4393145631). Phase A (commit `e7ba022c6`) widened `IntervalInt::ExactInterval` host repr from `i128` to `BigInt`; that unblocks the u128 row in `rust_pilot_primitives` whose `range_max_inclusive` is `u128::MAX` (`340282366920938463463374607431768211455`) — outside i128 representable scope. The `T-Int128 Slice B1` deferral marker in `primitives.dag` is now resolved. Changes: - `dsl/extdeps/languages/rust/primitives.dag`: add `IntegerPrimitive` row for `u128` (`SemiringAlgebra` over `Word128Carrier`, range `[0, 2^128 - 1]`, `TwoComplementWrap` overflow). Symmetric to the existing `i128` row above. - `src/v3/compiler/src/int_literal_ranges.rs`: bump `EXPECTED_INTEGER_ROWS` constant from 9 to 10. - `src/v3/compiler/tests/integration/int_literal_cardinality_test.rs`: bump `rust_pilot_primitives_integer_witnesses_are_unique` ratchet from 9 to 10 with updated message naming u128 inclusion. - `src/v3/compiler/tests/integration/extdeps_rust_primitives_loader_test.rs`: bump structural list ratchet from 11 to 12 (10 IntegerPrimitive + 2 NonIntegerPrimitive) with updated comment. - Bootstrap snapshots regenerated via `regen_bootstrap` (`bootstrap_generated.rs`, `bootstrap_generated_without_parse_surface.rs`, `bootstrap_std_generated.rs`). NOT in this commit (Phase B-2 follow-on within the same PR): - `src/v3/spec/rust.dag` `TargetIntegerTypeInhabitance` rows for u128 / isize / usize via `bound: StaticBoundFact` / `PlatformDependentFact`. - §1.8 ledger row `rust_primitive_full_coverage` receipt. - Cross-program handoff receipt to Grounding Mgr (#1745). Bundled-scope honored: Grounding G2 Phase 2 lowering rules NOT in this PR. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * feat(v3): add u128 TypeRealization + inhabitance row to spec/rust.dag (Phase B-2) R3 Substrate Rust-primitive-full-coverage Phase B step 2 (Director Path A RATIFIED at gunbc#1739 #issuecomment-4392731264; Option (ii) confirmed at gunbc#1761 #issuecomment-4393145631). Changes: - `src/v3/spec/rust.dag` import: add `UInt128` to `std.integer` import line (sibling to `Int128`). - `data rust_u128: TypeRealization`: u128 carrier realization. Symmetric to `rust_i128`; consumes the `UInt128` kernel integer (`Semiring<Word128>`) per `dsl/std/integer.dag:61`. - `data rust_integer_inhabit_u128_bound_unspecified: TargetIntegerTypeInhabitance`: inhabitance row for u128 mirroring the i128 `BoundUnspecified` shape above. Static interval bound facts on u128 (`StaticBoundFact(IntInterval)`) lower through the BigInt-backed `IntervalInt::ExactInterval` once narrowing consumers exercise them — Phase A's host-repr widening is what made this representable. - Bootstrap snapshots regenerated. The existing 5 `TargetIntegerTypeInhabitance` rows (u32 / i32 / int64 / i128 / i32_at_program_bound) are untouched per Mgr's confirmed Option (ii) dispatch path. NOT in this commit: - isize / usize rows: blocked on missing `IntSize` / `UIntSize` kernel integer types in `dsl/std/integer.dag` — `kernel_integer:` field on `TargetIntegerTypeInhabitance` requires a std integer type, but the std lacks platform-sized integer carriers. Surfacing as scope question to Substrate Mgr (#1739) before authoring; std-side IntSize/UIntSize introduction is substrate-fact-introduction, not in this slice's bundled scope. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs(v3): align EXPECTED_INTEGER_ROWS comment with live state (10 rows) Address non-blocking doc-drift finding on PR #1914 (cursor@b6989ec7): the block comment above `EXPECTED_INTEGER_ROWS` still said "9 rows" and referenced the resolved "Slice B2" deferral marker, but the const + the pilot list now carry 10 rows including u128. Refresh the comment to reflect live state per INVARIANTS.md "Documentation Describes Live State" spirit. Also update the docstring above to say "decimal BigInt" instead of "i128" (host repr widened in Phase A). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs(rust-primitives): refresh PILOT SCOPE / Slice B2 header to live state Address non-blocking finding on PR #1914 (codex@f994e1f3): file-level pilot-scope text still listed u128 as deferred even though the row is present in this PR. Refresh: - PILOT SCOPE list now includes u128. - T-Int128 SLICE B1/B2 block annotates the resolution: B2 deferral closed by Phase A `BigInt` host repr widening (commit `e7ba022c6`); u128 row landed in Phase B-1 (commit `59511503e`). - New FOLLOW-ON DEFERRAL block names the next cascade explicitly: isize/usize wait on substrate-side IntSize/UIntSize kernel integer introduction (Director-level scope per Mgr disposition Path 1 at gunbc#1761 #issuecomment-4393235461). Bootstrap snapshots + parse manifest regenerated to absorb the comment edit through the bootstrap span byte ranges. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * WIP: valiant-ibex-312 --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…fresh (post-#1914) (#2272) Mirror update for `RUST_PILOT_PRIMITIVES`: add `u128` `IntegerPrimitive` row (Semiring × Word128) symmetric to the existing `i128` row, and add the corresponding `DagType::UInt128` arm in `dag_type_facts` and the test routing tables. Refresh the stale `PROBE SCOPE` comment lines 6-8 — the prior `u128` deferral was closed by R3 Phase B-1 (commit `59511503e`) widening `IntervalInt::ExactInterval` to a BigInt host repr; consumer-side `u128` row already landed in `dsl/extdeps/languages/rust/primitives.dag:274` and the pilot mirror was the last parity gap. Brief #2224 narrowed to D2-only per Mgr disposition at #2063 issuecomment-4409325712 after grep-verified shape mismatches on D1/D3/D4: - D1 (`isize`/`usize` rows) HELD pending Director disposition on consumer-side platform-width carrier shape — `RustPrimitive`'s `TargetCarrier` / `IntegerAlgebra` are closed enums with no platform-width variant; substrate `IntPlatform`/`UIntPlatform` lives on `TargetIntegerTypeInhabitance`, a different row type. - D3 (cross-target meta cell coverage) MIS-ROUTED — `coverage.rs::ProjectionKey` is `{ target, dag_method }` for method template contracts; integer-row walker actually lives in `grounding_coercion_fold/fold.rs:69-86`. - D4 (`ReferenceModel<T>` Grounding wiring) HELD as fresh canvas-finding — zero scaffolding in `grounding_engine` to extend, would be substantive design slice. Tests: `cargo test -p v3-grounding-pilot` 12/12 green; `cargo test -p v3-grounding-engine` 6/6 green (first-row mirror invariant unchanged — `RUST_PILOT_PRIMITIVES[0]` is still `i8`). Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…nt (post-#1914) Refute BLOCKING inline review on PR #2272 at lib.rs:277 — reviewer claimed the consumer authority `dsl/extdeps/languages/rust/primitives.dag` "still defers u128", but the `u128` row is in fact present at line 274 (landed via R3 Phase B-1 commit `59511503e`, post-#1914 BigInt host repr widening). The misread was likely caused by a stale narrative comment on the *i128* row (lines 239-241 pre-fix) that still said "the symmetric u128 row is deferred to Slice B2", contradicting the declaration immediately below it. Refresh the i128-row comment to historical-tense and point at the resolved u128 row's location. No row data changes. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
R3 Substrate Rust-primitive-full-coverage — Phases A + B (u128 partial)
Authority: Director Path A RATIFIED at gunbc#1739 #issuecomment-4392731264. Bundled brief at
docs/briefs/r3-substrate-rust-primitive-full-coverage-worker.md(parent worktree, HEAD14f68b635+). Mgr disposition closing this PR at u128 + Phase A coverage per gunbc#1761 #issuecomment-4393235461.What's in this PR
Phase A —
IntervalInt::ExactIntervalBigInt host repr widening (commite7ba022c6)u128::MAXexceedsi128representable range. The previous narrowi128host repr forced an explicitu128row deferral indsl/extdeps/languages/rust/primitives.dag(substrate gap #1 surfaced by Grounding loyal-stag-699 STOP at gunbc#1907). Widening the internal host repr to arbitrary-precisionBigIntadmits any integer-width primitive structurally without per-width host-repr variant explosion.Cargo.tomlworkspace deps: addnum-bigint = { version = "0.4", default-features = false }— single, stable, widely-vendored crate.src/v3/compiler/Cargo.toml: consume the workspace dep.src/v3/compiler/src/int_literal_ranges.rs:IntervalInt::ExactInterval { min: BigInt, max: BigInt };contains_i64lifts i64 toBigInt; range-fact parsing usesBigInt::parse.infer.rs,lower.rs,int_literal_cardinality_test.rsare source-compatible.Phase B-1 — u128 IntegerPrimitive row (commit
59511503e)dsl/extdeps/languages/rust/primitives.dag: addIntegerPrimitiverow foru128(SemiringAlgebraoverWord128Carrier, range[0, 2^128 - 1],TwoComplementWrapoverflow). Symmetric to existingi128row.T-Int128 Slice B1deferral marker resolved.src/v3/compiler/src/int_literal_ranges.rs:EXPECTED_INTEGER_ROWSconst 9 → 10.int_literal_cardinality_test.rs::rust_pilot_primitives_integer_witnesses_are_unique: 9 → 10 with updated message naming u128.extdeps_rust_primitives_loader_test.rs::rust_pilot_primitives_value_body_is_structural_list: structural list ratchet 11 → 12 (10 IntegerPrimitive + 2 NonIntegerPrimitive).Phase B-2 — u128 TypeRealization + inhabitance row (commit
f994e1f30)src/v3/spec/rust.dagimport:UInt128added tostd.integerimport.data rust_u128: TypeRealization: u128 carrier realization (targetUInt128, carrier"u128", symmetric torust_i128).data rust_integer_inhabit_u128_bound_unspecified: TargetIntegerTypeInhabitance: u128 inhabitance row (BoundUnspecifiedmirroring i128 row above;StaticBoundFactlowering through BigInt-backedIntervalInt::ExactIntervalonce narrowing consumers exercise it).TargetIntegerTypeInhabitancerows (u32 / i32 / int64 / i128 / i32_at_program_bound) untouched per Mgr's confirmed Option (ii) dispatch path.Bootstrap regen + parse manifest
regen_bootstrapregeneratedbootstrap_std_generated.rs,bootstrap_generated.rs,bootstrap_generated_without_parse_surface.rs.parse_corpus_manifest.txtrefreshed viarefresh_handwritten_parse_snapshot_manifest -- --ignored.Closure gate
rust_primitive_full_coverageadvances DECLARED → PARTIAL on merge (not CONSUMER_LANDED — full coverage gates on isize/usize follow-on). §1.8 ledger entry notes the partial advancement.NOT in this PR (deferred to substrate-fact-introduction follow-on per Mgr disposition)
isize / usize rows in
src/v3/spec/rust.dagare deferred —TargetIntegerTypeInhabitance.kernel_integerrequires aDeclarationRefto a std integer type, butdsl/std/integer.daghas no platform-sized integer carrier. AddingIntSize/UIntSizeto std is load-bearing substrate-fact-introduction with canonical-naming + algebra-inhabitance questions — Director-level scope, not Mgr-tier bundled. Surfaced at gunbc#1761 #issuecomment-4393222862; Mgr disposition Path 1 (defer) ratified at #issuecomment-4393235461. Substrate Mgr commits to surfacing the cascade to Director with named scope (std-side IntSize/UIntSize).Cross-program handoff (Grounding Mgr #1745)
u128 coverage unblocks G2 Phase 2 partial coverage immediately on merge; isize/usize hold pending the substrate follow-on per Path A precedent (narrowed coverage on existing substrate). Substrate Mgr signals Grounding Mgr at merge.
Authority audit (5 questions)
IntervalInt::ExactIntervalexists; widened.BigIntadds vianum-bigintworkspace dep.UInt128exists indsl/std/integer.dag.IntSize/UIntSizedo NOT exist — named cascade dependency.docs/briefs/r3-substrate-rust-primitive-full-coverage-worker.md(parent worktree); Phases A + B (u128 only).CI status
cargo test -p v3-compilerpassing locally pre-merge-conflict; integration tests for int_literal + inhabitance all green (24 tests verified after Phase B-2 changes). Targeted ratchets (rust_pilot_primitives_integer_witnesses_are_unique,rust_pilot_primitives_value_body_is_structural_list,target_integer_type_inhabitance_rows_are_structural_slice_b_receipt) all green.SG-0 hand-path delta: 0