sync: resolve conflicts by taking main branch for all news indexes and sitemap by Copilot · Pull Request #494 · Hack23/riksdagsmonitor

Copilot · 2026-02-24T11:18:12Z

This branch had merge conflicts in 15 files (14 news index HTML files + sitemap.xml) that needed to be resolved by taking the current main branch versions.

Changes

news/index*.html (14 files) — replaced with origin/main versions for all language variants: en, ar, da, de, es, fi, fr, he, ja, ko, nl, no, sv, zh
sitemap.xml — replaced with origin/main version

All files were checked out directly via git checkout origin/main -- <files>, ensuring byte-for-byte parity with main before this branch's other changes are layered on top.

Original prompt

This section details on the original issue you should resolve

<issue_title>fix(i18n): Meta keywords not localized — English keywords used in all 14 language article variants</issue_title>
<issue_description>## 📋 Issue Type
Bug Fix / i18n Quality

🎯 Objective

Fix meta keywords in all news articles to be localized per language instead of using English-only keywords across all 14 language variants. Currently, Japanese, Arabic, Chinese, Korean, and other non-English articles all have identical English meta keywords, hurting SEO and article quality.

📊 Current State — Evidence from Recent PRs

Affected PRs: #480 (committee reports, merged), #482 (propositions, merged), #483 (motions, open), #485 (breaking news, open)

Example — Committee Reports (all 14 language files have identical English keywords):

<meta name="keywords" content="committee, reports, betänkanden, parliament, committees, reports, Swedish Parliament, Riksdag, politics, Sweden">


<meta name="keywords" content="committee, reports, betänkanden, parliament, committees, reports, Swedish Parliament, Riksdag, politics, Sweden">
Additional bug: Duplicate keyword "reports" appears twice in committee reports keywords.

Root cause: scripts/article-template.ts line ~318:
<meta name="keywords" content="${keywords.join(', ')}">
Keywords are passed in as a static English array from the news-type generators and never localized.

Affected article types: All types — committee-reports, government-propositions, opposition-motions, breaking-news, evening-analysis, week-ahead, month-ahead, weekly-review, monthly-review.

🚀 Desired State

Localize meta keywords for each language variant. For example:

ja: "委員会, 報告, 議会, スウェーデン議会, 政治, スウェーデン"

ar: "لجنة, تقارير, برلمان, البرلمان السويدي, سياسة, السويد"

de: "Ausschuss, Berichte, Parlament, Schwedisches Parlament, Riksdag, Politik, Schweden"

Remove duplicate keywords (e.g., "reports" appearing twice)

Add keyword localization map to the LABELS object in data-transformers.ts or a dedicated keyword translation module

🔧 Implementation Approach

Add keyword translations to LABELS object in scripts/data-transformers.ts for all 14 languages

Update scripts/article-template.ts to accept lang parameter and look up localized keywords

