Skip to content

Migrate to Vite - #707

Merged
timea-solid merged 1 commit into
SolidOS:stagingfrom
NoelDeMartin:vite
Jul 2, 2026
Merged

Migrate to Vite#707
timea-solid merged 1 commit into
SolidOS:stagingfrom
NoelDeMartin:vite

Conversation

@NoelDeMartin

@NoelDeMartinNoelDeMartin commented Jun 30, 2026

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Migrates the project’s build/dev/test toolchain from Webpack + Jest to Vite + Vitest, aligning the repo with solidos-toolkit’s Vite integration and introducing a Vite-based pane development sandbox.

Changes:

  • Remove Webpack/Babel/Jest configuration and legacy dev sandbox assets.
  • Add Vite/Vitest configuration (including coverage + setup file) and update unit tests/snapshots accordingly.
  • Update package entrypoints/exports and npm scripts to use Vite for build/dev/watch.

Reviewed changes

Copilot reviewed 25 out of 26 changed files in this pull request and generated 2 comments.

Show a summary per file
FileDescription
webpack.dev.config.mjsRemoved legacy Webpack dev-server configuration.
webpack.config.mjsRemoved legacy Webpack production bundling configuration.
vite.config.mtsAdded Vite + Vitest configuration and dev-sandbox aliases/plugins.
tsconfig.test.jsonUpdated test TS config to include Node types.
tsconfig.dev.jsonRemoved legacy dev TS config used by Webpack/ts-loader.
test/unit/trustedApplications/trustedApplications.test.tsMigrated test imports from Jest env to Vitest.
test/unit/trustedApplications/snapshots/trustedApplications.test.ts.snapUpdated snapshot header/keys for Vitest.
test/unit/social/triage.test.tsMigrated Jest mocking (jest.fn) to Vitest (vi.fn).
test/unit/outline/propertyViews.test.tsMigrated test imports from Jest env to Vitest.
test/unit/outline/manager.test.tsMigrated test imports from Jest env to Vitest.
test/setup.tsAdded Vitest setup (jest-dom integration + fetch mocking + globals).
test/helpers/setup.tsRemoved Jest-specific setup.
test/helpers/globals.jsRemoved Jest global shims previously needed for UMD/global expectations.
src/outline/manager.jsSwitched a global assignment from global to globalThis.
src/dev.tsAdded a Vite dev-sandbox pane entry that initializes the main page.
README.mdUpdated development instructions for the Vite dev server/port.
package.jsonSwitched scripts to Vite/Vitest and updated package entrypoints/exports.
jest.config.mjsRemoved Jest configuration.
dev/pane/index.tsRemoved legacy dev pane loader.
dev/loader.tsRemoved legacy Webpack-based dev loader.
dev/index.htmlRemoved legacy dev HTML shell for Webpack dev-server.
dev/dev-mash.cssRemoved legacy dev-only mash CSS bundle.
dev/dev-mash-utilities.cssRemoved legacy dev-only utility CSS bundle.
dev/dev-light.cssRemoved legacy dev-only light theme CSS bundle.
babel.config.mjsRemoved Babel configuration previously used for builds/tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadvite.config.mts Outdated
Comment threadsrc/dev.ts
@NoelDeMartin
NoelDeMartin marked this pull request as draft July 1, 2026 10:18
@NoelDeMartin
NoelDeMartin marked this pull request as ready for review July 1, 2026 10:22
@NoelDeMartinNoelDeMartin moved this from In progress to In review in SolidOS NLNet UIJul 1, 2026
@bourgeoa
bourgeoa requested a review from timea-solidJuly 1, 2026 10:40
@timea-solid

Copy link
Copy Markdown
Member

@NoelDeMartin take a look at copilot feedback.

@NoelDeMartin

Copy link
Copy Markdown
MemberAuthor

@NoelDeMartin take a look at copilot feedback.

I already did, and replied/resolved the issues it raised :).

@timea-solid
timea-solid merged commit c437907 into SolidOS:stagingJul 2, 2026
8 checks passed
@github-project-automationgithub-project-automationBot moved this from In review to Done in SolidOS NLNet UIJul 2, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Replace header v2 components Migrate to Vite

3 participants

@NoelDeMartin@timea-solid