Repository navigation
Conversation
Replace Parcel 1 with Vite for building docs-v1.js and styles-v1.css, and Jest with Vitest for the web resources tests. * One `vite.config.mjs` builds both files as a classic script plus CSS and fonts. The IE polyfills (details-polyfill, url-search-params-polyfill) are dropped. * `build_docs --open` runs `vite build --watch` (unminified, with a source map) and nginx serves its output directly instead of proxying to the Parcel dev server. There is no HMR any more, only rebuild and reload. * Node in the base image goes from 12 to 24, which Vite needs. * Imports use bare specifiers instead of paths into /node_modules. * Ramda is imported per function because Vitest chokes on its `then` export. Two files that contain JSX are renamed to .jsx. * lang-console no longer uses a regex `s` flag, which prettify drops and Parcel used to compile away. * The preview tests (still Jest) get nock 13 and error message regexes that work on current Node. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
* Use `ENV key=value` for the locale variables. * Install the nodesource and yarn apt keys as `signed-by` keyrings instead of with the deprecated `apt-key add`. The yarn key is now a binary keyring because apt on buster doesn't accept armored ones there. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… migrate-parcel-to-vite
Mpdreamz
requested changes
Nov 4, 2026
Mpdreamz
left a comment
Member
There was a problem hiding this comment.
This is great work! The reasons my previous attempt failed was that parcel 2 refused to run the tests the same way and I got into setup and concurrency issues amongst other things.
Let's park this and discuss offline what we want to do, we also have half a plan to migrate all the old asciidoc to markdown and docs-builder.
akira28
marked this pull request as draft
November 4, 2026 16:29
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replaces Parcel 1 with Vite 8 for building
docs-v1.jsandstyles-v1.css, and Jest with Vitest for theresources/webtests. The IE polyfills are dropped.What changed
resources/web/vite.config.mjsbuilds a classic (IIFE) script, the CSS and the fonts.lib/ES/Util.pmcallsvite build. Fonts keep their plain names instead of Parcel's hashed ones.build_docs --open): runsvite build --watch --minify false --sourcemapinto its own directory, and nginx serves it directly. There is no HMR any more, only rebuild and reload. Port 8001 is no longer published. Polling on Docker for Mac still works throughCHOKIDAR_USEPOLLING.resources/webtests run under Vitest, so there's no Parcel step for tests. The preview tests stay on Jest, withnock13 and error-message regexes that work on current Node./node_modules.thenexport).modal.jsandmount.jsrenamed to.jsx.sflag removed inlang-console.js. prettify drops regex flags, and Parcel's Babel step used to rewrite it.precssmangles inheading.pcss(&:has(h1.title)). The rule was already dead, so error recovery drops it.details-polyfilllicense. Removed the checks for Parcel HMR, the CSS source map and the JS test comment (Vite drops plain comments).Testing
make -C resources/web: 274 Vitest tests pass.make -C preview: 48 Jest tests pass.rubocopandpycodestyleare clean.To check before merging
baseline-widely-available, so IE and old browsers are no longer supported. Parcel's Babel step produced ES5 output.build_docs --openin a browser. The integration specs cover the watcher and nginx path.🤖 Generated with Claude Code