Skip to content

Normalize vulnerability summaries and preserve markdown details in jsrepository-master - #572

Closed
eoftedal with Copilot wants to merge 3 commits into
masterfrom
copilot/update-vulnerability-summaries
Closed

Normalize vulnerability summaries and preserve markdown details in jsrepository-master#572
eoftedal with Copilot wants to merge 3 commits into
masterfrom
copilot/update-vulnerability-summaries

Conversation

Copilot AI commented May 18, 2026

Copy link
Copy Markdown
Contributor

Some vulnerability entries used multiline markdown in summary, and a subset of generated summaries began with an artificial Summary prefix. This PR standardizes summary text for machine/human readability while preserving full advisory content.

  • Summary normalization

    • Rewrote affected summary fields to plain inline text (no markdown section blocks/newline-heavy formatting).
    • Removed leading Summary prefixes from generated summaries where present.
    • Kept multi-sentence summaries when useful, as long as they remain plain text.
  • Details preservation

    • Moved original multiline markdown content from summary into a new sibling details field on the vulnerability object.
    • Retains full advisory narrative without polluting concise summary output.
  • Schema + type support

    • Added optional details to vulnerability validation in node/src/repo.ts.
    • Added details?: string to Vulnerability in node/src/types.ts.
  • Repository artifact sync

    • Regenerated derived repository JSON files to keep all versioned outputs consistent with jsrepository-master.json.
{
  "summary": "Axios has an incomplete security patch for NO_PROXY hostname resolution.",
  "details": "## Summary\n\nThis report documents an incomplete security patch..."
}

@eoftedal
eoftedal marked this pull request as ready for review May 18, 2026 12:20
@eoftedal eoftedal closed this May 18, 2026
Copilot stopped work on behalf of eoftedal due to an error May 18, 2026 12:30
Copilot AI requested a review from eoftedal May 18, 2026 12:30
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