Skip to content
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

fix(qwikVite): stale QRL segments in dev mode #6945

Merged
merged 5 commits into from
Oct 15, 2024
Merged

Conversation

wmertens
Copy link
Member

@wmertens wmertens commented Oct 7, 2024

This simplifies the plugin code and tries harder to make updated QRL segments get into the browser.

Fix #6935

Copy link

changeset-bot bot commented Oct 7, 2024

🦋 Changeset detected

Latest commit: 6d70625

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@builder.io/qwik-city Patch
@builder.io/qwik Patch
eslint-plugin-qwik Patch
create-qwik Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@wmertens wmertens marked this pull request as ready for review October 15, 2024 15:19
@wmertens wmertens requested review from a team as code owners October 15, 2024 15:19
@wmertens wmertens force-pushed the refactor-plugin branch 5 times, most recently from 496e889 to d9aa89e Compare October 15, 2024 16:27
Copy link

pkg-pr-new bot commented Oct 15, 2024

Open in Stackblitz

npm i https://pkg.pr.new/@builder.io/qwik@6945
npm i https://pkg.pr.new/@builder.io/qwik-city@6945
npm i https://pkg.pr.new/eslint-plugin-qwik@6945
npm i https://pkg.pr.new/create-qwik@6945

commit: d9aa89e

Copy link
Contributor

github-actions bot commented Oct 15, 2024

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview d9aa89e

@wmertens wmertens enabled auto-merge October 15, 2024 17:38
@wmertens wmertens self-assigned this Oct 15, 2024
@wmertens wmertens added the COMP: DX Developer Experience related issue label Oct 15, 2024
this prevents duplicate vite versions
this is especially problematic in dev
- more robust dev server URL handling in resolveId
- remove ?_qrl_parent query and rely only on path
- remove foundQrls export, instead only keep track of generated segments' parents in parentIds
- enforce externals while building libraries, this is suddenly needed because of these changes
- better hot update invalidation
- more robust getSymbolHash
Nix now supports playwright 1.47.0, and the old version broke CI
@wmertens wmertens merged commit 8f806b1 into main Oct 15, 2024
16 checks passed
@wmertens wmertens deleted the refactor-plugin branch October 15, 2024 19:39
@github-actions github-actions bot mentioned this pull request Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
COMP: DX Developer Experience related issue
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[🐞] For each change we have to stop/start the dev server to refresh the stale cache in the browser
2 participants