PR5: Copilot Integration & Settings UI Enhancements (AST-109633)#321
Open
cx-rahul-pidde wants to merge 5 commits into
Open
PR5: Copilot Integration & Settings UI Enhancements (AST-109633)#321cx-rahul-pidde wants to merge 5 commits into
cx-rahul-pidde wants to merge 5 commits into
Conversation
- Refactor CopilotIntegration with improved error handling - Enhance CxAssistCopilotActions for better action handling - Add CxAssistConstants for centralized configuration - Improve CxOneAssistSettingsModule and UI - Enhance CxPreferencesUI for better settings management - Update CxWindowPackage with settings integration - Enhance CxConstants with new configuration options - Update CxAssistFindingsControl with better styling - Improve filter button styling for light/dark themes - Update CxAsca model - Optimize collapseall.png icon - Update project file with new references Base branch: feature/devassist-integration-branch Related to: AST-109633
|
Policy Management Violations (2)
|
cx-rahul-pidde
requested review from
cx-anurag-dalke,
cx-rakesh-kadu and
cx-umesh-waghode
cx-rakesh-kadu
requested changes
Apr 21, 2026
…ed state - Use IsScannerEnabled() instead of checking v.Scanner != 0, which was incorrect - Refactor to collect files for removal first, then remove them (avoid modifying dict during iteration) - Add clarifying comments and improve code structure - Remove unused exception variable in CxAssistFindingsControl Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
- Replace Debug.WriteLine with CxAssistOutputPane.WriteToOutputPane for clipboard errors - Aligns with codebase pattern: user-facing errors logged to Output Pane - Affects all three copy operations: JSON details, message, fix prompt - Users will now see clipboard failures in the Output Pane Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
- Replace Debug.WriteLine with CxAssistOutputPane for: * FINDINGS_WINDOW_INITIATED (when findings window loads) * FIX_PROMPT_COPIED (when fix prompt copied successfully) * ICONS_LOADED_FOR_THEME (when severity icons load for current theme) - Aligns with codebase pattern: lifecycle and success events to Output Pane - Leaves debug logging for exception details and diagnostic traces Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
cx-rakesh-kadu
approved these changes
Apr 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Copilot Integration & Settings UI
Files Changed (13 modified)
Related Issue
AST-109633