Migrate src/languages console statements to structured logger (#61076)

Author: Ebonsignori

eslint.config.ts

@@ -181,8 +181,6 @@ export default [
       'src/events/components/**/*.{ts,js}',
       'src/fixtures/**/*.{ts,js}',
       'src/journeys/**/*.{ts,js}',
-      'src/languages/**/*.{ts,js}',
-      'src/links/**/*.{ts,js}',
       'src/metrics/**/*.{ts,js}',
       'src/observability/lib/handle-package-not-found.ts',
       'src/search/**/*.{ts,js}',
@@ -200,6 +198,8 @@ export default [
       'src/workflows/**/*.{ts,js}',
       'src/content-linter/**/*.{ts,js}',
       '**/*.{tsx,jsx}',
+      // Client-side module that cannot use the server-only structured logger
+      'src/languages/lib/translation-utils.ts',
       // CLI help script — chalk-colored terminal output, not application logging
       'src/rest/docs.ts',
     ],

src/languages/lib/get-alert-titles.ts

@@ -1,8 +1,11 @@
 import fs from 'fs/promises'
 import path from 'path'
 import yaml from 'js-yaml'
+import { createLogger } from '@/observability/logger'
 import languages from './languages-server'
 
+const logger = createLogger(import.meta.url)
+
 interface AlertTitles {
   [key: string]: string
 }
@@ -26,7 +29,7 @@ export async function getAlertTitles(page: { languageCode: string }) {
       file = await fs.readFile(path.join(dir, `data/ui.yml`), 'utf-8')
       yamlFile = yaml.load(file) as UiYaml
     } catch (e) {
-      console.warn(`Failed to load translated alert titles`, e)
+      logger.warn('Failed to load translated alert titles', { error: e, languageCode })
     }
   }
   if (!file || !yamlFile.alerts) {

src/links/lib/extract-links.ts

@@ -8,11 +8,14 @@
 import fs from 'fs'
 import path from 'path'
 
+import { createLogger } from '@/observability/logger'
 import { allVersions } from '@/versions/lib/all-versions'
 import { latestStable } from '@/versions/lib/enterprise-server-releases'
 import { getDataByLanguage } from '@/data-directory/lib/get-data'
 import type { Context, Page } from '@/types'
 
+const logger = createLogger(import.meta.url)
+
 // Link patterns for Markdown
 const INTERNAL_LINK_PATTERN = /\]\(\/[^)]+\)/g
 const AUTOTITLE_LINK_PATTERN = /\[AUTOTITLE\]\(([^)]+)\)/g
@@ -329,7 +332,7 @@ export async function extractLinksWithLiquid(
   } catch (error) {
     // If Liquid rendering fails, fall back to raw extraction
     // This can happen with malformed templates
-    console.warn('Liquid rendering failed, falling back to raw extraction:', error)
+    logger.warn('Liquid rendering failed, falling back to raw extraction', { error })
     return extractLinksFromMarkdown(content)
   }
 }
@@ -347,7 +350,7 @@ export async function renderAndExtractLinks(
     const renderedMarkdown = await renderLiquid(content, context)
     return { renderedMarkdown, result: extractLinksFromMarkdown(renderedMarkdown) }
   } catch (error) {
-    console.warn('Liquid rendering failed, falling back to raw extraction:', error)
+    logger.warn('Liquid rendering failed, falling back to raw extraction', { error })
     return { renderedMarkdown: content, result: extractLinksFromMarkdown(content) }
   }
 }

src/links/lib/update-internal-links.ts

@@ -6,6 +6,7 @@ import { toMarkdown } from 'mdast-util-to-markdown'
 import yaml from 'js-yaml'
 import { type Node, type Nodes, type Definition, type Link } from 'mdast'
 
+import { createLogger } from '@/observability/logger'
 import frontmatter from '@/frame/lib/read-frontmatter'
 import {
   getPathWithLanguage,
@@ -20,6 +21,8 @@ import getRedirect, { splitPathByLanguage } from '@/redirects/lib/get-redirect'
 import nonEnterpriseDefaultVersion from '@/versions/lib/non-enterprise-default-version'
 import { deprecated } from '@/versions/lib/enterprise-server-releases'
 
+const logger = createLogger(import.meta.url)
+
 // That magical string that can be turned into the actual title when
 // we, at runtime, render out the links
 const AUTOTITLE = 'AUTOTITLE'
@@ -55,7 +58,7 @@ export async function updateInternalLinks(files: string[], options = {}) {
         ...(await updateFile(file, context, opts)),
       })
     } catch (err) {
-      console.warn(`The file it tried to process on exception was: ${file}`)
+      logger.warn('File processing failed', { file })
       throw err
     }
   }
@@ -149,7 +152,7 @@ async function updateFile(
       // bubble up to the CLI. And the CLI will mention which file it
       // was processing when it failed. But we have a valuable piece of
       // information here about which frontmatter key it was that failed.
-      console.warn(`The frontmatter key it processed and failed was '${key}'`)
+      logger.warn('Frontmatter key processing failed', { key })
       throw error
     }
   }
@@ -281,9 +284,10 @@ async function updateFile(
         newContent = newContent.replace(asMarkdown, newAsMarkdown)
       }
     } else if (opts.verbose) {
-      console.warn(
-        `Unable to find link as Markdown ('${asMarkdown}') in the source content (${file})`,
-      )
+      logger.warn('Unable to find link as Markdown in the source content', {
+        asMarkdown,
+        file,
+      })
     }
   })
 
@@ -394,7 +398,7 @@ function getNewFrontmatterLinkList(
   const better = []
   for (const entry of list) {
     if (/{%\s*else\s*%}/.test(entry)) {
-      console.warn(`Skipping frontmatter link with {% else %} in it: ${entry}. (file: ${file})`)
+      logger.warn('Skipping frontmatter link with {% else %} in it', { entry, file })
       better.push(entry)
       continue
     }
@@ -419,7 +423,7 @@ function getNewFrontmatterLinkList(
         if (opts.strict) {
           throw new Error(msg)
         }
-        console.warn(`WARNING: ${msg}`)
+        logger.warn(msg, { file, pure, lineNumber })
         better.push(entry)
       } else {
         // Perhaps it just redirected to a specific version
@@ -512,7 +516,7 @@ function getNewHref(
     if (opts.strict) {
       throw new Error(msg)
     } else {
-      console.warn(`WARNING: ${msg}`)
+      logger.warn(msg, { file, href: newHref })
       return
     }
   }
@@ -530,7 +534,7 @@ function getNewHref(
       if (opts.strict) {
         throw new Error(msg)
       } else {
-        console.warn(`WARNING: ${msg}`)
+        logger.warn(msg, { file, href })
         return
       }
     }
@@ -569,7 +573,7 @@ function getNewHref(
       if (opts.strict) {
         throw new Error(msg)
       } else {
-        console.warn(msg)
+        logger.warn(msg, { file })
         return
       }
     } else if (withoutLanguage.startsWith('/enterprise-server@latest')) {