Skip to content

Add ESM build#887

Merged
ext-helska merged 2 commits into
masterfrom
MOL-3496-dual-esm-build
Jun 4, 2026
Merged

Add ESM build#887
ext-helska merged 2 commits into
masterfrom
MOL-3496-dual-esm-build

Conversation

@ext-helska

@ext-helska ext-helska commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Gjør at @digitalarkivet/mui-theme bygges og publiseres både som CommonJS og ESM.

Betyr at eksisterende brukere som laster ned pakken via require fortsatt får dagens CommonJS-build fra dist/, mens moderne bundlere som Next/Turbopack kan velge ESM-builden fra dist/esm/ automatisk via package.json exports. Dette skal gi bedre tree-shaking, særlig for imports som @mui/material/locale, der CommonJS-builden kan dra inn mer kode enn nødvendig.

Lagt til et postbuild-steg for å gjøre ESM-outputen gyldig med .js på relative imports og "type": "module" i dist/esm, fordi mottak-web krasjet på ESM-entrypointen uten dette.

Ved verifisering i mottak-web ble resultatet (sammenlignet med tidligere):

develop:
.next/static: 3.8M
gzip JS/CSS: 825,121
største chunk: 208K

før da-mui-theme-fiks:
.next/static: 47M
gzip JS/CSS: 7,132,520
største chunks: 6 x 7.5M

etter 4.1.1:
.next/static: 5.3M
gzip JS/CSS: 1,485,233
største chunks: 6 x 560K

etter ESM-build:
.next/static: 2.6M
gzip JS/CSS: 845,356
største chunk: 208K

@ext-helska ext-helska marked this pull request as ready for review June 4, 2026 07:14
guffen
guffen previously approved these changes Jun 4, 2026

@guffen guffen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LG! Bump versjon i samme PR?

@ext-helska ext-helska merged commit f3aa694 into master Jun 4, 2026
1 check passed
@ext-helska ext-helska deleted the MOL-3496-dual-esm-build branch June 4, 2026 08:59
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.

2 participants