Skip to content

Upgrade vite to 8 and vitest to 5 - #45

Merged
Django1982 merged 1 commit into
mainfrom
claude/busy-hamilton-z2zym0
Oct 3, 2026
Merged

Django1982 merged 1 commit into
mainfrom
claude/busy-hamilton-z2zym0

Conversation

@Django1982

Copy link
Copy Markdown
Owner

Summary

Follow-up to #41. Takes the toolchain to the versions Dependabot proposes in #39/#40 (vite 8.3.2, vitest 5.0.3), but on top of current main. Those PRs' lockfiles predate #41 and conflict with it. After this merges, #36, #37, #39 and #40 are all superseded.

Related issue

Supersedes Dependabot PRs #36, #37, #39, #40.

Type of change

  • Refactoring or maintenance
  • Build, installer, or release change

Changes made

  • vite ^7.3.6 → ^8.3.2 (rolldown-based bundler), vitest ^4.1.11 → ^5.0.3, lockfile regenerated.
  • CLAUDE.md Node note updated (requirement unchanged: 20.19+ / 22.12+).

Testing performed

  • I checked for new console or build errors.

  • I added or updated relevant documentation.

  • Clean npm ci (npm 10), npm audit: 0 vulnerabilities

  • npm test: 478 passed

  • npm run build OK; dist/index.html still references assets via relative ./ paths (required for Tauri/file:///subpath hosting)

  • npm run build:server OK, npm run validate:printers OK

  • vite dev server serves / (200)

Test environment

Operating system: Linux (cloud container)
PerfectFit version or branch: 1.3.2 / claude/busy-hamilton-z2zym0
Slicer and version: n/a
Printer, if relevant: n/a

Compatibility and risks

The bundler changes from Rollup to Rolldown. Output was checked for the relative base, but the desktop app (npm run tauri build) was not built here, and nobody clicked through the bundle in a browser. Worth a quick smoke test of the built app before the next release.

Final checklist

  • My changes are focused on one issue or purpose.
  • I reviewed my own changes.
  • I did not include unrelated formatting or generated files.
  • Existing functionality continues to work.
  • I am ready to respond to review feedback.

🤖 Generated with Claude Code

https://claude.ai/code/session_012ogkkoq4TrDQskE34iXVzx


Generated by Claude Code

Brings the toolchain to the versions Dependabot proposes in #39/#40,
on top of the vite 7 / vitest 4 security bump. Vite 8 bundles with
rolldown; build output keeps the relative ./ base. Node requirement
is unchanged (20.19+ / 22.12+).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012ogkkoq4TrDQskE34iXVzx
@Django1982
Django1982 merged commit f3f197b into main Oct 3, 2026
7 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.

2 participants