Skip to content

Conversation

@refoo0
Copy link
Collaborator

@refoo0 refoo0 commented Jan 29, 2026

No description provided.

Signed-off-by: rafi <refaei.shikho@hotmail.com>
Copilot AI review requested due to automatic review settings January 29, 2026 17:53
Copy link
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

This pull request refactors risk metrics to use CVE PURL (Package URL) fields instead of the original severity field names. The refactoring updates how risk history data is aggregated and displayed across several pages of the application.

Changes:

  • Updated reduceRiskHistories function to accumulate new cvePurl* fields instead of legacy severity fields
  • Modified risk metric calculations in the overview page to read from new cvePurl* fields
  • Updated severity card displays in asset version pages to use new cvePurl* fields for both risk and CVSS modes

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/utils/view.ts Updated the reduceRiskHistories function to accumulate cvePurlLow/Medium/High/Critical and cvePurlLowCvss/MediumCvss/HighCvss/CriticalCvss fields instead of the legacy low/medium/high/critical and lowCvss/mediumCvss/highCvss/criticalCvss fields
src/app/(loading-group)/[organizationSlug]/projects/[projectSlug]/overview/page.tsx Updated four useMemo hooks (criticalAmount, highAmount, mediumAmount, lowAmount) to calculate totals using the new cvePurl-prefixed fields instead of legacy field names
src/app/(loading-group)/[organizationSlug]/projects/[projectSlug]/assets/[assetSlug]/refs/[assetVersionSlug]/page.tsx Updated SeverityCard components to display risk metrics from the new cvePurl-prefixed fields instead of legacy field names

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: rafi <refaei.shikho@hotmail.com>
@timbastin timbastin merged commit 26fa5d6 into main Jan 29, 2026
9 of 10 checks passed
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.

3 participants