Skip to content

Use the Vite+ Zed settings - #2413

Merged
kmcginnes merged 2 commits into
mainfrom
zed-vite-plus-settings
Oct 14, 2026
Merged

kmcginnes merged 2 commits into
mainfrom
zed-vite-plus-settings

Conversation

@kmcginnes

@kmcginnes kmcginnes commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Zed was formatting through an external command and had no lint server, while the docs already ask Zed users to install oxc-zed. This switches .zed/settings.json to the settings Vite+ generates for Zed, so format-on-save and inline lint come from the oxfmt and oxlint language servers, both reading vite.config.ts.

 {
-  "format_on_save": "on",
-  "formatter": [{ "external": { "command": "pnpm", "arguments": ["exec", "vp", "fmt", "--stdin-filepath", "{buffer_path}"] } }],
   "lsp": {
+    "oxlint": { "initialization_options": { "settings": { "run": "onType", "typeAware": true, ... } } },
+    "oxfmt": { "initialization_options": { "settings": { "fmt.configPath": "./vite.config.ts", "run": "onSave" } } },
     "tailwindcss-language-server": { ... }
   },
   "languages": {
+    // one entry per language Vite+ formats (17), each using the oxfmt language server
+    // JavaScript also runs source.fixAll.oxc
     "CSS": { "language_servers": ["tailwindcss-intellisense-css", "!vscode-css-language-server", "..."] }
   }
 }

The Tailwind lsp block and the CSS language_servers override are kept from the old file. The CSS list gains "..." so the oxfmt and oxlint servers still start for CSS; without it Zed runs only the servers listed by name.

Verified live: in Zed with oxc-zed, format on save and inline lint.

Merge Danger

Door: two-way

Impact: Zed users

Without the oxc-zed extension, Zed no longer formats on save. docs/development.md already tells Zed users to install it.

Related

@kmcginnes
kmcginnes merged commit 67a82f7 into main Oct 14, 2026
3 checks passed
@kmcginnes
kmcginnes deleted the zed-vite-plus-settings branch October 14, 2026 21:28
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