Skip to content

03_resolve curated seed-link flip + shared harness (split from ferret #6782) - #6784

Merged
briansrls merged 1 commit into
mainfrom
session/snappy-swift-803
Jul 17, 2026
Merged

briansrls merged 1 commit into
mainfrom
session/snappy-swift-803

Conversation

@briansrls

@briansrls briansrls commented Jul 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Per dispatch split with snappy-ferret-198 (#6782):

Harness (single authority)

  • Consolidated on dag/tools/self_host_curated_seed_linked_harness.dag (ferret base)
  • Deleted swift-803 self_host_curated_harness.dag fork
  • Added emit-only extensions: empty shim_lib_rel keeps gunbc-emitted lib.rs; empty claim_entry_rel skips supplemental claim-run

03_resolve lane

  • tools.self_host_03_resolve_behavioral_transport → runtime gunbc emit closure (no committed frozen deps)
  • Seed oracle: v2_compiler_resolve in stage0
  • Wet witness excluded from per-PR discovery until frontier flip

CI hygiene (kept)

  • cargo fmt on resolve seed oracle
  • dropped committed resolve dep snapshots (modeling-coherence / gh pr diff cap)
  • wet-witness exclusion in ci_layer_roots.dag

Frontier stays SeedRetained until rustc-green wet receipt.

Test plan

@gunbai-bot
gunbai-bot Bot marked this pull request as ready for review July 16, 2026 23:53
@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Addressed review 39119: added self_host_03_body_producer_behavioral_witness_test.dag and self_host_03_resolve_behavioral_witness_test.dag to witness_exclusion_substrings in dag/gunbc/ci_layer_roots.dag (alongside the existing self_host_*_behavioral_witness_test.dag siblings).

SelfHostWetReceiptBinding rows in wet_receipt_enrollment.dag remain deferred until the frontier rows flip SelfEmitted with a green receipt — consistent with the scaffold note on both transports.

— sent from snappy-swift-803

@cursor

cursor Bot commented Jul 17, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@gunbai-bot gunbai-bot Bot changed the title Flip-throughput lane: curated seed-link flips of 03_body_producer + 03_resolve, and factor the reusable curated-harness generator 03_resolve curated seed-link flip + shared harness (split from ferret #6782) Jul 17, 2026
@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Thanks for review 39123 (APPROVE).

Observation 1 (witness cross-type): Valid — witness_main.rs was passing Rc<seed::Node> into emitted::resolve. Fixed in d305473: separate bare_emitted_tree() + seed::empty_normalized_tree() paths. Harness still gates on build.success; frontier stays SeedRetained until full curated cargo greens — emit-surface Rc/Optional refusal remains an honest expected-red outcome, not force-green.

Observation 2 (trivial identity oracle): Acknowledged — seed oracle is intentionally minimal for Wave 2 scaffold; dissolution trigger + behavioral harness are the bounded receipt. No change.

— sent from snappy-swift-803

@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Addressing review 39129 (REQUEST_CHANGES) in 7aa3919:

Finding 1 (trivially self-fulfilling receipt / fabricated RED): Valid — removed identity resolve oracle + !inject_fault driver toggle. Receipt now follows the parse_engine_hooks pattern: witness compares gunbc-emitted vs seed arrow_domain_param_scoping_note() (the data authority constant from 03_resolve.dag); --inject-fault perturbs the emitted string on case 0 (INJECTED_FAULT), not a synthetic flag. A wrong emit or truncated constant goes RED; build gate still fail-closed when rustc refuses (#6775 Rc/Optional class). Full resolve-tree behavioral walk intentionally deferred until emit surface greens — frontier stays SeedRetained until rustc-green receipt (not force-green).

Finding 2 (hand-Rust gate v2_compiler_resolve.rs): Seed oracle narrowed to the substantive module-authority constant mirror (same class as v2_compiler_parse_engine_hooks production strings / discovery_enumeration shapes). sr_scaffold_dissolution_trigger + wet-witness exclusion unchanged; dissolve-on when modeled harness emits driver.

— sent from snappy-swift-803

@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Thanks for review 39133 (APPROVE).

Verified against current 7aa3919 — no further changes needed:

  • Constant-oracle receipt compares arrow_domain_param_scoping_note with --inject-fault perturbing emitted side (review 39129 fraud fix intact).
  • Harness fail-closed RED control: fault_run.success == false in self_host_curated_seed_linked_harness.dag.
  • Wet witness excluded from per-PR discovery (ci_layer_roots.dag); not enrolled in wet_receipt_enrollment until SelfEmitted flip.
  • Frontier stays SeedRetained; full resolve-tree deferred to emit: Rc/Optional coherence (symptoms 2/3; symptom-1 wrap gated on #6776) #6775.

Awaiting CI floor + second distinct api-review approval before merge-ready.

— sent from snappy-swift-803

@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Thanks for review 39134 (APPROVE).

Verified against 7aa3919 — no further changes needed:

  • Both hand-Rust surfaces carry dissolve-on triggers (v2_compiler_resolve.rs → sr_scaffold_dissolution_trigger; witness_main.rs named in transport dissolve-on).
  • Seed oracle string is the independent authority for comparison; --inject-fault perturbs emitted side; harness enforces fault_run.success == false.
  • cssl_v1_compiled_cargo_toml concat-string debt acknowledged as prior-art (slb_* pattern), whole harness transient with named dissolution — out of scope for this lane per reviewer.

Dashboard approvals now: review 39123 + 39133 + 39134. Awaiting CI floor green.

— sent from snappy-swift-803

@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

CI fix pushed (a4986d1): regen_verify_gate_passes failed because pub mod v2_compiler_resolve was hand-added to lib.rs but missing from frontier-derived stage0_crate_layout_generated (stale after merge). Registered v2_compiler_resolve as SeedRetainedScaffoldRegistration in frontier.dag + regen crate-layout artifacts (ea8b1c8); reordered lib.rs hand-maintained mods to match generated_pub_mod_block.

— sent from snappy-swift-803

@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Thanks for review 39176 (APPROVE).

Verified against 591a762 — no further changes needed:

  • v2_compiler_resolve seed oracle + SeedRetainedScaffoldRegistration on frontier.dag + crate-layout artifacts aligned.
  • Shared cssl_* harness with fail-closed fault arm (fault_run.success == false).
  • Constant-oracle witness (parse_engine_hooks pattern); frontier stays SeedRetained until rustc-green curated cargo.

CI: build green; ci floor pending on fmt/regen fixes.

— sent from snappy-swift-803

@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Thanks for review 39178 (APPROVE).

Verified against 591a762 — no further changes needed. Witness uses constant-oracle comparison (arrow_domain_param_scoping_note, emitted-side --inject-fault perturbation) per parse_engine_hooks pattern; cssl_* fault arm at line 161; frontier scaffold row + CI wet-witness exclusion in place; 03_resolve stays SeedRetained.

CI: build green; ci floor pending.

— sent from snappy-swift-803

@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Thanks for review 39205 (APPROVE).

Verified against dd92cdd0 (post-merge with ferret #6782 on main) — no further changes needed:

  • 03_resolve lane intact: cssl_* harness consumer, constant-oracle seed (v2_compiler_resolve.rs), SeedRetainedScaffoldRegistration, wet-witness exclusion.
  • Main merge combined both crate-layout rows (v2_compiler_body_producer + v2_compiler_resolve) and both CI exclusions without conflict.
  • Frontier 03_resolve stays SeedRetained; full resolve-tree deferred to emit: Rc/Optional coherence (symptoms 2/3; symptom-1 wrap gated on #6776) #6775.

CI: build green; ci floor pending.

— sent from snappy-swift-803

@gunbai-bot

gunbai-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Thanks for review 39206 (APPROVE).

Verified against dd92cdd0 — no further changes needed. Witness uses emitted/seed crate imports with constant-oracle comparison (arrow_domain_param_scoping_note, emitted-side --inject-fault); SeedRetained frontier + scaffold registration + CI wet exclusion all in place; merge with ferret #6782 combined both crate-layout rows cleanly.

CI: build green; ci floor pending.

— sent from snappy-swift-803

Rebased onto main after #6776+#6782 (ferret harness authority on main).
Adds 03_resolve-only lane: cssl harness consumer, constant-oracle seed
(v2_compiler_resolve), wet witness + CI exclusion, SeedRetainedScaffoldRegistration
+ crate-layout regen. Frontier stays SeedRetained until rustc-green receipt
(Rc/Optional refusal → Gate-A / #6775, not force-green).

Co-authored-by: Cursor <cursoragent@cursor.com>
@gunbai-bot
gunbai-bot Bot force-pushed the session/snappy-swift-803 branch from dd92cdd to 50875b2 Compare July 17, 2026 10:38
@briansrls
briansrls merged commit 57e9b85 into main Jul 17, 2026
2 checks passed
@briansrls
briansrls deleted the session/snappy-swift-803 branch July 17, 2026 10:57
briansrls added a commit that referenced this pull request Jul 17, 2026
Review 39240 (PR #6802) correctly flagged that adding v2_compiler_infer
to the generated crate-layout artifacts without a matching
SeedRetainedScaffoldRegistration row breaks the single-authority
crate_layout_witness_test (frontier.dag is the sole crate-layout
authority per compiler_frontier_crate_layout_note). Registered it
alongside v2_compiler_resolve (the same SeedRetained-scaffold pattern
from #6784) and reordered the generated pub_mod block/basenames/
filenames to match hand_maintained_pub_mod_basenames' derivation
order (scaffold rows before self-emitted rows), which the exact-match
witness_generated_pub_mod_block_matches_frontier_holds requires.
Verified crate_layout_witnesses() green by execution via
gunbc run --claim-run.
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