-
Notifications
You must be signed in to change notification settings - Fork 6
feat: ai debug mode #997
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: feat/ai-context
Are you sure you want to change the base?
feat: ai debug mode #997
Conversation
…e types for improved context handling
…k.json; add external Argo and Flux app types in AIAgent context handling
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request introduces AI debugging enhancements for the application status modal, enabling more detailed context and configuration for AI-powered debugging across different app types (including external Argo and Flux apps). The changes extend the type system to support flexible metadata and new app types, add feature flags for Athena debug mode, and update component prop interfaces to pass debug context and callbacks.
Key Changes
- Enhanced AI debugging with new
debugAgentContexttype that extendsAIAgentContextTypewith optional prompt field - Refactored
AppStatusBodyto construct and pass bothintelligenceConfiganddebugAgentContextto the AI button component - Extended type system to support external app types (
externalHelmChart,externalArgoApp,externalFluxApp) with flexible data structures using TypeScript's advanced mapped types
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/index.ts | Added new feature flag FEATURE_ATHENA_DEBUG_MODE_ENABLE for toggling Athena debug mode |
| src/Shared/types.ts | Updated IntelligenceConfig metadata to accept both string and number values |
| src/Shared/Providers/MainContextProvider/types.ts | Introduced AIAgentAppType, DebugAgentContextType, and refined AIAgentContextType with support for multiple external app types using advanced TypeScript type patterns |
| src/Shared/Components/AppStatusModal/types.ts | Updated prop types to include debugAgentContext and onClick callback for the AI button component, and added handleClose to AppStatusBodyProps |
| src/Shared/Components/AppStatusModal/AppStatusModal.component.tsx | Passed handleClose prop to AppStatusBody component |
| src/Shared/Components/AppStatusModal/AppStatusBody.tsx | Refactored to construct intelligenceConfig and debugAgentContext locally, extracting context from useMainContext, and passing both configs along with handleClose callback to the AI button |
| src/Common/Constants.ts | Added URL constants EXTERNAL_ARGO_APP and EXTERNAL_FLUX_APP for routing external apps |
| package.json | Bumped version from 1.22.0-beta-4 to 1.22.0-beta-5 |
| package-lock.json | Updated version references to match package.json |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…sage for improved error handling
…d error context management
Description
This pull request introduces several enhancements and refactors to support improved AI debugging functionality in the application status modal, along with some supporting type and constant updates. The primary focus is on enabling more detailed context and configuration for AI-powered debugging, especially for different app types and external integrations.
AI Debugging Enhancements
intelligenceConfigand introduceddebugAgentContextinAppStatusBodyand related components, allowing richer context for AI explanations and debugging. TheExplainWithAIButtonnow receives both configurations and a callback for closing the modal. [1] [2] [3] [4] [5]externalHelmChart,externalArgoApp,externalFluxApp), and added more flexible data structures for agent context and debug agent context, enabling more granular information to be passed for AI analytics and debugging. [1] [2]Type and Interface Updates
IntelligenceConfiginterface to allowmetadatavalues to be either strings or numbers, supporting more flexible metadata for AI analytics.AppStatusBodyandAppStatusModalto support the new AI debugging context and callback functionality. [1] [2]Constants and Feature Flags
URLSobject, supporting routing and identification for external Argo and Flux apps.FEATURE_ATHENA_DEBUG_MODE_ENABLEin the global environment interface, allowing toggling of Athena debug mode.Package Version Update
1.22.0-beta-4to1.22.0-beta-5inpackage.jsonto reflect these changes.Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist