fixup! fix(@angular/build): validate library outputPath and avoid duplicate secondary README assets

Author: alan-agius4

packages/angular/build/src/builders/library/options.ts

@@ -105,35 +105,6 @@ export async function normalizeLibraryOptions(
     { glob: 'README.md', input: projectRoot, output: '.' },
   ];
 
-  const primaryEntryPoint = entryPoints.get('.');
-  const primaryEntryDir = primaryEntryPoint
-    ? path.posix.dirname(primaryEntryPoint.entryFilePath)
-    : undefined;
-  const seenReadmeAssets = new Set<string>();
-
-  for (const entryPoint of entryPoints.values()) {
-    if (entryPoint.isPrimary) {
-      continue;
-    }
-
-    const entryDir = path.posix.dirname(entryPoint.entryFilePath);
-    if (entryDir === projectRoot || entryDir === primaryEntryDir) {
-      continue;
-    }
-
-    const readmeKey = `${entryDir}:${entryPoint.name}`;
-    if (seenReadmeAssets.has(readmeKey)) {
-      continue;
-    }
-    seenReadmeAssets.add(readmeKey);
-
-    defaultAssets.push({
-      glob: 'README.md',
-      input: entryDir,
-      output: entryPoint.name,
-    });
-  }
-
   const assets = normalizeAssetPatterns(
     [...defaultAssets, ...(rawAssets ?? [])],
     workspaceRoot,

packages/angular/build/src/builders/library/tests/options/assets_spec.ts

@@ -101,7 +101,7 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
       expect(harness.hasFile('dist/lib/testing/README.md')).toBeFalse();
     });
 
-    it('should copy secondary entry point README.md when located in its own directory', async () => {
+    it('should only copy the root README.md by default and not secondary entry point README.md files', async () => {
       await harness.writeFiles({
         'projects/lib/README.md': '# Root Library README',
         'projects/lib/testing/README.md': '# Testing Entry Point README',
@@ -122,7 +122,7 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
       expect(result?.success).toBeTrue();
 
       expect(harness.readFile('dist/lib/README.md')).toBe('# Root Library README');
-      expect(harness.readFile('dist/lib/testing/README.md')).toBe('# Testing Entry Point README');
+      expect(harness.hasFile('dist/lib/testing/README.md')).toBeFalse();
     });
   });
 });