-
Notifications
You must be signed in to change notification settings - Fork 1
Add prompts command with list, view, and delete subcommands #5
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
…delete subcommands
fe61e19 to
2b5741f
Compare
|
Latest install commands for this PR (updated for commit
|
ankrgyl
left a comment
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.
Thoughts from playing with it
- can we use
resolve_project(...)so that there's a project selector instead of having to include one? - I opened a prompt and it took me to
Ankurs-MacBook-Pro:~/projects/braintrust-cli-2 ankur$ bt prompts view --project prompt-test
✔ Select prompt · Testing 1234
✓ Opened https://staging-www.braintrust.dev/app/braintrustdata.com/p/prompt-test/prompts/ee6462eb-a0a6-445f-8dfc-08d215996c16 in browser
where does staging-www.braintrust.dev come from? i can't find it in the code.
- Also why does it instantly open a link in the UI? I'd rathe rsee the prompt definition in my CLI. I also think it's quite unintuitive for a coding agent that might try to run this
-n, --name <NAME_FLAG> Prompt name (flag)
name is ambiguous. we should have slug here IMO
| pub struct Prompt { | ||
| pub id: String, | ||
| pub name: String, | ||
| pub slug: String, | ||
| pub project_id: String, | ||
| #[serde(default)] | ||
| pub description: Option<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.
not for this PR but it would be nice if we could derive these from our openapi spec
Summary
promptscommand withlist,view, anddeletesubcommandsTest plan
braintrust prompts listshows prompts in a tablebraintrust prompts view <name>opens prompt in webbraintrust prompts delete <name>deletes after confirmation