Fix translated ifversion flag names causing hidden content in ja/zh/ko/de/pt (#62860)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: docs-bot

src/languages/lib/correct-translation-content.ts

@@ -463,6 +463,16 @@ export function correctTranslatedContentStrings(
     // `{% indented_data_reference 再利用可能.X.Y spaces=N %}` — translated path
     content = content.replace(/(\{%-?\s*indented_data_reference\s+)再利用可能\./g, '$1reusables.')
 
+    // `{% ifversion コマンド パレット %}` — translated flag name "command palette" = command-palette
+    content = content.replaceAll(
+      '{% ifversion コマンド パレット %}',
+      '{% ifversion command-palette %}',
+    )
+    content = content.replaceAll(
+      '{%- ifversion コマンド パレット %}',
+      '{%- ifversion command-palette %}',
+    )
+
     // [SCRAPE-6548] Per-file fixes for ja pages whose intro/title/shortTitle
     // Liquid was structurally scrambled (orphan endif, swapped tag order,
     // unclosed ifversion). Each replacement is scoped by the unique broken
@@ -649,6 +659,10 @@ export function correctTranslatedContentStrings(
     content = content.replace(/\{%-? (?:ifversion|elsif|if) [^%]*?ou [^%]*?%\}/g, (match) => {
       return match.replace(/ ou /g, ' or ')
     })
+    // Portuguese "não" for "not" in ifversion/elsif/if tags (e.g. `{% ifversion não ghes %}`)
+    content = content.replace(/\{%-? (?:ifversion|elsif|if) [^%]*?\bnão\b[^%]*?%\}/g, (match) => {
+      return match.replace(/\bnão\b/g, 'not')
+    })
     // Fully translated reusable path in audit log article:
     // `{% dados agrupados por categoria.complemento.audit_log.reference-grouped-by-category %}`
     content = content.replaceAll(
@@ -838,6 +852,16 @@ export function correctTranslatedContentStrings(
     content = content.replaceAll('{%- 行标头 %}', '{%- rowheaders %}')
     content = content.replaceAll('{% 行标题 %}', '{% rowheaders %}')
     content = content.replaceAll('{%- 行标题 %}', '{%- rowheaders %}')
+
+    // `{% ifversion 命令面板 %}` — translated flag name "command panel" = command-palette
+    content = content.replaceAll('{% ifversion 命令面板 %}', '{% ifversion command-palette %}')
+    content = content.replaceAll('{%- ifversion 命令面板 %}', '{%- ifversion command-palette %}')
+    // `{% ifversion 子问题 %}` — translated flag name "sub-issues" (子问题)
+    content = content.replaceAll('{% ifversion 子问题 %}', '{% ifversion sub-issues %}')
+    content = content.replaceAll('{%- ifversion 子问题 %}', '{%- ifversion sub-issues %}')
+    // `{% ifversion 问题类型 %}` — translated flag name "issue types" = issue-types
+    content = content.replaceAll('{% ifversion 问题类型 %}', '{% ifversion issue-types %}')
+    content = content.replaceAll('{%- ifversion 问题类型 %}', '{%- ifversion issue-types %}')
     // `{% 结束行标题 %}` / `{% 结束行标头 %}` / `{% 结束行头 %}` — endrowheaders
     content = content.replaceAll('{% 结束行标题 %}', '{% endrowheaders %}')
     content = content.replaceAll('{%- 结束行标题 %}', '{%- endrowheaders %}')
@@ -1636,6 +1660,31 @@ export function correctTranslatedContentStrings(
     content = content.replaceAll('{% 옥티콘 ', '{% octicon ')
     content = content.replaceAll('{%- 옥티콘 ', '{%- octicon ')
 
+    // `{% ifversion 명령 팔레트 %}` — translated flag name "command palette" = command-palette
+    content = content.replaceAll('{% ifversion 명령 팔레트 %}', '{% ifversion command-palette %}')
+    content = content.replaceAll('{%- ifversion 명령 팔레트 %}', '{%- ifversion command-palette %}')
+    // `{% ifversion 하위 문제 %}` — translated flag name "sub-issues" (하위 문제)
+    content = content.replaceAll('{% ifversion 하위 문제 %}', '{% ifversion sub-issues %}')
+    content = content.replaceAll('{%- ifversion 하위 문제 %}', '{%- ifversion sub-issues %}')
+    // `{% ifversion 리포지토리-규칙 관리 %}` — translated flag name "repository-rules management"
+    content = content.replaceAll(
+      '{% ifversion 리포지토리-규칙 관리 %}',
+      '{% ifversion repo-rules-management %}',
+    )
+    content = content.replaceAll(
+      '{%- ifversion 리포지토리-규칙 관리 %}',
+      '{%- ifversion repo-rules-management %}',
+    )
+    // `{% ifversion 업데이트 알림 설정-22 %}` — translated flag name "update notification settings-22"
+    content = content.replaceAll(
+      '{% ifversion 업데이트 알림 설정-22 %}',
+      '{% ifversion update-notification-settings-22 %}',
+    )
+    content = content.replaceAll(
+      '{%- ifversion 업데이트 알림 설정-22 %}',
+      '{%- ifversion update-notification-settings-22 %}',
+    )
+
     // `{% data Variables.` — capital V in "Variables" (Korean translator capitalised the word)
     content = content.replaceAll('{% data Variables.', '{% data variables.')
     content = content.replaceAll('{%- data Variables.', '{%- data variables.')
@@ -1936,6 +1985,16 @@ export function correctTranslatedContentStrings(
     // Translated tag name `{% eingerucktes_datenverweis ... %}` → `{% indented_data_reference ... %}`
     content = content.replaceAll('{% eingerucktes_datenverweis ', '{% indented_data_reference ')
     content = content.replaceAll('{%- eingerucktes_datenverweis ', '{%- indented_data_reference ')
+    // `{% ifversion unveränderliche Versionen %}` — translated flag name
+    // "immutable releases" = immutable-releases
+    content = content.replaceAll(
+      '{% ifversion unveränderliche Versionen %}',
+      '{% ifversion immutable-releases %}',
+    )
+    content = content.replaceAll(
+      '{%- ifversion unveränderliche Versionen %}',
+      '{%- ifversion immutable-releases %}',
+    )
 
     // [SCRAPE-6548] Per-file fix:
     // organizations/.../permissions-of-custom-organization-roles.md (intro):

src/languages/tests/correct-translation-content.ts

@@ -222,6 +222,14 @@ describe('correctTranslatedContentStrings', () => {
       expect(fix('{%- ifversion fpt または ghec %}', 'ja')).toBe('{%- ifversion fpt or ghec %}')
     })
 
+    test('fixes translated command-palette flag name', () => {
+      expect(fix('{% ifversion コマンド パレット %}', 'ja')).toBe('{% ifversion command-palette %}')
+      expect(fix('{%- ifversion コマンド パレット %}', 'ja')).toBe(
+        '{%- ifversion command-palette %}',
+      )
+      expect(fix('{% ifversion command-palette %}', 'ja')).toBe('{% ifversion command-palette %}')
+    })
+
     test('fixes trailing quote on YAML value', () => {
       expect(fix('  asked_too_many_times: some value"  ', 'ja')).toBe(
         '  asked_too_many_times: some value',
@@ -404,6 +412,12 @@ describe('correctTranslatedContentStrings', () => {
       expect(fix('{%– endif %}', 'pt')).toBe('{%- endif %}')
     })
 
+    test('fixes Portuguese não (not) in ifversion tags', () => {
+      expect(fix('{% ifversion não ghes %}', 'pt')).toBe('{% ifversion not ghes %}')
+      expect(fix('{%- ifversion não ghes %}', 'pt')).toBe('{%- ifversion not ghes %}')
+      expect(fix('{% ifversion not ghes %}', 'pt')).toBe('{% ifversion not ghes %}')
+    })
+
     test('fixes datavariables / dadosvariables (no space)', () => {
       // `{% datavariables` — no space between "data" and "variables" (post-translation)
       expect(fix('{% datavariables.product.github %}', 'pt')).toBe(
@@ -562,6 +576,18 @@ describe('correctTranslatedContentStrings', () => {
       expect(fix('{ 如果 ghec %}', 'zh')).toBe('{% if ghec %}')
     })
 
+    test('fixes translated flag names in ifversion tags', () => {
+      expect(fix('{% ifversion 命令面板 %}', 'zh')).toBe('{% ifversion command-palette %}')
+      expect(fix('{%- ifversion 命令面板 %}', 'zh')).toBe('{%- ifversion command-palette %}')
+      expect(fix('{% ifversion 子问题 %}', 'zh')).toBe('{% ifversion sub-issues %}')
+      expect(fix('{%- ifversion 子问题 %}', 'zh')).toBe('{%- ifversion sub-issues %}')
+      expect(fix('{% ifversion 问题类型 %}', 'zh')).toBe('{% ifversion issue-types %}')
+      expect(fix('{%- ifversion 问题类型 %}', 'zh')).toBe('{%- ifversion issue-types %}')
+      expect(fix('{% ifversion command-palette %}', 'zh')).toBe('{% ifversion command-palette %}')
+      expect(fix('{% ifversion sub-issues %}', 'zh')).toBe('{% ifversion sub-issues %}')
+      expect(fix('{% ifversion issue-types %}', 'zh')).toBe('{% ifversion issue-types %}')
+    })
+
     test('fixes stray Chinese then merged with HTML', () => {
       expect(fix('，则为 {%<div>', 'zh')).toBe('<div>')
     })
@@ -1172,6 +1198,33 @@ describe('correctTranslatedContentStrings', () => {
       )
     })
 
+    test('fixes translated flag names in ifversion tags', () => {
+      expect(fix('{% ifversion 명령 팔레트 %}', 'ko')).toBe('{% ifversion command-palette %}')
+      expect(fix('{%- ifversion 명령 팔레트 %}', 'ko')).toBe('{%- ifversion command-palette %}')
+      expect(fix('{% ifversion 하위 문제 %}', 'ko')).toBe('{% ifversion sub-issues %}')
+      expect(fix('{%- ifversion 하위 문제 %}', 'ko')).toBe('{%- ifversion sub-issues %}')
+      expect(fix('{% ifversion 리포지토리-규칙 관리 %}', 'ko')).toBe(
+        '{% ifversion repo-rules-management %}',
+      )
+      expect(fix('{%- ifversion 리포지토리-규칙 관리 %}', 'ko')).toBe(
+        '{%- ifversion repo-rules-management %}',
+      )
+      expect(fix('{% ifversion 업데이트 알림 설정-22 %}', 'ko')).toBe(
+        '{% ifversion update-notification-settings-22 %}',
+      )
+      expect(fix('{%- ifversion 업데이트 알림 설정-22 %}', 'ko')).toBe(
+        '{%- ifversion update-notification-settings-22 %}',
+      )
+      expect(fix('{% ifversion command-palette %}', 'ko')).toBe('{% ifversion command-palette %}')
+      expect(fix('{% ifversion sub-issues %}', 'ko')).toBe('{% ifversion sub-issues %}')
+      expect(fix('{% ifversion repo-rules-management %}', 'ko')).toBe(
+        '{% ifversion repo-rules-management %}',
+      )
+      expect(fix('{% ifversion update-notification-settings-22 %}', 'ko')).toBe(
+        '{% ifversion update-notification-settings-22 %}',
+      )
+    })
+
     test('fixes dada → data (via generic)', () => {
       expect(fix('{% dada variables.product.github %}', 'ko')).toBe(
         '{% data variables.product.github %}',
@@ -1291,6 +1344,18 @@ describe('correctTranslatedContentStrings', () => {
       expect(fix('{% ifversion fpt oder ghec %}', 'de')).toBe('{% ifversion fpt or ghec %}')
     })
 
+    test('fixes translated immutable-releases flag name', () => {
+      expect(fix('{% ifversion unveränderliche Versionen %}', 'de')).toBe(
+        '{% ifversion immutable-releases %}',
+      )
+      expect(fix('{%- ifversion unveränderliche Versionen %}', 'de')).toBe(
+        '{%- ifversion immutable-releases %}',
+      )
+      expect(fix('{% ifversion immutable-releases %}', 'de')).toBe(
+        '{% ifversion immutable-releases %}',
+      )
+    })
+
     test('fixes translated block tags', () => {
       expect(fix('{% Hinweis %}', 'de')).toBe('{% note %}')
       expect(fix('{%- Hinweis %}', 'de')).toBe('{%- note %}')