Skip to content

Commit 29a58e5

Browse files
clydinclydin
committed
build: move build-angular/build-webpack specific dependencies out of root
With the migration to `rules_js`, package specific dependencies now only need to be referenced with the source `package.json` for each specific package. Most of the `@angular-devkit/build-angular` and `@angular-devkit/build-webpack` specific dependencies have now been moved. This is not exhaustive and further changes will continue to move additional dependencies.
1 parent f4be831 commit 29a58e5

6 files changed

Lines changed: 36 additions & 127 deletions

File tree

‎package.json‎

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -67,14 +67,8 @@
6767
"@babel/helper-annotate-as-pure": "7.25.9",
6868
"@babel/helper-split-export-declaration": "7.24.7",
6969
"@babel/plugin-syntax-import-attributes": "7.26.0",
70-
"@babel/plugin-transform-async-generator-functions": "7.26.8",
71-
"@babel/plugin-transform-async-to-generator": "7.25.9",
72-
"@babel/plugin-transform-runtime": "7.26.10",
73-
"@babel/preset-env": "7.26.9",
74-
"@babel/runtime": "7.26.10",
7570
"@bazel/bazelisk": "1.25.0",
7671
"@bazel/buildifier": "8.0.3",
77-
"@discoveryjs/json-ext": "0.6.3",
7872
"@eslint/compat": "1.2.7",
7973
"@eslint/eslintrc": "3.3.0",
8074
"@eslint/js": "9.22.0",
@@ -116,15 +110,11 @@
116110
"ajv": "8.17.1",
117111
"ajv-formats": "3.0.1",
118112
"ansi-colors": "4.1.3",
119-
"autoprefixer": "10.4.21",
120-
"babel-loader": "10.0.0",
121113
"beasties": "0.2.0",
122114
"browser-sync": "3.0.3",
123115
"browserslist": "^4.21.5",
124116
"buffer": "6.0.3",
125117
"chokidar": "4.0.3",
126-
"copy-webpack-plugin": "13.0.0",
127-
"css-loader": "7.1.2",
128118
"debug": "^4.1.1",
129119
"esbuild": "0.25.1",
130120
"esbuild-wasm": "0.25.1",
@@ -153,14 +143,10 @@
153143
"karma-jasmine-html-reporter": "~2.1.0",
154144
"karma-source-map-support": "1.4.0",
155145
"less": "4.2.2",
156-
"less-loader": "12.2.0",
157-
"license-webpack-plugin": "4.0.2",
158146
"listr2": "8.2.5",
159147
"lmdb": "3.2.6",
160-
"loader-utils": "3.3.1",
161148
"lodash": "^4.17.21",
162149
"magic-string": "0.30.17",
163-
"mini-css-extract-plugin": "2.9.2",
164150
"mrmime": "2.0.1",
165151
"ng-packagr": "20.0.0-next.1",
166152
"npm": "^11.0.0",
@@ -173,26 +159,21 @@
173159
"picomatch": "4.0.2",
174160
"piscina": "4.8.0",
175161
"postcss": "8.5.3",
176-
"postcss-loader": "8.1.1",
177162
"prettier": "^3.0.0",
178163
"protractor": "~7.0.0",
179164
"puppeteer": "18.2.1",
180165
"quicktype-core": "23.0.171",
181-
"resolve-url-loader": "5.0.0",
182166
"rollup": "4.35.0",
183167
"rollup-license-plugin": "~3.0.1",
184168
"rollup-plugin-sourcemaps": "^0.6.0",
185169
"rxjs": "7.8.2",
186170
"sass": "1.85.1",
187-
"sass-loader": "16.0.5",
188171
"semver": "7.7.1",
189172
"shelljs": "^0.9.0",
190173
"source-map": "0.7.4",
191-
"source-map-loader": "5.0.0",
192174
"source-map-support": "0.5.21",
193175
"symbol-observable": "4.0.0",
194176
"tar": "^7.0.0",
195-
"terser": "5.39.0",
196177
"tinyglobby": "0.2.12",
197178
"tree-kill": "1.2.2",
198179
"ts-node": "^10.9.1",
@@ -203,11 +184,6 @@
203184
"verdaccio": "6.0.5",
204185
"verdaccio-auth-memory": "^10.0.0",
205186
"watchpack": "2.4.2",
206-
"webpack": "5.98.0",
207-
"webpack-dev-middleware": "7.4.2",
208-
"webpack-dev-server": "5.2.0",
209-
"webpack-merge": "6.0.1",
210-
"webpack-subresource-integrity": "5.1.0",
211187
"yargs": "17.7.2",
212188
"yargs-parser": "21.1.1",
213189
"zone.js": "^0.15.0"

