Migrate from webpack to vite by silverwind · Pull Request #37002 · go-gitea/gitea

silverwind · 2026-03-26T19:16:53Z

Replace webpack with Vite 8 as the frontend bundler. Frontend build is around 3-4 times faster than before. Will work on all platforms including riscv64 (via wasm).

iife.js is a classic render-blocking script in <head> (handles web components/early DOM setup). index.js is loaded as a type="module" script in the footer. All other JS chunks are also module scripts (supported in all browsers since 2018).

Entry filenames are content-hashed (e.g. index.C6Z2MRVQ.js) and resolved at runtime via the Vite manifest, eliminating the ?v= cache busting (which was unreliable in some scenarios like vscode dev build).

Replaces: #36896
Fixes: #17793
Docs: https://gitea.com/gitea/docs/pulls/364

Replace webpack with Vite 8 (Rolldown) as the frontend bundler. Key changes: - Replace webpack.config.ts with vite.config.ts using Rolldown - Use native ES modules with import maps for cache busting - Build web components as a separate blocking IIFE bundle to prevent flash of unstyled content (same behavior as webpack's blocking script) - Update all dynamic imports from webpack chunk comments to standard dynamic import() syntax - Replace process.env.TEST with import.meta.env.MODE - Add vite/client types to tsconfig.json - Update Monaco error suppression regex for new chunk names - Rename WEBPACK_* Makefile variables to FRONTEND_* Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Replace query-string cache busting (?v=) with content-hashed entry filenames for JS and CSS assets. Vite now generates a manifest (.vite/manifest.json) mapping unhashed to hashed paths. A new Go-side AssetPath function reads the manifest and resolves paths in templates. This eliminates the need for the importmap workaround that was required because chunks imported the entry module without the ?v= query parameter. - Add modules/public/manifest.go with mtime-based cache invalidation so dev mode auto-detects frontend rebuilds - Add AssetPath template function for hashed path resolution - Update all templates to use AssetPath instead of ?v=AssetVersion - Pass sharedWorkerPath via window.config for the SharedWorker URL - Rename eventsource.sharedworker to sharedworker - Fix markup_external_test.go for type="module" and hashed paths Co-Authored-By: Claude (Opus 4.6) <[email protected]>

- Add content hash to webcomponents.js IIFE build and append its entry to the Vite manifest - Use AssetPath for swagger asset paths in openapi.go renderer - Strip content hash from theme CSS filenames in webtheme.go to correctly extract internal theme names - Remove AssetVersion variable and template function, now fully replaced by content-hashed filenames via manifest - Rename AssetPath to GetAssetPath Co-Authored-By: Claude (Opus 4.6) <[email protected]>

- Rename AssetPath to GetAssetPath in template helper and all templates - Clean up stale webcomponents files before IIFE rebuild - Add comment clarifying TypeError catch for navigation-during-import Co-Authored-By: Claude (Opus 4.6) <[email protected]>

`build()` returns an array for IIFE lib builds, so the `'output' in result` check silently skipped the manifest append. Handle both array and single-object return types. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

The webcomponents directory had both index.ts and webcomponents-blocking.ts with identical content. Remove the duplicate and use index.ts as the entry. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

- Add comment explaining ENABLE_SOURCEMAP=reduced backward compatibility - Add dev-licenses-stub plugin so licenses.txt exists in dev builds Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

In reduced mode (the production default), only sourcemaps for the index chunks are kept. All other map files including worker maps are deleted after the build. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Co-authored-by: Lunny Xiao <[email protected]> Signed-off-by: silverwind <[email protected]>

In reduced mode (the production default), node_modules are excluded from sourcemaps via a transform hook that empties their mappings. A closeBundle hook then deletes map files with no own code and strips node_modules sourcesContent from mixed maps, reducing total map size from 68 MB to 1.5 MB. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Signed-off-by: silverwind <[email protected]>

Replace the separate webcomponents IIFE + deferred module index.js with a single blocking IIFE index.js (containing bootstrap, jQuery, webcomponents, user-settings, htmx) and a deferred module index-domready.js loaded in the footer. This matches webpack's behavior: index.js is a blocking script that sets up globals before body parsing, while heavy features load asynchronously via index-domready. - Build index.ts as IIFE via lib mode in closeBundle, CSS extracted by main build - Add index-domready as separate entry in main build - Redirect jquery imports in main build to window.jQuery (single instance) - Export initGlobalErrorHandler to prevent double-init from shared chunks - Revert inline script type=module changes (blocking index.js makes them unnecessary) - Add index-domready.css to head_style.tmpl for Vue component styles Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Replace `import $ from 'jquery'` in fomantic wrappers with global $ (set by globals.ts in the IIFE). This eliminates the need for the jquery-global Vite plugin since no code in the main build imports jquery. Move CSS imports from index.ts to index-domready.ts so index.ts is only in the IIFE build. Remove index from main build entries — no duplicate build, no strip-css plugin needed. Single CSS file (index-domready.css). Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Replace os.Stat/os.ReadFile with AssetFS().Open() which handles both disk files and embedded bindata, respecting StaticRootPath. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Use sync.Once + cached AssetFS for manifest loading. In production the manifest is loaded once with zero per-request overhead. In dev mode, mod time is checked for watch-frontend hot reload. Remove htmx import from index.ts (IIFE) — it was being loaded twice (IIFE + index-domready), causing htmx to process hx-* attributes twice and breaking pages with htmx content. htmx in index-domready (deferred module) executes before DOMContentLoaded, so the init timing is correct. IIFE size: 204KB -> 134KB. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

delvh · 2026-03-29T07:18:50Z

Gitpod is actually used by some maintainers.
That's the reason why it was added in the first place.

TheFox0x7 · 2026-03-29T07:19:53Z

Well egg on my face then. :)

wxiaoguang · 2026-03-29T07:31:49Z

https://github.com/go-gitea/gitea/commits/main/.gitpod.yml

@yp05327 @yardenshoham @anbraten @6543 @techknowlogick are you still using gitpod?

silverwind · 2026-03-29T07:32:40Z

Why don't we serve the vite assets via its port directly

That'd require CORS and CORS is a pain in the ass that I strongly prefer to avoid by using reverse proxies. There's two valid setups imho:

Have the reverse proxy in go. A bit complicated because of the needed URL filtering logic.
Have the reverse proxy in vite. No filtering logic needed, but imho not right because your client will not be talking directly to the go backend.

So, I think the current approach with the go proxy is the best way that avoid CORS. Also I think we should not test primarly with CORS. It's pre-flight requests are not representative on how the server is commonly ran.

wxiaoguang · 2026-03-29T07:37:54Z

Why don't we serve the vite assets via its port directly

So, I think the current approach with the go proxy is the best way that avoid CORS.

It should be good to keep it now. I have refactored the vite proxy middleware. It is 99% clear to me now, and the accesses are properly limited (unless vite dev server has security problems in its websocket handler)

vite.config.ts

yardenshoham · 2026-03-29T07:38:54Z

https://github.com/go-gitea/gitea/commits/main/.gitpod.yml

@yp05327 @yardenshoham @anbraten @6543 @techknowlogick are you still using gitpod?

No they killed that product. Now they are some AI product

TheFox0x7 · 2026-03-29T08:14:49Z

huh. I would've kept the branding but their pick.
https://ona.com/stories/gitpod-is-now-ona

Anyway the gitpod.yaml is dead - it uses devcontainers now

silverwind · 2026-03-29T08:49:28Z

Let's remove gitpod then.

Gitpod has shut down, remove the configuration file, badge links from all READMEs, labeler entry, and devcontainer comment reference. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

silverwind · 2026-03-29T08:57:58Z

Gitpod removed, lets merge it.

.devcontainer/devcontainer.json

Signed-off-by: silverwind <[email protected]>

wxiaoguang · 2026-03-29T16:15:13Z

