Skip to content

fix(compiler): keep consecutive imports adjacent (JUL-74) - #381

Merged
julia-script merged 2 commits into
mainfrom
julia/jul-74-keep-consecutive-import-declarations-adjacent-when
Sep 7, 2026
Merged

fix(compiler): keep consecutive imports adjacent (JUL-74)#381
julia-script merged 2 commits into
mainfrom
julia/jul-74-keep-consecutive-import-declarations-adjacent-when

Conversation

@julia-script

Copy link
Copy Markdown
Owner

Consecutive top-level imports now form a compact block, including across comments and authored blank lines. Import order and trailing comment attachment are preserved, and either boundary with a non-import retains one blank line.

Adds six formatter cases covering import order, duplicates, public imports, both boundaries, comments, module documentation, final newlines, clean reparsing, and unchanged second passes. Updates the two canonical pressure fixtures and generated compiler digest. The strict-validating OpenSpec change is synced to the main specification; archival remains pending successful CI.

Validation:

  • Passed locally: 57 SyntaxFormatter tests, pressure-fixture idempotence, typecheck, format:check, lint, strict OpenSpec validation, and all 135 maintained specifications.
  • The initial full suite found the corrected fixture mismatch and two 60-second timeouts (StoredCallableRuntime and StringLayout); both files passed in isolation. A subsequent full check hit a VectorAcceptance timeout.
  • Full local checks were stopped at the user's request because other local sessions are testing. CI owns the remaining full validation and release-candidate gate.

Closes JUL-74

@vercel

vercel Bot commented Sep 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
silk-effect-docs Ready Ready Preview Sep 7, 2026 6:01pm UTC

Request Review

@julia-script
julia-script merged commit ebaf717 into main Sep 7, 2026
15 checks passed
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