- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.5k
Vue template fe bumps #4417
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
base: v3-alpha
Are you sure you want to change the base?
Vue template fe bumps #4417
Conversation
| WalkthroughThe dependency versions in the  Changes
 Suggested labels
 Suggested reviewers
 Poem
 📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (1)
 🚧 Files skipped from review as they are similar to previous changes (1)
 ⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit: 
 SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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
 | 
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: 2
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
- v3/internal/templates/vue-ts/frontend/package.json(1 hunks)
- v3/internal/templates/vue/frontend/package.json(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
v3/internal/templates/vue-ts/frontend/package.json (1)
Learnt from: fbbdev
PR: wailsapp/wails#4001
File: v3/internal/generator/testdata/output/lang=TS/UseInterfaces=true/UseNames=false/github.com/wailsapp/wails/v3/internal/generator/testcases/function_single_internal/greetservice.ts:0-0
Timestamp: 2025-01-15T22:33:30.639Z
Learning: In the Wails framework's TypeScript bindings, the import path "/wails/runtime.js" is intentionally absolute and should not be changed to a relative path.
v3/internal/templates/vue/frontend/package.json (1)
Learnt from: fbbdev
PR: wailsapp/wails#4001
File: v3/internal/generator/testdata/output/lang=TS/UseInterfaces=true/UseNames=false/github.com/wailsapp/wails/v3/internal/generator/testcases/function_single_internal/greetservice.ts:0-0
Timestamp: 2025-01-15T22:33:30.639Z
Learning: In the Wails framework's TypeScript bindings, the import path "/wails/runtime.js" is intentionally absolute and should not be changed to a relative path.
🔇 Additional comments (2)
v3/internal/templates/vue-ts/frontend/package.json (1)
12-15: Double-check Vue 3.5.x compatibility with Wails runtimeJumping from 3.2.x → 3.5.x is a sizable leap (multiple minor releases).
Please confirm that:
- The Wails bridge (
@wailsio/runtime) still functions with Vue 3.5 APIs (no breaking changes in reactivity/SSR APIs you rely on).- Any custom code in
src/main.tsthat relied on removed deprecations has been updated.Small local builds can pass yet runtime integration may surface edge cases.
Consider at least smoke-testing a freshly generated app under Windows & Linux before merging.v3/internal/templates/vue/frontend/package.json (1)
12-15: Verify runtime behaviour after Vue core bumpSame caution as the TS template: upgrade from 3.2.x → 3.5.x may surface subtle breaking changes (e.g.
isCustomElementconfig defaults, SFC macro tweaks).Run
npm run dev&npm run buildon a clean template, then launch withwails devto ensure no runtime regressions.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
| 
 | 
| Thanks for this. I'm keen to upgrade vite across all templates. If you're willing to do that - awesome. If you don't have time then that's fine, let's turn this into an issue and we can add it to the backlog 👍 | 
| Thanks 🙏 Please could you add an entry to the changelog located at  | 
| 
 I agree with this approach 😄 Give me another week or so to see if I find a moment to knock it out, otherwise I can close this PR and create an issue? | 



Description
Updates vue template frontend dependencies
Fixes # (issue)
Type of change
Dependency bump
Please select the option that is relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration using
wails doctor.If you checked Linux, please specify the distro and version.
Test Configuration
Checklist:
website/src/pages/changelog.mdxwith details of this PRSummary by CodeRabbit