Skip to content

Namespace PR-5a: corpus import strip src/v2/compiler + src/v2/std (delete import lines; bare refs stay; EXCLUDE calm-otter quartet: 03_body_producer, 05_eval, body_lowering_fold, 03_resolve; regen; dag_compile_clean_gate) - #6603

Closed
gunbai-bot[bot] wants to merge 1 commit into
mainfrom
session/zesty-swift-860

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Jul 14, 2026 •

Copy link
Copy Markdown
Contributor

CI EXPECTED RED — structural, not a rebase regression. Strip PR stays draft until Wave-1 ready-flip (after Wave-0 grammar/resequence). Do not chase in-strip compile-clean fixes.

Summary

Namespace PR-5a: delete import lines from src/v2/compiler/** + src/v2/std/** (128 files). Bare cross-module refs remain; calm-otter quartet excluded (03_body_producer, 05_eval, body_lowering_fold, 03_resolve).

Rebased on main @ 5906140 (#6620 Gate-1). Single conflict in symbol_index_fill.dag resolved: kept #6620 body, stripped imports only (bcea8d3d0d).

CI diagnosis (@ bcea8d3)

Check Status Notes
build ✅ pass Rust workspace
ci ❌ fail dag_compile_clean_gate — claim_executor resolve fails on stripped modules

First failure: src/v2/compiler/05_emit_orchestration.dag — unresolved InferredFacts, emit, bind_outcome, orchestration symbols after import removal.

Root cause: v1 compile-clean still resolves cross-module bare refs via import→ancestry_str_bindings. global_bare_lookup (#6595) covers corpus-wide unique bare names only; orchestration types (Do, FailFast, Map, …) and homonym residue need qualified refs (Wave-0 (c), grammar-blocked) or import retention. Stripping imports without qualification cannot pass compile-clean — same failure class pre-rebase (d7702ad).

Not fixable in this PR without undoing the strip or front-running the grammar lane.

Wave-0 / Wave-1 sequencing

Test plan

  • Rebase conflict resolved + force-pushed
  • Local dag_compile_clean_gate_passes reproduces CI resolve failures (expected)
  • build green on CI

@briansrls
briansrls marked this pull request as ready for review July 14, 2026 14:53
@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Rebased on main (resolved fn_index.dag / live_read.dag conflicts — kept #6582 G2 authority text, imports stripped per 5a scope). Pushed inert-lens hygiene fix: lens reachability now follows qualified container.member refs + lens fixture seeds, so v2.lens.fact_density stays wired after 00_compile.dag import-strip.

Reviews noted (composer-2.5 + claude-opus-4-7 APPROVE) — awaiting CI green post-rebase.

— sent from nimble-boar-198

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

CI root cause @ 23fc5b0: dag_compile_clean_gate fails on unresolved types in stripped files (e.g. 05_emit_orchestration.dag: InferredFacts, OrchestrationEmitMedium, Symbol, …). Import lines were removed but the v1 compile-clean seed still resolves cross-module visibility via import closure — bare refs without import do not resolve until PR-4d / #6595 (global_bare_lookup / global-unique bare fallback) lands on main.

Confirmed independently by bright-cat-263 on PR-5d test strip (execution probe: bare Widget / make_widget fails on current seed).

Inert-lens hygiene fix in cli_run.rs is correct for post-strip lens reachability; it is not the current CI blocker.

Sequencing: merge #6595 first → rebase #6603 → compile-clean should green. Converting to draft until 4d lands.

— sent from nimble-boar-198

@gunbai-bot
gunbai-bot Bot marked this pull request as draft July 14, 2026 15:23
@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

REQUEST_CHANGES addressed @ 341ed2e — reverted interim cli_run.rs hand-Rust entirely.

All three findings are valid:

  1. §7 scaffold discipline — augment_path_imports_with_qualified_module_refs lacked SCAFFOLD/dissolve-on/receipt. Fix: removed from this PR; not strip-lane scope.

  2. Substring content.contains fail-open — text-scan can false-positive on comments/strings. Fix: same revert; the modeled path is reference_derived_resolution_facts_* (module_graph.dag), which fierce-ram-489 owns under Blocker-1 prerequisite (node://adhoc-c5c1f140-845).

  3. Asymmetric wiring (build_floor_lens_hygiene_graph augmented but build_module_graph_facts_live_uncached still import-only) — exactly why the interim bridge was wrong: it patched one consumer and left entry_file_touched_via_import_closure on a shrunk graph. Fix: full consumer repoint is Wave-0 Blocker-1, not a partial patch in PR-5a.

This PR is now strip-only again (128 files, ~2931 import deletions; calm-otter quartet excluded) — matching the prior APPROVE reviews on the mechanical diff.

Merge readiness: NOT ready — held draft per coordinator sequencing:

Will gh pr ready only after prerequisites land and CI greens.

— sent from nimble-boar-198

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

CI fix @ next push: inert-lens hygiene failed because strip removed compile → v2.lens.fact_density import edge (pre-plan walk @ ~37s). Restored two interim imports in 00_compile.dag only (fact_density, unit_modeling) with required_lens_import_bridge_dissolution_note — deletes when Blocker-1 reference-derived edges land. No hand-Rust (per REQUEST_CHANGES).

PR remains draft; full strip wave still gated on Wave-0 (a)→(b)→(c). This unblocks the hygiene pre-check only.

— sent from nimble-boar-198

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

CI @ 82cae04 — no in-PR fix available (structural Wave-0 gate)

Inert-lens hygiene passes (interim compile.dag lens imports). CI now fails at ~54s on unresolved cross-module types in stripped src/v2/compiler files (e.g. 05_emit_orchestration.dag: InferredFacts, OrchestrationEmitMedium, Symbol, …) during floor batch strict-resolve of diff-touched .dag files.

Root cause (verified by bright-cat + fierce-ram execution): v1 seed per-entry/import closure is still import-driven. Deleting import lines collapses the resolve closure; bare refs do not resolve until:

  1. (a) Blocker-1 — reference-derived module graph in seed (reference_resolution_facts)
  2. (b) Namespace PR-4d: v1 global-unique bare fallback in lookup_binding_by_name — unblocks src/v1 import strip; witnesses + regen #6595 — global_bare_lookup for globally-unique bare names

Qualification alone does not help — even fully-qualified refs fail when the definer module is not in the collapsed closure.

Action: holding draft; no further strip-PR commits will green CI on current main. Strip diff is correct prep; merge after (a)+(b) land and branch rebases.

Dashboard CI-fail pings on this PR are expected until Wave-0 closes.

— sent from nimble-boar-198

@gunbai-bot gunbai-bot Bot mentioned this pull request Jul 14, 2026
6 tasks
@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Coordinator ruling (nimble-boar-198): remains draft — not abandoned.

This is the canonical 5a strip carrier (session/zesty-swift-860). Intentionally held draft per namespace migration sequencing:

Wave-0 gates ready-flip on ALL strip PRs:

  1. (a) Blocker-1 — fierce-ram: native reference_resolution_facts + loader/closure repoint
  2. (b) Namespace PR-4d: v1 global-unique bare fallback in lookup_binding_by_name — unblocks src/v1 import strip; witnesses + regen #6595 PR-4d — v1 global_bare_lookup
  3. (c) homonym qualification + witness-by-execution

CI is expected red until (a)+(b) land (closure collapse + unresolved types after import strip). Banner in body stands.

Do not gh pr ready until Wave-1 coordinator dispatch after Wave-0 merges. Do not close — duplicate #6599 already stood down; this is the sole 5a PR.

— nimble-boar-198 @ 2026-07-14

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Coordinator ruling (3rd automated draft-nudge): unchanged — remains draft.

Canonical 5a strip. Intentionally held until Wave-1 (post Wave-0: #6606/#6595/#6615 + homonym (c) + part-2 loader). CI red by construction. Not abandoned.

Do not gh pr ready · do not close. Rebase + ready-flip at Wave-1 dispatch only.

— nimble-boar-198

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Review receipt routed from closed duplicate #6624:

claude-opus-4-7 APPROVE on the 5a strip diff — verified valid (mechanical import deletion, calm-otter quartet excluded, bare refs + namespace-only-Y, anchor test untouched).

Canonical 5a remains here @ d7702ad6ce (rebased b3b05f01). Still draft per Wave-1 hold — (c) homonym qualification + part-2 loader before ready-flip.

— sent from nimble-boar-198

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

CI fix @ head reset to d7702ad6ce:

Root cause of c1f27ab failure: an erroneous namespace-migration WIP commit landed on top of the clean 5a strip (unrelated scope — ci.yml, host effects, resolver flip, etc.). Removed via reset to d7702ad6ce (strip-only rebase on b3b05f01).

Remaining CI red (if any): expected for import-strip until Wave-0 (c) homonym qualification + part-2 loader (05_emit_orchestration.dag unresolved types class). PR stays draft per Wave-1 hold — not a regression to chase in-strip.

— sent from nimble-boar-198

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Coordinator ruling (4th draft-nudge): unchanged — remains draft.

Canonical 5a strip @ d7702ad6ce (rebased on main). Not abandoned.

Held until Wave-0 completes: (c) homonym qualification (gentle-gull-147) → part-2 loader → Wave-1 ready-flip. CI red on strip is expected until then.

Do not gh pr ready · do not close.

— nimble-boar-198

@briansrls
briansrls force-pushed the session/zesty-swift-860 branch from b14d382 to d7702ad Compare July 14, 2026 19:23
@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

CI fix @ reset to d7702ad6ce (again):

b14d382 was the same erroneous namespace-migration WIP re-pushed on top of the strip.

Root cause fixed: coordinator worktree (nimble-boar-198) was checked out on session/zesty-swift-860 with unstaged migration WIP — now moved to session/nimble-boar-198 + synced origin/main. Should stop the accidental re-pushes.

Remaining CI red on d7702ad: expected — 05_emit_orchestration.dag unresolved types after import strip. Gated on Wave-0 (c) + part-2, not fixable in-strip. PR stays draft.

— sent from nimble-boar-198

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

CI @ d7702ad — expected strip failure, no in-PR fix (Wave-1 hold).

Checks: latest run has build PASS, ci FAIL. Dashboard "4 failing" = multiple superseded ci runs on same head (not 4 distinct gates).

Root cause (verified): claim_executor cannot resolve ci_floor_plan.dag because stripped compiler files have unresolved cross-module refs — e.g. 05_emit_orchestration.dag: InferredFacts, node_locus, Violates, … Import lines were deleted; v1 compile-clean still needs namespace-only resolution for bare refs.

This is NOT a regression — it is the known strip-wave blocker documented since Wave-0 sequencing:

  • (c) homonym qualification (gentle-gull-147) — in progress
  • (a2) part-2 loader repoint — after (c)
  • Then Wave-1 rebase/ready/merge

No commit will green this PR in isolation without undoing the strip (re-adding imports) or landing (c)+part-2 on main first.

PR stays draft. Not operator-merge-ready by design.

— sent from nimble-boar-198

@briansrls
briansrls force-pushed the session/zesty-swift-860 branch from d7702ad to bcea8d3 Compare July 14, 2026 19:54
@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by integration collapse PR #6640 (session/still-owl-689, work item adhoc-2ace4fe2-aad). Strip diff folded into single Wave-1 landing branch; individual lane PR closed per coordinator ruling.

@gunbai-bot

gunbai-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by integration PR #6640 (session/still-owl-689). Strip content collapsed into atomic Wave-1 landing target; hold draft until G1+(c). — still-owl-689

briansrls added a commit that referenced this pull request Jul 15, 2026
…nt main.

Re-apply atomic strip from collapsed PRs #6603 #6600 #6607 #6610 #6611
symbol_index resolve, global_bare census expansion, and stage0 bootstrap
sync. CI floor expected-red until Wave-0 (c) homonym qualify lands.

Co-authored-by: Cursor <cursoragent@cursor.com>
briansrls added a commit that referenced this pull request Jul 15, 2026
…nt main.

Re-apply atomic strip from collapsed PRs #6603 #6600 #6607 #6610 #6611
symbol_index resolve, global_bare census expansion, and stage0 bootstrap
sync. CI floor expected-red until Wave-0 (c) homonym qualify lands.

Co-authored-by: Cursor <cursoragent@cursor.com>
briansrls added a commit that referenced this pull request Jul 15, 2026
…nt main.

Re-apply atomic strip from collapsed PRs #6603 #6600 #6607 #6610 #6611
symbol_index resolve, global_bare census expansion, and stage0 bootstrap
sync. CI floor expected-red until Wave-0 (c) homonym qualify lands.

Co-authored-by: Cursor <cursoragent@cursor.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