[AI Bundle] Add chat console command for interactive AI agent conversations #395
Conversation
carsonbot
added
AI Bundle
OskarStark
force-pushed
the
feature/chat-command
branch
2 times, most recently
from
be7fc49 to
7429039
Compare
|
Ready for review |
GromNaN
left a comment
GromNaN
left a comment
There was a problem hiding this comment.
Nice feature!
I think there are small details we can improve.
|
Ready to merge from my side |
OskarStark
force-pushed
the
feature/chat-command
branch
2 times, most recently
from
3a9bf1a to
73ba017
Compare
|
oh, i wonder, did you try to use streaming? otherwise maybe a follow up |
chr-hertel
left a comment
chr-hertel
left a comment
There was a problem hiding this comment.
found that one bug about the ChoiceQuestion
No not yet, lets follow up |
GromNaN
left a comment
GromNaN
left a comment
There was a problem hiding this comment.
I think we can simplify by using short agent names everywhere.
|
I agree with @GromNaN that we can simplify this a lot if we only allow |
|
It works when I provide a correct one, or a wrong one. But it does not work when I provide none... investigating. |
|
Works now! |
|
Common, i feel like you're trolling me ... 😂 
I'm going to bed now - sad :D |
|
|
Got a 👍 from @chr-hertel via WhatsApp 😄 |
OskarStark
force-pushed
the
feature/chat-command
branch
from
2872221 to
6309a0b
Compare
| protected function interact(InputInterface $input, OutputInterface $output): void | ||
| { | ||
| // Skip interaction in non-interactive mode | ||
| if (!$input->isInteractive()) { |
There was a problem hiding this comment.
useless check. The parent class already skips calling the interact method entirely.
| protected function configure(): void | ||
| { | ||
| $this | ||
| ->addArgument('agent', InputArgument::OPTIONAL, 'The name of the agent to chat with') |
There was a problem hiding this comment.
The argument should be marked as required, as you use the interact method to fill it anyway. this provides better documentation (and standard error reporting)
| } | ||
|
|
||
| $agentArg = $input->getArgument('agent'); | ||
| $this->agentName = \is_string($agentArg) ? $agentArg : ''; |
There was a problem hiding this comment.
Why using instance properties (making the service stateful) when they are only used locally ? You should use local variables instead.
|
Thanks for your feedback @stof, makes sense |
This PR was squashed before being merged into the main branch. Discussion ---------- [AI Bundle] Improve `ChatCommand` | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | Docs? | no | Issues | Follows #395 | License | MIT cc `@stof` Commits ------- c0e7f3c [AI Bundle] Improve `ChatCommand`
…(OskarStark) This PR was merged into the main branch. Discussion ---------- [AI Bundle] Rename `ai:chat` command to `ai:agent:call` | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | Docs? | no | Issues | Follows #395 | License | MIT I think we can have ai:chat command soon after #675 Commits ------- ec7018f Rename ai:chat command to ai:agent:call with backward compatibility
6 participants
Implement a new console command
bin/console ai:chatthat allows users to interactively chat with different AI agents (blog, stream, youtube, wikipedia, audio). The command supports custom system prompts and maintains conversation history.Demo