Skip to content

Conversation

@refoo0
Copy link
Collaborator

@refoo0 refoo0 commented Jan 27, 2026

No description provided.

Signed-off-by: rafi <refaei.shikho@hotmail.com>
Copilot AI review requested due to automatic review settings January 27, 2026 16:47
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 extracts the vulnerability and affected component details UI into a reusable AffectedComponentDetails component, improving code organization and maintainability.

Changes:

  • Added new TypeScript interfaces for PURL inspection API responses (PackageURL, PurlMatchContext, AffectedComponent, VulnInPackage, PURLInspectResponse)
  • Created AffectedComponentDetails component that fetches and displays vulnerability details and affected component information via the PURL inspection API
  • Refactored the vulnerability details page to use the new component, removing duplicated UI code

Reviewed changes

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

File Description
src/types/api/api.ts Added type definitions for PURL inspection API response structures
src/components/AffectedComponent.tsx New component for displaying vulnerability and affected component details with API integration
src/app/(loading-group)/[organizationSlug]/projects/[projectSlug]/assets/[assetSlug]/refs/[assetVersionSlug]/dependency-risks/[vulnId]/page.tsx Replaced inline UI code with new AffectedComponentDetails component

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

Signed-off-by: rafi <refaei.shikho@hotmail.com>
Signed-off-by: rafi <refaei.shikho@hotmail.com>
Signed-off-by: rafi <refaei.shikho@hotmail.com>
Signed-off-by: rafi <refaei.shikho@hotmail.com>
@timbastin timbastin merged commit 13340ac into main Jan 29, 2026
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