Skip to content

Organize imports across Astro frontmatter and scripts - #17916

Merged
matthewp merged 1 commit into
mainfrom
issue-17909-organize-imports
Sep 8, 2026
Merged

matthewp merged 1 commit into
mainfrom
issue-17909-organize-imports

Conversation

@matthewp

@matthewp matthewp commented Sep 5, 2026 •

Copy link
Copy Markdown
Contributor

Changes

organize-imports-fixed

Testing

  • Adds full-document organize-import coverage for frontmatter and two script blocks.

Docs

  • No docs update needed; this restores expected editor behavior.

@changeset-bot

changeset-bot Bot commented Sep 5, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c0f70ca

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@matthewp
matthewp marked this pull request as ready for review September 5, 2026 19:03

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't we send a patch to volar instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not being actively maintained. My understanding is that after all of the content mapping stuff is done we'll be able to remove Volar from usage in our vscode plugin.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a pity

@matthewp
matthewp merged commit 8eaa4f2 into main Sep 8, 2026
28 checks passed
@matthewp
matthewp deleted the issue-17909-organize-imports branch September 8, 2026 16:16
@astrobot-houston astrobot-houston mentioned this pull request Sep 8, 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.

[Astro VS CODE] source.organizeImports doesn't run property if the component contains both server and client imports

2 participants