Migrate src/data-directory console statements to structured logger (#61074)

Author: Ebonsignori

eslint.config.ts

@@ -177,7 +177,6 @@ export default [
       'src/article-api/**/*.{ts,js}',
       'src/audit-logs/**/*.{ts,js}',
       'src/color-schemes/**/*.{ts,js}',
-      'src/data-directory/**/*.{ts,js}',
       'src/dev-toc/**/*.{ts,js}',
       'src/events/components/**/*.{ts,js}',
       'src/fixtures/**/*.{ts,js}',

src/data-directory/lib/get-data.ts

@@ -7,6 +7,9 @@ import { merge, get } from 'lodash-es'
 
 import languages from '@/languages/lib/languages-server'
 import { correctTranslatedContentStrings } from '@/languages/lib/correct-translation-content'
+import { createLogger } from '@/observability/logger'
+
+const logger = createLogger(import.meta.url)
 
 interface YAMLException extends Error {
   mark?: any
@@ -128,7 +131,7 @@ export const getDataByLanguage = memoize((dottedPath: string, langCode: string):
       // would have caused a YAMLException
       if (langCode !== 'en') {
         if (DEBUG_JIT_DATA_READS) {
-          console.warn(`Unable to parse Yaml in (${langCode}) '${dottedPath}': ${error.message}`)
+          logger.warn('Unable to parse Yaml in translation', { langCode, dottedPath, error })
         }
         // Give it one more chance, but use English this time
         return getDataByDir(dottedPath, languages.en.dir)
@@ -208,7 +211,7 @@ function getDataByDir(
         return content
       }
     } else {
-      console.warn(`Unable to find variables Yaml file ${fullPath.join(path.sep)}`)
+      logger.warn('Unable to find variables Yaml file', { filePath: fullPath.join(path.sep) })
     }
     return undefined
   }
@@ -347,7 +350,7 @@ const getFileContent = (
   englishRoot?: string,
 ): string => {
   const filePath = root ? path.join(root, relPath) : relPath
-  if (DEBUG_JIT_DATA_READS) console.log('READ', filePath)
+  if (DEBUG_JIT_DATA_READS) logger.info('READ', { filePath })
   try {
     return fs.readFileSync(filePath, 'utf-8')
   } catch (err) {