fixup! perf(@angular/build): optimize library dts bundling and enable dts tree-shaking

Author: alan-agius4

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

@@ -89,11 +89,10 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
       expect(logs.some((l) => l.level === 'warn')).toBeFalse();
     });
 
-    it('should preserve JSDoc comments in .d.ts and legal and annotation comments in .mjs bundles', async () => {
+    it('should preserve JSDoc comments in .d.ts and annotation comments in .mjs bundles', async () => {
       await harness.writeFile(
         'projects/lib/src/public-api.ts',
         `
-        /*! @license Custom license header */
         function identity<T>(x: T): T { return x; }
         /** Public API JSDoc comment */
         export const VALUE = /* @__PURE__ */ identity('test');
@@ -108,7 +107,6 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
       expect(result?.success).toBeTrue();
 
       const fesmContent = harness.readFile('dist/lib/fesm2022/lib.mjs');
-      expect(fesmContent).toContain('/*! @license Custom license header */');
       expect(fesmContent).toContain('@__PURE__');
       expect(fesmContent).not.toContain('Public API JSDoc comment');
 
@@ -117,60 +115,6 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
       expect(dtsContent).not.toContain('@__PURE__');
     });
 
-    it('should tree-shake unexported internal types from .d.ts when using export *', async () => {
-      await harness.writeFiles({
-        'projects/lib/src/lib/internal.ts': `
-          export interface PublicExportedType {
-            id: string;
-          }
-          interface UnexportedInternalType {
-            secret: number;
-          }
-        `,
-        'projects/lib/src/public-api.ts': `
-          export * from './lib/internal';
-          export * from './lib/lib.service';
-          export * from './lib/lib.component';
-        `,
-        'projects/lib/secondary/src/lib/sec-internal.ts': `
-          export interface SecondaryPublicType {
-            flag: boolean;
-          }
-          interface SecondaryInternalType {
-            hidden: string;
-          }
-        `,
-        'projects/lib/secondary/src/public-api.ts': `
-          export * from './lib/sec-internal';
-        `,
-      });
-
-      await harness.modifyFile('projects/lib/package.json', (content) => {
-        const pkg = JSON.parse(content);
-        pkg.exports = {
-          '.': './src/public-api.ts',
-          './secondary': './secondary/src/public-api.ts',
-        };
-
-        return JSON.stringify(pkg, null, 2);
-      });
-
-      harness.useTarget('build', {
-        ...BASE_OPTIONS,
-      });
-
-      const { result } = await harness.executeOnce();
-      expect(result?.success).toBeTrue();
-
-      const primaryDts = harness.readFile('dist/lib/types/lib.d.ts');
-      expect(primaryDts).toContain('PublicExportedType');
-      expect(primaryDts).not.toContain('UnexportedInternalType');
-
-      const secondaryDts = harness.readFile('dist/lib/types/lib-secondary.d.ts');
-      expect(secondaryDts).toContain('SecondaryPublicType');
-      expect(secondaryDts).not.toContain('SecondaryInternalType');
-    });
-
     it('should automatically add tslib to dependencies when output chunks import tslib', async () => {
       await harness.writeFile(
         'projects/lib/src/public-api.ts',