Skip to content

Conversation

@cx-rahul-pidde
Copy link
Contributor

@cx-rahul-pidde cx-rahul-pidde commented Sep 30, 2025

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

This PR addresses the issue where CxOne Assist in VS Code did not support light mode properly in the following areas:

  • The popover UI was not styled correctly for light mode.
  • The Ignored Vulnerabilities log page had visibility issues in light mode.

Previously, these elements displayed correctly only in dark mode.

References

AST-114409 - VS Code | CxOne Assist | VS Code resulted in not supporting light mode | ignored vulnerabilities issue
AST-114410 - VS Code | CxOne Assist | VS Code resulted in not supporting light mode | popover issue
AST-115359 - VS Code | CxOne Assist Disabled When Checkmarx MCP Enabled on CxOne Web Portal and Logging in via OAuth Using VS Code Plugin

Testing

Case 1: Verify Popover UI in Light Mode

  • VS Code theme set to Light Mode.
  • Open CxOne Assist popover.
  • Observe the UI elements (background, text, buttons).
    Expected Result: - Popover UI should be clearly visible with proper contrast and styling consistent with the light theme design.

Case 2: Verify Popover UI in Dark Mode

  • VS Code theme set to Dark Mode.
  • Open CxOne Assist popover.
  • Observe the UI elements.
    Expected Result: - Popover UI should remain unchanged and properly styled for dark mode.

Case 3: Verify Ignored Vulnerabilities Log Page in Light Mode

  • VS Code theme set to Light Mode.
  • Navigate to the Ignored Vulnerabilities log page.
  • Review the readability and UI consistency.
    Expected Result: - Text and UI elements should be easily readable with appropriate colors and no visual glitches.

Case 4: Verify Ignored Vulnerabilities Log Page in Dark Mode

  • VS Code theme set to Dark Mode.
  • Navigate to the Ignored Vulnerabilities log page.
  • Review the readability and UI consistency.
    Expected Result: - Log page should display correctly as before with no regressions.

Case 5: Switching Themes on the Fly

  • VS Code is open with CxOne Assist active.
  • Toggle VS Code theme from Light to Dark and vice versa.
  • Observe the popover and ignored vulnerabilities page.
    Expected Result:- UI should dynamically update without requiring a restart, maintaining correct styling in both modes.

Case 6: CxOne Assist Enable in Welcome page

  • Enable Checkmarx Model Context Protocol (MCP) in the CxOne Web Portal as an admin.
  • Open Visual Studio Code.
  • Install and launch the CxOne VS Code plugin (if not already installed).
  • Log in using OAuth via the CxOne plugin.
  • Check the plugin’s welcome page for CxOne Assist availability.

Case 7: CxOne Assist Enable in Welcome page

  • Disable Checkmarx Model Context Protocol (MCP) in the CxOne Web Portal as an admin.
  • Open Visual Studio Code.
  • Install and launch the CxOne VS Code plugin (if not already installed).
  • Log in using OAuth via the CxOne plugin.
  • Check the plugin’s welcome page for CxOne Assist availability. (Checkbox will not display)

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@github-actions
Copy link
Contributor

github-actions bot commented Sep 30, 2025

Logo
Checkmarx One – Scan Summary & Detailsfd82ae54-d0fe-41d9-a79f-f2eb5ca53220

Great job! No new security vulnerabilities introduced in this pull request

Copy link
Collaborator

@cx-anurag-dalke cx-anurag-dalke left a comment

Choose a reason for hiding this comment

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

ok

@cx-rahul-pidde cx-rahul-pidde merged commit abd0656 into main Oct 6, 2025
7 checks passed
@cx-rahul-pidde cx-rahul-pidde deleted the feature/AST-114409 branch October 6, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants