chore(hok): command description tweaks

Signed-off-by: Chawye Hsu <[email protected]>

src/clap_app.rs

@@ -54,7 +54,7 @@ pub fn build() -> Command {
         )
         .subcommand(
             Command::new("cache")
-                .about("List or remove download caches")
+                .about("Package cache management")
                 .arg_required_else_help(true)
                 .subcommand(
                     Command::new("list")
@@ -84,7 +84,7 @@ pub fn build() -> Command {
         )
         .subcommand(
             Command::new("cat")
-                .about("Display manifest content of a package")
+                .about("Inspect the manifest of a package")
                 .arg_required_else_help(true)
                 .arg(
                     Arg::new("package")
@@ -111,7 +111,7 @@ pub fn build() -> Command {
         )
         .subcommand(
             Command::new("config")
-                .about("Configuration manipulations")
+                .about("Configuration management")
                 .arg_required_else_help(true)
                 .subcommand(
                     Command::new("edit").about("Edit the config file [default editor: notepad]"),
@@ -145,7 +145,7 @@ pub fn build() -> Command {
         )
         .subcommand(
             Command::new("hold")
-                .about("Hold package(s) to disable updates")
+                .about("Hold package(s) to disable changes")
                 .arg_required_else_help(true)
                 .arg(
                     Arg::new("package")
@@ -156,17 +156,17 @@ pub fn build() -> Command {
         )
         .subcommand(
             Command::new("home")
-                .about("Open the homepage of given package")
+                .about("Browse the homepage of a package")
                 .arg_required_else_help(true)
                 .arg(Arg::new("package").help("The package name").required(true)),
         )
         .subcommand(
             Command::new("info")
-                .about("Display information about a package")
+                .about("Show package(s) basic information")
                 .arg_required_else_help(true)
                 .arg(
-                    Arg::new("package")
-                        .help("Name of the package to be inspected")
+                    Arg::new("query")
+                        .help("The query string (regex supported)")
                         .required(true),
                 ),
         )
@@ -323,7 +323,7 @@ pub fn build() -> Command {
         )
         .subcommand(
             Command::new("unhold")
-                .about("Unhold package(s) to enable updates")
+                .about("Unhold package(s) to enable changes")
                 .arg_required_else_help(true)
                 .arg(
                     Arg::new("package")
@@ -381,7 +381,7 @@ pub fn build() -> Command {
         )
         .subcommand(
             Command::new("update")
-                .about("Fetch and update all buckets")
+                .about("Fetch and update subscribed buckets")
                 .alias("u"),
         )
         .subcommand(

src/cmd/config.rs

@@ -31,6 +31,7 @@ pub fn cmd_config(matches: &ArgMatches, session: &Session) -> Result<()> {
                 .map(|s| s.as_str())
                 .unwrap_or_default();
             operation::config_set(session, key, value)?;
+            println!("Config '{}' has been set to '{}'", key, value);
             Ok(())
         }
         Some(("unset", args)) => {
@@ -39,6 +40,7 @@ pub fn cmd_config(matches: &ArgMatches, session: &Session) -> Result<()> {
                 .map(|s| s.as_str())
                 .unwrap_or_default();
             operation::config_set(session, key, "")?;
+            println!("Config '{}' has been unset", key);
             Ok(())
         }
         _ => unreachable!(),

src/cmd/hold.rs

@@ -11,7 +11,7 @@ pub fn cmd_hold(matches: &ArgMatches, session: &Session) -> Result<()> {
         .unwrap_or_default();
 
     for name in packages {
-        print!("Holding {}... ", name);
+        print!("Holding {}...", name);
         match operation::package_hold(session, name, true) {
             Ok(..) => {
                 println!("{}", "Ok".green());

src/cmd/info.rs

@@ -4,18 +4,23 @@ use libscoop::{operation, Session};
 use crate::Result;
 
 pub fn cmd_info(matches: &ArgMatches, session: &Session) -> Result<()> {
-    if let Some(query) = matches.get_one::<String>("package") {
+    if let Some(query) = matches.get_one::<String>("query") {
         let queries = vec![query.as_str()];
         let options = vec![];
         let packages = operation::package_query(session, queries, options, false)?;
         let length = packages.len();
         match length {
             0 => eprintln!("Could not find package for query '{}'.", query),
             _ => {
-                println!("Found {} package(s) for query '{}':", length, query);
+                if length == 1 {
+                    println!("Found 1 package for query '{}':", query);
+                } else {
+                    println!("Found {} package(s) for query '{}':", length, query);
+                }
+
                 for (idx, pkg) in packages.iter().enumerate() {
                     // Ident
-                    // println!("Identity: {}/{}", pkg.bucket, pkg.name);
+                    println!("Identity: {}", pkg.ident());
                     // Name
                     println!("Name: {}", pkg.name());
                     // Bucket
@@ -30,7 +35,7 @@ pub fn cmd_info(matches: &ArgMatches, session: &Session) -> Result<()> {
                     // Homepage
                     println!("Homepage: {}", pkg.homepage());
                     // License
-                    // println!("License: {}", pkg.license);
+                    println!("License: {}", pkg.license());
                     // Binaries
                     println!(
                         "Shims: {}",

src/cmd/unhold.rs

@@ -11,7 +11,7 @@ pub fn cmd_unhold(matches: &ArgMatches, session: &Session) -> Result<()> {
         .unwrap_or_default();
 
     for name in packages {
-        print!("Unholding {}... ", name);
+        print!("Unholding {}...", name);
         match operation::package_hold(session, name, false) {
             Ok(..) => {
                 println!("{}", "Ok".green());