Skip to content

fix: replace broken JSDoc {@link CommitteeTheme} with inline type values - #547

Merged
pethers merged 2 commits into
news/committee-reports-2026-03-12-0fff6247d0620371from
copilot/sub-pr-530
Mar 12, 2026
Merged

fix: replace broken JSDoc {@link CommitteeTheme} with inline type values#547
pethers merged 2 commits into
news/committee-reports-2026-03-12-0fff6247d0620371from
copilot/sub-pr-530

Conversation

Copilot AI commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

{@link CommitteeTheme} in categorizeAdoptedText's JSDoc produces a broken reference in generated docs because TypeScript type aliases are erased during compilation to JS. JSDoc runs against the compiled scripts/ output.

  • Replaced {@link CommitteeTheme} with inline literal values that survive compilation:
- * @returns Committee theme key — one of the {@link CommitteeTheme} values
+ * @returns Committee theme key — one of `'ENVI'` | `'ECON'` | `'AFET'` | `'LIBE'` | `'AGRI'` | `'OTHER'`

The CommitteeTheme union type itself remains in the TypeScript source for compile-time safety — only the JSDoc cross-reference is replaced.


🔒 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.

…ues in categorizeAdoptedText

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title [WIP] Chore: EU Parliament committee-reports articles 2026-03-12 fix: replace broken JSDoc {@link CommitteeTheme} with inline type values Mar 12, 2026
@pethers
pethers marked this pull request as ready for review March 12, 2026 11:17
@pethers
pethers merged commit 3052d0a into news/committee-reports-2026-03-12-0fff6247d0620371 Mar 12, 2026
@pethers
pethers deleted the copilot/sub-pr-530 branch March 12, 2026 11: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