enhance(cli): Add conversation subcommand aliases (#454)

Each subcommand in the `conversation` command now has one or more short
aliases. Visible aliases (shown in `--help`) are single- letter
shortcuts: `l` for `ls`, `s` for `show`, `u` for `use`, `e` for `edit`,
`f` for `fork`, `g` for `grep`, and `p` for `print`. Hidden extras
include `list` for `ls`, `rg` for `grep`, and `remove`, `rem`, `delete`,
`del` for `rm`.

Also replaces a direct equality check against `QuestionTarget::User`
with the idiomatic `target.is_user()` helper in the tool coordinator.

Signed-off-by: Jean Mertz <git@jeanmertz.com>

Author: JeanMertz

crates/jp_cli/src/cmd/conversation.rs

@@ -34,32 +34,35 @@ impl Conversation {
 #[derive(Debug, clap::Subcommand)]
 enum Commands {
     /// Remove conversations.
-    #[command(name = "rm")]
+    #[command(name = "rm", aliases = ["remove", "rem", "delete", "del"])]
     Remove(rm::Rm),
 
     /// List conversations.
-    #[command(name = "ls")]
+    #[command(name = "ls", alias = "list", visible_alias = "l")]
     List(ls::Ls),
 
     /// Show conversation details.
-    #[command(name = "show")]
+    #[command(name = "show", visible_alias = "s")]
     Show(show::Show),
 
     /// Set the active conversation.
-    #[command(name = "use")]
+    #[command(name = "use", visible_alias = "u")]
     Use(use_::Use),
 
     /// Edit conversation details.
-    #[command(name = "edit")]
+    #[command(name = "edit", visible_alias = "e")]
     Edit(edit::Edit),
 
     /// Fork a conversation.
+    #[command(name = "fork", visible_alias = "f")]
     Fork(fork::Fork),
 
     /// Search through conversation history.
+    #[command(name = "grep", alias = "rg", visible_alias = "g")]
     Grep(grep::Grep),
 
     /// Print conversation history to the terminal.
+    #[command(name = "print", visible_alias = "p")]
     Print(print::Print),
     // /// Merge a conversation.
     // Merge(merge::Merge),

crates/jp_cli/src/cmd/query/tool/coordinator.rs

@@ -1021,7 +1021,7 @@ impl ToolCoordinator {
                     "Tool question received, routing to target",
                 );
 
-                if is_tty && target == QuestionTarget::User {
+                if is_tty && target.is_user() {
                     if *prompt_active {
                         pending_prompts.push_back(PendingPrompt::Question { index, question });
                     } else {