feat: add filtering options to `ls-remote` (#1063)

* feat: add remote version sorting and filtering

* use `UserVersion` for filtering

* add additional lts filter

* refactor: use `Vec::retain` instead of filtering and collecting

* fix docstring for sort option

* change docstring for filter argument

* refactor vec filtering

Co-authored-by: Brennan Kinney <5098581+polarathene@users.noreply.github.com>

* refactor to use `sort_by_key` and `reverse`

Co-authored-by: Brennan Kinney <5098581+polarathene@users.noreply.github.com>

* add latest flag

* make filter an option as well

* move sort into command as it is presentational

* fix getting latest version

* refactors

* update docs/commands.md

* sort upon installation

* fix clippy

* don't mention rust structs in the cli docs

* sort by default

* remove leftover temp dir from test

---------

Co-authored-by: Brennan Kinney <5098581+polarathene@users.noreply.github.com>
Co-authored-by: Gal Schlezinger <gal@spitfire.co.il>

.changeset/fifty-emus-type.md

@@ -0,0 +1,5 @@
+---
+"fnm": minor
+---
+
+feat: add remote version sorting and filtering

docs/commands.md

@@ -81,6 +81,9 @@ List all remote Node.js versions
 Usage: fnm list-remote [OPTIONS]
 
 Options:
+      --filter <FILTER>
+          Filter versions by a user-defined version or a semver range
+
       --node-dist-mirror <NODE_DIST_MIRROR>
           <https://nodejs.org/dist/> mirror
 
@@ -92,6 +95,21 @@ Options:
 
           [env: FNM_DIR]
 
+      --lts [<LTS>]
+          Show only LTS versions (optionally filter by LTS codename)
+
+      --sort <SORT>
+          Version sorting order
+
+          [default: asc]
+
+          Possible values:
+          - desc: Sort versions in descending order (latest to earliest)
+          - asc:  Sort versions in ascending order (earliest to latest)
+
+      --latest
+          Only show the latest matching version
+
       --log-level <LOG_LEVEL>
           The log level of fnm commands
 

src/commands/ls_remote.rs

@@ -1,20 +1,79 @@
 use crate::config::FnmConfig;
 use crate::remote_node_index;
+use crate::user_version::UserVersion;
+
+use colored::Colorize;
 use thiserror::Error;
 
 #[derive(clap::Parser, Debug)]
-pub struct LsRemote {}
+pub struct LsRemote {
+    /// Filter versions by a user-defined version or a semver range
+    #[arg(long)]
+    filter: Option<UserVersion>,
+
+    /// Show only LTS versions (optionally filter by LTS codename)  
+    #[arg(long)]
+    #[allow(clippy::option_option)]
+    lts: Option<Option<String>>,
+
+    /// Version sorting order
+    #[arg(long, default_value = "asc")]
+    sort: SortingMethod,
+
+    /// Only show the latest matching version
+    #[arg(long)]
+    latest: bool,
+}
+
+#[derive(clap::ValueEnum, Clone, Debug, PartialEq)]
+pub enum SortingMethod {
+    #[clap(name = "desc")]
+    /// Sort versions in descending order (latest to earliest)
+    Descending,
+    #[clap(name = "asc")]
+    /// Sort versions in ascending order (earliest to latest)
+    Ascending,
+}
 
 impl super::command::Command for LsRemote {
     type Error = Error;
 
     fn apply(self, config: &FnmConfig) -> Result<(), Self::Error> {
-        let all_versions = remote_node_index::list(&config.node_dist_mirror)?;
+        let mut all_versions = remote_node_index::list(&config.node_dist_mirror)?;
+
+        if let Some(lts) = &self.lts {
+            match lts {
+                Some(codename) => all_versions.retain(|v| {
+                    v.lts
+                        .as_ref()
+                        .is_some_and(|v_lts| v_lts.eq_ignore_ascii_case(codename))
+                }),
+                None => all_versions.retain(|v| v.lts.is_some()),
+            };
+        }
+
+        if let Some(filter) = &self.filter {
+            all_versions.retain(|v| filter.matches(&v.version, config));
+        }
+
+        if self.latest {
+            all_versions.truncate(1);
+        }
+
+        all_versions.sort_by_key(|v| v.version.clone());
+        if let SortingMethod::Descending = self.sort {
+            all_versions.reverse();
+        }
+
+        if all_versions.is_empty() {
+            eprintln!("{}", "No versions were found!".red());
+            return Ok(());
+        }
 
-        for version in all_versions {
+        for version in &all_versions {
             print!("{}", version.version);
             if let Some(lts) = &version.lts {
-                print!(" ({lts})");
+                print!("{}", format!(" ({lts})").cyan());
             }
             println!();
         }