euparliamentmonitor 0.8.50 → 0.8.52

package/README.md

@@ -136,7 +136,7 @@ The published site is the audience-facing companion to this npm/TypeScript packa
 
 **MCP Server Integration**: The project uses the
 [European-Parliament-MCP-Server](https://github.com/Hack23/European-Parliament-MCP-Server)
-v1.2.15 for accessing real EU Parliament data via the Model Context Protocol.
+v1.2.18 for accessing real EU Parliament data via the Model Context Protocol.
 
 - **MCP Server Status**: ✅ Fully operational — 60+ EP data tools available
   (feeds, direct lookups, analytical tools, intelligence correlation)
@@ -426,7 +426,7 @@ import type { ArticleCategory, LanguageCode } from 'euparliamentmonitor/types';
 
 ## 🔌 Data Sources
 
-**Primary — European Parliament MCP Server** ([Hack23/European-Parliament-MCP-Server](https://github.com/Hack23/European-Parliament-MCP-Server) v1.2.15+, fully operational):
+**Primary — European Parliament MCP Server** ([Hack23/European-Parliament-MCP-Server](https://github.com/Hack23/European-Parliament-MCP-Server) v1.2.18+, fully operational):
 
 - 🗳️ Plenary sessions, voting records, roll-call votes
 - 📜 Adopted texts, motions, resolutions, urgency files

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "euparliamentmonitor",
-  "version": "0.8.50",
+  "version": "0.8.52",
   "type": "module",
   "description": "European Parliament Intelligence Platform - Monitor political activity with systematic transparency",
   "main": "scripts/index.js",
@@ -33,14 +33,6 @@
     "./generators/*": {
       "import": "./scripts/generators/*.js",
       "types": "./scripts/generators/*.d.ts"
-    },
-    "./generators/pipeline/*": {
-      "import": "./scripts/generators/pipeline/*.js",
-      "types": "./scripts/generators/pipeline/*.d.ts"
-    },
-    "./generators/strategies/*": {
-      "import": "./scripts/generators/strategies/*.js",
-      "types": "./scripts/generators/strategies/*.d.ts"
     }
   },
   "files": [
@@ -56,7 +48,8 @@
     "registry": "https://registry.npmjs.org/"
   },
   "scripts": {
-    "prebuild": "node scripts/generators/news-indexes.js && node scripts/generators/sitemap.js",
+    "prebuild": "node scripts/generators/build-info.js && node scripts/generators/news-indexes.js && node scripts/generators/sitemap.js",
+    "generate-build-info": "node scripts/generators/build-info.js",
     "build": "tsc",
     "build:check": "tsc --noEmit",
     "build:check-tests": "tsc --project tsconfig.test.json --noEmit",
@@ -84,6 +77,9 @@
     "test:e2e:report": "playwright show-report",
     "lint": "eslint src/",
     "lint:fix": "eslint src/ --fix",
+    "knip": "knip",
+    "knip:production": "knip --production",
+    "knip:fix": "knip --fix",
     "lint:report": "eslint src/ --format json --output-file builds/test-results/eslint-report.json",
     "lint:report:html": "eslint src/ --format html --output-file builds/test-results/eslint-report.html",
     "format": "prettier --write \"src/**/*.ts\"",
@@ -158,6 +154,7 @@
     "htmlhint": "1.9.2",
     "husky": "9.1.7",
     "jscpd": "4.0.9",
+    "knip": "^6.7.0",
     "lint-staged": "16.4.0",
     "mermaid": "11.14.0",
     "papaparse": "5.5.3",
@@ -172,7 +169,7 @@
     "node": ">=25"
   },
   "dependencies": {
-    "european-parliament-mcp-server": "1.2.15",
+    "european-parliament-mcp-server": "1.2.18",
     "markdown-it": "^14.1.1",
     "markdown-it-anchor": "^9.2.0",
     "markdown-it-attrs": "^4.3.1",

package/scripts/aggregator/analysis-aggregator.d.ts

@@ -1,27 +1,5 @@
 import { type ArtifactSection } from './artifact-order.js';
-import { type Manifest, type ManifestFiles as _ManifestFiles, type ManifestHistoryEntry as _ManifestHistoryEntry } from './manifest/index.js';
-/**
- * Raw manifest shape as committed by the analysis pipeline.
- *
- * @deprecated Use {@link Manifest} from `aggregator/manifest/index.js`.
- *   This alias is preserved for back-compat with the existing test suite
- *   and external curators that import `AnalysisManifest` from this module.
- */
-export type AnalysisManifest = Manifest;
-/**
- * `manifest.files` can be nested category → paths or flat path → description.
- *
- * @deprecated Use {@link _ManifestFiles} (`ManifestFiles`) from
- *   `aggregator/manifest/index.js`.
- */
-export type ManifestFiles = _ManifestFiles;
-/**
- * One entry in `manifest.history[]`; only fields we read are typed.
- *
- * @deprecated Use {@link _ManifestHistoryEntry} (`ManifestHistoryEntry`) from
- *   `aggregator/manifest/index.js`.
- */
-export type ManifestHistoryEntry = _ManifestHistoryEntry;
+import { type Manifest, type ManifestFiles } from './manifest/index.js';
 /** Result of {@link aggregateAnalysisRun}. */
 export interface AggregatedRun {
     /** Final Markdown document (provenance + sections + appendices). */
@@ -100,7 +78,7 @@ export declare function flattenManifestFiles(files: ManifestFiles | undefined):
  * @param manifest - Parsed manifest object
  * @returns The latest non-PENDING gate result, or `"PENDING"` when none found
  */
-export declare function latestGateResult(manifest: AnalysisManifest): string;
+export declare function latestGateResult(manifest: Manifest): string;
 /**
  * Expand an `artifacts` entry from {@link ArtifactSection} into a list of
  * concrete artifact paths. Exact paths are kept as-is; directory prefixes
@@ -174,7 +152,7 @@ export declare function renderAnalysisIndex(included: readonly IncludedArtifact[
  */
 export declare function renderReaderIntelligenceGuide(sections: readonly TocSection[], included: readonly IncludedArtifact[]): string;
 /**
- * Resolve the article-type slug from a manifest, tolerating legacy schemas.
+ * Resolve the article-type slug from a manifest, tolerating historic schemas.
  *
  * Thin re-export of {@link _resolveArticleType} from
  * `aggregator/manifest/index.js`. Resolution order: `articleType` →
@@ -183,7 +161,7 @@ export declare function renderReaderIntelligenceGuide(sections: readonly TocSect
  * @param manifest - Parsed manifest (any of the supported schemas)
  * @returns Article-type slug usable as a filename component
  */
-export declare function resolveArticleTypeFromManifest(manifest: AnalysisManifest): string;
+export declare function resolveArticleTypeFromManifest(manifest: Manifest): string;
 /**
  * Read, clean, and concatenate every artifact declared by the run's manifest
  * (with discovery fallback when manifest.files is missing), returning a

package/scripts/aggregator/analysis-aggregator.js

@@ -141,7 +141,7 @@ function collectRunArtifacts(runDir) {
             const full = path.join(dir, entry.name);
             const rel = prefix ? `${prefix}/${entry.name}` : entry.name;
             if (entry.isDirectory()) {
-                // Skip raw payloads, legacy run snapshots, and Pass-1 work-in-progress
+                // Skip raw payloads, prior-run snapshots, and Pass-1 work-in-progress
                 // snapshots so they are not rendered as supplementary artifacts.
                 if (entry.name === 'data' || entry.name === 'runs' || entry.name === 'pass1')
                     continue;
@@ -448,7 +448,7 @@ function appendSection(runDir, runDirRelPath, sectionId, sectionTitle, paths, se
     sectionMarkdown.push('');
 }
 /**
- * Resolve the article-type slug from a manifest, tolerating legacy schemas.
+ * Resolve the article-type slug from a manifest, tolerating historic schemas.
  *
  * Thin re-export of {@link _resolveArticleType} from
  * `aggregator/manifest/index.js`. Resolution order: `articleType` →

package/scripts/aggregator/article-generator.d.ts

@@ -102,11 +102,11 @@ export declare function extractDefaultDescription(markdown: string): string;
  * @returns Summary of the generated artefacts ({@link GenerateResult})
  */
 export declare function generateArticle(opts: CliOptions, runSuffix?: string, articleCountOverride?: number): GenerateResult;
-/** Candidate run discovered under `analysis/daily/`. */
 /**
  * One run discovered by {@link discoverAnalysisRuns}.
  *
- * @deprecated Re-exported from `aggregator/runs/index.js` for back-compat.
+ * Thin re-export of {@link _DiscoveredRun} from `aggregator/runs/index.js`,
+ * preserved here as the public type for `article-generator` consumers.
  */
 export type DiscoveredRun = _DiscoveredRun;
 /**

package/scripts/aggregator/article-generator.js

@@ -261,7 +261,7 @@ const FALLBACK_DESCRIPTION = 'EU Parliament intelligence summary derived from co
  */
 export function extractDefaultDescription(markdown) {
     // Suppress unused warning: keep `shouldSkipDescriptionLine` for any
-    // legacy consumer importing it transitively.
+    // historic consumer importing it transitively.
     void shouldSkipDescriptionLine;
     const strong = extractStrongProseLine(markdown);
     return strong.length > 0 ? strong : FALLBACK_DESCRIPTION;

package/scripts/aggregator/article-html.js

@@ -19,7 +19,8 @@
  * browser and CloudFront caches automatically.
  */
 import { BASE_URL, MERMAID_VERSION } from '../constants/config.js';
-import { ALL_LANGUAGES, LANGUAGE_NAMES, LANGUAGE_FLAGS, PAGE_TITLES, SKIP_LINK_TEXTS, TOC_ARIA_LABELS, getLocalizedString, getTextDirection, } from '../constants/languages.js';
+import { buildHeadFreshnessTags } from '../constants/build-info-meta.js';
+import { ALL_LANGUAGES, LANGUAGE_NAMES, LANGUAGE_FLAGS, PAGE_TITLES, SKIP_LINK_TEXTS, TOC_ARIA_LABELS, UPDATE_AVAILABLE_LABELS, UPDATE_REFRESH_CTA_LABELS, UPDATE_DISMISS_LABELS, getLocalizedString, getTextDirection, } from '../constants/languages.js';
 import { escapeHTML } from '../utils/file-utils.js';
 import { buildSiteFooter, buildSiteHeader, buildPageBanner, } from '../templates/section-builders.js';
 /**
@@ -189,6 +190,10 @@ ${hreflangLinks}
   <link rel="manifest" href="../site.webmanifest">
   <meta name="theme-color" content="#003399">
   <link rel="stylesheet" href="../styles.css">
+  <meta name="ep-i18n-update-text" content="${escapeHTML(getLocalizedString(UPDATE_AVAILABLE_LABELS, safeLang))}">
+  <meta name="ep-i18n-update-cta" content="${escapeHTML(getLocalizedString(UPDATE_REFRESH_CTA_LABELS, safeLang))}">
+  <meta name="ep-i18n-dismiss" content="${escapeHTML(getLocalizedString(UPDATE_DISMISS_LABELS, safeLang))}">
+${buildHeadFreshnessTags('../')}
   <script type="application/ld+json">${jsonLdString}</script>
   <script type="module" src="../js/mermaid-init.js?v=${MERMAID_VERSION}" defer></script>
   <script src="../js/article-runtime.js" defer></script>

package/scripts/aggregator/article-metadata.d.ts

@@ -8,9 +8,9 @@ export interface ResolvedMetadataEntry {
 export type ResolvedMetadata = LanguageMap<ResolvedMetadataEntry>;
 /**
  * Raw manifest subset consumed by the resolver. Deliberately narrower
- * than the full {@link AnalysisManifest} shape so the resolver stays
- * usable for backport (which only has the manifest in text form) and for
- * callers that don't need the full typed structure.
+ * than the full {@link import('./manifest/types.js').Manifest} shape so
+ * the resolver stays usable for backport (which only has the manifest in
+ * text form) and for callers that don't need the full typed structure.
  */
 export interface MetadataManifest {
     readonly articleType?: string;
@@ -40,7 +40,7 @@ export interface ResolveMetadataOptions {
     readonly date: string;
     /** Aggregated Markdown document body (after provenance/header). */
     readonly markdown: string;
-    /** Parsed analysis manifest (may be empty for legacy/backport callers). */
+    /** Parsed analysis manifest (may be empty for historic/backport callers). */
     readonly manifest?: MetadataManifest;
     /**
      * Absolute path to the analysis run directory so the resolver can

package/scripts/aggregator/article-metadata.js

@@ -24,7 +24,7 @@
  * 3. **Aggregated-markdown H1** — the first `# …` heading in the aggregator
  *    output, accepted under the same non-generic rule. In practice this
  *    tier rarely fires because the aggregator itself writes the generic
- *    default, but it covers hand-edited or legacy aggregates.
+ *    default, but it covers hand-edited or historic aggregates.
  * 4. **First strong prose paragraph** — the first line of the aggregated
  *    Markdown that survives {@link shouldSkipDescriptionLine}. Used for
  *    `description`; also used for `title` as a last editorial-content
@@ -313,7 +313,7 @@ export function isGenericHeading(heading, articleType, date) {
         `${human} ${date}`,
     ];
     // Also accept the collision-suffix pattern (e.g. `Breaking Breaking — …`)
-    // and the auto-generated "EU Parliament <Type> — <date>" legacy form.
+    // and the auto-generated "EU Parliament <Type> — <date>" historic form.
     const humanRedundant = `${human} ${human}`;
     for (const p of patterns) {
         if (normalized === p)

package/scripts/aggregator/cli/parse.d.ts

@@ -39,7 +39,7 @@ export declare const HELP_TEXT: string;
  * - `{kind:'options', value}` — argv parsed cleanly; `value` is ready to
  *   pass to `generateArticle` / `generateAllArticles`.
  *
- * Compared to the legacy `parseCliArgs` in `article-generator.ts` (which
+ * Compared to the original `parseCliArgs` in `article-generator.ts` (which
  * throws and calls `process.exit` on `--help`), this entry point keeps
  * tests self-contained.
  *

package/scripts/aggregator/cli/parse.js

@@ -3,7 +3,7 @@
 /**
  * @module Aggregator/Cli/Parse
  * @description Pure CLI parser that returns a discriminated union instead
- * of calling `process.exit` mid-parse. The legacy `parseCliArgs` entry
+ * of calling `process.exit` mid-parse. The original `parseCliArgs` entry
  * point in `article-generator.ts` is preserved for backward compatibility
  * with existing callers and tests; new callers and unit tests should
  * prefer {@link parseCliArgsSafe} so the `--help` and error branches are
@@ -202,7 +202,7 @@ function processArgvToken(argv, index, acc) {
  * - `{kind:'options', value}` — argv parsed cleanly; `value` is ready to
  *   pass to `generateArticle` / `generateAllArticles`.
  *
- * Compared to the legacy `parseCliArgs` in `article-generator.ts` (which
+ * Compared to the original `parseCliArgs` in `article-generator.ts` (which
  * throws and calls `process.exit` on `--help`), this entry point keeps
  * tests self-contained.
  *

package/scripts/aggregator/infra/github-urls.d.ts

@@ -7,7 +7,7 @@
  * `githubRawUrl`) and `article-generator.ts` (which embedded the same slug
  * literally inside an `isBasedOn` template string).
  *
- * Every consumer should import from here; the legacy entry points in
+ * Every consumer should import from here; the original entry points in
  * `clean-artifact.ts` are preserved as thin re-export shims for back-compat.
  */
 /** Hack23 repo slug used when building blob/raw/tree URLs. */

package/scripts/aggregator/infra/github-urls.js

@@ -9,7 +9,7 @@
  * `githubRawUrl`) and `article-generator.ts` (which embedded the same slug
  * literally inside an `isBasedOn` template string).
  *
- * Every consumer should import from here; the legacy entry points in
+ * Every consumer should import from here; the original entry points in
  * `clean-artifact.ts` are preserved as thin re-export shims for back-compat.
  */
 /** Hack23 repo slug used when building blob/raw/tree URLs. */

package/scripts/aggregator/manifest/resolver.d.ts

@@ -10,12 +10,12 @@ import type { Manifest, ManifestFiles } from './types.js';
 /** Sentinel used when no schema variant supplies a usable article type. */
 export declare const UNKNOWN_ARTICLE_TYPE = "unknown";
 /**
- * Resolve the article-type slug from a manifest, tolerating legacy schemas.
+ * Resolve the article-type slug from a manifest, tolerating historic schemas.
  *
  * Resolution order (highest precedence first):
  *   1. `articleType` — canonical singular field
  *   2. `articleTypes[0]` — pre-aggregator-pipeline plural array
- *   3. `runType` — legacy field on older breaking-run manifests
+ *   3. `runType` — historic field on older breaking-run manifests
  *
  * Falls back to `'unknown'` when none of the above is a non-empty string.
  *

package/scripts/aggregator/manifest/resolver.js

@@ -3,12 +3,12 @@
 /** Sentinel used when no schema variant supplies a usable article type. */
 export const UNKNOWN_ARTICLE_TYPE = 'unknown';
 /**
- * Resolve the article-type slug from a manifest, tolerating legacy schemas.
+ * Resolve the article-type slug from a manifest, tolerating historic schemas.
  *
  * Resolution order (highest precedence first):
  *   1. `articleType` — canonical singular field
  *   2. `articleTypes[0]` — pre-aggregator-pipeline plural array
- *   3. `runType` — legacy field on older breaking-run manifests
+ *   3. `runType` — historic field on older breaking-run manifests
  *
  * Falls back to `'unknown'` when none of the above is a non-empty string.
  *

package/scripts/aggregator/manifest/types.d.ts

@@ -2,8 +2,8 @@
  * @module Aggregator/Manifest/Types
  * @description Canonical manifest schema for analysis runs and the narrower
  * projection consumed by the editorial-metadata resolver. Centralises every
- * historic schema variant (canonical `articleType`, legacy plural
- * `articleTypes[]`, very-legacy `runType`) into one type that downstream
+ * historic schema variant (canonical `articleType`, plural
+ * `articleTypes[]`, original `runType`) into one type that downstream
  * modules can read against.
  */
 import type { LanguageCode } from '../../types/index.js';
@@ -28,20 +28,22 @@ export type ManifestMetadataOverride = string | Partial<Record<LanguageCode, str
 /**
  * Raw manifest shape as committed by the analysis pipeline. Matches every
  * schema variant the pipeline has ever emitted; readers consult
- * {@link resolveArticleType} rather than `articleType` directly so legacy
+ * {@link resolveArticleType} rather than `articleType` directly so historic
  * runs stay readable.
  */
 export interface Manifest {
     /** Canonical singular form (current pipeline). */
     readonly articleType?: string;
     /**
-     * Legacy plural form emitted by some pre-aggregator-pipeline workflows.
-     * When present, `articleTypes[0]` is treated as the article type.
+     * Plural form emitted by some pre-aggregator-pipeline workflows (historic
+     * schema variant). When present, `articleTypes[0]` is treated as the
+     * article type.
      */
     readonly articleTypes?: readonly string[];
     /**
-     * Very-legacy field on older breaking-run manifests. Used as the last
-     * fallback when neither `articleType` nor `articleTypes` is present.
+     * Original field on older breaking-run manifests (historic schema variant).
+     * Used as the last fallback when neither `articleType` nor `articleTypes`
+     * is present.
      */
     readonly runType?: string;
     /** Stable run identifier; falls back to the run-dir basename. */
@@ -65,7 +67,7 @@ export interface Manifest {
  * Narrower manifest projection consumed by {@link resolveArticleMetadata}
  * in `aggregator/article-metadata.ts`. The metadata resolver only needs a
  * subset; keeping this projection separate means string-only callers
- * (backport, legacy curators) don't have to construct a full {@link Manifest}.
+ * (backport, historic curators) don't have to construct a full {@link Manifest}.
  */
 export interface MetadataManifest {
     readonly articleType?: string;

package/scripts/aggregator/prior-run-diff.js

@@ -3,24 +3,33 @@
 // SPDX-License-Identifier: Apache-2.0
 
 /**
- * Prior-run diff helper for the re-run merge rule.
+ * Prior-run diff helper for the re-run improve/extend rule.
  *
  * Reads `manifest.json.history[]` from a same-day analysis folder and
- * classifies every artifact as **at-floor** (carry-forward) or **below-floor**
- * (rewrite). The result — a `priorRunDiff` plan — is written to stdout as
- * JSON and can be consumed by Stage A of the analysis workflow.
+ * classifies every artifact as **at-floor** (must-extend / carry-forward) or
+ * **below-floor** (rewrite). The result — a `priorRunDiff` plan with
+ * `mode: "improve-and-extend"` — is written to stdout as JSON and is
+ * consumed by Stage B of the analysis workflow.
  *
- * Controlled by the `ENABLE_PRIOR_RUN_MERGE` environment variable:
- *   - `ENABLE_PRIOR_RUN_MERGE=true`  → normal operation (produce plan)
- *   - unset / any other value        → short-circuit: emit plan with
- *                                       `enabled: false` and empty arrays
+ * **Re-run semantics (never no-op).** Entries listed under `carryForward[]`
+ * are **NOT** skipped on re-runs — they are must-extend targets. Stage B
+ * MUST raise their depth: each prior artifact's `priorLines` becomes the new
+ * floor and the agent must add ≥1 new section, ≥3 new evidence citations, or
+ * ≥1 new chart, ending at `lines >= max(floor, priorLines + 20)`. Entries in
+ * `rewrite[]` are still written from scratch to the catalog floor.
+ *
+ * Always-on. The `ENABLE_PRIOR_RUN_MERGE` environment variable is no longer
+ * read — the helper runs unconditionally so re-runs cannot accidentally
+ * regress to the legacy "skip-write" behaviour. The `buildPriorRunDiff(..,
+ * enabled)` parameter is kept for back-compat with unit tests but the CLI
+ * always passes `true`.
  *
  * Invocation:
  *   node scripts/aggregator/prior-run-diff.js <runDir>
  *   npm run prior-run-diff -- analysis/daily/2026-04-26/week-in-review
  *
  * Exit codes:
- *   0  — plan emitted successfully (or feature disabled)
+ *   0  — plan emitted successfully
  *   1  — runDir missing or invalid
  *   2  — bad CLI usage
  *
@@ -28,6 +37,7 @@
  * ```json
  * {
  *   "enabled": true,
+ *   "mode": "improve-and-extend",
  *   "runDir": "analysis/daily/2026-04-26/week-in-review",
  *   "articleType": "week-in-review",
  *   "priorRunId": "week-in-review-run-1714128000",
@@ -35,8 +45,10 @@
  *     {
  *       "relativePath": "intelligence/synthesis-summary.md",
  *       "lines": 250,
+ *       "priorLines": 250,
  *       "floor": 180,
- *       "source": "carry-forward-from:week-in-review-run-1714128000"
+ *       "extendFloor": 270,
+ *       "source": "extend-from-prior:week-in-review-run-1714128000"
  *     }
  *   ],
  *   "rewrite": [
@@ -50,10 +62,13 @@
  * }
  * ```
  *
- * The `source` value on each carry-forward entry follows the schema:
- *   `"carry-forward-from:<runId>"`
- * which Stage B writes into `manifest.json.artifactSources` (additive,
- * backward-compatible with the existing schema).
+ * - `priorLines` exposes the prior-run line count so Stage B knows the lower
+ *   bound it must beat.
+ * - `extendFloor` = `max(floor, priorLines + 20)` — the minimum line count
+ *   the new pass MUST reach for this artifact.
+ * - The `source` value follows the schema `"extend-from-prior:<runId>"`,
+ *   which Stage B writes into `manifest.json.artifactSources` (additive,
+ *   back-compat with prior `"carry-forward-from:<runId>"` consumers).
  */
 
 import fs from 'node:fs';
@@ -63,6 +78,7 @@ import { fileURLToPath } from 'node:url';
 
 const ROOT = process.cwd();
 const DEFAULT_MIN_LINES = 30;
+const EXTEND_DELTA_LINES = 20;
 
 // Artifacts that must contain at least one Mermaid fenced block.
 // Mirrors the directory-based heuristic in validate-analysis-completeness.js.
@@ -86,9 +102,10 @@ function usage(code = 2) {
     '',
     '  <runDir>   Path to analysis/daily/<date>/<slug>/',
     '',
-    'Environment:',
-    '  ENABLE_PRIOR_RUN_MERGE=true   Enable the carry-forward classifier',
-    '                                (default: disabled — emits empty plan)',
+    'Always-on. The helper unconditionally classifies prior-run artifacts as',
+    'must-extend (carryForward[]) or below-floor rewrite (rewrite[]) so re-runs',
+    'can never accidentally no-op. The legacy ENABLE_PRIOR_RUN_MERGE env flag',
+    'is no longer read.',
     '',
     'Example:',
     '  npm run prior-run-diff -- analysis/daily/2026-04-26/week-in-review',
@@ -190,9 +207,14 @@ export function classifyArtifact(runDir, relativePath, floor, mermaidRequiredLis
 /**
  * Build the `priorRunDiff` plan for a same-day analysis folder.
  *
+ * Mode is always **improve-and-extend**: `carryForward[]` entries are
+ * must-extend targets (their `priorLines` and `extendFloor` exposed), not
+ * skip-write targets. The `enabled` parameter is preserved for back-compat
+ * with the legacy unit-test signature; the CLI always passes `true`.
+ *
  * @param {string} runDir              - Absolute path to the run folder.
  * @param {object|null} thresholdsJson - Parsed reference-quality-thresholds.json.
- * @param {boolean} enabled            - Whether the feature is enabled.
+ * @param {boolean} enabled            - Whether the feature is enabled (CLI: always true).
  * @returns {object} The diff plan (serialisable to JSON).
  */
 export function buildPriorRunDiff(runDir, thresholdsJson, enabled) {
@@ -210,6 +232,7 @@ export function buildPriorRunDiff(runDir, thresholdsJson, enabled) {
   if (!enabled) {
     return {
       enabled: false,
+      mode: 'improve-and-extend',
       runDir: relRunDir,
       articleType,
       priorRunId: null,
@@ -222,6 +245,7 @@ export function buildPriorRunDiff(runDir, thresholdsJson, enabled) {
   if (history.length === 0) {
     return {
       enabled: true,
+      mode: 'improve-and-extend',
       runDir: relRunDir,
       articleType,
       priorRunId: null,
@@ -248,11 +272,14 @@ export function buildPriorRunDiff(runDir, thresholdsJson, enabled) {
     const floor = Math.max(DEFAULT_MIN_LINES, perArtifactFloors[relativePath] ?? 0);
     const result = classifyArtifact(runDir, relativePath, floor, mermaidRequiredList);
     if (result.atFloor) {
+      const extendFloor = Math.max(floor, result.lines + EXTEND_DELTA_LINES);
       carryForward.push({
         relativePath,
         lines: result.lines,
+        priorLines: result.lines,
         floor: result.floor,
-        source: `carry-forward-from:${priorRunId}`,
+        extendFloor,
+        source: `extend-from-prior:${priorRunId}`,
       });
     } else {
       rewrite.push({
@@ -266,6 +293,7 @@ export function buildPriorRunDiff(runDir, thresholdsJson, enabled) {
 
   return {
     enabled: true,
+    mode: 'improve-and-extend',
     runDir: relRunDir,
     articleType,
     priorRunId,
@@ -338,7 +366,11 @@ function main() {
     process.exit(1);
   }
 
-  const enabled = process.env['ENABLE_PRIOR_RUN_MERGE'] === 'true';
+  // Re-run improve/extend rule is always-on. The legacy ENABLE_PRIOR_RUN_MERGE
+  // env flag is no longer read — re-runs cannot accidentally regress to the
+  // pre-2026-05 skip-write behaviour. See .github/prompts/02-analysis-protocol.md
+  // §"Re-run improve/extend rule".
+  const enabled = true;
   const thresholdsJson = loadThresholds(opts.thresholdsPath);
   const plan = buildPriorRunDiff(runDir, thresholdsJson, enabled);
 

package/scripts/aggregator/runs/discover.d.ts

@@ -34,7 +34,7 @@ export declare function readRunCandidate(runDir: string): DiscoveredRun | null;
  * The walk stops descending into a directory the moment it sees a
  * `manifest.json`, so nested artifact subdirectories never get reported
  * as separate runs. Results are sorted by date ascending then by path
- * lexically — the same order used by the legacy implementation in
+ * lexically — the same order used by the previous in-line implementation in
  * `article-generator.ts`.
  *
  * @param repoRoot - Absolute repository root

package/scripts/aggregator/runs/discover.js

@@ -52,7 +52,7 @@ export function readRunCandidate(runDir) {
  * The walk stops descending into a directory the moment it sees a
  * `manifest.json`, so nested artifact subdirectories never get reported
  * as separate runs. Results are sorted by date ascending then by path
- * lexically — the same order used by the legacy implementation in
+ * lexically — the same order used by the previous in-line implementation in
  * `article-generator.ts`.
  *
  * @param repoRoot - Absolute repository root

package/scripts/backport-article-seo.js

@@ -388,7 +388,7 @@ function extractBodyFirstProse(articleHtml) {
  * 1. If a manifest.json for the run exists (aggregator cohort), use the
  *    full {@link resolveArticleMetadata} pipeline — this picks up manifest
  *    overrides and artefact H1s.
- * 2. Otherwise (legacy cohort), derive from the rendered body:
+ * 2. Otherwise (historic cohort), derive from the rendered body:
  *    - Title = non-generic `<h1>` from the body, else first sentence of
  *      the first strong prose paragraph.
  *    - Description = first strong prose paragraph (full, not the same
@@ -412,7 +412,7 @@ function deriveMetadataForFile(file, html) {
   // fallback for `committee-reports` renders realistic abbreviations
   // (`ENVI, ECON, AFET, LIBE, AGRI`) instead of the placeholder
   // `Main Committees`. This keeps the localized template consistent
-  // with the legacy format even when the manifest is missing.
+  // with the historic format even when the manifest is missing.
   const committee = extractCommitteeCodes(bodyH1) || extractCommitteeCodes(bodyProse);
 
   const resolved = resolveArticleMetadata({
@@ -425,7 +425,7 @@ function deriveMetadataForFile(file, html) {
 
   if (file.lang !== 'en') {
     // NON-ENGLISH files: The article body may be in a different language
-    // than the file claims to be — legacy files have localized H1/chrome
+    // than the file claims to be — historic files have localized H1/chrome
     // but English body prose; aggregator PR#1404 files have English H1
     // AND English body in every language variant. We accept body content
     // only when it is plausibly in the file's language, and fall back to
@@ -658,9 +658,9 @@ function buildSyntheticMarkdown(h1, prose) {
 
 /**
  * Choose the final title text. Prefers a non-generic body H1. When the
- * H1 is generic (e.g. legacy "Legislative Procedures: European Parliament
+ * H1 is generic (e.g. historic "Legislative Procedures: European Parliament
  * Monitor"), falls back to the first sentence of body prose — this is
- * the single biggest SEO win for legacy files.
+ * the single biggest SEO win for historic files.
  *
  * @param {string} bodyH1 - First H1 from the body
  * @param {string} bodyProse - First strong prose paragraph
@@ -681,7 +681,7 @@ function chooseTitle(bodyH1, bodyProse, templateTitle, file) {
 
 /**
  * Extend {@link isGenericHeading} with a few extra patterns specific to
- * legacy-era titles (pre-aggregator pipeline) so those files get
+ * historic-era titles (pre-aggregator pipeline) so those files get
  * replaced during backport. Also catches the pure `<Title-Case-Phrase>
  * — <ISO-date>` form that the default aggregator title emits when the
  * articleType slug has a run suffix (e.g. `breaking-190`) that
@@ -695,7 +695,7 @@ function chooseTitle(bodyH1, bodyProse, templateTitle, file) {
 function isGenericBodyH1(h1, articleType, date) {
   if (isGenericHeading(h1, articleType, date)) return true;
   const normalized = h1.trim();
-  const legacyTemplates = [
+  const historicTemplates = [
     'Legislative Procedures: European Parliament Monitor',
     'EU Parliament Committee Activity Report',
     'EU Parliament Breaking',
@@ -703,7 +703,7 @@ function isGenericBodyH1(h1, articleType, date) {
     'Plenary Votes & Resolutions',
     'Plenary Votes and Resolutions',
   ];
-  for (const t of legacyTemplates) {
+  for (const t of historicTemplates) {
     if (normalized === t || normalized.startsWith(`${t} `) || normalized.startsWith(`${t}:`)) {
       return true;
     }
@@ -814,7 +814,7 @@ function rewriteHtml(html, metadata) {
 /**
  * Replace `<meta name="<name>" content="…">` in-place. When absent the
  * document is returned unchanged — we never inject new tags during
- * backport so legacy files retain their original meta-tag order.
+ * backport so historic files retain their original meta-tag order.
  *
  * The `content` match is quote-aware: the content of a double-quoted
  * attribute value may contain apostrophes (e.g. `Parliament's`), so the

package/scripts/constants/analysis-constants.d.ts

@@ -27,7 +27,7 @@ export declare const AI_MARKER = "[AI_ANALYSIS_REQUIRED]";
  *
  * Recognises three marker formats:
  * - `[AI_ANALYSIS_REQUIRED]` — the current standard marker (v3.0+)
- * - `[REQUIRED]` — legacy marker used in template stubs before v3.0
+ * - `[REQUIRED]` — historic marker used in template stubs before v3.0
  * - `[?]` — shorthand used in some early methodology templates
  *
  * @param text - Text to test