Improve article intelligence projection and generated article UX by Copilot · Pull Request #1994 · Hack23/riksdagsmonitor

Copilot · 2026-04-25T09:27:25Z

Fix Reader Intelligence Guide table headers (Reader need / What you'll get / Source artifact) — column 2 now matches the readerValue content
Replace bespoke anchorForTitle slugger with github-slugger (the same library rehype-slug uses) so guide anchors stay in sync with rendered heading IDs across punctuation/Unicode/duplicate cases
Replace rehype-slug with a custom rehypeSlugWithPrefix plugin that pre-prefixes heading IDs with rm-, keep rehype-sanitize's DOM-clobbering mitigation enabled (clobberPrefix: 'rm-', id dropped from clobber to avoid double-prefix), and align rehype-autolink-headings hrefs and the aggregator's Reader Intelligence Guide anchors to the same prefix
Translate article trust badges and aria-label across all 14 languages via LANGUAGE_META[lang].translations.articleTrust*
Wrap rendered <table> elements in <div class="rm-table-wrap"> via a custom rehype plugin and remove display: block from .rm-article-body table to preserve native table layout/semantics for assistive tech; idempotence guard accepts both string and string[] HAST className shapes so raw-HTML wrappers are not double-wrapped
Scope @media (prefers-color-scheme: light) rules to html:not([data-theme]) so an explicit data-theme="light" / data-theme="dark" always wins over the OS preference
Add github-slugger@^2.0.0 and hast-util-to-string to direct dependencies (no advisory issues)
Regenerate 25 analysis/daily/*/article.md aggregations and 50 EN/SV HTML articles to reflect the new prefixed slugs, table wrappers and translated badges
Verify build and 2132 tests pass

Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/8f235d03-a6f3-452d-ba7f-9142163c1167 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

github-actions · 2026-04-25T09:28:03Z

🏷️ Automatic Labeling Summary

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

Applied Labels: documentation,html-css,translation,testing,refactor,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 · 2026-04-25T09:28:37Z

🔍 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

…nts roadmap Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/19038edc-3385-422b-ac60-96e9a8643afc Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

github-actions · 2026-04-25T09:36:04Z

🔍 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 updates the article aggregation + rendering pipeline to better project “reader-intelligence-first” narratives and improve the generated article UX, while keeping provenance traceable.

Changes:

Reorders AGGREGATION_ORDER and injects a deterministic Reader Intelligence Guide into aggregated article.md.
Enhances rendered HTML article headers with a dek (description) and trust/provenance badges, plus CSS updates for the new layout.
Tightens provenance/source enumeration by excluding generated article*.md and pass1/ snapshots from artifact lists; updates docs/prompts accordingly.

Reviewed changes

Copilot reviewed 21 out of 97 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/render-lib.test.ts	Updates aggregation-order expectations and adds coverage for reader guide text, link-stripping in metadata extraction, and dek rendering.
tests/render-lib-architecture.test.ts	Adjusts architecture assertions to match new aggregation ordering and new header CSS hooks.
styles.css	Adds styling for article dek + trust badges, refines article container, and light-scheme overrides.
scripts/render-lib/article.ts	Renders the dek and trust/provenance badges inside the article header.
scripts/render-lib/aggregator.ts	Re-documents and changes aggregation order; adds Reader Intelligence Guide generation and improved inline-markdown→text cleaning.
scripts/render-articles.ts	Excludes `pass1/` and generated `article*.md` from artifact enumeration used for source footers / JSON-LD `isBasedOn`.
political-intelligence.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_sv.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_da.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_no.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_fi.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_de.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_fr.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_es.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_nl.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_ar.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_he.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_ja.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_ko.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
political-intelligence_zh.html	Regenerates collection page (counts + new stream listing + updated modified timestamps).
analysis/daily/2026-04-21/realtime-1353/article.md	Shows the new Reader Intelligence Guide insertion and reordered sections in a real generated article.
Article-Generation.md	Updates narrative-order documentation, fixes description example, and adds the “Future Improvements Roadmap” section + ToC entry.
.github/prompts/06-article-generation.md	Updates workflow prompt contract to include reader guide insertion, new ordering, and refined source enumeration rules.

Copilot · 2026-04-25T09:43:05Z

+    '## Reader Intelligence Guide',
+    '',
+    'Use this guide to read the article as a political-intelligence product rather than a raw artifact dump. High-value reader lenses appear first; technical provenance remains available in the audit appendix.',
+    '',
+    '| Reader need | Where to go | Source artifact |',
+    '|---|---|---|',
+    ...entries,


The Reader Intelligence Guide table header doesn’t match the row content: the second column currently contains the “readerValue” (what the reader gets), not “Where to go”. This is confusing in the rendered article—either rename the header to reflect the content (e.g. “What you’ll get”) or change the row layout so the “Where to go” column is the anchor link and the first column is the reader need label.

Copilot · 2026-04-25T09:43:05Z

+function anchorForTitle(title: string): string {
+  return title
+    .toLowerCase()
+    .normalize('NFKD')
+    .replace(/[^\p{L}\p{N}\s-]/gu, '')
+    .trim()
+    .replace(/\s+/g, '-');
+}


anchorForTitle re-implements heading slug generation, but the actual HTML IDs are produced by rehype-slug (GitHub slugger). If these algorithms diverge (punctuation/Unicode/duplicate-heading suffixes), the Reader Intelligence Guide links will be broken. Consider using the same slugger library as rehype-slug, or emit explicit stable IDs for section headings (e.g. based on the artifact filename) and link to those.

Copilot · 2026-04-25T09:43:05Z

+          <ul class="rm-article-trust-badges" aria-label="Article provenance and quality controls">
+            <li>Public sources</li>
+            <li>AI-FIRST review</li>
+            <li>Traceable artifacts</li>
+          </ul>


The new trust-badge copy is hard-coded in English (and so is the aria-label). Since the site renders 14 languages and already has a LANGUAGE_META[lang].translations mechanism in chrome, these strings should be sourced from translations as well; otherwise non-English article pages will show English badges in the header.

Copilot · 2026-04-25T09:43:05Z

+  display: block;
+  overflow-x: auto;
+  -webkit-overflow-scrolling: touch;
+  border: 1px solid color-mix(in srgb, var(--primary-cyan, #00d9ff) 12%, transparent);
+  border-radius: 8px;


Setting .rm-article-body table { display: block; } can break table layout/column sizing and may degrade table semantics for assistive tech. A more robust pattern is to keep the table’s native display and wrap it in a scroll container (or apply overflow to a wrapper element) so responsiveness doesn’t sacrifice table behavior.

pethers · 2026-04-25T09:43:39Z

@copilot apply changes based on the comments in this thread

…le headers Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/a3b09f19-a6f7-4c22-acd8-1507e8b70dde Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Copilot · 2026-04-25T09:54:54Z

@copilot apply changes based on the comments in this thread

Applied all four review points in 36414d7:

Reader Intelligence Guide column header is now What you'll get (matches the readerValue content) and the link in column 1 is the reader-need.
anchorForTitle now delegates to github-slugger (the same library rehype-slug uses) and rehype-sanitize's clobberPrefix is set to '' so the rendered heading IDs match the guide anchors exactly across punctuation/Unicode.
Trust badges and aria-label are sourced from LANGUAGE_META[lang].translations.articleTrust* for all 14 languages (fr/ar correctly placed after fixing a regex that broke on 'Pages d\'index…').
Tables are now wrapped in <div class="rm-table-wrap"> via a small rehype plugin; display: block is removed from .rm-article-body table so the native table layout and a11y semantics are preserved while the wrapper handles horizontal overflow.

All 25 aggregations and 50 EN/SV HTML articles regenerated; 2132 tests pass.

github-actions · 2026-04-25T09:55:44Z

🔍 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 21 out of 100 changed files in this pull request and generated 3 comments.

Copilot · 2026-04-25T10:02:45Z

+  // Disable rehype-sanitize's default `user-content-` ID prefix so heading
+  // anchors generated by `rehype-slug` (and matched by the aggregator's
+  // Reader Intelligence Guide via `github-slugger`) resolve as written.
+  // The aggregator owns the entire markdown body — there is no untrusted
+  // user content here that could weaponise DOM-clobbering on element IDs.
+  clobberPrefix: '',
  attributes: {


Setting clobberPrefix: '' disables the DOM-clobbering mitigation in rehype-sanitize. Even if content is “repo-owned”, headings/IDs still come from AI-generated artifacts and can be influenced by external inputs; keeping the prefix (or another non-empty prefix) is the safer default. Consider restoring a non-empty clobberPrefix and updating the Reader Intelligence Guide anchor generation to include that prefix so links still resolve.

Copilot · 2026-04-25T10:02:45Z

+      // Skip if already wrapped (idempotent).
+      if (
+        parent.type === 'element' &&
+        (parent as Element).tagName === 'div' &&
+        Array.isArray((parent as Element).properties?.className) &&
+        ((parent as Element).properties!.className as string[]).includes('rm-table-wrap')
+      ) {
+        return;
+      }


rehypeWrapTables() checks properties.className only when it is an array. If a table is already wrapped via raw HTML (<div class="rm-table-wrap">), className can be a string in HAST, and this will fail the idempotence check and double-wrap tables. Consider accepting both string and string[] forms when detecting an existing wrapper.

Copilot · 2026-04-25T10:02:46Z

+@media (prefers-color-scheme: light) {
+  html:not([data-theme="dark"]) .rm-article-body,
+  html:not([data-theme="dark"]) body.rm-article-body {
+    color: var(--text-color, #1a1a1a);
+    background: var(--bg-color, #f8f9fa);
+  }
+
+  html:not([data-theme="dark"]) .rm-article {
+    background: var(--card-bg, #ffffff);
+    border-color: var(--card-border, #dee2e6);
+    box-shadow: 0 16px 48px rgba(0, 0, 0, 0.08);
+  }
+
+  html:not([data-theme="dark"]) .rm-article-body p,
+  html:not([data-theme="dark"]) .rm-article-body li,
+  html:not([data-theme="dark"]) .rm-article-dek {
+    color: var(--text-color, #1a1a1a);
+  }
+
+  html:not([data-theme="dark"]) .rm-article-body strong { color: #000000; }
+  html:not([data-theme="dark"]) .rm-article-body a { color: var(--link-color, #007744); }
+  html:not([data-theme="dark"]) .rm-article-body a:hover { color: var(--primary-color, #006633); }
+
+  html:not([data-theme="dark"]) .rm-site-header,
+  html:not([data-theme="dark"]) .rm-site-footer {
+    background: #ffffff;
+    color: var(--text-secondary, #4a4a4a);
+  }
+
+  html:not([data-theme="dark"]) .rm-site-header {
+    border-bottom-color: var(--card-border, #dee2e6);
+  }


The new @media (prefers-color-scheme: light) rules target html:not([data-theme="dark"]) ..., which also matches html[data-theme="light"]. Because these rules come after the existing html[data-theme="light"] block, they override explicit light-theme styling (e.g. .rm-site-header background gradient becomes plain white) whenever the OS prefers light. If the intent is only an OS-level fallback, scope these selectors to html:not([data-theme]) (so manual theme choice always wins), or align the overridden values with the existing html[data-theme="light"] rules.

pethers · 2026-04-25T10:04:28Z

@copilot apply changes based on the comments in this thread

…string className, scope OS dark/light to no-explicit-theme Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/6e4d9e98-5993-43a2-8aaa-a8fca0d66ffb Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Copilot · 2026-04-25T10:12:13Z

@copilot apply changes based on the comments in this thread

Applied all three points in d467d92:

Restored a non-empty clobberPrefix (rm-). Heading IDs are now pre-prefixed by a custom rehypeSlugWithPrefix plugin (uses github-slugger, mirrors rehype-slug), and id is dropped from the sanitiser's clobber list so it doesn't double-prefix. name and aria-attribute clobbering protection is preserved, autolink-headings hrefs (href="#rm-…") match the rendered IDs, and the Reader Intelligence Guide stays in sync via ${HEADING_ID_PREFIX}${slug} in the aggregator.
rehypeWrapTables idempotence check now accepts both string and string[] HAST className shapes, so raw-HTML <div class="rm-table-wrap"> wrappers no longer get double-wrapped.
@media (prefers-color-scheme: light) rules are now scoped to html:not([data-theme]), so an explicit data-theme="light" or data-theme="dark" always wins over the OS preference.

25 aggregations + 50 EN/SV articles regenerated; build, lint and 2132 tests pass.

github-actions · 2026-04-25T10:13:08Z

🔍 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 and others added 2 commits April 25, 2026 09:08

Improve article intelligence aggregation and chrome

051e68f

Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/8f235d03-a6f3-452d-ba7f-9142163c1167 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Regenerate article outputs with reader-first projection

72bf1cd

Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/8f235d03-a6f3-452d-ba7f-9142163c1167 Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Copilot AI assigned Copilot and pethers Apr 25, 2026

Copilot created this pull request from a session on behalf of pethers April 25, 2026 09:27 View session

github-actions Bot added documentation Documentation updates html-css HTML/CSS changes translation Translation updates testing Test coverage refactor Code refactoring news News articles and content generation size-xl Extra large change (> 1000 lines) labels Apr 25, 2026

Copilot started work on behalf of pethers April 25, 2026 09:29 View session

Update Article-Generation.md with current status and future improveme…

679bfd9

…nts roadmap Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/19038edc-3385-422b-ac60-96e9a8643afc Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Copilot finished work on behalf of pethers April 25, 2026 09:35

Copilot AI requested a review from pethers April 25, 2026 09:35

pethers marked this pull request as ready for review April 25, 2026 09:37

Copilot AI review requested due to automatic review settings April 25, 2026 09:37

Copilot started reviewing on behalf of pethers April 25, 2026 09:39 View session

Copilot AI reviewed Apr 25, 2026

View reviewed changes

Copilot started work on behalf of pethers April 25, 2026 09:43 View session

Address PR review: i18n trust badges, slug parity, table wrapper, tab…

36414d7

…le headers Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/a3b09f19-a6f7-4c22-acd8-1507e8b70dde Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

github-actions Bot added the dependencies Dependency updates label Apr 25, 2026

Copilot finished work on behalf of pethers April 25, 2026 09:55

pethers requested a review from Copilot April 25, 2026 09:56

Copilot started reviewing on behalf of pethers April 25, 2026 09:57 View session

Copilot AI reviewed Apr 25, 2026

View reviewed changes

Copilot started work on behalf of pethers April 25, 2026 10:04 View session

Copilot finished work on behalf of pethers April 25, 2026 10:13

pethers merged commit bea7119 into main Apr 25, 2026
27 checks passed

pethers deleted the copilot/improve-political-intelligence-quality branch April 25, 2026 10:18

Conversation

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

Uh oh!

Uh oh!

github-actions Bot commented Apr 25, 2026

🏷️ Automatic Labeling Summary

Label Categories

Uh oh!

github-actions Bot commented Apr 25, 2026

🔍 Lighthouse Performance Audit

Uh oh!

github-actions Bot commented Apr 25, 2026

🔍 Lighthouse Performance Audit

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

Uh oh!

pethers commented Apr 25, 2026

Uh oh!

Copilot AI commented Apr 25, 2026

Uh oh!

github-actions Bot commented Apr 25, 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 Apr 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

Uh oh!

pethers commented Apr 25, 2026

Uh oh!

Copilot AI commented Apr 25, 2026

Uh oh!

github-actions Bot commented Apr 25, 2026

🔍 Lighthouse Performance Audit

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Apr 25, 2026 •

edited

Loading