Skip to content

Commit 34d6745

Browse files
authored
chore(templates): minify: false in package-builder esbuild configs (#1214)
Both package-builder templates (cli-package and cli-sentry-package) shipped with minify: true in their esbuild.index.mts. Flip to false to match the repo-wide rule that shipped bundles never minify. Minification breaks ESM/CJS interop and makes debugging harder; the esbuild-minify validation scripts elsewhere in the org enforce this for finished packages. These templates should default to the same.
1 parent a01b649 commit 34d6745

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

packages/package-builder/templates/cli-package/.config/esbuild.index.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ const cliPath = path.resolve(__dirname, '../../cli')
1818
const config = createIndexConfig({
1919
entryPoint: path.join(cliPath, 'src', 'index.mts'),
2020
outfile: path.join(rootPath, 'dist', 'index.js'),
21-
minify: true,
21+
minify: false,
2222
})
2323

2424
if (fileURLToPath(import.meta.url) === process.argv[1]) {

packages/package-builder/templates/cli-sentry-package/.config/esbuild.index.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ const cliPath = path.resolve(__dirname, '../../cli')
1818
const config = createIndexConfig({
1919
entryPoint: path.join(cliPath, 'src', 'index.mts'),
2020
outfile: path.join(rootPath, 'dist', 'index.js'),
21-
minify: true,
21+
minify: false,
2222
})
2323

2424
if (fileURLToPath(import.meta.url) === process.argv[1]) {

0 commit comments

Comments
 (0)