feat: move package types under /types

All package type have been moved into `/types`

Author: alan-agius4

src/ng_package/angular_package_format.bzl

@@ -251,16 +251,14 @@ def _angular_package_format_impl(ctx):
         ] if _ != ""])
 
         bundle_name_base = primary_bundle_name if is_primary_entry_point else entry_point
-        dts_bundle_name_base = "index" if is_primary_entry_point else "%s/index" % entry_point
 
         # Store the collected entry point in a list of all entry-points. This
         # can be later passed to the packager as a manifest.
         collected_entry_points.append(struct(
             module_name = module_name,
             es2022_entry_point = es2022_entry_point,
             fesm2022_file = "fesm2022/%s.mjs" % bundle_name_base,
-            # TODO(devversion): Put all types under `/types/` folder. Breaking change in v20.
-            dts_bundle_relative_path = "%s.d.ts" % dts_bundle_name_base,
+            dts_bundle_relative_path = "types/%s.d.ts" % bundle_name_base,
             typings_entry_point = typings_entry_point,
             # TODO: Determine if we can just remove this as we are always "guessing" now
             guessed_paths = True,

src/ng_package/packager/index.ts

@@ -150,7 +150,7 @@ function main(args: string[]): void {
     return getEntryPointSubpath(moduleName) !== '';
   }
 
-  const crossEntryPointFailures = esm2022.flatMap((file) =>
+  const crossEntryPointFailures = esm2022.flatMap(file =>
     analyzeFileAndEnsureNoCrossImports(file, metadata),
   );
 
@@ -161,16 +161,13 @@ function main(args: string[]): void {
 
   // Copy all FESM files (and their potential shared chunks) into the package output.
   const fesmFiles = globSync('**/*', {cwd: metadata.fesmBundlesOut.path});
-  fesmFiles.forEach((f) =>
+  fesmFiles.forEach(f =>
     copyFile(path.join(metadata.fesmBundlesOut.path, f), path.join('fesm2022', f)),
   );
 
   // Copy all dts files (and their potential shared chunks) into the package output.
   const dtsFiles = globSync('**/*', {cwd: metadata.dtsBundlesOut.path});
-  dtsFiles.forEach((f) =>
-    // TODO(devversion): Put all types under `/types/` folder. Breaking change in v20.
-    copyFile(path.join(metadata.dtsBundlesOut.path, f), f),
-  );
+  dtsFiles.forEach(f => copyFile(path.join(metadata.dtsBundlesOut.path, f), path.join('types', f)));
 
   for (const file of staticFiles) {
     // We copy all files into the package output while preserving the sub-path from
@@ -326,10 +323,10 @@ function main(args: string[]): void {
 
     const sideEffects = packageJson.sideEffects as undefined | false | string[];
     const neededSideEffects = sideEffectEntryPoints.map(
-      (entryPointModule) => `./${metadata.entryPoints[entryPointModule].fesm2022RelativePath}`,
+      entryPointModule => `./${metadata.entryPoints[entryPointModule].fesm2022RelativePath}`,
     );
     const missingSideEffects = neededSideEffects.filter(
-      (p) =>
+      p =>
         // It's missing, if the whole package is marked as having no side effects.
         sideEffects === false ||
         // Alternatively, it's missing if the explicit list doesn't contain the pattern.
@@ -348,9 +345,7 @@ function main(args: string[]): void {
     // of the `ng_package` known entry points.
     const unexpectedExtra =
       sideEffects !== false
-        ? (sideEffects ?? []).filter(
-            (p) => p.includes('fesm2022') && !neededSideEffects.includes(p),
-          )
+        ? (sideEffects ?? []).filter(p => p.includes('fesm2022') && !neededSideEffects.includes(p))
         : [];
     if (unexpectedExtra.length > 0) {
       throw Error(

src/ng_package/rollup/rollup.config.js

@@ -165,7 +165,7 @@ const input = {};
 for (const info of Object.values(entrypointMetadata)) {
   const entryFile = dtsMode ? info.typingsEntryPoint.path : info.index.path;
   const chunkName = dtsMode
-    ? info.dtsBundleRelativePath.replace(/\.d\.ts$/, '')
+    ? info.dtsBundleRelativePath.replace(/\.d\.ts$/, '').replace('types/', '')
     : info.fesm2022RelativePath.replace(/\.mjs$/, '').replace('fesm2022/', '');
 
   input[chunkName] = entryFile;