From 4b14887d846684ba4202d950ef3276126d44bb24 Mon Sep 17 00:00:00 2001 From: Charles Lyding <19598772+clydin@users.noreply.github.com> Date: Wed, 29 Oct 2025 12:37:33 -0400 Subject: [PATCH] refactor(@angular/build): remove Angular packages from externals for browser tests To facilitate the future implementation of more comprehensive Vite-based prebundling for browser-based unit tests, this change removes Angular packages from the list of external dependencies when a browser is configured. --- .../src/builders/unit-test/runners/vitest/build-options.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/packages/angular/build/src/builders/unit-test/runners/vitest/build-options.ts b/packages/angular/build/src/builders/unit-test/runners/vitest/build-options.ts index 1faffcc3fe7c..5df9aafe0d57 100644 --- a/packages/angular/build/src/builders/unit-test/runners/vitest/build-options.ts +++ b/packages/angular/build/src/builders/unit-test/runners/vitest/build-options.ts @@ -98,7 +98,12 @@ export async function getVitestBuildOptions( }); entryPoints.set('init-testbed', 'angular:test-bed-init'); - const externalDependencies = new Set(['vitest', ...ANGULAR_PACKAGES_TO_EXTERNALIZE]); + const externalDependencies = new Set(['vitest']); + if (!options.browsers?.length) { + // Only add for non-browser setups. + // Comprehensive browser prebundling will be handled separately. + ANGULAR_PACKAGES_TO_EXTERNALIZE.forEach((dep) => externalDependencies.add(dep)); + } if (baseBuildOptions.externalDependencies) { baseBuildOptions.externalDependencies.forEach((dep) => externalDependencies.add(dep)); }