Skip to content

emit: Measure-alias count peel (E0560) - #5396

Closed
gunbai-bot[bot] wants to merge 12 commits into
mainfrom
session/bright-badger-311
Closed

gunbai-bot[bot] wants to merge 12 commits into
mainfrom
session/bright-badger-311

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Auto-opened by session-dashboard for session bright-badger-311.
Pushing to session/bright-badger-311 advances this PR.

Worker attestation

Before flipping this PR to ready for review, confirm each item:

  • Title describes the change (not the session id or branch).
  • PR body summarises what and why (replace the TODO below).
  • Tests run: name the command (e.g. npm test, cargo test) and the result.
  • If this closes a work item, the body contains a Closes #N directive.
  • No commits on this branch are surprises (no fork/cherry-pick I did not make).
  • No secrets / credentials / large binaries staged.

Summary

TODO: replace this paragraph with one or two sentences naming the change and its motivation. Reviewers read this first.

Test plan

  • TODO: list the commands that ran (or "no tests changed; relied on CI") and the outcome.

briansrls and others added 6 commits June 20, 2026 17:09
The auto-commit captured a local regen of src/v1/stage0/. Per the
regen-lockstep repair hygiene, the committed seed must stay main's until
the green co-land (PR 5325); this slice ships the emitter .dag change only.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@gunbai-bot
gunbai-bot Bot marked this pull request as ready for review June 20, 2026 18:15
briansrls and others added 6 commits June 20, 2026 18:21
… struct (cursor 31502)

Construction peel kept naming/boxing/phantom on the AUTHORED alias (tn,
e.g. Gibibyte) while feeding the expanded resolved_type — so for a struct
alias the phantom field was skipped (E0063) and display could become
Measure::Gibibyte. Fix: pass the peeled concrete struct name (from
expanded_rt) as emit_typed_record_lit's type_name so the literal name,
Box<M> boxing, and PhantomData population all agree on Measure; and guard
the effective_parent fallback with variant_belongs_to_enum so a struct is
never prefixed as Enum::Variant. .dag-only; seed stays main's.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…sources

The discriminating witness ran the real emit pipeline via the interpreter,
but `compile_sources` hits 'not yet implemented: method chars' in the
interpreter, so it cannot execute (and would RED the CI floor if enrolled).
Removing it; execution-verification of the construction peel rides the
integration-branch full build. Keeps the PR .dag-only.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@briansrls

Copy link
Copy Markdown
Contributor

Consolidated into #5413 (freeze consolidation of the durable gunbc lanes). Branch retained; reopen if the consolidation is unwound.

@briansrls briansrls closed this Jun 20, 2026
briansrls added a commit that referenced this pull request Jun 20, 2026
…) (#5413)

Consolidates the durable freeze lanes onto one branch (each constituent closed in favor of this):
- #5402 dsl_compile_clean dual-root gate-widen (Strategy A keystone)
- #5395 .length() -> .count() dissolution
- #5407 anemia CONFIRM eval corpus (additive)
- #5400 phantom type-tag derives
- #5396 Measure-alias count peel (E0560)

Plus a carried fix for a pre-existing main defect (from #5406): clippy::manual_is_multiple_of +
fmt in v2_whole_tree_parse_scan.rs (main's rust_monolith gate is skipped on main pushes, so it
went uncaught; this is the first post-#5406 PR to touch a .rs and run clippy against it).

Local receipt: cargo fmt --all --check clean; cargo clippy --all-targets -- -D warnings exit 0;
cargo test -p v1-compiler-tests interp_recorded_fixture/wet_hermetic/resolve_expr_types_retraversal
= 28 passed, 0 failed. All other floor gates (dsl_compile_clean, source_root_ingest, emit_host,
layering, 557-witness corpus) were green on the prior runs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant