Skip to content

Rolldown-Vite #84

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

Draft
wants to merge 231 commits into
base: main
Choose a base branch
from
Draft

Rolldown-Vite #84

wants to merge 231 commits into from

Conversation

sapphi-red
Copy link
Member

@sapphi-red sapphi-red commented Mar 7, 2025

Description

Current Status

  • Vite repo test status
    • ✅ unit tests
    • e2e tests
      • ✅ external: changed the test for intended behavior change
      • ❌ legacy: system output is not supported yet
      • ⚠️ object-hooks: sequential: false object hooks are not supported
      • ✅ resolve: skipped some browser field related tests as those cases were only supported by esbuild and Vite, also skipped some test that does fallback based on the file content which is Vite only and I believe it should not be done
  • ecosystem-ci (see https://github.com/vitejs/vite-ecosystem-ci/blob/rolldown-vite/README-temp.md)

additional features

  • tsconfig paths support (feat: add resolve.tsconfigPaths option to resolve with tsconfig paths #277)
    • When resolve.tsconfigPaths: true is set, resolution respecting tsconfig paths is supported
    • This option requires experimental.enableNativePlugin to be set to 'resolver' or true
    • Currently this feature only supports basic cases. If the tsconfig.json uses references/include/exclude, it won't work as expected.
  • moduleType is supported in hook filters (experimental)

Intended behavior differences

Known missing features

TODO list for me
  • set rolldown's minify target
  • update docs

Copy link

pkg-pr-new bot commented Mar 7, 2025

Open in StackBlitz

npm i https://pkg.pr.new/vitejs/rolldown-vite@84

commit: 18e54bf

@sapphi-red sapphi-red force-pushed the rolldown-vite branch 2 times, most recently from 9ce2683 to a974a4a Compare March 17, 2025 03:57
@sapphi-red sapphi-red changed the title WIP Rolldown-Vite Mar 17, 2025
@sapphi-red sapphi-red mentioned this pull request Mar 17, 2025
3 tasks
`function(dep, importerUrl) { return new URL(dep, importerUrl).href }`
: // If the base isn't relative, then the deps are relative to the projects `outDir` and the base
// is appended inside __vitePreload too.
`function(dep) { return ${JSON.stringify(environment.config.base)}+dep }`

Check warning

Code scanning / CodeQL

Improper code sanitization

Code construction depends on an [improperly sanitized value](1).
@christianjuth
Copy link

christianjuth commented Mar 25, 2025

Found this PR in the Vite docs. Curious how stable rolldown-vite is. On a scale of "Next.js would call this is stable" to "this will actually work 99% of the time", were are we at?

@sapphi-red sapphi-red force-pushed the rolldown-vite branch 2 times, most recently from 51fd297 to 3fc2ac5 Compare March 26, 2025 02:29
@sapphi-red
Copy link
Member Author

Found this PR in the Vite docs. Curious how stable rolldown-vite is. On a scale of "Next.js would call this is stable" to "this will actually work 99% of the time", were are we at?

I'd say it would work for basic cases as it is passing most tests through out the ecosystem.

sapphi-red and others added 17 commits March 26, 2025 14:58
Co-authored-by: underfin <likui.underfin@gmail.com>
Co-authored-by: underfin <likui.underfin@gmail.com>
Co-authored-by: Hiroshi Ogawa <hi.ogawa.zz@gmail.com>
Co-authored-by: Yury <YurySolovyov@users.noreply.github.com>
Co-authored-by: underfin <2218301630@qq.com>
Co-authored-by: IWANABETHATGUY <iwanabethatguy@qq.com>
Co-authored-by: IWANABETHATGUY <iwanabethatguy@qq.com>
Co-authored-by: IWANABETHATGUY <iwanabethatguy@qq.com>
Co-Authored-By: Evan You <evan@vuejs.org>
shulaoda and others added 30 commits June 23, 2025 11:00
…tform is `browser` (#290)

Co-authored-by: 翠 <green@sapphi.red>
Co-authored-by: Hiroshi Ogawa <hi.ogawa.zz@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants