fix(scripts-prettier): escape file names that are being passed to pre…

…ttier bin via shell (#31889)
microsoft · Jul 2, 2024 · 88db367 · 88db367
1 parent 1e591f2
commit 88db367
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/scripts/prettier/src/prettier-helpers.js b/scripts/prettier/src/prettier-helpers.js
@@ -47,10 +47,18 @@ function runPrettier(files, config = {}) {
 
   const prettierSupportedFiles = fileIsGlob
     ? files
-    : files.filter(file => {
+    : files.reduce((acc, file) => {
         const ext = path.extname(file).replace('.', '');
-        return prettierSupportedFileExtensions.includes(ext);
-      });
+
+        if (prettierSupportedFileExtensions.includes(ext)) {
+          // WHY IGNORE IS NEEDED?: prettier removes one of the '\' within replaceValue
+          // prettier-ignore
+          const escapedFileName = `"${file.replace(/\$/g, '\\\$')}"`;
+
+          acc.push(escapedFileName);
+        }
+        return acc;
+      }, /** @type {string[]} */ ([]));
 
   if (!prettierSupportedFiles.length) {
     console.log('prettier: No supported files found');