Skip to content

Comments

Fix with CxOne assist#386

Merged
cx-anand-nandeshwar merged 8 commits intofeature/ignore_AST-109610from
feature/Cxone-Assist-Fix
Jan 13, 2026
Merged

Fix with CxOne assist#386
cx-anand-nandeshwar merged 8 commits intofeature/ignore_AST-109610from
feature/Cxone-Assist-Fix

Conversation

@cx-amol-mane
Copy link
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

Describe the purpose of this PR along with any background information and the impacts of the proposed change.

References

Include supporting link to GitHub Issue/PR number

Testing

Describe how this change was tested. Be specific about anything not tested and reasons why. If this solution has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

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

…es directly; updated constant for fix prompt naming.
…notifications

- Introduced `AIIntegrationService` and `AIProvider` interface for managing AI providers.
- Implemented `CopilotProvider` to handle interactions with GitHub Copilot, including prompt sending and UI automation.
- Updated `RemediationManager` to utilize AI integration for automated fixes, with fallback to clipboard if Copilot is unavailable.
- Added new resources for success and error notifications related to AI integration.
- Refactored `DevAssistUtils` to include a method for sending prompts to Copilot.
- Enhanced logging and error handling throughout the integration process.
…om fixWithAI method

- Removed success and fallback notification parameters from `fixWithAI` method in `DevAssistUtils`.
- Updated `RemediationManager` to reflect changes in `fixWithAI`, eliminating unnecessary notifications.
- Cleaned up resource files by deleting unused constants related to AI fix notifications.
…faces

- Deleted `AIIntegrationResult`, `AIIntegrationService`, `AIProvider`, `AIProviderInitializer`, `AIProviderRegistry`, and `CopilotProvider` classes to streamline the codebase.
- Updated `DevAssistUtils` to directly utilize `CopilotIntegration` for AI prompt handling, removing the need for the now-removed AI provider architecture.
- Cleaned up related imports and references throughout the project.
…or improved verbosity during integration workflow
…tomizable delays via system properties. Updated delay constants to use Integer.getInteger for dynamic configuration, improving flexibility for UI automation troubleshooting.
@github-actions
Copy link

github-actions bot commented Jan 13, 2026

Logo
Checkmarx One – Scan Summary & Detailsf9c7307f-5912-4108-b18e-e0c16755f907

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

…rity and functionality

- Introduced constants for Copilot UI components and chat modes in `CopilotIntegration` to enhance readability and maintainability.
- Updated `RemediationManager` to build remediation prompts based on scan engine type, improving the structure of the `fixWithCxOneAssist` method.
- Moved AI integration logic from `DevAssistUtils` to `RemediationManager`, streamlining the process of sending prompts to Copilot.
- Enhanced logging for remediation processes to provide clearer insights during execution.
Copy link
Contributor

@cx-anand-nandeshwar cx-anand-nandeshwar left a comment

Choose a reason for hiding this comment

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

Approved

@cx-anand-nandeshwar cx-anand-nandeshwar merged commit 90129b3 into feature/ignore_AST-109610 Jan 13, 2026
4 of 7 checks passed
@cx-anand-nandeshwar cx-anand-nandeshwar deleted the feature/Cxone-Assist-Fix branch January 13, 2026 16:10
@cx-anand-nandeshwar cx-anand-nandeshwar restored the feature/Cxone-Assist-Fix branch January 13, 2026 16:11
@cx-anand-nandeshwar cx-anand-nandeshwar deleted the feature/Cxone-Assist-Fix branch January 13, 2026 16:13
cx-aniket-shinde added a commit that referenced this pull request Jan 14, 2026
*  Added  GitHub Copilot AI integration on click of Fix with Checkmarx One Assist option

Co-authored-by: Amol Mane <22643905+cx-amol-mane@users.noreply.github.com>
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