-
Notifications
You must be signed in to change notification settings - Fork 25
Migrate the AI Agent creation form to LS #309
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: main
Are you sure you want to change the base?
Migrate the AI Agent creation form to LS #309
Conversation
…eManager as a parameter
b8b52d6
to
8a7bf71
Compare
…isible symbols instead of module nodes
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 PR migrates the AI Agent creation form to use the Language Server (LS) approach, implementing a more comprehensive agent configuration interface with additional properties and improved property management.
- Enhanced agent property handling with support for system prompts, tools, model providers, and advanced configuration options
- Migrated from basic agent configuration to a full template-based approach using the language server
- Added support for role and instructions as separate configurable properties instead of being embedded in system prompts
Reviewed Changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
test JSON files | Updated test configurations to reflect the new agent property structure and template-based approach |
FlowModelGeneratorService.java | Added WorkspaceManager parameter to ModelGenerator constructor calls |
AgentsManagerService.java | Updated method call to include WorkspaceManager parameter |
ParamUtils.java | Added utility method for removing string delimiters from values |
FlowNodeUtil.java | Added comprehensive utility methods for property copying and node building |
FileSystemUtils.java | Added method to resolve file paths from codedata |
CallBuilder.java | Enhanced template data handling with additional codedata properties |
AgentCallBuilder.java | Major refactor to implement full agent creation workflow with property management |
NodeBuilder.java | Updated template data handling to properly set isNew flag from context |
Core classes | Updated constructors and method signatures to include WorkspaceManager parameter |
Comments suppressed due to low confidence (1)
flow-model-generator/modules/flow-model-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/model/node/AgentCallBuilder.java:1
- The word 'mutiply' is misspelled. It should be 'multiply'.
/*
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
...-generator-ls-extension/src/test/resources/agents_manager/config/agent_call_flow_node_1.json
Outdated
Show resolved
Hide resolved
...-generator-ls-extension/src/test/resources/agents_manager/config/agent_call_flow_node_2.json
Outdated
Show resolved
Hide resolved
...el-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/utils/FlowNodeUtil.java
Outdated
Show resolved
Hide resolved
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
Copilot reviewed 23 out of 23 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
...odel-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/utils/ParamUtils.java
Outdated
Show resolved
Hide resolved
...tor-core/src/main/java/io/ballerina/flowmodelgenerator/core/model/node/AgentCallBuilder.java
Show resolved
Hide resolved
...el-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/utils/FlowNodeUtil.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <[email protected]>
Purpose
Resolves wso2/product-ballerina-integrator#1130
This PR introduces an update to the
AGENT_CALL
template, allowing it to include fields from theAGENT
node template. This allows us to render the agent form in the frontend with agent fields likeRole
,Instructions
,Max Iterations
etc as well as agent call related fields likeQuery
,Session ID
etc. This updated node template can be used to generate source code for the default model provider + agent definition + agent call.It also includes support for editing the agent definition in both local and module level scopes in the low code UI.
Goals
Approach
User stories
Release note
Documentation
Training
Certification
Marketing
Automation tests
Security checks
Samples
Related PRs
Migrations (if applicable)
Test environment
Learning