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

@NoelDeMartin NoelDeMartin commented Jun 30, 2026

Copy link
Copy Markdown
Member

@timea-solid timea-solid linked an issue Jun 30, 2026 that may be closed by this pull request
@timea-solid timea-solid moved this to In progress in SolidOS NLNet UI Jul 1, 2026
@timea-solid timea-solid requested a review from Copilot July 1, 2026 09:26

Copilot AI 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
File Description
webpack.dev.config.mjs Removed legacy Webpack dev-server configuration.
webpack.config.mjs Removed legacy Webpack production bundling configuration.
vite.config.mts Added Vite + Vitest configuration and dev-sandbox aliases/plugins.
tsconfig.test.json Updated test TS config to include Node types.
tsconfig.dev.json Removed legacy dev TS config used by Webpack/ts-loader.
test/unit/trustedApplications/trustedApplications.test.ts Migrated test imports from Jest env to Vitest.
test/unit/trustedApplications/snapshots/trustedApplications.test.ts.snap Updated snapshot header/keys for Vitest.
test/unit/social/triage.test.ts Migrated Jest mocking (jest.fn) to Vitest (vi.fn).
test/unit/outline/propertyViews.test.ts Migrated test imports from Jest env to Vitest.
test/unit/outline/manager.test.ts Migrated test imports from Jest env to Vitest.
test/setup.ts Added Vitest setup (jest-dom integration + fetch mocking + globals).
test/helpers/setup.ts Removed Jest-specific setup.
test/helpers/globals.js Removed Jest global shims previously needed for UMD/global expectations.
src/outline/manager.js Switched a global assignment from global to globalThis.
src/dev.ts Added a Vite dev-sandbox pane entry that initializes the main page.
README.md Updated development instructions for the Vite dev server/port.
package.json Switched scripts to Vite/Vitest and updated package entrypoints/exports.
jest.config.mjs Removed Jest configuration.
dev/pane/index.ts Removed legacy dev pane loader.
dev/loader.ts Removed legacy Webpack-based dev loader.
dev/index.html Removed legacy dev HTML shell for Webpack dev-server.
dev/dev-mash.css Removed legacy dev-only mash CSS bundle.
dev/dev-mash-utilities.css Removed legacy dev-only utility CSS bundle.
dev/dev-light.css Removed legacy dev-only light theme CSS bundle.
babel.config.mjs Removed Babel configuration previously used for builds/tests.

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

Comment thread vite.config.mts Outdated
Comment thread src/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
@NoelDeMartin NoelDeMartin moved this from In progress to In review in SolidOS NLNet UI Jul 1, 2026
@bourgeoa bourgeoa requested a review from timea-solid July 1, 2026 10:40
@timea-solid

Copy link
Copy Markdown
Member

@NoelDeMartin take a look at copilot feedback.

@NoelDeMartin

Copy link
Copy Markdown
Member Author

@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:staging Jul 2, 2026
8 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in SolidOS NLNet UI Jul 2, 2026
Sign up for free to 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