@freckle/i18n-scripts 1.1.23 → 2.0.0

package/dist/check-i18n-variables.js

@@ -1,16 +1,131 @@
 #!/usr/bin/env node
+//
+// Checks all locize keys (in the codebase) with known variables against key
+// values in locize ensuring that variables in locize refer to known vars.
+//
+const i18next = require('i18next')
+const ICU = require('i18next-icu')
+const HttpBackend = require('i18next-http-backend')
+const {IntlMessageFormat} = require('intl-messageformat')
 
-const {spawn} = require('node:child_process')
-const path = require('path')
+const {
+  extractTranslationKeysAndVariables,
+  KNOWN_VARIABLES_TAG
+} = require('./translation-extraction.js')
+const {getFiles, filterJsFiles, supportedLngs} = require('./helpers.js')
 
-const [_nodeBin, scriptPath, ...args] = process.argv
+const [_nodeBin, _scriptPath, ...projects] = process.argv
 
-const proc = spawn(`${path.dirname(scriptPath)}/check-i18n-variables`, args)
+if (projects.length < 1) {
+  console.error('Usage: check-i18n-variables <path>')
+  process.exit(64)
+}
 
-proc.stdout.on('data', data => {
-  process.stdout.write(data)
-})
+// TODO: Pass in via ENV?
+const ns = ['school', 'student', 'console', 'classroom', 'common']
 
-proc.stderr.on('data', data => {
-  process.stderr.write(data)
-})
+const translationKeysWithKnownVariablesInProject = async project => {
+  const files = await getFiles(project)
+  const jsFiles = files.filter(filterJsFiles)
+  const results = []
+
+  jsFiles.forEach(({path}) => {
+    extractTranslationKeysAndVariables(path)
+      .filter(({tag}) => tag === KNOWN_VARIABLES_TAG)
+      .forEach(keyWithVars => results.push(keyWithVars))
+  })
+
+  return results
+}
+
+// The AST is undocumented, however, its type definitions are here
+//    https://github.com/formatjs/formatjs/blob/82a2e5d07f9a999624ffb6387a58464f9a5f7399/packages/icu-messageformat-parser/types.ts#L122-L131
+const extractVariablesFromICUAst = ast => {
+  // Some AST field values are `null`
+  if (!ast) {
+    return []
+  }
+
+  if (ast.constructor === Array) {
+    return ast.flatMap(extractVariablesFromICUAst)
+  }
+  if (ast.constructor === Object) {
+    const subVariables = Object.values(ast).flatMap(extractVariablesFromICUAst)
+    // `type`s that aren't 0 (string literals) are variables when `.value` exists
+    const hasVariable = ast.type && typeof ast.type === 'number' && ast.type !== 0 && ast.value
+
+    return (hasVariable ? [ast.value] : []).concat(subVariables)
+  }
+
+  return []
+}
+
+const main = async () => {
+  await i18next
+    .use(ICU)
+    .use(HttpBackend)
+    .init({
+      supportedLngs,
+      allowMultiLoading: true,
+      keySeparator: false,
+      ns,
+      backend: {
+        loadPath: 'https://translations.freckle.com/latest/{{lng}}/{{ns}}'
+      }
+    })
+
+  let anyErrors = false
+
+  // Project key extraction is not cheap so cache per project
+  const keysPerProject = {}
+
+  await Promise.all(
+    projects.map(
+      async project =>
+        (keysPerProject[project] = await translationKeysWithKnownVariablesInProject(project))
+    )
+  )
+
+  for (const lang of supportedLngs) {
+    await i18next.changeLanguage(lang)
+    const translations = i18next.getDataByLanguage(lang)
+
+    for (const [project, keys] of Object.entries(keysPerProject)) {
+      for (const {i18nKey, variables: codeVariables} of keys) {
+        const keyValue =
+          (translations[project] && translations[project][i18nKey]) || translations.common[i18nKey]
+
+        try {
+          if (keyValue) {
+            const locizeVariables = extractVariablesFromICUAst(
+              new IntlMessageFormat(keyValue, lang, {}, {ignoreTag: true}).getAst()
+            )
+
+            for (const locizeVariable of locizeVariables) {
+              if (!codeVariables.includes(locizeVariable)) {
+                anyErrors = true
+                console.error(
+                  `${project}:${lang}:${i18nKey} does not pass the variable ${locizeVariable}`
+                )
+              }
+            }
+          }
+        } catch {
+          // We have another script that checks that `IntlMessageFormat` can
+          // successfully parse the ICU text so let's not fail for that reason.
+        }
+      }
+    }
+  }
+
+  if (anyErrors) {
+    console.error(
+      'The above keys have variables not passed by the code and need to be fixed in Locize.'
+    )
+    process.exit(1)
+  } else {
+    process.exit(0)
+  }
+}
+
+main()

package/dist/check-i18next-icu-parser.js

@@ -1,16 +1,62 @@
 #!/usr/bin/env node
+//
+// Checks all keys in Locize to make sure that they can be parsed by the `i18next` `t` function
+//
+const i18next = require('i18next')
+const ICU = require('i18next-icu')
+const HttpBackend = require('i18next-http-backend')
+const {supportedLngs} = require('./helpers.js')
 
-const {spawn} = require('node:child_process')
-const path = require('path')
+const VALID_NAMESPACES = ['school', 'student', 'console', 'classroom', 'common']
 
-const [_nodeBin, scriptPath, ...args] = process.argv
+const ns = process.argv.slice(2)
 
-const proc = spawn(`${path.dirname(scriptPath)}/check-i18next-icu-parser`, args)
+for (const arg of ns) {
+  if (!VALID_NAMESPACES.some(ns => ns === arg)) {
+    throw new Error(`Invalid arguments: ${arg} is not a namespace`)
+  }
+}
 
-proc.stdout.on('data', data => {
-  process.stdout.write(data)
-})
+const main = async () => {
+  const t = await i18next
+    .use(ICU)
+    .use(HttpBackend)
+    .init({
+      supportedLngs,
+      allowMultiLoading: true,
+      keySeparator: false,
+      ns,
+      backend: {
+        loadPath: 'https://translations.freckle.com/latest/{{lng}}/{{ns}}'
+      }
+    })
 
-proc.stderr.on('data', data => {
-  process.stderr.write(data)
-})
+  let anyErrors = false
+
+  for (const l of supportedLngs) {
+    await i18next.changeLanguage(l)
+    for (const n of ns) {
+      for (const key of Object.keys(i18next.getDataByLanguage(l)[n])) {
+        try {
+          t(`${n}:${key}`)
+        } catch (error) {
+          if (!isInterpolationError(error)) {
+            anyErrors = true
+            console.error(`${n}:${l}:${key}`)
+          }
+        }
+      }
+    }
+  }
+
+  if (anyErrors) {
+    console.error('The above keys cannot be parsed as ICU and need to be fixed in Locize.')
+    process.exit(1)
+  } else {
+    process.exit(0)
+  }
+}
+
+const isInterpolationError = error => /^The intl string context variable/.test(error.message)
+
+main()

package/dist/check-missing-translations

