chore: update dependencies + 7d minimum release age#2052
Draft
maxffarrell wants to merge 9 commits into
Draft
Conversation
Add min-version-age=7d to .npmrc so pnpm update only upgrades packages published at least 7 days ago. Update all dependencies to latest versions. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Downgrade vitest, @vitest/browser, and vitest-browser-svelte back to their v3/v1 series to maintain test stability. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… v4 upgrades
- Fix no-useless-assignment errors: remove redundant null initializers, refactor
event reassignment in dismissible layer, add eslint-disable for $bindable() rune
false positives (mounted.svelte, switch-demo-custom.svelte)
- Fix preserve-caught-error: add { cause: error } to thrown errors in build scripts
- Fix shiki v4 breaking change: replace getHighlighter with createHighlighter
- Suppress vite v8/vitest v3 Plugin type conflict in packages/bits-ui/vite.config.ts
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…8, and vitest/vite type conflict - Fix velite v0.3 double import attributes: skip post-processing if attributes already present - Fix flexsearch v0.8 API: replace namespace usage with named imports (Index, Id) - Fix tests/vite.config.ts: import defineConfig from vitest/config to support test property; suppress vite v8/vitest v3 plugin type mismatch Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
- Upgrade vitest to ^4.1.6, vitest-browser-svelte to ^2.1.1, vite to ^8.0.12 - Add @vitest/browser-playwright package for the new playwright() provider API - Update vite.config.ts: use playwright() function, maxWorkers instead of poolOptions.forks.maxForks - Fix type errors from vitest-browser-svelte v2: render() now returns PromiseLike so setup functions use await render() - Fix Element vs HTMLElement type errors across 13 test files (expect.element() now requires Locator|HTMLElement|SVGElement) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Vitest v4 rejects test.environment="browser" when browser.enabled is set. Remove the redundant environment key; browser.enabled: true is the correct API. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Svelte 5.55.5 introduced a change ("don't mark deriveds while an effect is
updating", sveltejs/svelte#18124) that breaks svelte-toolbelt's boxWith()
ref resolution, causing NavigationMenu refs to return a non-Element value
and failing the onFocusOutside/onkeydown guards. Tests pass with 5.55.4.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Undo the vitest v4 migration: restore v3 config, test files, and vitest-browser-svelte v1. Vitest v3 runs with provider: "playwright" (string) and environment: "browser", which is valid in v3. Note: vitest@3 declares vite@^7 as a peer; vite@8 is used here under the same pnpm override pattern established earlier in this branch. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
With vite 8, svelte-toolbelt's .svelte.js files get pre-bundled by esbuild (optimizeDeps) before vite-plugin-svelte can transform them, leaving Svelte runes ($derived, $state) uncompiled and undefined. Two-part fix: - optimizeDeps.exclude: ["svelte-toolbelt"] prevents esbuild from consuming the files before the Svelte compiler can see them. - vitePlugin.include: [/node_modules\/svelte-toolbelt/] in svelte.config.js opts svelte-toolbelt into vite-plugin-svelte's transform, which is normally skipped for node_modules. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
min-version-age=7dto.npmrcsopnpm updateonly upgrades packages published at least 7 days ago (guards against pulling in immediately-yanked releases)Notable upgrades
Test plan
pnpm build:packagespasses cleanlypnpm build(docs build has a pre-existing Node 26 / velite ESMwith { type: 'json' }issue unrelated to this PR)pnpm test:browser— run browser tests against updated deps🤖 Generated with Claude Code