Skip to content

Enhance Settings & Welcome Pages and MCP Installation Flow (AST-107883)#310

Open
cx-rahul-pidde wants to merge 20 commits intofeature/devassist-integration-branchfrom
feature/AST-107883
Open

Enhance Settings & Welcome Pages and MCP Installation Flow (AST-107883)#310
cx-rahul-pidde wants to merge 20 commits intofeature/devassist-integration-branchfrom
feature/AST-107883

Conversation

@cx-rahul-pidde
Copy link
Copy Markdown
Contributor

@cx-rahul-pidde cx-rahul-pidde commented Mar 31, 2026

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 introduces several improvements and fixes across the Visual Studio extension

  • Settings Page:
    - Refactored logic for scanner enable/disable and user preferences.
    - Improved error handling and configuration management.
    - Added/updated UI elements for better user experience.

  • Welcome Page:

    • Fixed icon resource loading using constants and correct namespaces.
    • Ensured robust handling of embedded resources (.ico/.png).
    • Improved UI theming and contrast for better accessibility.
  • MCP Installation:

    • Refactored MCP install/uninstall/config logic for reliability.
    • Enhanced error handling and user feedback during installation.
    • Improved config file management and safe dictionary access.

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
Copy Markdown

github-actions bot commented Mar 31, 2026

Logo
Checkmarx One – Scan Summary & Details4fbdbbb5-11a2-45ab-ba17-1323ad20a6d5


New Issues (20) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 303
detailsMethod GetConfigSnapshot at line 303 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
2 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 302
detailsMethod GetConfigSnapshot at line 302 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
3 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 303
detailsMethod GetConfigSnapshot at line 303 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
4 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 290
detailsMethod GetCxConfig at line 290 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
5 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 291
detailsMethod GetCxConfig at line 291 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
6 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 302
detailsMethod GetConfigSnapshot at line 302 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
7 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 303
detailsMethod GetConfigSnapshot at line 303 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
8 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 290
detailsMethod GetCxConfig at line 290 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
9 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 291
detailsMethod GetCxConfig at line 291 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
10 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 302
detailsMethod GetConfigSnapshot at line 302 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
11 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 303
detailsMethod GetConfigSnapshot at line 303 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
12 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 290
detailsMethod GetCxConfig at line 290 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
13 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 291
detailsMethod GetCxConfig at line 291 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
14 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 302
detailsMethod GetConfigSnapshot at line 302 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
15 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 290
detailsMethod GetCxConfig at line 290 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
16 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 291
detailsMethod GetCxConfig at line 291 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
17 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 290
detailsMethod GetCxConfig at line 290 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
18 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 291
detailsMethod GetCxConfig at line 291 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This elemen...
Attack Vector
19 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 302
detailsMethod GetConfigSnapshot at line 302 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector
20 LOW Log_Forging /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs: 303
detailsMethod GetConfigSnapshot at line 303 of /ast-visual-studio-extension/CxPreferences/CxPreferencesUI.cs gets user input from element Text. This ...
Attack Vector

Policy Management Violations (2)
Policy Name Rule(s) Break Build
[SAST-ML0] Not allowed NEW Sast vulnerabilities true
Commercial packages true

@cx-rahul-pidde cx-rahul-pidde changed the title test (AST-107883) Enhance Settings & Welcome Pages and MCP Installation Flow (AST-107883) Mar 31, 2026
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.

3 participants