Skip to content

build: include vinext in docker build#34535

Merged
hyoban merged 15 commits intomainfrom
4-3-prod-for-vinext
Apr 8, 2026
Merged

build: include vinext in docker build#34535
hyoban merged 15 commits intomainfrom
4-3-prod-for-vinext

Conversation

@hyoban
Copy link
Copy Markdown
Member

@hyoban hyoban commented Apr 3, 2026

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

Fixes #34534

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran make lint and make type-check (backend) and cd web && npx lint-staged (frontend) to appease the lint gods

@github-actions github-actions Bot added the web This relates to changes on the web. label Apr 3, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.75%. Comparing base (4d4265f) to head (a80781b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #34535      +/-   ##
==========================================
- Coverage   83.75%   83.75%   -0.01%     
==========================================
  Files        4179     4179              
  Lines      176784   176788       +4     
  Branches    35366    35367       +1     
==========================================
+ Hits       148065   148067       +2     
- Misses      25839    25840       +1     
- Partials     2880     2881       +1     
Flag Coverage Δ
api 82.36% <ø> (-0.01%) ⬇️
web 85.46% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

hyoban and others added 10 commits April 7, 2026 11:57
- Enable build-time precompression (.br/.gz/.zst) via vinext precompress option,
  eliminating per-request compression overhead on the production server
- Set build target to es2022 to reduce unnecessary syntax downleveling
- Use lightningcss for CSS minification
- Remove dead commented-out reactGrabOpenFilePlugin code
Convert elkjs from a static top-level import to a dynamic import()
with lazy singleton initialization. Remove elk-layout from the
workflow utils barrel to prevent the 2.5MB elkjs bundle from
polluting the shared chunk loaded by all pages.

The utils chunk drops from 1,553KB to 330KB. elkjs is now an
independent 1,432KB chunk loaded only when users trigger the
workflow auto-layout feature.
The elk-layout functions are now imported directly from
utils/elk-layout instead of the barrel, so the vi.mock target
must match the new import path.
@hyoban hyoban marked this pull request as ready for review April 8, 2026 07:15
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 8, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 8, 2026
@hyoban hyoban enabled auto-merge April 8, 2026 07:20
@hyoban hyoban added this pull request to the merge queue Apr 8, 2026
Merged via the queue into main with commit aad0b3c Apr 8, 2026
42 checks passed
@hyoban hyoban deleted the 4-3-prod-for-vinext branch April 8, 2026 07:36
HanqingZ pushed a commit to HanqingZ/dify that referenced this pull request Apr 23, 2026
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: yyh <92089059+lyzno1@users.noreply.github.com>
Co-authored-by: yyh <yuanyouhuilyz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files. web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide vinext build for prod as an option

2 participants