Skip to content

Commit f38f67d

Browse files
committed
feat!: remove webpack-based EdgeBundler in favor of ESBuild
BREAKING CHANGE: The webpack-based EdgeBundler has been removed. All edge bundling now uses ESBuild via EdgeESBuildBundler. Users must update their CLI invocations from `--bundler webpack` to `--bundler esbuild`. - Remove src/EdgeBundler.js - Remove @adobe/helix-deploy-plugin-webpack peer dependency - Update all tests to use esbuild bundler - ESBuild provides faster bundling with smaller output 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Lars Trieloff <lars@trieloff.net>
1 parent 241d7a7 commit f38f67d

11 files changed

+21
-806
lines changed

package-lock.json

Lines changed: 13 additions & 623 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,7 @@
5656
"*.cjs": "eslint"
5757
},
5858
"peerDependencies": {
59-
"@adobe/helix-deploy": "^12.0.0 || ^13.0.0",
60-
"@adobe/helix-deploy-plugin-webpack": "^1.0.2"
59+
"@adobe/helix-deploy": "^12.0.0 || ^13.0.0"
6160
},
6261
"dependencies": {
6362
"@adobe/fastly-native-promises": "3.1.0",

src/EdgeBundler.js

Lines changed: 0 additions & 170 deletions
This file was deleted.

src/index.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,12 @@
1212

1313
import ComputeAtEdgeDeployer from './ComputeAtEdgeDeployer.js';
1414
import FastlyGateway from './FastlyGateway.js';
15-
import EdgeBundler from './EdgeBundler.js';
1615
import EdgeESBuildBundler from './EdgeESBuildBundler.js';
1716
import CloudflareDeployer from './CloudflareDeployer.js';
1817

1918
export const plugins = [
2019
ComputeAtEdgeDeployer,
2120
FastlyGateway,
2221
CloudflareDeployer,
23-
EdgeBundler,
2422
EdgeESBuildBundler,
2523
];

test/build.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ describe('Edge Build Test', () => {
7676
.prepare([
7777
'--target', 'wsk',
7878
'--plugin', path.resolve(__rootdir, 'src', 'index.js'),
79-
'--bundler', 'webpack',
79+
'--bundler', 'esbuild',
8080
'--esm', 'false',
8181
'--arch', 'edge',
8282
'--verbose',

test/cloudflare.integration.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ describe('Cloudflare Integration Test', () => {
6868
'--test', '/foo',
6969
'--directory', testRoot,
7070
'--entryFile', 'src/index.js',
71-
'--bundler', 'webpack',
71+
'--bundler', 'esbuild',
7272
'--esm', 'false',
7373
]);
7474
builder.cfg._logger = new TestLogger();

test/computeatedge.integration.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ describe('Fastly Compute@Edge Integration Test', () => {
6969
'--test', '/201',
7070
'--directory', testRoot,
7171
'--entryFile', 'src/index.js',
72-
'--bundler', 'webpack',
72+
'--bundler', 'esbuild',
7373
'--esm', 'false',
7474
]);
7575
builder.cfg._logger = new TestLogger();

test/deploy.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ describe('Deploy Test', () => {
7575
'--cloudflare-account-id', '123',
7676
'--cloudflare-auth', 'test-token',
7777
'--name', 'test-worker',
78-
'--bundler', 'webpack',
78+
'--bundler', 'esbuild',
7979
'--esm', 'false',
8080
]);
8181
builder.cfg._logger = new TestLogger();
@@ -141,7 +141,7 @@ describe('Deploy Test', () => {
141141
'--cloudflare-account-id', '123',
142142
'--cloudflare-auth', 'test-token',
143143
'--name', 'test-worker',
144-
'--bundler', 'webpack',
144+
'--bundler', 'esbuild',
145145
'--esm', 'false',
146146
]);
147147
builder.cfg._logger = new TestLogger();

test/edge-integration.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ describe('Edge Integration Test', () => {
8080
'-p', 'FOO=bar',
8181
'--directory', testRoot,
8282
'--entryFile', 'src/index.js',
83-
'--bundler', 'webpack',
83+
'--bundler', 'esbuild',
8484
'--esm', 'false',
8585
]);
8686
builder.cfg._logger = new TestLogger();

test/gateway.integration.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ describe.skip('Gateway Integration Test', () => {
6666
'-l', 'latest',
6767
'-l', 'major',
6868
'-l', 'minor',
69-
'--bundler', 'webpack',
69+
'--bundler', 'esbuild',
7070
'--esm', 'false',
7171
]);
7272
builder.cfg._logger = new TestLogger();

0 commit comments

Comments
 (0)