Skip to content

Conversation

@dotNomad
Copy link
Collaborator

@dotNomad dotNomad commented Jun 27, 2025

This PR addresses feedback allowing the Package Vulnerability Scanner to show more than just packages with vulnerabilities. Now it can show all packages, but still defaults to only showing vulnerable packages.

To show packages the "Stat Buttons" that show the number of different type of packages are click-able with hover stats to indicate that.

Preview
CleanShot.2025-06-27.at.12.28.49.mp4

Fixes #207

These changes have been published to our internal Dogfood server here.


To facilitate this I had to change how a bit about how the packages flow between components.

Some notes:

  • VulnerabilityCard has become PackageCard since it no longer only shows vulnerabilities
  • VulnerabilityList has becomePackageList for a similar reason
  • The construction of the packages has been moved a bit to the vulnsStore with the getDetailsForPackageVersion method.
    • Most of the functionality here is very similar. I'd recommend comparing the removed code from VulnerabilityChecker with the new store code.
  • Some HTML templating has been moved around a bit for better understanding of state and easier styling

Copy link
Contributor

@marcosnav marcosnav left a comment

Choose a reason for hiding this comment

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

I really like how you structured the components here 💯

@dotNomad dotNomad merged commit 7b83f24 into main Jul 1, 2025
17 checks passed
@dotNomad dotNomad deleted the dotnomad/207 branch July 1, 2025 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show all packages for content not just vulnerable packages

3 participants