Skip to content

Conversation

@LakshanWeerasinghe
Copy link
Contributor

@LakshanWeerasinghe LakshanWeerasinghe commented May 29, 2025

Purpose

$subject

Fixes Agent manager block is not rendering in BI

Screen.Recording.2025-05-29.at.10.27.16.mov

@NipunaRanasinghe NipunaRanasinghe requested a review from Copilot May 29, 2025 05:17
Copy link
Contributor

Copilot AI left a 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 PR fixes the issue with rendering agent nodes by explicitly specifying the package name in JSON config files and updating the core generators to correctly distinguish between moduleName and packageName. The key changes include:

  • Adding the "packageName": "ai" property to various JSON configuration files.
  • Updating AvailableNodesGenerator and AgentsGenerator to supply packageName correctly and use moduleName from module IDs.
  • Adjusting method calls in getActions to extract and use moduleName and packageName separately.

Reviewed Changes

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

File Description
/config/.json Added "packageName": "ai" across multiple JSON configuration files for agent nodes.
AvailableNodesGenerator.java Inserted .packageName(AI_AGENT) to correctly set the package name for agent call nodes.
AgentsGenerator.java Revised method chaining to correctly distinguish and pass moduleName and packageName.
agents_manager/*.json Updated JSON configs to specify packageName for different provider initializations.

@LakshanWeerasinghe LakshanWeerasinghe merged commit 6ac1953 into ballerina-platform:2201.12.x May 29, 2025
3 checks passed
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