Update all news-type generators (scripts/news-types/*.ts) to pass language-aware keywords

De-duplicate any existing keyword arrays

Run npx vitest run to verify

🤖 Recommended Agent

code-quality-engineer — systematic fix across article template and all news-type generators

✅ Acceptance Criteria

Meta keywords are localized for all 14 languages in generated articles

No duplicate keywords in any article

Keywords include topic-specific terms (not just generic "parliament, politics")

Existing tests pass (npx vitest run)

At least core languages (EN, SV, DE, FR, ES, JA, AR, ZH, KO) have keyword translations

📚 References

Template: scripts/article-template.ts line ~318

LABELS: scripts/data-transformers.ts lines 400–1400

Example bad output: news/2026-02-24-committee-reports-ja.html — English keywords in Japanese article

PR evidence: 📋 Committee Reports: Energy and Migration Dominate Riksdag Agenda (2026-02-24) #480 (merged), feat(news): Government propositions analysis — 2026-02-24 (14 languages) #482 (merged), feat(news): Opposition Motions Analysis – 24 February 2026 #483, 🔴 Breaking: Sweden files five propositions on civilian defence, psychological violence — 2026-02-24 #485 (open)</issue_description>

<agent_instructions>Fix meta keywords localization in all news articles.

Key tasks:

Add keyword translations to the LABELS object in scripts/data-transformers.ts for all 14 languages. Create language-specific keyword sets for each article type (committee-reports, government-propositions, opposition-motions, etc.).

Update scripts/article-template.ts to use localized keywords based on the lang parameter.

Remove duplicate keywords (e.g., "reports" appears twice in committee reports).

Update all news-type generators in scripts/news-types/*.ts to pass language-aware keywords.

Run npx vitest run to verify all tests pass.

The current code at scripts/article-template.ts line ~318 just does keywords.join(', ') without any localization.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes fix(i18n): Meta keywords not localized — English keywords used in all 14 language article variants #488

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: pethers <[email protected]>

github-actions · 2026-02-24T11:50:09Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot

Pull request overview

This PR fixes a critical i18n bug where all 14 language variants of articles were emitting identical English <meta name="keywords"> tags, negatively impacting SEO for non-English content. The fix introduces comprehensive keyword localization infrastructure and eliminates a duplicate "reports" keyword that appeared in committee-reports articles.

Changes:

Added ARTICLE_TYPE_KEYWORDS map with localized keywords for all 8 article types across all 14 supported languages
Added COMMON_KEYWORDS map for localized shared footer keywords (e.g., "Swedish Parliament", "Riksdag", "politics", "Sweden")
Refactored generateMetadata to look up keywords by language with English fallback, replacing hardcoded English keyword pushes in the switch statement
Implemented Set-based deduplication to eliminate duplicate keywords

Copilot · 2026-02-24T11:51:42Z

scripts/data-transformers.ts

+    zh: ['议会', '下周展望', '日历', '活动'],
+  },
+  'committee-reports': {
+    en: ['committee', 'committee reports', 'parliament', 'legislation'],


The keyword 'committee reports' contains a space, making it a multi-word keyword. Consider whether this should be two separate keywords 'committee' and 'reports' for consistency with other entries, or if the multi-word phrase is intentional for SEO purposes.

Split into two separate single-word entries 'committee' and 'reports' (commit 605bfea), consistent with all other keyword arrays in the map.

Copilot · 2026-02-24T11:51:42Z

scripts/data-transformers.ts

+  const typeKw: string[] =
+    ARTICLE_TYPE_KEYWORDS[type]?.[lang] ??
+    ARTICLE_TYPE_KEYWORDS[type]?.en ??
+    [];


The nested optional chaining with double fallback could be simplified using a helper function to improve readability, especially since this pattern is repeated for both typeKw and commonKw.

Extracted a getLocalizedKeywords(map, type, lang) helper (commit 605bfea) that encapsulates the double-fallback. The typeKw lookup now becomes a single call; COMMON_KEYWORDS uses a compact inline expression since it has a different shape.

pethers · 2026-02-24T11:55:14Z

@copilot apply changes based on the comments in this thread

github-actions · 2026-02-24T11:56:28Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Co-authored-by: pethers <[email protected]>

github-actions · 2026-02-24T12:30:49Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Copilot · 2026-02-24T12:33:02Z

scripts/data-transformers.ts

+  // Deduplicate while preserving insertion order (also eliminates duplicate 'reports'
+  // that previously arose when data.reports was present alongside the committee-reports type)


The deduplication comment mentions the 'reports' duplication fix, but there's still an i18n issue: lines 3280 and 3283 (just above the changed region) add English-only keywords ('calendar', 'events', 'debates', 'committees', 'reports') when data.events or data.reports are present. These will appear in all 14 language variants. Consider creating a DATA_DRIVEN_KEYWORDS map similar to ARTICLE_TYPE_KEYWORDS to localize these data-driven keywords as well.

Added DATA_DRIVEN_KEYWORDS map (commit e8e344f) with entries for 'events' (calendar/events/debates) and 'reports' (committees/reports) in all 14 languages. The if (data.events) and if (data.reports) blocks now call getLocalizedKeywords(DATA_DRIVEN_KEYWORDS, …, lang) instead of pushing English strings directly.

pethers · 2026-02-24T12:35:58Z

@copilot apply changes based on the comments in this thread

…EYWORDS map Co-authored-by: pethers <[email protected]>

github-actions · 2026-02-24T14:13:46Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.

…sitemap and news from main Co-authored-by: pethers <[email protected]>

…s, tests, index_ja.html, sitemap.xml Co-authored-by: pethers <[email protected]>

pethers · 2026-02-24T20:48:47Z

@copilot — This PR has merge conflicts in 15 files that must all be resolved before it can merge. For each conflicting file listed below, resolve the conflict by taking the version from main since these are all generated output files:

news/index.html
news/index_ar.html
news/index_da.html
news/index_de.html
news/index_es.html
news/index_fi.html
news/index_fr.html
news/index_he.html
news/index_ja.html
news/index_ko.html
news/index_nl.html
news/index_no.html
news/index_sv.html
news/index_zh.html
sitemap.xml
For each file: read the current version from main branch and update the file on this branch (copilot/fix-meta-keywords-localization) to match main's version exactly. Do this for ALL 15 files — do not stop after a few.

…nd sitemap.xml Co-authored-by: pethers <[email protected]>

Copilot · 2026-02-24T21:11:22Z

@copilot — This PR has merge conflicts in 15 files that must all be resolved before it can merge. For each conflicting file listed below, resolve the conflict by taking the version...

All 15 files have been resolved by taking the main branch versions — done in commit b4720f0. Running git diff origin/main against all 15 files produces no output, confirming byte-for-byte parity with main.

github-actions · 2026-02-24T21:29:32Z

🔍 Lighthouse Performance Audit

Category	Score	Status
Performance	85/100	🟡
Accessibility	95/100	🟢
Best Practices	90/100	🟢
SEO	95/100	🟢

📥 Download full Lighthouse report

Budget Compliance: Performance budgets enforced via budget.json

Copilot AI assigned Copilot and pethers Feb 24, 2026

Copilot started work on behalf of pethers February 24, 2026 11:18 View session

Copilot AI mentioned this pull request Feb 24, 2026

Investigate poor quality news articles: create 4 issues with root cause analysis #486

Merged

fix(i18n): Localize meta keywords for all 14 language article variants

d21f431

Co-authored-by: pethers <[email protected]>

Copilot AI changed the title ~~[WIP] Fix localized meta keywords for all language articles~~ fix(i18n): Localize meta keywords for all 14 language article variants Feb 24, 2026

Copilot finished work on behalf of pethers February 24, 2026 11:27

pethers requested a review from Copilot February 24, 2026 11:48

github-actions bot added deployment Deployment configuration refactor Code refactoring size-xl Extra large change (> 1000 lines) labels Feb 24, 2026

Copilot started reviewing on behalf of pethers February 24, 2026 11:49 View session

Copilot AI reviewed Feb 24, 2026

View reviewed changes

Merge branch 'main' into copilot/fix-meta-keywords-localization

96915f8

github-actions bot added the size-m Medium change (50-250 lines) label Feb 24, 2026

Copilot started work on behalf of pethers February 24, 2026 11:56 View session

fix: split multi-word keyword and extract getLocalizedKeywords helper

605bfea

Co-authored-by: pethers <[email protected]>

Copilot finished work on behalf of pethers February 24, 2026 12:00

pethers requested a review from Copilot February 24, 2026 12:28

Copilot started reviewing on behalf of pethers February 24, 2026 12:29 View session

Copilot AI reviewed Feb 24, 2026

View reviewed changes

Copilot started work on behalf of pethers February 24, 2026 13:01 View session

fix: localize data-driven keywords (events/reports) via DATA_DRIVEN_K…

e8e344f

…EYWORDS map Co-authored-by: pethers <[email protected]>

Copilot finished work on behalf of pethers February 24, 2026 13:05

pethers requested a review from Copilot February 24, 2026 13:11

Copilot started reviewing on behalf of pethers February 24, 2026 14:12 View session

Copilot AI reviewed Feb 24, 2026

View reviewed changes

Copilot started work on behalf of pethers February 24, 2026 14:24 View session

fix(ts): cast type as ArticleType in getLocalizedKeywords call; sync …

f012e91

…sitemap and news from main Co-authored-by: pethers <[email protected]>

Copilot finished work on behalf of pethers February 24, 2026 14:32

github-actions bot added html-css HTML/CSS changes i18n Internationalization/localization translation Translation updates rtl RTL language support (Arabic, Hebrew) news News articles and content generation labels Feb 24, 2026

Copilot started work on behalf of pethers February 24, 2026 15:39 View session

sync: resolve conflicts by taking origin/main for data-transformers.t…

4957502

…s, tests, index_ja.html, sitemap.xml Co-authored-by: pethers <[email protected]>

Copilot AI changed the title ~~fix(i18n): Localize meta keywords for all 14 language article variants~~ sync: resolve merge conflicts and TS2345 in data-transformers.ts Feb 24, 2026

Copilot finished work on behalf of pethers February 24, 2026 15:44

Copilot started work on behalf of pethers February 24, 2026 16:31 View session

Copilot finished work on behalf of pethers February 24, 2026 17:31

Copilot started work on behalf of pethers February 24, 2026 20:23 View session

sync: copy main branch versions of all conflicting news index files a…

b4720f0

…nd sitemap.xml Co-authored-by: pethers <[email protected]>

Copilot AI changed the title ~~sync: resolve merge conflicts and TS2345 in data-transformers.ts~~ sync: resolve conflicts by taking main branch for all news indexes and sitemap Feb 24, 2026

Copilot finished work on behalf of pethers February 24, 2026 21:08

Copilot started work on behalf of pethers February 24, 2026 21:09 View session

Copilot finished work on behalf of pethers February 24, 2026 21:12

Merge branch 'main' into copilot/fix-meta-keywords-localization

dc3c751

github-actions bot added the size-xs Extra small change (< 10 lines) label Feb 24, 2026

pethers closed this Feb 24, 2026

pethers deleted the copilot/fix-meta-keywords-localization branch February 24, 2026 21:41

		// Deduplicate while preserving insertion order (also eliminates duplicate 'reports'
		// that previously arose when data.reports was present alongside the committee-reports type)

Comments

Conversation

Copilot AI commented Feb 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

🎯 Objective

📊 Current State — Evidence from Recent PRs

🚀 Desired State

🔧 Implementation Approach

🤖 Recommended Agent

✅ Acceptance Criteria

📚 References

Comments on the Issue (you are @copilot in this section)

Uh oh!

github-actions bot commented Feb 24, 2026

🔍 Lighthouse Performance Audit

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

pethers commented Feb 24, 2026

Uh oh!

github-actions bot commented Feb 24, 2026

🔍 Lighthouse Performance Audit

Uh oh!

github-actions bot commented Feb 24, 2026

🔍 Lighthouse Performance Audit

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

pethers commented Feb 24, 2026

Uh oh!

github-actions bot commented Feb 24, 2026

🔍 Lighthouse Performance Audit

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

pethers commented Feb 24, 2026

Uh oh!

Copilot AI commented Feb 24, 2026

Uh oh!

github-actions bot commented Feb 24, 2026

🔍 Lighthouse Performance Audit

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Feb 24, 2026 •

edited

Loading