Miscellaneous CLI fixes (#346)

* show latest_major_change_at in the cli

* fix broken `meta` cli command

Author: pietroalbini

rfd-cli/src/main.rs

@@ -430,6 +430,7 @@ impl ProgenitorCliConfig for Context {
                 .printer()
                 .unwrap()
                 .output_oauth_secret(reserialize(value)),
+            "RfdWithoutContent" => self.printer().unwrap().output_rfd_meta(reserialize(value)),
             "Array_of_RfdWithoutContent" => {
                 self.printer().unwrap().output_rfd_list(reserialize(value))
             }

rfd-cli/src/printer/json.rs

@@ -71,6 +71,10 @@ impl CliOutput for RfdJsonPrinter {
         println!("{}", serde_json::to_string(&value).unwrap())
     }
 
+    fn output_rfd_meta(&self, value: types::RfdWithoutContent) {
+        println!("{}", serde_json::to_string(&value).unwrap())
+    }
+
     fn output_rfd_list(&self, value: Vec<types::RfdWithoutContent>) {
         println!("{}", serde_json::to_string(&value).unwrap())
     }

rfd-cli/src/printer/mod.rs

@@ -31,6 +31,7 @@ pub trait CliOutput {
     fn output_oauth_redirect_uri(&self, value: types::OAuthClientRedirectUri) {}
     fn output_oauth_secret_initial(&self, value: types::InitialOAuthClientSecretResponse) {}
     fn output_oauth_secret(&self, value: types::OAuthClientSecret) {}
+    fn output_rfd_meta(&self, value: types::RfdWithoutContent) {}
     fn output_rfd_list(&self, value: Vec<types::RfdWithoutContent>) {}
     fn output_rfd_full(&self, value: types::RfdWithRaw) {}
     fn output_rfd(&self, value: types::Rfd) {}
@@ -164,6 +165,13 @@ impl CliOutput for Printer {
         }
     }
 
+    fn output_rfd_meta(&self, value: types::RfdWithoutContent) {
+        match self {
+            Self::Json(printer) => printer.output_rfd_meta(value),
+            Self::Tab(printer) => printer.output_rfd_meta(value),
+        }
+    }
+
     fn output_rfd_list(&self, value: Vec<types::RfdWithoutContent>) {
         match self {
             Self::Json(printer) => printer.output_rfd_list(value),

rfd-cli/src/printer/tab.rs

@@ -101,6 +101,10 @@ impl CliOutput for RfdTabPrinter {
         self.print_cli_output(&value, None);
     }
 
+    fn output_rfd_meta(&self, value: types::RfdWithoutContent) {
+        self.print_cli_output(&value, None);
+    }
+
     fn output_rfd_list(&self, value: Vec<types::RfdWithoutContent>) {
         self.print_cli_output(&value, Some("rfds".to_string()));
     }
@@ -162,6 +166,15 @@ impl TabDisplay for types::Rfd {
         printer.print_field(tw, level, "id", &self.id);
         printer.print_field(tw, level, "rfd_number", &self.rfd_number);
         printer.print_field(tw, level, "visibility", &self.visibility.to_string());
+        printer.print_field(
+            tw,
+            level,
+            "latest_major_change_at",
+            &self
+                .latest_major_change_at
+                .map(|d| d.to_string())
+                .unwrap_or_else(|| "--".to_string()),
+        );
         printer.print_field(
             tw,
             level,
@@ -484,6 +497,15 @@ impl TabDisplay for RfdWithoutContent {
                 .map(|d| d.to_string())
                 .unwrap_or_else(|| "--".to_string()),
         );
+        printer.print_field(
+            tw,
+            level,
+            "latest_major_change_at",
+            &self
+                .latest_major_change_at
+                .map(|d| d.to_string())
+                .unwrap_or_else(|| "--".to_string()),
+        );
     }
 }
 
@@ -557,6 +579,15 @@ impl TabDisplay for RfdWithRaw {
                 .map(|d| d.to_string())
                 .unwrap_or_else(|| "--".to_string()),
         );
+        printer.print_field(
+            tw,
+            level,
+            "latest_major_change_at",
+            &self
+                .latest_major_change_at
+                .map(|d| d.to_string())
+                .unwrap_or_else(|| "--".to_string()),
+        );
         writeln!(tw, "");
         if let Some(content) = &self.content {
             writeln!(tw, "{}", content);