diff --git a/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/browser.ts b/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/browser.ts index 195cc9be93b2..3aeb9438ca24 100644 --- a/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/browser.ts +++ b/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/browser.ts @@ -8,7 +8,7 @@ import * as webpack from 'webpack'; import { CommonJsUsageWarnPlugin } from '../../plugins/webpack'; import { WebpackConfigOptions } from '../build-options'; -import { getSourceMapDevTool, isPolyfillsEntry, normalizeExtraEntryPoints } from './utils'; +import { getSourceMapDevTool } from './utils'; const SubresourceIntegrityPlugin = require('webpack-subresource-integrity'); const LicenseWebpackPlugin = require('license-webpack-plugin').LicenseWebpackPlugin; @@ -22,7 +22,6 @@ export function getBrowserConfig(wco: WebpackConfigOptions): webpack.Configurati extractLicenses, vendorChunk, commonChunk, - styles, allowedCommonJsDependencies, } = buildOptions; @@ -59,9 +58,6 @@ export function getBrowserConfig(wco: WebpackConfigOptions): webpack.Configurati )); } - const globalStylesBundleNames = normalizeExtraEntryPoints(styles, 'styles') - .map(style => style.bundleName); - let crossOriginLoading: 'anonymous' | 'use-credentials' | false = false; if (subresourceIntegrity && crossOrigin === 'none') { crossOriginLoading = 'anonymous'; @@ -95,17 +91,11 @@ export function getBrowserConfig(wco: WebpackConfigOptions): webpack.Configurati priority: 5, }, vendors: false, - vendor: !!vendorChunk && { + defaultVendors: !!vendorChunk && { name: 'vendor', - chunks: 'initial', + chunks: (chunk) => chunk.name === 'main', enforce: true, - test: (module: { nameForCondition?: Function }, chunks: Array<{ name: string }>) => { - const moduleName = module.nameForCondition ? module.nameForCondition() : ''; - - return /[\\/]node_modules[\\/]/.test(moduleName) - && !chunks.some(({ name }) => isPolyfillsEntry(name) - || globalStylesBundleNames.includes(name)); - }, + test: /[\\/]node_modules[\\/]/, }, }, }, diff --git a/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/test.ts b/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/test.ts index 166b5d5cf2fc..b85d12b20f23 100644 --- a/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/test.ts +++ b/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/test.ts @@ -74,18 +74,14 @@ export function getTestConfig( plugins: extraPlugins, optimization: { splitChunks: { - chunks: ((chunk: { name: string }) => !isPolyfillsEntry(chunk.name)), + chunks: (chunk) => !isPolyfillsEntry(chunk.name), cacheGroups: { vendors: false, - vendor: { + defaultVendors: { name: 'vendor', - chunks: 'initial', - test: (module: { nameForCondition?: () => string }, chunks: { name: string }[]) => { - const moduleName = module.nameForCondition ? module.nameForCondition() : ''; - - return /[\\/]node_modules[\\/]/.test(moduleName) - && !chunks.some(({ name }) => isPolyfillsEntry(name)); - }, + chunks: (chunk) => chunk.name === 'main', + enforce: true, + test: /[\\/]node_modules[\\/]/, }, }, },