When running "make watch-frontend", it doesn't generate "manifest.json" sometimes

silverwind · 2026-03-29T16:35:35Z

dev server does not generate a manifest, it does not need one. manifest is for prod build only.

wxiaoguang · 2026-03-29T16:36:23Z

dev server does not generate a manifest, it does not need one.

But, you made the "theme" functions depend on manifest.

You will see no theme from fresh start.

wxiaoguang · 2026-03-29T16:37:35Z

silverwind · 2026-03-29T16:40:12Z

Hmm yes that's an issue, we have to make dev mode themes work without a previous build. And they should be HMR-able. I can check a bit later.

* main: (35 commits) Correct swagger annotations for enums, status codes, and notification state (go-gitea#37030) Update Nix flake (go-gitea#37024) Bump go and python versions in nix flake (go-gitea#37031) Make task list checkboxes clickable in the preview tab (go-gitea#37010) Add support for in_progress event in workflow_run webhook (go-gitea#36979) Fix various problems (go-gitea#37029) Update AI Contribution Policy (go-gitea#37022) Migrate from webpack to vite (go-gitea#37002) Upgrade yaml (go-gitea#37015) Fix issue label deletion with Actions tokens (go-gitea#37013) Hide delete branch or tag buttons in mirror or archived repositories. (go-gitea#37006) Update AGENTS.md with additional guidelines (go-gitea#37018) Optimize 'refreshAccesses' to perform update without removing then adding (go-gitea#35702) Fix relative-time RangeError (go-gitea#37021) Restyle Workflow Graph (go-gitea#36912) Update message severity colors, fix navbar double border (go-gitea#37019) Clean up checkbox cursor styles (go-gitea#37016) add missing cron tasks to example ini (go-gitea#37012) Add e2e tests for server push events (go-gitea#36879) Update JS dependencies (go-gitea#37001) ...

* origin/main: (69 commits) Correct swagger annotations for enums, status codes, and notification state (go-gitea#37030) Update Nix flake (go-gitea#37024) Bump go and python versions in nix flake (go-gitea#37031) Make task list checkboxes clickable in the preview tab (go-gitea#37010) Add support for in_progress event in workflow_run webhook (go-gitea#36979) Fix various problems (go-gitea#37029) Update AI Contribution Policy (go-gitea#37022) Migrate from webpack to vite (go-gitea#37002) Upgrade yaml (go-gitea#37015) Fix issue label deletion with Actions tokens (go-gitea#37013) Hide delete branch or tag buttons in mirror or archived repositories. (go-gitea#37006) Update AGENTS.md with additional guidelines (go-gitea#37018) Optimize 'refreshAccesses' to perform update without removing then adding (go-gitea#35702) Fix relative-time RangeError (go-gitea#37021) Restyle Workflow Graph (go-gitea#36912) Update message severity colors, fix navbar double border (go-gitea#37019) Clean up checkbox cursor styles (go-gitea#37016) add missing cron tasks to example ini (go-gitea#37012) Add e2e tests for server push events (go-gitea#36879) Update JS dependencies (go-gitea#37001) ... # Conflicts: # package.json # pnpm-lock.yaml # tests/e2e/utils.ts # web_src/css/themes/theme-gitea-dark.css # web_src/css/themes/theme-gitea-light.css # web_src/js/bootstrap.ts # web_src/js/features/codeeditor.ts # web_src/js/modules/errors.test.ts # webpack.config.ts

silverwind and others added 30 commits March 13, 2026 16:59

Merge branch 'main' into vite

a3d579e

cleanup

581bd1b

Fix webcomponents manifest entry not written

a419e10

`build()` returns an array for IIFE lib builds, so the `'output' in result` check silently skipped the manifest append. Handle both array and single-object return types. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

inline

e701562

Remove duplicate webcomponents-blocking.ts

2a198b0

The webcomponents directory had both index.ts and webcomponents-blocking.ts with identical content. Remove the duplicate and use index.ts as the entry. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Address review comments on vite.config.ts

6533de7

- Add comment explaining ENABLE_SOURCEMAP=reduced backward compatibility - Add dev-licenses-stub plugin so licenses.txt exists in dev builds Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Add upstream issue link for ENABLE_SOURCEMAP comment

e9554c7

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

add ref

df3c17a

Simplify ENABLE_SOURCEMAP handling

6215c5e

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Restore ENABLE_SOURCEMAP=reduced mode with Vite plugin

d6b34b3

In reduced mode (the production default), only sourcemaps for the index chunks are kept. All other map files including worker maps are deleted after the build. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Restore original ENABLE_SOURCEMAP variable names from webpack config

17ab78e

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Merge branch 'main' into vite

c160f2b

Update modules/public/manifest.go

5fb3d15

Co-authored-by: Lunny Xiao <[email protected]> Signed-off-by: silverwind <[email protected]>

Apply suggestion from @silverwind

3fcea7d

Signed-off-by: silverwind <[email protected]>

Use AssetFS() for Vite manifest reading

e13f106

Replace os.Stat/os.ReadFile with AssetFS().Open() which handles both disk files and embedded bindata, respecting StaticRootPath. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Use Vite's Manifest type for manifest handling

70568dd

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Add jQuery to vitest setup for global $ in tests

e0b1ad8

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Update stale webpack references to vite

14744ac

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

forbid jquery imports because of single global instance

ef8811f

Suppress plugin timing warnings for worker builds

fee04f3

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

Extract shared rolldownOptions for main, IIFE and worker builds

93ce326

Co-Authored-By: Claude (Opus 4.6) <[email protected]>

use function

ae24785

Merge branch 'main' into vite

bc22661

silverwind commented Mar 29, 2026

View reviewed changes

vite.config.ts Show resolved Hide resolved

Remove Gitpod integration

214ab0c

Gitpod has shut down, remove the configuration file, badge links from all READMEs, labeler entry, and devcontainer comment reference. Co-Authored-By: Claude (Opus 4.6) <[email protected]>

silverwind added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Mar 29, 2026

Merge branch 'main' into vite

ca37154

silverwind enabled auto-merge (squash) March 29, 2026 08:58

silverwind commented Mar 29, 2026

View reviewed changes

.devcontainer/devcontainer.json Outdated Show resolved Hide resolved

silverwind and others added 3 commits March 29, 2026 11:00

Apply suggestion from @silverwind

f2d119a

Signed-off-by: silverwind <[email protected]>

Merge branch 'main' into vite

743458b

Merge branch 'main' into vite

1027575

silverwind merged commit 0ec66b5 into go-gitea:main Mar 29, 2026
26 checks passed

silverwind deleted the vite branch March 29, 2026 10:24

GiteaBot added this to the 1.26.0 milestone Mar 29, 2026

GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Mar 29, 2026

Uh oh!

Conversation

silverwind commented Mar 26, 2026

Uh oh!

delvh commented Mar 29, 2026

Uh oh!

TheFox0x7 commented Mar 29, 2026

Uh oh!

wxiaoguang commented Mar 29, 2026

Uh oh!

silverwind commented Mar 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

wxiaoguang commented Mar 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

yardenshoham commented Mar 29, 2026

Uh oh!

TheFox0x7 commented Mar 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

silverwind commented Mar 29, 2026

Uh oh!

silverwind commented Mar 29, 2026

Uh oh!

Uh oh!

Uh oh!

wxiaoguang commented Mar 29, 2026

Uh oh!

silverwind commented Mar 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

wxiaoguang commented Mar 29, 2026

Uh oh!

wxiaoguang commented Mar 29, 2026

Uh oh!

silverwind commented Mar 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

silverwind commented Mar 29, 2026 •

edited

Loading

wxiaoguang commented Mar 29, 2026 •

edited

Loading

TheFox0x7 commented Mar 29, 2026 •

edited

Loading

silverwind commented Mar 29, 2026 •

edited

Loading

silverwind commented Mar 29, 2026 •

edited

Loading