perf(@angular/build): overlap library asset resolution with bundling and disk emission

packages/angular/build/src/builders/library/pipeline/build-action.ts

@@ -71,6 +71,7 @@ export async function buildAction(actionContext: BuildActionContext): Promise<Ex
       return result;
     }
 
+    let collectedAssets = actionContext.collectedAssets;
     const filesToEmit: OutputFile[] = [];
 
     if (shouldCompileEntryPoints) {
@@ -102,13 +103,14 @@ export async function buildAction(actionContext: BuildActionContext): Promise<Ex
       const itemsToBundle = collectEntryPointsToBundle(options, buildState);
       let bundleOutput: BundleEntryPointsOutput;
       try {
-        [bundleOutput] = await Promise.all([
+        [bundleOutput, , collectedAssets] = await Promise.all([
           bundleEntryPoints(itemsToBundle, esmFiles, dtsFiles, {
             ...options,
             declarationMap,
             compilationMode,
           }),
           diagnosePromise,
+          collectedAssets ?? buildState.collectAssets(options, modifiedFiles),
         ]);
       } catch (error) {
         assertIsError(error);
@@ -121,10 +123,10 @@ export async function buildAction(actionContext: BuildActionContext): Promise<Ex
         return result;
       }
 
-      result.addWarnings(bundleOutput.warnings);
       if (result.hasErrors) {
         return result;
       }
+      result.addWarnings(bundleOutput.warnings);
 
       filesToEmit.push(...bundleOutput.filesToEmit);
       buildState.recordBundleOutput(bundleOutput.bundleResults);
@@ -145,19 +147,17 @@ export async function buildAction(actionContext: BuildActionContext): Promise<Ex
       filesToEmit.push(...manifestFiles);
     }
 
-    const collectedAssets =
-      actionContext.collectedAssets ?? (await buildState.collectAssets(options, modifiedFiles));
+    collectedAssets ??= await buildState.collectAssets(options, modifiedFiles);
     filesToEmit.push(...collectedAssets.filesToEmit);
 
-    await emitFilesToDisk(filesToEmit, (file) => path.posix.join(options.outputPath, file.path));
-
     const filesToDelete = buildState.recordEmittedOutputs(collectedAssets, manifestFiles);
-
     if (filesToDelete.size) {
       await deleteFilesFromDisk(
         Array.from(filesToDelete, (file) => path.posix.join(options.outputPath, file)),
       );
     }
+
+    await emitFilesToDisk(filesToEmit, (file) => path.posix.join(options.outputPath, file.path));
   } catch (error) {
     assertIsError(error);
     result.addError(error.message);
@@ -178,7 +178,7 @@ function verifyAllowedDependencies(
   options: NormalizedLibraryOptions,
   result: ExecutionResult,
 ): boolean {
-  const { packageJson, allowedNonPeerDependencies } = options;
+  const { packageJson, packageJsonPath, allowedNonPeerDependencies } = options;
   const dependencies = {
     ...(packageJson.dependencies ?? {}),
     ...(packageJson.optionalDependencies ?? {}),
@@ -189,7 +189,7 @@ function verifyAllowedDependencies(
     if (!allowedNonPeerDependencies.some((regex) => regex.test(dep))) {
       result.addError(
         `Dependency '${dep}' must be explicitly allowed using the 'allowedNonPeerDependencies' option, ` +
-          `or moved to 'peerDependencies' in 'package.json'.`,
+          `or moved to 'peerDependencies' in '${packageJsonPath}'.`,
       );
       isValid = false;
     }

packages/angular/build/src/builders/library/tests/behavior/build_spec.ts

@@ -86,7 +86,7 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
 
       const { result, logs } = await harness.executeOnce();
       expect(result?.success).toBeTrue();
-      expect(logs.some((l) => l.level === 'warn')).toBeFalse();
+      expect(logs).not.toContain(jasmine.objectContaining({ level: 'warn' }));
     });
 
     it('should preserve JSDoc comments in .d.ts and annotation comments in .mjs bundles', async () => {
@@ -204,10 +204,13 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
       );
     });
 
-    it('should log failure on build error', async () => {
+    it('should log failure on build error without secondary bundler warnings', async () => {
       await harness.writeFile(
         'projects/lib/src/public-api.ts',
-        `export const title: number = 'invalid type';`,
+        `
+        export const title: number = 'invalid type';
+        export function runDynamic(code: string) { return eval(code); }
+        `,
       );
 
       harness.useTarget('build', {
@@ -230,6 +233,7 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
           ),
         }),
       );
+      expect(logs).not.toContain(jasmine.objectContaining({ level: 'warn' }));
       expect(logs).not.toContain(
         jasmine.objectContaining({
           message: jasmine.stringMatching(/Output location:/),

packages/angular/build/src/builders/library/tests/options/allowed-non-peer-dependencies_spec.ts

@@ -29,7 +29,9 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
       expect(logs).toContain(
         jasmine.objectContaining({
           level: 'error',
-          message: jasmine.stringMatching(/lodash-es.*allowedNonPeerDependencies/),
+          message: jasmine.stringMatching(
+            /lodash-es.*allowedNonPeerDependencies.*projects\/lib\/package\.json/,
+          ),
         }),
       );
     });