From 7d3fb75bdae663293c6764f12bc4f4c664f3d062 Mon Sep 17 00:00:00 2001
From: Aaron Shim <aaronshim@google.com>
Date: Fri, 18 Oct 2024 10:25:20 +0000
Subject: [PATCH] More descriptive comments and readability changes.

---
 packages/angular/build/src/utils/index-file/auto-csp.ts | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/packages/angular/build/src/utils/index-file/auto-csp.ts b/packages/angular/build/src/utils/index-file/auto-csp.ts
index 5437a6c7a2e6..ef7ec3ea527b 100644
--- a/packages/angular/build/src/utils/index-file/auto-csp.ts
+++ b/packages/angular/build/src/utils/index-file/auto-csp.ts
@@ -142,7 +142,10 @@ export async function autoCsp(html: string): Promise<string> {
       }
     }
     // We are encountering the first start tag that's not <script src="..."> after a string of
-    // consecutive <script src="...">. The first place when we can determine this to be the case is
+    // consecutive <script src="...">. <script> tags without a src attribute will also end a chain
+    // of src attributes that can be loaded in a single loader script, so those will end up here.
+    //
+    // The first place when we can determine this to be the case is
     // during the first opening tag that's not <script src="...">, where we need to insert the
     // dynamic loader script before continuing on with writing the rest of the tags.
     // (One edge case is where there are no more opening tags after the last <script src="..."> is
@@ -280,7 +283,7 @@ function createLoaderScript(srcList: SrcScriptTag[], enableTrustedTypes = false)
   const srcListFormatted = srcList
     .map(
       (s) =>
-        `['${encodeURI(s.src).replaceAll("'", "\\'")}', ${s.type ? "'" + encodeURI(s.type) + "'" : undefined}, ${s.async ? 'true' : 'false'}, ${s.defer ? 'true' : 'false'}]`,
+        `['${encodeURI(s.src).replaceAll("'", "\\'")}', ${s.type ? "'" + s.type + "'" : undefined}, ${s.async ? 'true' : 'false'}, ${s.defer ? 'true' : 'false'}]`,
     )
     .join();
   return enableTrustedTypes