@@ -9,7 +9,7 @@ translation_keys_in_code() {
   git ls-files "$project_path" |
     grep -E '\.(js|ts|tsx)$' |
     grep -v '\.min\.js$' |
-    "$script_dir/translation-keys"
+    "$script_dir/translation-keys.js"
 }
 
 translation_keys_in_locize() {

package/{src/translation-keys → dist/translation-keys.js}

@@ -7,21 +7,19 @@
 //   git ls-files |
 //      grep '\.js$' |
 //      grep -v '\.min\.js$' |
-//     ./scripts/translation-keys
+//     ./scripts/translation-keys.js
 //
 // We stream out the keys as seen. We do no sorting or de-duplication.
 //
 const {extractTranslationKeysAndVariables} = require('./translation-extraction.js')
 const readline = require('readline')
 
-const interface = readline.createInterface({
+const lines = readline.createInterface({
   input: process.stdin,
   output: process.stdout,
   terminal: false
 })
 
-interface.on('line', path => {
-  extractTranslationKeysAndVariables(path).forEach(({i18nKey}) =>
-    console.log(i18nKey)
-  )
+lines.on('line', path => {
+  extractTranslationKeysAndVariables(path).forEach(({i18nKey}) => console.log(i18nKey))
 })

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@freckle/i18n-scripts",
-  "version": "1.1.23",
+  "version": "2.0.0",
   "description": "Scripts for i18n",
   "author": "Renaissance Learning Inc",
   "license": "MIT",
   "bin": {
     "check-i18n-variables": "./dist/check-i18n-variables.js",
     "check-i18next-icu-parser": "./dist/check-i18next-icu-parser.js",
-    "check-missing-translations": "./dist/check-missing-translations.js"
+    "check-missing-translations": "./dist/check-missing-translations"
   },
   "main": "./dist/index.js",
   "repository": {
@@ -22,7 +22,7 @@
   },
   "dependencies": {
     "esprima-walk": "0.1.0",
-    "flow-parser": "0.332.0",
+    "flow-parser": "0.333.0",
     "i18next": "26.4.2",
     "i18next-http-backend": "^4.0.2",
     "i18next-icu": "^2.4.4",
@@ -32,7 +32,7 @@
     "@freckle/eslint-config": "^1.1.0",
     "eslint": "^10.11.0",
     "globals": "^17.12.0",
-    "prettier": "^3.9.8"
+    "prettier": "^3.9.9"
   },
   "packageManager": "pnpm@12.6.0+sha512.3ef68f951cb111ac204b4a5a16f0b2ddf0da56a96e0413e81d855d9f0b55ef926714709028e1cd00c405c2c5fb7b9e8ec4dc46777c805d0373c2f2ff00fd20ec"
 }

package/src/check-i18n-variables.js

@@ -1,16 +1,131 @@
 #!/usr/bin/env node
+//
+// Checks all locize keys (in the codebase) with known variables against key
+// values in locize ensuring that variables in locize refer to known vars.
+//
+const i18next = require('i18next')
+const ICU = require('i18next-icu')
+const HttpBackend = require('i18next-http-backend')
+const {IntlMessageFormat} = require('intl-messageformat')
 
-const {spawn} = require('node:child_process')
-const path = require('path')
+const {
+  extractTranslationKeysAndVariables,
+  KNOWN_VARIABLES_TAG
+} = require('./translation-extraction.js')
+const {getFiles, filterJsFiles, supportedLngs} = require('./helpers.js')
 
-const [_nodeBin, scriptPath, ...args] = process.argv
+const [_nodeBin, _scriptPath, ...projects] = process.argv
 
-const proc = spawn(`${path.dirname(scriptPath)}/check-i18n-variables`, args)
+if (projects.length < 1) {
+  console.error('Usage: check-i18n-variables <path>')
+  process.exit(64)
+}
 
-proc.stdout.on('data', data => {
-  process.stdout.write(data)
-})
+// TODO: Pass in via ENV?
+const ns = ['school', 'student', 'console', 'classroom', 'common']
 
-proc.stderr.on('data', data => {
-  process.stderr.write(data)
-})
+const translationKeysWithKnownVariablesInProject = async project => {
+  const files = await getFiles(project)
+  const jsFiles = files.filter(filterJsFiles)
+  const results = []
+
+  jsFiles.forEach(({path}) => {
+    extractTranslationKeysAndVariables(path)
+      .filter(({tag}) => tag === KNOWN_VARIABLES_TAG)
+      .forEach(keyWithVars => results.push(keyWithVars))
+  })
+
+  return results
+}
+
+// The AST is undocumented, however, its type definitions are here
+//    https://github.com/formatjs/formatjs/blob/82a2e5d07f9a999624ffb6387a58464f9a5f7399/packages/icu-messageformat-parser/types.ts#L122-L131
+const extractVariablesFromICUAst = ast => {
+  // Some AST field values are `null`
+  if (!ast) {
+    return []
+  }
+
+  if (ast.constructor === Array) {
+    return ast.flatMap(extractVariablesFromICUAst)
+  }
+  if (ast.constructor === Object) {
+    const subVariables = Object.values(ast).flatMap(extractVariablesFromICUAst)
+    // `type`s that aren't 0 (string literals) are variables when `.value` exists
+    const hasVariable = ast.type && typeof ast.type === 'number' && ast.type !== 0 && ast.value
+
+    return (hasVariable ? [ast.value] : []).concat(subVariables)
+  }
+
+  return []
+}
+
+const main = async () => {
+  await i18next
+    .use(ICU)
+    .use(HttpBackend)
+    .init({
+      supportedLngs,
+      allowMultiLoading: true,
+      keySeparator: false,
+      ns,
+      backend: {
+        loadPath: 'https://translations.freckle.com/latest/{{lng}}/{{ns}}'
+      }
+    })
+
+  let anyErrors = false
+
+  // Project key extraction is not cheap so cache per project
+  const keysPerProject = {}
+
+  await Promise.all(
+    projects.map(
+      async project =>
+        (keysPerProject[project] = await translationKeysWithKnownVariablesInProject(project))
+    )
+  )
+
+  for (const lang of supportedLngs) {
+    await i18next.changeLanguage(lang)
+    const translations = i18next.getDataByLanguage(lang)
+
+    for (const [project, keys] of Object.entries(keysPerProject)) {
+      for (const {i18nKey, variables: codeVariables} of keys) {
+        const keyValue =
+          (translations[project] && translations[project][i18nKey]) || translations.common[i18nKey]
+
+        try {
+          if (keyValue) {
+            const locizeVariables = extractVariablesFromICUAst(
+              new IntlMessageFormat(keyValue, lang, {}, {ignoreTag: true}).getAst()
+            )
+
+            for (const locizeVariable of locizeVariables) {
+              if (!codeVariables.includes(locizeVariable)) {
+                anyErrors = true
+                console.error(
+                  `${project}:${lang}:${i18nKey} does not pass the variable ${locizeVariable}`
+                )
+              }
+            }
+          }
+        } catch {
+          // We have another script that checks that `IntlMessageFormat` can
+          // successfully parse the ICU text so let's not fail for that reason.
+        }
+      }
+    }
+  }
+
+  if (anyErrors) {
+    console.error(
+      'The above keys have variables not passed by the code and need to be fixed in Locize.'
+    )
+    process.exit(1)
+  } else {
+    process.exit(0)
+  }
+}
+
+main()

package/src/check-i18next-icu-parser.js

@@ -1,16 +1,62 @@
 #!/usr/bin/env node
+//
+// Checks all keys in Locize to make sure that they can be parsed by the `i18next` `t` function
+//
+const i18next = require('i18next')
+const ICU = require('i18next-icu')
+const HttpBackend = require('i18next-http-backend')
+const {supportedLngs} = require('./helpers.js')
 
-const {spawn} = require('node:child_process')
-const path = require('path')
+const VALID_NAMESPACES = ['school', 'student', 'console', 'classroom', 'common']
 
-const [_nodeBin, scriptPath, ...args] = process.argv
+const ns = process.argv.slice(2)
 
-const proc = spawn(`${path.dirname(scriptPath)}/check-i18next-icu-parser`, args)
+for (const arg of ns) {
+  if (!VALID_NAMESPACES.some(ns => ns === arg)) {
+    throw new Error(`Invalid arguments: ${arg} is not a namespace`)
+  }
+}
 
-proc.stdout.on('data', data => {
-  process.stdout.write(data)
-})
+const main = async () => {
+  const t = await i18next
+    .use(ICU)
+    .use(HttpBackend)
+    .init({
+      supportedLngs,
+      allowMultiLoading: true,
+      keySeparator: false,
+      ns,
+      backend: {
+        loadPath: 'https://translations.freckle.com/latest/{{lng}}/{{ns}}'
+      }
+    })
 
-proc.stderr.on('data', data => {
-  process.stderr.write(data)
-})
+  let anyErrors = false
+
+  for (const l of supportedLngs) {
+    await i18next.changeLanguage(l)
+    for (const n of ns) {
+      for (const key of Object.keys(i18next.getDataByLanguage(l)[n])) {
+        try {
+          t(`${n}:${key}`)
+        } catch (error) {
+          if (!isInterpolationError(error)) {
+            anyErrors = true
+            console.error(`${n}:${l}:${key}`)
+          }
+        }
+      }
+    }
+  }
+
+  if (anyErrors) {
+    console.error('The above keys cannot be parsed as ICU and need to be fixed in Locize.')
+    process.exit(1)
+  } else {
+    process.exit(0)
+  }
+}
+
+const isInterpolationError = error => /^The intl string context variable/.test(error.message)
+
+main()

package/src/check-missing-translations

@@ -9,7 +9,7 @@ translation_keys_in_code() {
   git ls-files "$project_path" |
     grep -E '\.(js|ts|tsx)$' |
     grep -v '\.min\.js$' |
-    "$script_dir/translation-keys"
+    "$script_dir/translation-keys.js"
 }
 
 translation_keys_in_locize() {

package/{dist/translation-keys → src/translation-keys.js}

@@ -7,21 +7,19 @@
 //   git ls-files |
 //      grep '\.js$' |
 //      grep -v '\.min\.js$' |
-//     ./scripts/translation-keys
+//     ./scripts/translation-keys.js
 //
 // We stream out the keys as seen. We do no sorting or de-duplication.
 //
 const {extractTranslationKeysAndVariables} = require('./translation-extraction.js')
 const readline = require('readline')
 
-const interface = readline.createInterface({
+const lines = readline.createInterface({
   input: process.stdin,
   output: process.stdout,
   terminal: false
 })
 
-interface.on('line', path => {
-  extractTranslationKeysAndVariables(path).forEach(({i18nKey}) =>
-    console.log(i18nKey)
-  )
+lines.on('line', path => {
+  extractTranslationKeysAndVariables(path).forEach(({i18nKey}) => console.log(i18nKey))
 })

package/dist/check-i18n-variables

@@ -1,133 +0,0 @@
-#!/usr/bin/env node
-//
-// Checks all locize keys (in the codebase) with known variables against key
-// values in locize ensuring that variables in locize refer to known vars.
-//
-const {promises: fs} = require('fs')
-const p = require('path')
-const i18next = require('i18next')
-const ICU = require('i18next-icu')
-const HttpBackend = require('i18next-http-backend')
-const {IntlMessageFormat} = require('intl-messageformat')
-
-const {
-  extractTranslationKeysAndVariables,
-  KNOWN_VARIABLES_TAG
-} = require('./translation-extraction.js')
-const {getFiles, filterJsFiles, supportedLngs} = require('./helpers.js')
-
-const [_nodeBin, _scriptPath, ...projects] = process.argv
-
-if (projects.length < 1) {
-  console.log('Usage: check-i18n-variables <path>')
-  return
-}
-
-// TODO: Pass in via ENV?
-const ns = ['school', 'student', 'console', 'classroom', 'common']
-
-const translationKeysWithKnownVariablesInProject = async project => {
-  const files = await getFiles(project)
-  const jsFiles = files.filter(filterJsFiles)
-  const results = []
-
-  jsFiles.forEach(({path}) => {
-    extractTranslationKeysAndVariables(path)
-      .filter(({tag}) => tag === KNOWN_VARIABLES_TAG)
-      .forEach(keyWithVars => results.push(keyWithVars))
-  })
-
-  return results
-}
-
-// The AST is undocumented, however, its type definitions are here
-//    https://github.com/formatjs/formatjs/blob/82a2e5d07f9a999624ffb6387a58464f9a5f7399/packages/icu-messageformat-parser/types.ts#L122-L131
-const extractVariablesFromICUAst = ast => {
-  // Some AST field values are `null`
-  if (!ast) {
-    return []
-  }
-
-  if (ast.constructor === Array) {
-    return ast.flatMap(extractVariablesFromICUAst)
-  }
-  if (ast.constructor === Object) {
-    const subVariables = Object.values(ast).flatMap(extractVariablesFromICUAst)
-    // `type`s that aren't 0 (string literals) are variables when `.value` exists
-    const hasVariable = ast.type && typeof ast.type === 'number' && ast.type !== 0 && ast.value
-
-    return (hasVariable ? [ast.value] : []).concat(subVariables)
-  }
-
-  return []
-}
-
-const main = async () => {
-  const t = await i18next
-    .use(ICU)
-    .use(HttpBackend)
-    .init({
-      supportedLngs,
-      allowMultiLoading: true,
-      keySeparator: false,
-      ns,
-      backend: {
-        loadPath: 'https://translations.freckle.com/latest/{{lng}}/{{ns}}'
-      }
-    })
-
-  let anyErrors = false
-
-  // Project key extraction is not cheap so cache per project
-  const keysPerProject = {}
-
-  await Promise.all(
-    projects.map(
-      async project =>
-        (keysPerProject[project] = await translationKeysWithKnownVariablesInProject(project))
-    )
-  )
-
-  for (const lang of supportedLngs) {
-    await i18next.changeLanguage(lang)
-    const translations = i18next.getDataByLanguage(lang)
-
-    for (const [project, keys] of Object.entries(keysPerProject)) {
-      for (const {i18nKey, variables: codeVariables} of keys) {
-        const keyValue =
-          (translations[project] && translations[project][i18nKey]) || translations.common[i18nKey]
-
-        try {
-          if (keyValue) {
-            const locizeVariables = extractVariablesFromICUAst(
-              new IntlMessageFormat(keyValue, lang, {}, {ignoreTag: true}).getAst()
-            )
-
-            for (const locizeVariable of locizeVariables) {
-              if (!codeVariables.includes(locizeVariable)) {
-                anyErrors = true
-                console.error(
-                  `${project}:${lang}:${i18nKey} does not pass the variable ${locizeVariable}`
-                )
-              }
-            }
-          }
-        } catch (e) {
-          // We have another script that checks that `IntlMessageFormat` can
-          // successfully parse the ICU text so let's not fail for that reason.
-        }
-      }
-    }
-  }
-
-  if (anyErrors) {
-    console.error(
-      'The above keys have variables not passed by the code and need to be fixed in Locize.'
-    )
-    process.exit(1)
-  } else {
-    process.exit(0)
-  }
-}
-
-main()

package/dist/check-i18next-icu-parser

@@ -1,64 +0,0 @@
-#!/usr/bin/env node
-// @flow
-//
-// Checks all keys in Locize to make sure that they can be parsed by the `i18next` `t` function
-//
-const i18next = require('i18next')
-const ICU = require('i18next-icu')
-const HttpBackend = require('i18next-http-backend')
-const {supportedLngs} = require('./helpers.js')
-
-const VALID_NAMESPACES = ['school', 'student', 'console', 'classroom', 'common']
-
-const ns = process.argv.slice(2)
-
-for (const arg of ns) {
-  if (!VALID_NAMESPACES.some(ns => ns === arg)) {
-    throw new Error(`Invalid arguments: ${arg} is not a namespace`)
-  }
-}
-
-
-const main = async () => {
-  const t = await i18next
-    .use(ICU)
-    .use(HttpBackend)
-    .init({
-      supportedLngs,
-      allowMultiLoading: true,
-      keySeparator: false,
-      ns,
-      backend: {
-        loadPath: 'https://translations.freckle.com/latest/{{lng}}/{{ns}}'
-      }
-    })
-
-  let anyErrors = false
-
-  for (const l of supportedLngs) {
-    await i18next.changeLanguage(l)
-    for (const n of ns) {
-      for (const key of Object.keys(i18next.getDataByLanguage(l)[n])) {
-        try {
-          t(`${n}:${key}`)
-        } catch (error) {
-          if (!isInterpolationError(error)) {
-            anyErrors = true
-            console.error(`${n}:${l}:${key}`)
-          }
-        }
-      }
-    }
-  }
-
-  if (anyErrors) {
-    console.error('The above keys cannot be parsed as ICU and need to be fixed in Locize.')
-    process.exit(1)
-  } else {
-    process.exit(0)
-  }
-}
-
-const isInterpolationError = error => /^The intl string context variable/.test(error.message)
-
-main()

package/dist/check-missing-translations.js

@@ -1,16 +0,0 @@
-#!/usr/bin/env node
-
-const {spawn} = require('node:child_process')
-const path = require('path')
-
-const [_nodeBin, scriptPath, ...args] = process.argv
-
-const proc = spawn(`${path.dirname(scriptPath)}/check-missing-translations`, args)
-
-proc.stdout.on('data', data => {
-  process.stdout.write(data)
-})
-
-proc.stderr.on('data', data => {
-  process.stderr.write(data)
-})

package/src/check-i18n-variables

@@ -1,133 +0,0 @@
-#!/usr/bin/env node
-//
-// Checks all locize keys (in the codebase) with known variables against key
-// values in locize ensuring that variables in locize refer to known vars.
-//
-const {promises: fs} = require('fs')
-const p = require('path')
-const i18next = require('i18next')
-const ICU = require('i18next-icu')
-const HttpBackend = require('i18next-http-backend')
-const {IntlMessageFormat} = require('intl-messageformat')
-
-const {
-  extractTranslationKeysAndVariables,
-  KNOWN_VARIABLES_TAG
-} = require('./translation-extraction.js')
-const {getFiles, filterJsFiles, supportedLngs} = require('./helpers.js')
-
-const [_nodeBin, _scriptPath, ...projects] = process.argv
-
-if (projects.length < 1) {
-  console.log('Usage: check-i18n-variables <path>')
-  return
-}
-
-// TODO: Pass in via ENV?
-const ns = ['school', 'student', 'console', 'classroom', 'common']
-
-const translationKeysWithKnownVariablesInProject = async project => {
-  const files = await getFiles(project)
-  const jsFiles = files.filter(filterJsFiles)
-  const results = []
-
-  jsFiles.forEach(({path}) => {
-    extractTranslationKeysAndVariables(path)
-      .filter(({tag}) => tag === KNOWN_VARIABLES_TAG)
-      .forEach(keyWithVars => results.push(keyWithVars))
-  })
-
-  return results
-}
-
-// The AST is undocumented, however, its type definitions are here
-//    https://github.com/formatjs/formatjs/blob/82a2e5d07f9a999624ffb6387a58464f9a5f7399/packages/icu-messageformat-parser/types.ts#L122-L131
-const extractVariablesFromICUAst = ast => {
-  // Some AST field values are `null`
-  if (!ast) {
-    return []
-  }
-
-  if (ast.constructor === Array) {
-    return ast.flatMap(extractVariablesFromICUAst)
-  }
-  if (ast.constructor === Object) {
-    const subVariables = Object.values(ast).flatMap(extractVariablesFromICUAst)
-    // `type`s that aren't 0 (string literals) are variables when `.value` exists
-    const hasVariable = ast.type && typeof ast.type === 'number' && ast.type !== 0 && ast.value
-
-    return (hasVariable ? [ast.value] : []).concat(subVariables)
-  }
-
-  return []
-}
-
-const main = async () => {
-  const t = await i18next
-    .use(ICU)
-    .use(HttpBackend)
-    .init({
-      supportedLngs,
-      allowMultiLoading: true,
-      keySeparator: false,
-      ns,
-      backend: {
-        loadPath: 'https://translations.freckle.com/latest/{{lng}}/{{ns}}'
-      }
-    })
-
-  let anyErrors = false
-
-  // Project key extraction is not cheap so cache per project
-  const keysPerProject = {}
-
-  await Promise.all(
-    projects.map(
-      async project =>
-        (keysPerProject[project] = await translationKeysWithKnownVariablesInProject(project))
-    )
-  )
-
-  for (const lang of supportedLngs) {
-    await i18next.changeLanguage(lang)
-    const translations = i18next.getDataByLanguage(lang)
-
-    for (const [project, keys] of Object.entries(keysPerProject)) {
-      for (const {i18nKey, variables: codeVariables} of keys) {
-        const keyValue =
-          (translations[project] && translations[project][i18nKey]) || translations.common[i18nKey]
-
-        try {
-          if (keyValue) {
-            const locizeVariables = extractVariablesFromICUAst(
-              new IntlMessageFormat(keyValue, lang, {}, {ignoreTag: true}).getAst()
-            )
-
-            for (const locizeVariable of locizeVariables) {
-              if (!codeVariables.includes(locizeVariable)) {
-                anyErrors = true
-                console.error(
-                  `${project}:${lang}:${i18nKey} does not pass the variable ${locizeVariable}`
-                )
-              }
-            }
-          }
-        } catch (e) {
-          // We have another script that checks that `IntlMessageFormat` can
-          // successfully parse the ICU text so let's not fail for that reason.
-        }
-      }
-    }
-  }
-
-  if (anyErrors) {
-    console.error(
-      'The above keys have variables not passed by the code and need to be fixed in Locize.'
-    )
-    process.exit(1)
-  } else {
-    process.exit(0)
-  }
-}
-
-main()

package/src/check-i18next-icu-parser

@@ -1,64 +0,0 @@
-#!/usr/bin/env node
-// @flow
-//
-// Checks all keys in Locize to make sure that they can be parsed by the `i18next` `t` function
-//
-const i18next = require('i18next')
-const ICU = require('i18next-icu')
-const HttpBackend = require('i18next-http-backend')
-const {supportedLngs} = require('./helpers.js')
-
-const VALID_NAMESPACES = ['school', 'student', 'console', 'classroom', 'common']
-
-const ns = process.argv.slice(2)
-
-for (const arg of ns) {
-  if (!VALID_NAMESPACES.some(ns => ns === arg)) {
-    throw new Error(`Invalid arguments: ${arg} is not a namespace`)
-  }
-}
-
-
-const main = async () => {
-  const t = await i18next
-    .use(ICU)
-    .use(HttpBackend)
-    .init({
-      supportedLngs,
-      allowMultiLoading: true,
-      keySeparator: false,
-      ns,
-      backend: {
-        loadPath: 'https://translations.freckle.com/latest/{{lng}}/{{ns}}'
-      }
-    })
-
-  let anyErrors = false
-
-  for (const l of supportedLngs) {
-    await i18next.changeLanguage(l)
-    for (const n of ns) {
-      for (const key of Object.keys(i18next.getDataByLanguage(l)[n])) {
-        try {
-          t(`${n}:${key}`)
-        } catch (error) {
-          if (!isInterpolationError(error)) {
-            anyErrors = true
-            console.error(`${n}:${l}:${key}`)
-          }
-        }
-      }
-    }
-  }
-
-  if (anyErrors) {
-    console.error('The above keys cannot be parsed as ICU and need to be fixed in Locize.')
-    process.exit(1)
-  } else {
-    process.exit(0)
-  }
-}
-
-const isInterpolationError = error => /^The intl string context variable/.test(error.message)
-
-main()

package/src/check-missing-translations.js

@@ -1,16 +0,0 @@
-#!/usr/bin/env node
-
-const {spawn} = require('node:child_process')
-const path = require('path')
-
-const [_nodeBin, scriptPath, ...args] = process.argv
-
-const proc = spawn(`${path.dirname(scriptPath)}/check-missing-translations`, args)
-
-proc.stdout.on('data', data => {
-  process.stdout.write(data)
-})
-
-proc.stderr.on('data', data => {
-  process.stderr.write(data)
-})