Add support for commands in the CLI/TUI/API - #377
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 <chrisjpetito@gmail.com>
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