Skip to content

Checkmarx AST plugin custom state filters bug AST-141818#426

Closed
cx-anand-nandeshwar wants to merge 2 commits into
mainfrom
bug/filter_state_AST-141818
Closed

Checkmarx AST plugin custom state filters bug AST-141818#426
cx-anand-nandeshwar wants to merge 2 commits into
mainfrom
bug/filter_state_AST-141818

Conversation

@cx-anand-nandeshwar

Copy link
Copy Markdown
Contributor

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 contains the bug-fix for Checkmarx AST plugin. The plugin experienced critical filter persistence issues that prevented filters from being properly saved and restored across IDE restarts.

References

Bug: https://checkmarx.atlassian.net/browse/AST-141818
RCA and changes : https://checkmarx.sharepoint.com/:t:/s/Sypher-Plugins/IQCsMJtTV_8dRbG_rRJp9QmAAftUaT8b2mEMAzll_aNblQA?e=ktKQNS

Testing

Test 1: First Launch

  • Defaults severity and custom state filters initialized and persisted to storage

Test 2: User Selection

  • Changes synced immediately, reflected in UI in real-time and Persisted on shutdown
    Restored on restart

Test 3: Multiple Restarts

  • Settings survive multiple restarts and consistent behaviour across sessions

Test 4: Always initialize severity and custom state filters

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 github-actions Bot added the bug Something isn't working label Mar 20, 2026
@cx-anand-nandeshwar cx-anand-nandeshwar self-assigned this Mar 20, 2026
@github-actions

github-actions Bot commented Mar 20, 2026

Copy link
Copy Markdown

Logo
Checkmarx One – Scan Summary & Details125d4718-2ac5-43a7-9a98-38995a4f3bcb

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

@cx-anand-nandeshwar

Copy link
Copy Markdown
Contributor Author

This code is merged in PR: #429

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant