Skip to content

Conversation

@sapphi-red
Copy link
Member

import analysis plugin was rewriting from import '/home/path/to/file.js' to import '/path/to/file.js' when the base is /home/. But since we treat the ids in imports as file paths instead of URLs, we shouldn't trim the base here. (import analysis plugin rewrites file paths into URLs)

fixes vitejs/vite-plugin-vue#25
close #7224

@sapphi-red sapphi-red added the p2-edge-case Bug, but has workaround or limited in scope (priority) label Dec 5, 2025
@sapphi-red
Copy link
Member Author

/ecosystem-ci run

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 5, 2025

Open in StackBlitz

npm i https://pkg.pr.new/vitejs/vite/@vitejs/plugin-legacy@21221
npm i https://pkg.pr.new/vitejs/vite@21221

commit: ed1ec62

@vite-ecosystem-ci
Copy link

📝 Ran ecosystem CI on f1d979b: Open

suite result latest scheduled
histoire failure success
nuxt failure success
astro failure failure
analogjs success failure
one failure success
qwik failure failure
rakkas failure success
react-router failure ⏹️ cancelled
sveltekit failure success
unocss success failure
storybook failure success
vike failure failure
vite-plugin-cloudflare failure failure
vite-plugin-pwa failure success
vite-plugin-svelte failure success
vitest failure success

ladle, marko, laravel, quasar, vite-plugin-vue, vite-environment-examples, vite-setup-catalogue, vuepress, vitepress, waku, vite-plugin-rsc, vite-plugin-react

@sapphi-red
Copy link
Member Author

probably better to make a PR to v7 branch and then run the ecosystem-ci
also need to add tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p2-edge-case Bug, but has workaround or limited in scope (priority) trigger: preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vue style import will conflict with the base url config if base url is the prefix of project's full path

1 participant