-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(deps): update dependency vite to v6 #738
Conversation
WalkthroughThis pull request updates the Changes
Possibly related PRs
Suggested labels
Poem
📜 Recent review detailsConfiguration used: .coderabbit.yaml ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
🔇 Additional comments (4)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Dependency ReviewThe following issues were found:
Vulnerabilitiesrefarch-frontend/package-lock.json
OpenSSF ScorecardScorecard details
Scanned Files
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (1)
refarch-frontend/package-lock.json
is excluded by!**/package-lock.json
📒 Files selected for processing (1)
refarch-frontend/package.json
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build (refarch-backend)
🔇 Additional comments (2)
refarch-frontend/package.json (2)
41-41
: Verify compatibility with Vite 6.0.7.The upgrade from Vite 5 to 6 is a major version bump that requires careful verification:
- Node.js compatibility: Your current Node.js version (>=22.11 <=22) meets Vite 6's requirement of Node.js 18+.
- Vite plugins compatibility needs verification.
Run this script to check compatibility of current dependencies with Vite 6:
41-41
: Review breaking changes and required actions.Please review the Vite 6 Migration Guide for breaking changes that might affect your build configuration:
- The
build.minify
option now defaults toesbuild
- Changes to SSR externalization heuristics
- Changes to CSS handling
Let's check your Vite configuration for potential breaking changes:
✅ Verification successful
No configuration changes required for Vite 6 upgrade
Your Vite configurations are using standard features and don't override any of the options affected by Vite 6 breaking changes. The default changes in Vite 6 are safe for your current setup.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Search for Vite configuration that might be affected by v6 breaking changes # Look for vite.config files echo "Searching for Vite configuration files..." fd -e ts -e js -g 'vite.config.*' # Check for usage of affected options echo "Checking for potentially affected configuration..." rg -l 'build.minify|ssr|css' -g 'vite.config.*'Length of output: 357
Script:
#!/bin/bash echo "=== Content of refarch-frontend/vite.config.ts ===" cat refarch-frontend/vite.config.ts echo -e "\n=== Content of refarch-webcomponent/vite.config.ts ===" cat refarch-webcomponent/vite.config.tsLength of output: 2920
This PR also requires the bump to Vitest 3.
and check what breaking changes are relevant to us. We should also replace the usage of deprecated API. |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
Vitest:
Besides this small change no other changes where necessary here. This is mostly due to the RefArch only containing one simple unit test. |
Vite:
The more interesting breaking changes like Environment and Runtime API are experimental for now and thus we don't inspect them. They will become stable with Vite 7 and then we need to respect those breaking changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR contains the following updates:
5.4.14
->6.0.7
6.0.11
(+3)Release Notes
vitejs/vite (vite)
v6.0.7
Compare Source
minify
whenbuilder.sharedPlugins: true
(#19025) (f7b1964), closes #19025vite-ignore
attribute for inline script (#19062) (a492253), closes #19062v6.0.6
Compare Source
fetchModule
-side resolve (#18361) (9f10261), closes #18361build.target
(#19047) (0e9e81f), closes #19047v6.0.5
Compare Source
v6.0.4
Compare Source
this.resolve
skipSelf should not skip for differentid
orimport
(#18903) (4727320), closes #18903pluginContainer.getModuleInfo
(#18895) (258cdd6), closes #18895url()
when lightningcss is used (#18997) (3734f80), closes #18997true
(#18899) (8a6bb4e), closes #18899handleInvoke
(#18902) (27f691b), closes #18902ModuleRunnerTransport
invoke
API (#18865) (e5f5301), closes #18865v6.0.3
Compare Source
ModuleRunnerTransport#invoke
more explicit (#18851) (a75fc31), closes #18851environments.ssr.resolve
with rootssr
config (#18857) (3104331), closes #18857v6.0.2
Compare Source
node_modules
if deno (#18823) (a20267b), closes #18823style.css
as a key for the style file forcssCodesplit: false
(#18820) (ec51115), closes #18820external
by default (#18821) (2250ffa), closes #18821ssr.target: 'webworker'
defaults as fallback (#18827) (b39e696), closes #18827v6.0.1
Compare Source
proxy
prevents starting http2 server (#18788) (bbaf514), closes #18788applyToEnvironment
hooks on worker build (#18793) (0c6cdb0), closes #18793v6.0.0
Compare Source
createRunnableDevEnvironment
returnsRunnableDevEnvironment
, notDevEnvironment
(#18673) (74221c3), closes #18673getModulesByFile
should return aserverModule
(#18715) (b80d5ec), closes #18715SIGTERM
(#18741) (cc55e36), closes #18741InterceptorOptions
type (#18766) (6252c60), closes #18766__vite_ssr_identity__
with(0, ...)
and inject;
between statements (#18748) (94546be), closes #18748Object.keys(import.meta.glob(...))
/ `Object.values(import.meta.glob( (ed99a2c), closes #18666module-sync
condition when loading config if enabled (#18650) (cf5028d), closes #18650HotUpdateContext
toHotUpdateOptions
(#18718) (824c347), closes #18718playground/json/__tests__/ssr
(#18701) (f731ca2), closes #18701Configuration
📅 Schedule: Branch creation - "* 0-3 * * 1" in timezone Europe/Berlin, Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.
Summary by CodeRabbit
it
function withtest
function in the test cases for improved compatibility with the Vitest framework, ensuring continued validation of component behavior.