Skip to content

Comments

Fix inconsistent article type classification for weekend articles across all 14 language indexes#439

Merged
pethers merged 2 commits intonews-realtime/2026-02-22-weekend-analysis-751b87f663d6aa99from
copilot/sub-pr-436
Feb 22, 2026
Merged

Fix inconsistent article type classification for weekend articles across all 14 language indexes#439
pethers merged 2 commits intonews-realtime/2026-02-22-weekend-analysis-751b87f663d6aa99from
copilot/sub-pr-436

Conversation

Copy link
Contributor

Copilot AI commented Feb 22, 2026

classifyArticleType() keyword-matched "analysis" (and translations like "analys", "分析") from article body content in 7 language versions of the weekend-security-preparedness article, overriding the intended prospective classification. The other 7 languages happened not to trigger the analysis keywords and classified correctly.

Changes

  • scripts/generate-news-indexes.ts: Added fileName.includes('weekend') to the prospective filename guard — evaluated before content keyword scanning — so all weekend-* articles are classified consistently:
    if (fileName.includes('week-ahead') || fileName.includes('weekend') || prospectiveKeywords.some(...))
      return 'prospective';
  • 7 news index files (index.html, index_es, index_fi, index_fr, index_ko, index_nl, index_zh): Updated "type": "analysis""type": "prospective" for the 2026-02-22-weekend-security-preparedness article to align with the 7 language versions already correct.

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI changed the title [WIP] Update weekend analysis on Sweden's security measures Fix inconsistent article type classification for weekend articles across all 14 language indexes Feb 22, 2026
Copilot AI requested a review from pethers February 22, 2026 16:09
@pethers pethers marked this pull request as ready for review February 22, 2026 16:16
@pethers pethers merged commit 55254b1 into news-realtime/2026-02-22-weekend-analysis-751b87f663d6aa99 Feb 22, 2026
@pethers pethers deleted the copilot/sub-pr-436 branch February 22, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants