-
-
Notifications
You must be signed in to change notification settings - Fork 19
feat: add configurable agent namespaces #26
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
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
feat: add configurable agent namespaces #26
Conversation
This allows the agent:chat command to find agent classes on namespaces specified in the config.
src/Commands/AgentChatCommand.php
Outdated
@@ -4,6 +4,7 @@ | |||
|
|||
use Illuminate\Console\Command; | |||
use Illuminate\Support\Str; | |||
use LarAgent\Tests\Fakes\FakeLlmDriver; |
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.
Thanks @johalternate for your PR! All looks good 💪 Only this line seems like an extra, please remove it and I will merge the PR 👍
'namespaces' => [ | ||
'App\\AiAgents\\', | ||
'App\\Agents\\', | ||
], |
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.
Can we also add a comment for this config to make sure everyone understands what is it for? Later, I will make sure to pick the first one from here while creating a new agent with command
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.
You got it. I will get on it later today.
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.
Thank you @johalternate for this PR! 🙏 If you already joined the LarAgent discord community, ping me there, I will give you a contributor role! 💪
This allows the agent:chat command to find agent classes on namespaces specified in the config.