Skip to content

Document the article generation pipeline#1992

Merged
pethers merged 5 commits intomainfrom
copilot/create-article-generation-docs
Apr 25, 2026
Merged

Document the article generation pipeline#1992
pethers merged 5 commits intomainfrom
copilot/create-article-generation-docs

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 25, 2026 •
edited
Loading

  • Inspect PR review comment and relevant Article-Generation.md section
  • Confirm aggregator exclusion behavior in scripts/render-lib/aggregator.ts
  • Update the Aggregator input/output table to state README.md, article.md, and article.<lang>.md exclusions explicitly
  • Add a concise note that README.md is gate-required but not published in article.md
  • Run targeted markdown/whitespace validation
  • Commit and push the documentation fix
  • Reply to the new PR comment with the addressing commit hash

Copilot AI and others added 4 commits April 25, 2026 08:00
@pethers
Add article generation documentation
1973c22 
Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/1e00f029-b486-441b-b7b4-6eace6944b58

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
@pethers
Refine article generation documentation
35a6b29 
Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/1e00f029-b486-441b-b7b4-6eace6944b58

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
@pethers
Polish article generation documentation wording
27e0327 
Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/1e00f029-b486-441b-b7b4-6eace6944b58

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
@pethers
Address final documentation review comments
3a6610d 
Agent-Logs-Url: https://github.com/Hack23/riksdagsmonitor/sessions/1e00f029-b486-441b-b7b4-6eace6944b58

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
@github-actions github-actions Bot added documentation Documentation updates size-l Large change (250-1000 lines) labels Apr 25, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2026

🏷️ Automatic Labeling Summary

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

Applied Labels: documentation,size-l

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.

@pethers pethers marked this pull request as ready for review April 25, 2026 08:10
Copilot AI review requested due to automatic review settings April 25, 2026 08:10
@github-actions
Copy link
Copy Markdown
Contributor

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

Copilot AI reviewed Apr 25, 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.

Pull request overview

Adds a new, end-to-end documentation artifact describing how Riksdagsmonitor turns analysis artifacts into published multilingual news articles, using the 2026-04-24 interpellations run as the worked example.

Changes:

  • Introduces Article-Generation.md documenting the agentic workflow → analysis artifacts → aggregation (article.md) → rendering (sanitized HTML) → deployment (Vite + S3/CloudFront) pipeline.
  • Details the 23-artifact analysis contract, analysis gate, provenance model (sources footer + JSON-LD isBasedOn), and multilingual rendering/translation approach.
  • Documents UI/rendering constraints (Mermaid supported; Chart.js/D3 via trusted code + JSON artifacts, not inline scripts).

Comment thread Article-Generation.md Outdated
Comment on lines +415 to +418
| `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/*.md` | `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/article.md` |
| `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/documents/*.md` | Included under `## Per-document intelligence` |
| Supplementary `.md` files | Appended after the canonical sequence |

Copy link

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

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

The “Aggregator input and output” table implies all analysis/daily/$ARTICLE_DATE/$SUBFOLDER/*.md files feed into article.md, but the current aggregator intentionally excludes README.md (and article.md / article.<lang>.md) from aggregation. Consider adjusting the table to avoid implying that README.md becomes part of the published article, or add an explicit note that README.md is required for the 23-artifact gate but is not included in article.md output.

Suggested change
| `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/*.md` | `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/article.md` |
| `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/documents/*.md` | Included under `## Per-document intelligence` |
| Supplementary `.md` files | Appended after the canonical sequence |
| Canonical analysis `.md` files in `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/` (excluding `README.md`, `article.md`, and `article.<lang>.md`) | `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/article.md` |
| `analysis/daily/$ARTICLE_DATE/$SUBFOLDER/documents/*.md` | Included under `## Per-document intelligence` |
| Supplementary `.md` files in the subfolder, excluding `README.md`, `article.md`, and `article.<lang>.md` | Appended after the canonical sequence |
> **Note:** `README.md` is required for the 23-artifact analysis gate and repository readability, but it is intentionally not aggregated into the published `article.md`. Existing `article.md` and `article.<lang>.md` files are also excluded from aggregation.

Copilot uses AI. Check for mistakes.
@pethers
Copy link
Copy Markdown
Member

pethers commented Apr 25, 2026

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 25, 2026

@copilot apply changes based on the comments in this thread

Applied the requested documentation change in 2f670d1. The aggregator table now explicitly excludes README.md, article.md, and article.<lang>.md, and notes that README.md is gate-required but not published into article.md. No UI screenshot applicable; this is a documentation-only change.

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

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

Copilot AI requested a review from pethers April 25, 2026 08:38
@pethers pethers merged commit 256b9d4 into main Apr 25, 2026
12 checks passed
@pethers pethers deleted the copilot/create-article-generation-docs branch April 25, 2026 08:39
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 size-l Large change (250-1000 lines) size-xl Extra large change (> 1000 lines)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pethers