-
Notifications
You must be signed in to change notification settings - Fork 68
feat(agent): add support for agent collab, orch and FM in prompt #1011
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
Conversation
The following code snippet:
|
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.
Overall, the interfaces appear well-designed and intuitive. My only suggestion pertains to the Agent Descriptor. In my opinion, it would be more consistent and clear to replace it with IAgentAlias
. This aligns better the naming convention used in the console, which features a simple Agent Alias and makes no mention of AgentDescriptor.
Apart from this minor adjustment, the work is pretty neat. Great job!
Hi folks, will this PR be merged anytime soon ? |
any update on this request ? |
Testing custom orchestration: with the following code snippet (lambda code from here:
The stack deploys correctly and custom orchestration is correctly applied: ![]() Custom logic is correctly invoked: ![]() |
PR is open now, once reviewed it will be in |
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.
LGTM
Fixes #998 and #999 and #1011
See comments in the PR below to get output of tests
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.