Reset known and unique broken translation files sequentially (#22709)

* fix: de-duplicate and filter non-existent broken files

* fix: sequentially reset known broken translation files

When a known broken translation is reset, a Git operation performs the
underlying reset. If more than one Git operation is running at a time,
Git will throw an error:

> Another git process seems to be running in this repository [...]

This change ensures that we reset each broken translation file
sequentially to not run into this error.

* Skip reset of a file that does not exist

Co-authored-by: Marcelo Jacobus <marcelo.jacobus@gmail.com>

Author: francisfuzz

script/i18n/reset-known-broken-translation-files.js

@@ -9,12 +9,12 @@
 
 import dotenv from 'dotenv'
 import Github from '../helpers/github.js'
-import { promisify } from 'util'
-import ChildProcess from 'child_process'
+import { execSync } from 'child_process'
+import uniq from 'lodash/uniq.js'
+import { existsSync } from 'fs'
 
 dotenv.config()
 const github = Github()
-const exec = promisify(ChildProcess.exec)
 
 // Check for required PAT
 if (!process.env.GITHUB_TOKEN) {
@@ -37,26 +37,28 @@ async function main() {
   // Get the list of broken files from the body text.
   const brokenFiles = body.replace(/^[\s\S]*?## List of Broken Translations/m, '').trim()
 
-  // Turn it into a simple array of files.
-  const brokenFilesArray = brokenFiles
-    .split('\n')
-    .filter((line) => !line.toLowerCase().startsWith('- [x]'))
-    .map((line) => line.replace('- [ ] ', '').trim())
-
-  // Run the script to revert them.
-  await Promise.all(
-    brokenFilesArray.map(async (file) => {
-      console.log(`resetting ${file}`)
-      await exec(`script/i18n/reset-translated-file.js --prefer-main ${file}`)
-    })
+  // De-duplicate the list of broken files and filter out any that don't exist in the repo.
+  const brokenFilesArray = uniq(
+    brokenFiles
+      .split('\n')
+      .filter((line) => !line.toLowerCase().startsWith('- [x]'))
+      .map((line) => line.replace('- [ ] ', '').trim())
+      .filter((line) => existsSync(line))
   )
 
+  // Revert each of the broken files.
+  // This is done sequentially to ensure only one Git operation is running at any given time.
+  brokenFilesArray.forEach((file) => {
+    console.log(`Resetting ${file}`)
+    execSync(`node script/i18n/reset-translated-file.js ${file}`)
+  })
+
   // Print a message with next steps.
   console.log(`
 Success!
-  
-Verify changes with git status and then run: 
-  
+
+Verify changes with git status and then run:
+
 git commit --no-verify -m "Reset broken translated files to English"
 `)
 }

script/i18n/reset-translated-file.js

@@ -37,11 +37,18 @@ const resetToEnglishSource = (translationFilePath) => {
     translationFilePath.startsWith('translations/'),
     'path argument must be in the format `translations/<lang>/path/to/file`'
   )
-  assert(fs.existsSync(translationFilePath), `file does not exist: ${translationFilePath}`)
+
+  if (!fs.existsSync(translationFilePath)) {
+    return
+  }
 
   const relativePath = translationFilePath.split(path.sep).slice(2).join(path.sep)
   const englishFile = path.join(process.cwd(), relativePath)
-  assert(fs.existsSync(englishFile), `file does not exist: ${englishFile}`)
+
+  if (!fs.existsSync(englishFile)) {
+    fs.unlinkSync(translationFilePath)
+    return
+  }
 
   // replace file with English source
   const englishContent = fs.readFileSync(englishFile, 'utf8')