‎packages/angular_devkit/build_angular/BUILD.bazel‎

Lines changed: 27 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,33 @@ ts_project(
134134
":node_modules/@angular-devkit/core",
135135
":node_modules/@angular/build",
136136
":node_modules/@angular/ssr",
137+
":node_modules/@babel/plugin-transform-async-generator-functions",
138+
":node_modules/@babel/plugin-transform-async-to-generator",
139+
":node_modules/@babel/plugin-transform-runtime",
140+
":node_modules/@babel/preset-env",
141+
":node_modules/@babel/runtime",
142+
":node_modules/@discoveryjs/json-ext",
137143
":node_modules/@ngtools/webpack",
138144
":node_modules/@vitejs/plugin-basic-ssl",
145+
":node_modules/autoprefixer",
146+
":node_modules/babel-loader",
147+
":node_modules/copy-webpack-plugin",
148+
":node_modules/css-loader",
149+
":node_modules/less",
150+
":node_modules/less-loader",
151+
":node_modules/license-webpack-plugin",
152+
":node_modules/loader-utils",
153+
":node_modules/mini-css-extract-plugin",
154+
":node_modules/postcss-loader",
155+
":node_modules/resolve-url-loader",
156+
":node_modules/sass-loader",
157+
":node_modules/source-map-loader",
158+
":node_modules/terser",
159+
":node_modules/webpack",
160+
":node_modules/webpack-dev-middleware",
161+
":node_modules/webpack-dev-server",
162+
":node_modules/webpack-merge",
163+
":node_modules/webpack-subresource-integrity",
139164
"//:node_modules/@ampproject/remapping",
140165
"//:node_modules/@angular/common",
141166
"//:node_modules/@angular/compiler-cli",
@@ -147,12 +172,6 @@ ts_project(
147172
"//:node_modules/@babel/generator",
148173
"//:node_modules/@babel/helper-annotate-as-pure",
149174
"//:node_modules/@babel/helper-split-export-declaration",
150-
"//:node_modules/@babel/plugin-transform-async-generator-functions",
151-
"//:node_modules/@babel/plugin-transform-async-to-generator",
152-
"//:node_modules/@babel/plugin-transform-runtime",
153-
"//:node_modules/@babel/preset-env",
154-
"//:node_modules/@babel/runtime",
155-
"//:node_modules/@discoveryjs/json-ext",
156175
"//:node_modules/@types/babel__core",
157176
"//:node_modules/@types/babel__generator",
158177
"//:node_modules/@types/browser-sync",
@@ -166,12 +185,8 @@ ts_project(
166185
"//:node_modules/@web/test-runner",
167186
"//:node_modules/ajv",
168187
"//:node_modules/ansi-colors",
169-
"//:node_modules/autoprefixer",
170-
"//:node_modules/babel-loader",
171188
"//:node_modules/browser-sync",
172189
"//:node_modules/browserslist",
173-
"//:node_modules/copy-webpack-plugin",
174-
"//:node_modules/css-loader",
175190
"//:node_modules/esbuild",
176191
"//:node_modules/esbuild-wasm",
177192
"//:node_modules/fast-glob",
@@ -180,33 +195,18 @@ ts_project(
180195
"//:node_modules/jsonc-parser",
181196
"//:node_modules/karma",
182197
"//:node_modules/karma-source-map-support",
183-
"//:node_modules/less",
184-
"//:node_modules/less-loader",
185-
"//:node_modules/license-webpack-plugin",
186-
"//:node_modules/loader-utils",
187-
"//:node_modules/mini-css-extract-plugin",
188198
"//:node_modules/ng-packagr",
189199
"//:node_modules/open",
190200
"//:node_modules/ora",
191201
"//:node_modules/piscina",
192202
"//:node_modules/postcss",
193-
"//:node_modules/postcss-loader",
194-
"//:node_modules/resolve-url-loader",
195203
"//:node_modules/rxjs",
196204
"//:node_modules/sass",
197-
"//:node_modules/sass-loader",
198205
"//:node_modules/semver",
199-
"//:node_modules/source-map-loader",
200206
"//:node_modules/source-map-support",
201-
"//:node_modules/terser",
202207
"//:node_modules/tree-kill",
203208
"//:node_modules/tslib",
204209
"//:node_modules/typescript",
205-
"//:node_modules/webpack",
206-
"//:node_modules/webpack-dev-middleware",
207-
"//:node_modules/webpack-dev-server",
208-
"//:node_modules/webpack-merge",
209-
"//:node_modules/webpack-subresource-integrity",
210210
],
211211
)
212212

@@ -227,11 +227,11 @@ ts_project(
227227
deps = [
228228
":build_angular",
229229
":build_angular_test_utils",
230+
":node_modules/webpack",
230231
"//:node_modules/@types/node",
231232
"//:node_modules/fast-glob",
232233
"//:node_modules/prettier",
233234
"//:node_modules/typescript",
234-
"//:node_modules/webpack",
235235
"//packages/angular_devkit/architect/testing",
236236
"//packages/angular_devkit/core",
237237
],
@@ -335,7 +335,7 @@ LARGE_SPECS = {
335335
"//:node_modules/karma-jasmine",
336336
"//:node_modules/karma-jasmine-html-reporter",
337337
"//:node_modules/puppeteer",
338-
"//:node_modules/webpack",
338+
":node_modules/webpack",
339339
],
340340
},
341341
"protractor": {

‎packages/angular_devkit/build_webpack/BUILD.bazel‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,10 @@ ts_project(
4848
module_name = "@angular-devkit/build-webpack",
4949
deps = [
5050
":node_modules/@angular-devkit/architect",
51+
":node_modules/webpack",
52+
":node_modules/webpack-dev-server",
5153
"//:node_modules/@types/node",
5254
"//:node_modules/rxjs",
53-
"//:node_modules/webpack",
54-
"//:node_modules/webpack-dev-server",
5555
],
5656
)
5757

‎packages/angular_devkit/build_webpack/package.json‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@
2222
"devDependencies": {
2323
"@angular-devkit/core": "workspace:0.0.0-PLACEHOLDER",
2424
"@ngtools/webpack": "workspace:0.0.0-PLACEHOLDER",
25-
"webpack": "5.98.0"
25+
"webpack": "5.98.0",
26+
"webpack-dev-server": "5.2.0"
2627
},
2728
"peerDependencies": {
2829
"webpack": "^5.30.0",

‎packages/ngtools/webpack/BUILD.bazel‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,10 @@ ts_project(
3131
],
3232
module_name = "@ngtools/webpack",
3333
deps = [
34+
":node_modules/webpack",
3435
"//:node_modules/@angular/compiler-cli",
3536
"//:node_modules/@types/node",
3637
"//:node_modules/typescript",
37-
"//:node_modules/webpack",
3838
],
3939
)
4040

@@ -61,6 +61,7 @@ jasmine_test(
6161
data = [
6262
":webpack_test_lib",
6363
# Needed at runtime for runtime TS compilations performed by tests.
64+
":node_modules/webpack",
6465
"//:node_modules/tslib",
6566
],
6667
)

0 commit comments

Comments
 (0)