Skip to content

Commit 921c283

Browse files
committed
fix size limit
1 parent ee1ec19 commit 921c283

File tree

3 files changed

+23
-17
lines changed

3 files changed

+23
-17
lines changed

.size-limit.js

Lines changed: 20 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -2,28 +2,28 @@ module.exports = [
22
// Main browser webpack builds
33
{
44
name: '@sentry/browser (incl. Tracing, Replay, Feedback) - Webpack (gzipped)',
5-
path: 'packages/browser/build/npm/esm/index.js',
5+
path: 'packages/browser/build/npm/esm/index.mjs',
66
import: '{ init, Replay, browserTracingIntegration, Feedback }',
77
gzip: true,
88
limit: '90 KB',
99
},
1010
{
1111
name: '@sentry/browser (incl. Tracing, Replay) - Webpack (gzipped)',
12-
path: 'packages/browser/build/npm/esm/index.js',
12+
path: 'packages/browser/build/npm/esm/index.mjs',
1313
import: '{ init, Replay, browserTracingIntegration }',
1414
gzip: true,
1515
limit: '75 KB',
1616
},
1717
{
1818
name: '@sentry/browser (incl. Tracing, Replay with Canvas) - Webpack (gzipped)',
19-
path: 'packages/browser/build/npm/esm/index.js',
19+
path: 'packages/browser/build/npm/esm/index.mjs',
2020
import: '{ init, Replay, browserTracingIntegration, ReplayCanvas }',
2121
gzip: true,
2222
limit: '90 KB',
2323
},
2424
{
2525
name: '@sentry/browser (incl. Tracing, Replay) - Webpack with treeshaking flags (gzipped)',
26-
path: 'packages/browser/build/npm/esm/index.js',
26+
path: 'packages/browser/build/npm/esm/index.mjs',
2727
import: '{ init, Replay, browserTracingIntegration }',
2828
gzip: true,
2929
limit: '75 KB',
@@ -42,35 +42,35 @@ module.exports = [
4242
},
4343
{
4444
name: '@sentry/browser (incl. Tracing) - Webpack (gzipped)',
45-
path: 'packages/browser/build/npm/esm/index.js',
45+
path: 'packages/browser/build/npm/esm/index.mjs',
4646
import: '{ init, browserTracingIntegration }',
4747
gzip: true,
4848
limit: '35 KB',
4949
},
5050
{
5151
name: '@sentry/browser (incl. browserTracingIntegration) - Webpack (gzipped)',
52-
path: 'packages/browser/build/npm/esm/index.js',
52+
path: 'packages/browser/build/npm/esm/index.mjs',
5353
import: '{ init, browserTracingIntegration }',
5454
gzip: true,
5555
limit: '35 KB',
5656
},
5757
{
5858
name: '@sentry/browser (incl. Feedback) - Webpack (gzipped)',
59-
path: 'packages/browser/build/npm/esm/index.js',
59+
path: 'packages/browser/build/npm/esm/index.mjs',
6060
import: '{ init, Feedback }',
6161
gzip: true,
6262
limit: '50 KB',
6363
},
6464
{
6565
name: '@sentry/browser (incl. sendFeedback) - Webpack (gzipped)',
66-
path: 'packages/browser/build/npm/esm/index.js',
66+
path: 'packages/browser/build/npm/esm/index.mjs',
6767
import: '{ init, sendFeedback }',
6868
gzip: true,
6969
limit: '50 KB',
7070
},
7171
{
7272
name: '@sentry/browser - Webpack (gzipped)',
73-
path: 'packages/browser/build/npm/esm/index.js',
73+
path: 'packages/browser/build/npm/esm/index.mjs',
7474
import: '{ init }',
7575
gzip: true,
7676
limit: '28 KB',
@@ -137,37 +137,43 @@ module.exports = [
137137
// React
138138
{
139139
name: '@sentry/react (incl. Tracing, Replay) - Webpack (gzipped)',
140-
path: 'packages/react/build/esm/index.js',
140+
path: 'packages/react/build/esm/index.mjs',
141141
import: '{ init, browserTracingIntegration, Replay }',
142142
gzip: true,
143143
limit: '75 KB',
144144
},
145145
{
146146
name: '@sentry/react - Webpack (gzipped)',
147-
path: 'packages/react/build/esm/index.js',
147+
path: 'packages/react/build/esm/index.mjs',
148148
import: '{ init }',
149149
gzip: true,
150150
limit: '30 KB',
151151
},
152152

153153
// Next.js
154+
// We ignore next/router and next/constants because they break size-limit calculation
155+
// with new *.mjs bundles in v8
154156
{
155157
name: '@sentry/nextjs Client (incl. Tracing, Replay) - Webpack (gzipped)',
156-
path: 'packages/nextjs/build/esm/client/index.js',
158+
path: 'packages/nextjs/build/esm/client/index.mjs',
157159
import: '{ init, browserTracingIntegration, Replay }',
160+
ignore: ['next/router', 'next/constants'],
158161
gzip: true,
159162
limit: '110 KB',
160163
},
161164
{
162165
name: '@sentry/nextjs Client - Webpack (gzipped)',
163-
path: 'packages/nextjs/build/esm/client/index.js',
166+
path: 'packages/nextjs/build/esm/client/index.mjs',
164167
import: '{ init }',
168+
ignore: ['next/router', 'next/constants'],
165169
gzip: true,
166170
limit: '57 KB',
167171
},
172+
173+
// Feedback
168174
{
169175
name: '@sentry-internal/feedback - Webpack (gzipped)',
170-
path: 'packages/feedback/build/npm/esm/index.js',
176+
path: 'packages/feedback/build/npm/esm/index.mjs',
171177
import: '{ Feedback }',
172178
gzip: true,
173179
limit: '25 KB',

dev-packages/rollup-utils/npmHelpers.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ export function makeBaseNPMConfig(options = {}) {
118118
}
119119

120120
export function makeNPMConfigVariants(baseConfig, options = {}) {
121-
const { emitMjs = true, entryFileNameWithoutExtension = '[name]'} = options;
121+
const { emitMjs = true, entryFileNameWithoutExtension = '[name]' } = options;
122122
const variantSpecificConfigs = [
123123
{ output: { format: 'cjs', dir: path.join(baseConfig.output.dir, 'cjs') } },
124124
{ output: { format: 'esm', dir: path.join(baseConfig.output.dir, 'esm') } },

packages/nextjs/rollup.npm.config.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ export default [
5353
},
5454
}),
5555
// Preserve the original file structure (i.e., so that everything is still relative to `src`)
56-
{ entryFileNameWithoutExtension: 'config/templates/[name]' }
56+
{ entryFileNameWithoutExtension: 'config/templates/[name]' },
5757
),
5858
...makeNPMConfigVariants(
5959
makeBaseNPMConfig({
@@ -68,6 +68,6 @@ export default [
6868
},
6969
}),
7070
// Preserve the original file structure (i.e., so that everything is still relative to `src`)
71-
{ entryFileNameWithoutExtension: 'config/loaders/[name]' }
71+
{ entryFileNameWithoutExtension: 'config/loaders/[name]' },
7272
),
7373
];

0 commit comments

Comments
 (0)