-
Notifications
You must be signed in to change notification settings - Fork 132
Add support for commands in the CLI/TUI/API #377
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
I think I would prefer if we could run a command with And maybe have Things got merged and this has conflicts now, sorry |
e95c5a3
to
72fd7de
Compare
conflicts resolved. i'm fine without the -c, but do we still want to support messages? right now i can add the |
-c without a command currently does the listing like i assume you want |
What about: $ cagent command ls agent.yaml $ cagent command run agent.yaml <command> I'm only asking mind you :) |
Oh listing with --command/-c wasn't meant as the way to list available commands, but more as a ux thing for users who mess up so they don't have to type another command to see what commands are available. I'm ok with either approach really, having a dedicated cagent command for commands (lol) feels a bit overkill to me right now for what we're trying to do, which is just have a default first message from the agent file sent automatically, but I can make those changes if that's what we agree upon. Would love to hear from the others as well but if I don't then I'll just make these changes and move on with other things :) |
a0c4b83
to
67b6883
Compare
Unless there are hard feelings around this, I'd honestly prefer if we merge this as is for now (meaning without a new top level CLI command) so this doesn't remain stuck here, and then iterate if we need to. After using it a bit we can decide if we feel the need to make a top level |
closes docker#128 Signed-off-by: Christopher Petito <[email protected]>
67b6883
to
763e8de
Compare
Let's go |
Proposal for adding support for commands in the CLI/TUI/API
Example yaml file
Example CLI usage
closes #128