refactor(@angular/build): simplify Set operations with ES2025 Set methods (#34505)

PR Close #34505

Author: alan-agius4

packages/angular/build/src/builders/library/builder-state.ts

@@ -256,10 +256,8 @@ export class BuilderState {
       return true;
     }
 
-    for (const file of this.#pendingCompilationFiles) {
-      if (this.#watchedCompilationFiles.has(file)) {
-        return true;
-      }
+    if (!this.#pendingCompilationFiles.isDisjointFrom(this.#watchedCompilationFiles)) {
+      return true;
     }
 
     this.#pendingCompilationFiles.clear();
@@ -281,12 +279,7 @@ export class BuilderState {
       return undefined;
     }
 
-    const files = new Set(modifiedFiles);
-    for (const file of this.#pendingCompilationFiles) {
-      files.add(file);
-    }
-
-    return files;
+    return modifiedFiles.union(this.#pendingCompilationFiles);
   }
 
   /**
@@ -380,7 +373,7 @@ export class BuilderState {
     collectedAssets: CollectedAssets,
     manifestFiles?: readonly OutputFile[],
   ): ReadonlySet<string> {
-    const filesToDelete = new Set<string>();
+    let filesToDelete = new Set<string>();
 
     if (this.#hasUpdatedBundles) {
       this.#hasUpdatedBundles = false;
@@ -394,21 +387,14 @@ export class BuilderState {
         }
       }
 
-      for (const prevFile of this.#emittedBundleFiles) {
-        if (!nextBundleFiles.has(prevFile)) {
-          filesToDelete.add(prevFile);
-        }
-      }
-
+      filesToDelete = this.#emittedBundleFiles.difference(nextBundleFiles);
       this.#emittedBundleFiles = nextBundleFiles;
     }
 
     if (manifestFiles) {
       const nextManifestFiles = new Set(manifestFiles.map((file) => file.path));
-      for (const prevFile of this.#emittedManifestFiles) {
-        if (!nextManifestFiles.has(prevFile)) {
-          filesToDelete.add(prevFile);
-        }
+      for (const prevFile of this.#emittedManifestFiles.difference(nextManifestFiles)) {
+        filesToDelete.add(prevFile);
       }
 
       this.#emittedManifestFiles = nextManifestFiles;

packages/angular/build/src/builders/library/pipeline/entry-points.ts

@@ -308,10 +308,10 @@ export function collectEntryPointsToBundle(
     const previousBundleResult = previousBundleResults.get(entryPoint.name);
     const hasEsmChanges =
       !previousBundleResult ||
-      hasEntryPointChanges(previousBundleResult.esmModuleIds, pendingChangedEsmFiles);
+      !previousBundleResult.esmModuleIds.isDisjointFrom(pendingChangedEsmFiles);
     const hasDtsChanges =
       !previousBundleResult ||
-      hasEntryPointChanges(previousBundleResult.dtsModuleIds, pendingChangedDtsFiles);
+      !previousBundleResult.dtsModuleIds.isDisjointFrom(pendingChangedDtsFiles);
 
     if (hasEsmChanges || hasDtsChanges) {
       executionResult.addLog(`Compiling ${entryPoint.displayName}...`);
@@ -326,27 +326,3 @@ export function collectEntryPointsToBundle(
 
   return itemsToBundle;
 }
-
-/**
- * Checks whether any file in `changedFiles` belongs to the given entry point's `moduleIds`.
- *
- * @param moduleIds Set of virtual POSIX module IDs referenced by the entry point's previous bundle.
- * @param changedFiles Set of POSIX file paths that changed during compilation.
- * @returns `true` if at least one changed file is part of `moduleIds`; otherwise `false`.
- */
-function hasEntryPointChanges(
-  moduleIds: ReadonlySet<string>,
-  changedFiles: ReadonlySet<string>,
-): boolean {
-  if (changedFiles.size === 0) {
-    return false;
-  }
-
-  for (const file of changedFiles) {
-    if (moduleIds.has(file)) {
-      return true;
-    }
-  }
-
-  return false;
-}

packages/angular/build/src/tools/sass/rebasing-importer.ts

@@ -209,8 +209,8 @@ export class RelativeUrlRebasingImporter extends UrlRebasingImporter {
       // If there is a preprocessed cache of the directory, perform an intersection of the potentials
       // and the directory files.
       const { files, directories } = cachedEntries;
-      foundDefaults = [...defaultPotentials].filter((potential) => files.has(potential));
-      foundImports = [...importPotentials].filter((potential) => files.has(potential));
+      foundDefaults = [...defaultPotentials.intersection(files)];
+      foundImports = [...importPotentials.intersection(files)];
       hasPotentialIndex = checkDirectory && !hasStyleExtension && directories.has(filename);
     } else {
       // If no preprocessed cache exists, get the entries from the file system and, while searching,