Skip to content

Import strip: src/v2/test + src/v2/lens (464 files) - #6951

Closed
briansrls wants to merge 5 commits into
mainfrom
test/loader-plus-strip
Closed

briansrls wants to merge 5 commits into
mainfrom
test/loader-plus-strip

Conversation

@briansrls

Copy link
Copy Markdown
Contributor

Auto-opened by session-dashboard for session calm-deer-217.
Pushing to test/loader-plus-strip 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.

Brian Searls and others added 5 commits July 20, 2026 19:28
…files)

Brace-depth-aware removal of import declarations only — no reference
qualification. Files that fail compile-clean after strip keep their imports
(held for parent lane): affected_set.dag, application.dag,
corpus_dependency_view.dag, entry_selection.dag, module_graph.dag.

Excluded from strip: layering_scan plant fixtures (import is the planted
violation). Whole-tree compile: 0 hard diagnostics.

Co-authored-by: Cursor <cursoragent@cursor.com>
@gunbai-bot

gunbai-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Superseded by #6937 — this draft branch had an incomplete early cherry-pick of the loader fix (tree_bare_census only, not the full both-closure gate-loader fix in #6950). The complete merge is on session/calm-deer-217 at 9234ad3.

@gunbai-bot gunbai-bot Bot closed this Jul 20, 2026
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