Upgrade Vite 8 and Svelte tooling

[mtsgrd]

Summary

• Bumps Vite from 6.4.1 → 8.0.0, @sveltejs/kit 2.52.2 → 2.55.0, and @sveltejs/vite-plugin-svelte 6.2.4 → 7.0.0
• Fixes CJS interop regressions in Vite 8/Rolldown for redux-persist: imports persistStore from the main ESM entry and replaces the lib/storage import with an inline localStorage wrapper
• Skips oxc TypeScript preprocessing for node_modules Svelte files to prevent cross-block imports (e.g. createCommand) from being stripped
• Sets css: "injected" for node_modules Svelte files to avoid PostCSS issues
• Excludes svelte-lexical from pre-bundling and explicitly includes all @lexical/* packages (Vite 8 no longer auto-handles process.env.NODE_ENV for transitive deps of excluded packages)
• Removes the now-invalid manualChunks rollup option (Rolldown replaces Rollup in Vite 8)
• Fixes PrTemplateSection.svelte to use get(path) instead of a top-level $store subscription (required by Svelte 5 runes mode)

Test plan

• pnpm dev starts without errors in apps/desktop
• App renders and initializes correctly (layout mounts, redux-persist rehydrates)
• Lexical editor (svelte-lexical) works without createCommand reference errors
• PR template section loads and toggles correctly

🤖 Generated with Claude Code

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
gitbutler-web	Ready	Preview, Comment	Mar 22, 2026 5:37pm

[Copilot]

Pull request overview

Upgrades the Desktop app’s Vite/SvelteKit toolchain to Vite 8 and adjusts Vite/Svelte preprocessing and dependency optimization to work around Vite 8/Rolldown regressions (notably around CJS interop and Svelte preprocessing of node_modules).

Changes:

• Bump Vite to 8.0.0 and update SvelteKit / vite-plugin-svelte versions in the workspace catalog.
• Update Desktop Vite/Svelte config to avoid new Vite 8 + tooling pitfalls (dependency optimization, preprocessing, CSS handling for node_modules).
• Adjust redux-persist integration and a Svelte component store access pattern for compatibility.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
pnpm-workspace.yaml	Updates catalog versions for Vite and Svelte tooling.
package.json	Adds a pnpm override to force Vite 8.0.0 resolution.
apps/desktop/vite.config.ts	Updates optimizeDeps handling for svelte-lexical/Lexical and removes manualChunks config.
apps/desktop/svelte.config.js	Filters TS preprocessing for node_modules and injects CSS for node_modules Svelte files.
apps/desktop/src/lib/state/clientState.svelte.ts	Changes redux-persist imports and replaces lib/storage with an inline localStorage wrapper.
apps/desktop/src/components/PrTemplateSection.svelte	Switches to get(store) instead of using $store in script logic (Svelte 5 runes compatibility).

[Copilot]

optimizeDeps.include lists @lexical/dragon and @lexical/extension, but these packages don’t appear to be installed (no entries in pnpm-lock.yaml). With Vite, unresolved entries in optimizeDeps.include typically fail dependency optimization / dev server startup. Remove the non-existent entries, or add them as explicit dependencies (and pin via overrides/catalog) so they resolve consistently.

Suggested change

	"@lexical/dragon",
	"@lexical/extension",

[Copilot]

Pull request overview

Copilot reviewed 8 out of 10 changed files in this pull request and generated 2 comments.

[Copilot]

The inline redux-persist storage wrapper calls localStorage.* inside Promise.resolve(...), which will throw synchronously if access fails (e.g., storage disabled/quota exceeded) instead of returning a rejected promise. Wrap the localStorage operations in try/catch (or use an async function) so failures are surfaced as promise rejections and don’t crash store initialization.

Suggested change

	getItem: (key: string) => Promise.resolve(localStorage.getItem(key)),
	setItem: (key: string, value: string) => Promise.resolve(localStorage.setItem(key, value)),
	removeItem: (key: string) => Promise.resolve(localStorage.removeItem(key)),
	getItem: async (key: string) => {
	return localStorage.getItem(key);
	},
	setItem: async (key: string, value: string) => {
	localStorage.setItem(key, value);
	},
	removeItem: async (key: string) => {
	localStorage.removeItem(key);
	},

[Copilot]

PR description mentions excluding svelte-lexical from Vite pre-bundling, but optimizeDeps.exclude here only lists the local @gitbutler packages. If svelte-lexical still needs to be excluded for Vite 8, add it to optimizeDeps.exclude; otherwise update the PR description to match the actual behavior.