Skip to content

Optimise Vite config for production builds (hosted app)#1486

Merged
dcblundell merged 3 commits into
devfrom
chore/chunk-hosted-js
May 14, 2026
Merged

Optimise Vite config for production builds (hosted app)#1486
dcblundell merged 3 commits into
devfrom
chore/chunk-hosted-js

Conversation

@dcblundell
Copy link
Copy Markdown
Contributor

@dcblundell dcblundell commented May 14, 2026

Chunks JS files and turns off some features we don't need (source maps and compression stats).

Closes #727.

@dcblundell dcblundell marked this pull request as ready for review May 14, 2026 16:04
@dcblundell dcblundell requested a review from Copilot May 14, 2026 16:04
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Vite production build configuration for the hosted app to reduce bundle size impact by splitting out several heavy third-party dependencies into dedicated chunks, and to streamline production build outputs.

Changes:

  • Disable CSS code-splitting to emit a single CSS bundle for production builds.
  • Disable JS source maps and compressed-size reporting to reduce build overhead/output.
  • Add Rollup manualChunks rules to split Plotly, Vis, XYFlow, and BlueprintJS into separate vendor chunks.

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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread vite.config.ts
@dcblundell dcblundell merged commit 63cc414 into dev May 14, 2026
15 checks passed
@dcblundell dcblundell deleted the chore/chunk-hosted-js branch May 14, 2026 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hosted App: Investigate dynamically importing weighty packages

3 participants