Skip to content

UI Fixes, Bug Fixes & Code Improvements#395

Open
cx-atish-jadhav wants to merge 8 commits intomainfrom
other/ui_fixes
Open

UI Fixes, Bug Fixes & Code Improvements#395
cx-atish-jadhav wants to merge 8 commits intomainfrom
other/ui_fixes

Conversation

@cx-atish-jadhav
Copy link
Contributor

@cx-atish-jadhav cx-atish-jadhav commented Feb 2, 2026

Description

This PR includes multiple UI fixes, bug fixes, and code improvements for the JetBrains Checkmarx plugin:

1. CLI | Fix IAC engine Path env for MAC (AST-128580)

  • Type: Task
  • Summary: Fixed the IAC engine PATH environment variable configuration for macOS systems to ensure proper scanner functionality.

2. JetBrains: CxOne Assist UI shows promotional panel instead of realtime scan results after plugin install/update (AST-131866)

  • Type: Bug Fix
  • Summary: Fixed an issue where users with valid realtime scanner license (Dev Assist) AND scan results license (One Assist) would see the promotional panel instead of actual realtime scan findings after plugin installation or update.
  • Root Cause: License flags (oneAssistLicenseEnabled, devAssistLicenseEnabled) were only fetched during authentication flow, not during plugin initialization.
  • Fix: Added a new startup activity class to refresh license status during plugin initialization when user is already authenticated.

3. JetBrains Plugin - Promotional Panels UI Implementation (AST-132978)

  • Type: Task
  • Summary: Implemented promotional panels UI to match Figma design specifications.
  • Changes:
    • ScanResultsUpsellPanel: Title (Inter Bold, 15px), Description (Inter Regular, 13px, theme-aware colors), Button (400x32px, #0081E1 blue, 8px border-radius)
    • DevAssistPromotionalPanel: Title (Inter Bold, 15px), Description (Inter Regular, 13px with line break), Contact text (Inter Semi Bold, 13px, #787C87), 32px gap between description and contact text
    • Used JBColor for theme-aware color handling

4. JetBrains: Multiple 'Session Expired' notifications appear on logout (AST-132334)

  • Type: Task (Bug Fix)
  • Summary: Fixed an issue where multiple 'Session Expired' notifications (3-4 or more) appeared when clicking the Logout button.
  • Root Cause: Multiple background services detected expired token and each triggered Utils.notifySessionExpired().
  • Fix: Implemented a flag-based mechanism to ensure only one notification is shown per session expiry event.

5. JetBrains Plugin - Align columns in Ignored Findings window (AST-132279)

  • Type: Task
  • Summary: Fixed column alignment in the Ignored Findings window so that Last Updated and Actions columns align with the middle section of the Risk column.
  • Additional Improvements:
    • Comprehensive code refactoring on CxIgnoredFindings.java (reduced from 1,496 to 1,317 lines - 12% reduction)
    • Replaced java.awt.Color with JBColor for proper theme support
    • Fixed multiple IDE warnings

References

| Jira Ticket | Type | Summary |
|-------------|------|---------||
| AST-128580 | Task | CLI - Fix IAC engine Path env for MAC |
| AST-131866 | Bug | CxOne Assist UI shows promotional panel instead of realtime scan results |
| AST-132978 | Task | Promotional Panels UI Implementation (Figma design) |
| AST-132334 | Task | Multiple 'Session Expired' notifications on logout |
| AST-132279 | Task | Align columns in Ignored Findings window |

Testing

Manual Testing Performed:

  1. AST-128580 (IAC Path Fix):

    • Verified IAC scanner works correctly on macOS with proper PATH environment
  2. AST-131866 (License Refresh):

    • Installed plugin fresh with valid Dev Assist + One Assist licenses
    • Verified realtime scan findings appear correctly (not promotional panel)
    • Verified "Checkmarx One Assist" child node appears in Settings tree
    • Verified "Go to Checkmarx One Assist" link appears in tool window
  3. AST-132978 (Promotional Panels UI):

    • Verified ScanResultsUpsellPanel matches Figma design (fonts, colors, button styling)
    • Verified DevAssistPromotionalPanel matches Figma design
    • Tested in both light and dark themes for proper color switching
  4. AST-132334 (Session Expired Notifications):

    • Clicked Logout button and verified only ONE 'Session Expired' notification appears
    • Logged back in and logged out again to verify flag reset works correctly
  5. AST-132279 (Ignored Findings Alignment):

    • Opened Ignored Findings window with multiple entries
    • Verified Last Updated column aligns with description section
    • Verified Actions column (Revive button) aligns with description section
    • Tested in both light and dark themes

Files Modified:

  • ScanResultsUpsellPanel.java
  • DevAssistPromotionalPanel.java
  • CxIgnoredFindings.java
  • Utils.java
  • GlobalSettingsComponent.java
  • Resource.java
  • CxBundle.properties
  • New startup activity class for license refresh

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

github-actions bot commented Feb 2, 2026

Logo
Checkmarx One – Scan Summary & Details5bcc8f98-364a-42c5-8c11-32622749a9de

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

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.

4 participants