Skip to content

Downgrade dev tooling to svelte-3-compatible versions#2

Open
Copilot wants to merge 3 commits intomainfrom
copilot/debug-deploy-multi-version-docs
Open

Downgrade dev tooling to svelte-3-compatible versions#2
Copilot wants to merge 3 commits intomainfrom
copilot/debug-deploy-multi-version-docs

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 28, 2026

The lockfile on main was stale from when the project used svelte 4. The "legacy version bump" commit downgraded svelte to ^3.0.0 in package.json but left all dev tooling at svelte-4+-only versions (SvelteKit 2, svelte-preprocess 6, vite 5). The previous fix bumped svelte back to ^4.0.0 — this instead keeps svelte 3 and downgrades the tooling to match.

Dev dependency downgrades

  • @sveltejs/kit: ^2.5.17^1.0.0 (kit 1 supports svelte ≥3.54)
  • @sveltejs/adapter-auto: ^3.2.2^2.0.0
  • @sveltejs/adapter-static: ^3.0.2^2.0.0
  • svelte-preprocess: ^6.0.1^5.0.0 (v6 requires svelte 4+)
  • vite: 5.3^4.0.0 (kit 1 peers on vite 4)
  • typescript: ^5.4.5~5.4.5 (TS 5.5+ removed importsNotUsedAsValues/preserveValueImports that kit 1's generated tsconfig emits)

Config fix

svelte-preprocess 5 uses a default export; 6 introduced the named export:

// before (v6)
import { sveltePreprocess } from "svelte-preprocess";
// after (v5)
import sveltePreprocess from "svelte-preprocess";

Lockfile

Deleted stale lockfile and regenerated. Resolves to svelte 3.59.2, kit 1.30.4, vite 4.5.14. npm ci and npm run build both pass.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…4.0.0

The deploy-multi-version docs workflow was failing because `npm ci`
requires package.json and package-lock.json to be in sync. The lockfile
had svelte@4.2.18 but the devDependency specified ^3.0.0 (which only
allows 3.x). Updated to ^4.0.0 to match reality and regenerated the
lockfile.

Co-authored-by: thinkle-iacs <15049315+thinkle-iacs@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix deploy-multi-version docs failing for svelte5 commits Fix deploy-multi-version docs: sync svelte devDependency with lockfile Feb 28, 2026
@thinkle-iacs
Copy link
Copy Markdown
Collaborator

Sorry did you up the svelte version on Main? Main is meant for svelte 3-4 -- it's the svelte5 branch that needs a newer svelte. I'm wondering if I accidentally committed the lock file from the svelte5 branch into the main branch and that's the root problem?

@thinkle-iacs thinkle-iacs marked this pull request as ready for review February 28, 2026 01:05
Copilot AI review requested due to automatic review settings February 28, 2026 01:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the repo’s Svelte devDependency spec to align with the existing lockfile resolution, preventing npm ci failures in the deploy-docs workflow on Node 20 / npm 10+ due to dependency spec/lock mismatch.

Changes:

  • Bump devDependencies.svelte from ^3.0.0 to ^4.0.0 in package.json.
  • Regenerate package-lock.json to synchronize root package metadata and dependency specs (including the Svelte range).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Updates Svelte devDependency range to allow the locked Svelte 4.x version.
package-lock.json Syncs lockfile root metadata and dependency specs with package.json (including Svelte).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI changed the title Fix deploy-multi-version docs: sync svelte devDependency with lockfile Fix package.json/lockfile sync to unbreak deploy-multi-version docs Feb 28, 2026
…ing svelte

- @sveltejs/kit: ^2.5.17 -> ^1.0.0 (kit 1 supports svelte 3)
- @sveltejs/adapter-auto: ^3.2.2 -> ^2.0.0
- @sveltejs/adapter-static: ^3.0.2 -> ^2.0.0
- svelte-preprocess: ^6.0.1 -> ^5.0.0 (v5 supports svelte 3)
- vite: 5.3 -> ^4.0.0 (kit 1 requires vite 4)
- typescript: ^5.4.5 -> ~5.4.5 (pin to 5.4.x; TS 5.5+ removed kit 1 tsconfig options)
- svelte.config.js: use default import for svelte-preprocess 5
- Regenerated package-lock.json fresh

Co-authored-by: thinkle-iacs <15049315+thinkle-iacs@users.noreply.github.com>
Copilot AI changed the title Fix package.json/lockfile sync to unbreak deploy-multi-version docs Downgrade dev tooling to svelte-3-compatible versions Feb 28, 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.

3 participants