Skip to content

Regenerate all article.md and HTML output for all 14 languages + fix HTML validation#2195

Merged
pethers merged 3 commits intomainfrom
copilot/improve-article-generation
May 2, 2026
Merged

Regenerate all article.md and HTML output for all 14 languages + fix HTML validation#2195
pethers merged 3 commits intomainfrom
copilot/improve-article-generation

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 2, 2026 •
edited
Loading

Re-runs the full article generation pipeline against all existing analysis artifacts following improvements to aggregate-analysis.ts and render-articles.ts, and fixes a pre-existing HTML validation bug.

Changes

  • article.md regeneration — aggregate-analysis --all re-aggregated 74 analysis subfolders into fresh article.md files under analysis/daily/
  • HTML regeneration — render-articles --all --lang all rendered 1,036 HTML files (74 subfolders × 14 languages: en, sv, da, no, fi, de, fr, es, nl, ar, he, ja, ko, zh)
  • HTML validation fix — Fixed a regex bug in scripts/normalize-static-html-chrome.ts where the hero-stat label regex used a lazy [\s\S]*? that stopped at the first </span> (the inner icon span), leaving duplicate label text and unpaired </span> tags. This caused 70 tag-pair HTMLHint errors across all 14 index*.html files. The simplified regex now matches from the number span through the </div> boundary and rebuilds the label cleanly.
  • All 14 root-level index*.html files corrected — zero HTMLHint errors across all 3,761 scanned files

@pethers pethers marked this pull request as ready for review May 2, 2026 23:24
Copilot AI review requested due to automatic review settings May 2, 2026 23:24
@github-actions github-actions Bot added documentation Documentation updates html-css HTML/CSS changes translation Translation updates rtl RTL language support (Arabic, Hebrew) news News articles and content generation labels May 2, 2026
Copilot AI reviewed May 2, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@github-actions github-actions Bot added the size-xl Extra large change (> 1000 lines) label May 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

🏷️ Automatic Labeling Summary

This PR has been automatically labeled based on the files changed and PR metadata.

Applied Labels: documentation,html-css,translation,rtl,size-xl,news

Label Categories

  • 🗳️ Content: news, dashboard, visualization, intelligence
  • 💻 Technology: html-css, javascript, workflow, security
  • 📊 Data: cia-data, riksdag-data, data-pipeline, schema
  • 🌍 I18n: i18n, translation, rtl
  • 🔒 ISMS: isms, iso-27001, nist-csf, cis-controls
  • 🏗️ Infrastructure: ci-cd, deployment, performance, monitoring
  • 🔄 Quality: testing, accessibility, documentation, refactor
  • 🤖 AI: agent, skill, agentic-workflow

For more information, see .github/labeler.yml.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

🔍 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

@pethers
Copy link
Copy Markdown
Member

pethers commented May 2, 2026

@copilot analyse html validation, errors do we need to fix html generation to output better/correct html and regenerate html ?

See Quality Checks / html-validation (pull_request)
Quality Checks / html-validation (pull_request)Failing after 1m

@github-actions github-actions Bot added i18n Internationalization/localization refactor Code refactoring labels May 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

🔍 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

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 2, 2026

@copilot analyse html validation, errors do we need to fix html generation to output better/correct html and regenerate html ?

See [Quality Checks / html-validation (pull_request)...

Analysed the CI failure — all 70 errors were tag-pair violations in the 14 root-level index*.html files (not in news/). The bug was in scripts/normalize-static-html-chrome.ts: the hero-stat label regex used a lazy [\s\S]*? that stopped at the first </span> (the inner icon span), leaving the old label text and an unpaired </span> dangling after each run.

Fixed in e2b1ee4:

  • Simplified the regex to match from the number span through the </div> boundary, then rebuild the label cleanly
  • Fixed all 14 index HTML files — now 0 htmlhint errors across all 3,761 files
  • Verified the fix is idempotent (re-running normalize produces no new errors)

Copilot AI changed the title Regenerate all article.md and HTML output for all 14 languages Regenerate all article.md and HTML output for all 14 languages + fix HTML validation May 2, 2026
Copilot finished work on behalf of pethers May 2, 2026 23:45
Copilot AI requested a review from pethers May 2, 2026 23:45
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

🔍 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

@pethers pethers merged commit f3690f8 into main May 2, 2026
24 checks passed
@pethers pethers deleted the copilot/improve-article-generation branch May 2, 2026 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@pethers pethers Awaiting requested review from pethers

Assignees

@pethers pethers

Copilot code review Copilot

Labels

documentation Documentation updates html-css HTML/CSS changes i18n Internationalization/localization news News articles and content generation refactor Code refactoring rtl RTL language support (Arabic, Hebrew) size-xl Extra large change (> 1000 lines) translation Translation updates

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pethers