Skip to content

Conversation

@arunjaindev
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Checklist

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@arunjaindev arunjaindev self-assigned this Nov 24, 2025
@arunjaindev arunjaindev requested a review from Copilot November 24, 2025 12:28
Copilot finished reviewing on behalf of arunjaindev November 24, 2025 12:30
Copy link

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 PR introduces support for CVE (Common Vulnerabilities and Exposures) listing in the security center, restructuring the security scanning interface to focus on vulnerability management.

Key changes include:

  • Reorganized security center navigation from a single "scans" view to separate "vulnerabilities" and "security-enablement" sections
  • Added new filtering enums for vulnerability discovery age and fix availability
  • Extended component library with new utility functions and UI enhancements
  • Renamed constants for better clarity (SEVERITIES → SEVERITIES_LABEL_COLOR_MAP)

Reviewed changes

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

Show a summary per file
File Description
src/Shared/Hooks/useUserPreferences/types.ts Updated navigation IDs to reflect new security center structure
src/Shared/Components/SelectPicker/utils.ts Added multi-value option filtering utility
src/Shared/Components/SelectPicker/index.ts Exported new utility function
src/Shared/Components/Security/types.tsx Added enums for vulnerability filtering criteria
src/Shared/Components/Security/index.tsx Exported new security-related components and types
src/Shared/Components/Security/SecurityModal/utils.tsx Updated to use renamed constant
src/Shared/Components/Security/SecurityModal/index.ts Exported renamed constant
src/Shared/Components/Security/SecurityModal/constants.tsx Renamed SEVERITIES to SEVERITIES_LABEL_COLOR_MAP
src/Shared/Components/Security/SecurityDetailsCards/SecurityCard.tsx Updated to use renamed constant
src/Common/Types.ts Added id field to app list type
src/Common/SegmentedControl/types.ts Added support for large size
src/Common/SegmentedControl/constants.ts Added styling for large size
src/Common/SegmentedBarChart/types.ts Added conditional type for hideTotal feature
src/Common/SegmentedBarChart/SegmentedBarChart.tsx Implemented hideTotal logic
src/Common/Constants.ts Updated security center URLs
package.json Version bump
package-lock.json Version bump

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

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