From 4d515bbbd6f30d85063e8eb30ba4322dfb698cc8 Mon Sep 17 00:00:00 2001 From: Alan Agius Date: Thu, 25 Jan 2024 08:52:17 +0000 Subject: [PATCH] fix(@angular-devkit/build-angular): add `required` modules as externals imports Prior to this change any module which was used using `require` was not listed as an external. Closes #26833 --- .../build_angular/src/tools/esbuild/bundler-context.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/angular_devkit/build_angular/src/tools/esbuild/bundler-context.ts b/packages/angular_devkit/build_angular/src/tools/esbuild/bundler-context.ts index 454778cebae5..64103284e7d5 100644 --- a/packages/angular_devkit/build_angular/src/tools/esbuild/bundler-context.ts +++ b/packages/angular_devkit/build_angular/src/tools/esbuild/bundler-context.ts @@ -331,7 +331,9 @@ export class BundlerContext { for (const importData of imports) { if ( !importData.external || - (importData.kind !== 'import-statement' && importData.kind !== 'dynamic-import') + (importData.kind !== 'import-statement' && + importData.kind !== 'dynamic-import' && + importData.kind !== 'require-call') ) { continue; }