Merge pull request #176 from MordechaiHadad/fix/capacity-overflow

Fix capacity overflow for list command

src/handlers/list_handler.rs

@@ -1,6 +1,6 @@
 use anyhow::{anyhow, Result};
 use regex::Regex;
-use std::fs;
+use std::{fs, path::PathBuf};
 use yansi::Paint;
 
 use crate::{
@@ -11,10 +11,10 @@ use crate::{
 pub async fn start(config: Config) -> Result<()> {
  let downloads_dir = directories::get_downloads_directory(&config).await?;
 
- let paths = fs::read_dir(downloads_dir)?
- .filter_map(|e| e.ok())
+ let paths: Vec<PathBuf> = fs::read_dir(downloads_dir)?
+ .filter_map(Result::ok)
  .map(|entry| entry.path())
- .collect::<Vec<_>>();
+ .collect();
 
  if paths.is_empty() {
  return Err(anyhow!("There are no versions installed"));

src/helpers/version/mod.rs

@@ -54,13 +54,9 @@ pub async fn parse_version_type(client: &Client, version: &str) -> Result<Parsed
  });
  }
 
- let alphanumeric_regex = Regex::new(r"^[a-zA-Z0-9]{8}$")?;
- let separated_version: Vec<&str> = version.split('-').collect();
+ let rollback_regex = Regex::new(r"nightly-[a-zA-Z0-9]{7,8}")?;
 
- if separated_version[0] == "nightly"
- && (hash_regex.is_match(separated_version[1])
- || alphanumeric_regex.is_match(separated_version[1]))
- {
+ if rollback_regex.is_match(version) {
  return Ok(ParsedVersion {
  tag_name: version.to_string(),
  version_type: VersionType::NightlyRollback,

src/helpers/version/nightly.rs

@@ -21,7 +21,7 @@ pub async fn produce_nightly_vec(config: &Config) -> Result<Vec<LocalNightly>> {
  let downloads_dir = directories::get_downloads_directory(config).await?;
  let mut paths = fs::read_dir(&downloads_dir).await?;
 
- let regex = Regex::new(r"nightly-[a-zA-Z0-9]{8}")?;
+ let regex = Regex::new(r"nightly-[a-zA-Z0-9]{7,8}")?;
 
  let mut nightly_vec: Vec<LocalNightly> = Vec::new();