-
Notifications
You must be signed in to change notification settings - Fork 8.5k
[agent builder] agent workflow step #244957
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?
Conversation
…nto feature/onechat-workflow-step # Conflicts: # x-pack/platform/plugins/shared/onechat/server/step_types/run_agent_step.ts
| /** | ||
| * Output schema for the run agent step. | ||
| */ | ||
| export const OutputSchema = z.object({ | ||
| /** | ||
| * The agent's response message. | ||
| */ | ||
| output: z.union([z.string(), z.any()]), | ||
| }); |
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.
The output property is already set by the execution engine, this will make the step agent response be inside step.agent.output.output.
Maybe it would be better to just do:
| /** | |
| * Output schema for the run agent step. | |
| */ | |
| export const OutputSchema = z.object({ | |
| /** | |
| * The agent's response message. | |
| */ | |
| output: z.union([z.string(), z.any()]), | |
| }); | |
| /** | |
| * Output schema for the run agent step. | |
| */ | |
| export const OutputSchema = z.union([z.string(), z.any()]); |
| /** | ||
| * The user input message to send to the agent. | ||
| */ | ||
| input: z.string(), |
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.
NIT: Similar, we will have step.agent.input.input, maybe better to use message?
💔 Build Failed
Failed CI StepsHistory
|
| observabilityOnboarding: 12872 | ||
| observabilityShared: 75115 | ||
| onechat: 25223 | ||
| onechat: 88464 |
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.
Are we able to bring this back down?
Summary
resolves https://github.com/elastic/search-team/issues/11874