Skip to content

chore: update version to 1.22.5-beta-1 in package.json and package-lock.json; add CommonContextDataType to types.ts#1005

Merged
AbhishekA1509 merged 1 commit into
fix/ai-fixesfrom
fix/ai-qa
Jan 15, 2026
Merged

chore: update version to 1.22.5-beta-1 in package.json and package-lock.json; add CommonContextDataType to types.ts#1005
AbhishekA1509 merged 1 commit into
fix/ai-fixesfrom
fix/ai-qa

Conversation

@AbhishekA1509

Copy link
Copy Markdown
Member

Description

This pull request introduces a minor version bump and refactors the context type definitions to improve consistency and maintainability. The main focus is on standardizing shared context data structures for AI agent contexts.

Version update:

  • Bumped the package version in package.json from 1.22.5-beta-0 to 1.22.5-beta-1.

Type definition improvements:

  • Added a new CommonContextDataType type in types.ts to encapsulate shared context properties, including an optional uiMarkup field.
  • Updated the AIAgentContextType type in types.ts to use CommonContextDataType for both APP_DETAILS and RESOURCE_BROWSER_CLUSTER sources, replacing the previous use of Record<string, unknown>.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

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

  • Test A
  • Test B

Checklist

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

…ck.json; add CommonContextDataType to types.ts
@AbhishekA1509 AbhishekA1509 self-assigned this Jan 15, 2026
Copilot AI review requested due to automatic review settings January 15, 2026 11:20

Copilot AI left a comment

Copy link
Copy Markdown

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 bumps the package version to 1.22.5-beta-1 and introduces a new shared type definition CommonContextDataType to standardize context data structures with optional uiMarkup support for AI agent contexts.

Changes:

  • Updated package version from 1.22.5-beta-0 to 1.22.5-beta-1 in package.json and package-lock.json
  • Added CommonContextDataType type definition to encapsulate shared context properties with optional uiMarkup field
  • Applied CommonContextDataType to RESOURCE_BROWSER_CLUSTER and externalFluxApp context types

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
package.json Version bump to 1.22.5-beta-1
package-lock.json Version bump to 1.22.5-beta-1
src/Shared/Providers/MainContextProvider/types.ts Added CommonContextDataType and applied it to two AI agent context source types
Comments suppressed due to low confidence (1)

src/Shared/Providers/MainContextProvider/types.ts:99

  • The CommonContextDataType is only applied to externalFluxApp (line 104) but not to the other app types (devtronApp, devtronHelmChart, externalHelmChart, externalArgoApp). This creates inconsistency where some app types support uiMarkup and other dynamic properties while others don't. Consider applying CommonContextDataType to all app types within the APP_DETAILS source for consistency, or document why only externalFluxApp requires this additional data structure.
              | AIAgentAppDataType<
                    'devtronApp' | 'devtronHelmChart',
                    'appId' | 'appName' | 'envId' | 'envName' | 'clusterId'
                >
              | AIAgentAppDataType<'externalHelmChart', 'appId' | 'appName' | 'clusterId' | 'namespace'>
              | AIAgentAppDataType<'externalArgoApp', 'appName' | 'clusterId' | 'namespace'>

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AbhishekA1509 AbhishekA1509 merged commit d837685 into fix/ai-fixes Jan 15, 2026
9 checks passed
@AbhishekA1509 AbhishekA1509 deleted the fix/ai-qa branch January 15, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants