fix(@angular-devkit/build-angular): ensure directories are properly ignored in esbuild builder

When using the esbuild-based browser application builder in watch mode, the underlying file
watcher based on chokidar would previously not fully ignore the output path if the path contained
a trailing slash. To workaround this, directory paths based on supplied options are now
normalized to remove any trailing slashes.

Author: clydin

packages/angular_devkit/build_angular/src/builders/browser-esbuild/options.ts

@@ -36,17 +36,18 @@ export async function normalizeOptions(
 ) {
   const workspaceRoot = context.workspaceRoot;
   const projectMetadata = await context.getProjectMetadata(projectName);
-  const projectRoot = path.join(workspaceRoot, (projectMetadata.root as string | undefined) ?? '');
-  const projectSourceRoot = path.join(
-    workspaceRoot,
-    (projectMetadata.sourceRoot as string | undefined) ?? 'src',
+  const projectRoot = normalizeDirectoryPath(
+    path.join(workspaceRoot, (projectMetadata.root as string | undefined) ?? ''),
+  );
+  const projectSourceRoot = normalizeDirectoryPath(
+    path.join(workspaceRoot, (projectMetadata.sourceRoot as string | undefined) ?? 'src'),
   );
 
   const cacheOptions = normalizeCacheOptions(projectMetadata, workspaceRoot);
 
   const mainEntryPoint = path.join(workspaceRoot, options.main);
   const tsconfig = path.join(workspaceRoot, options.tsConfig);
-  const outputPath = path.join(workspaceRoot, options.outputPath);
+  const outputPath = normalizeDirectoryPath(path.join(workspaceRoot, options.outputPath));
   const optimizationOptions = normalizeOptimization(options.optimization);
   const sourcemapOptions = normalizeSourceMaps(options.sourceMap ?? false);
   const assets = options.assets?.length
@@ -222,3 +223,18 @@ function findTailwindConfigurationFile(
 
   return undefined;
 }
+
+/**
+ * Normalize a directory path string.
+ * Currently only removes a trailing slash if present.
+ * @param path A path string.
+ * @returns A normalized path string.
+ */
+function normalizeDirectoryPath(path: string): string {
+  const last = path[path.length - 1];
+  if (last === '/' || last === '\\') {
+    return path.slice(0, -1);
+  }
+
+  return path;
+}