fix(install): use deterministic hash for package without checksum

Author: QaidVoid

Cargo.lock

@@ -25,7 +25,6 @@ miette = { workspace = true }
 minisign-verify = "0.2.4"
 nu-ansi-term = "0.50.3"
 once_cell = "1.21.3"
-rand = "0.9.2"
 rayon = { workspace = true }
 regex = { workspace = true }
 semver = "1.0.27"

crates/soar-cli/Cargo.toml

@@ -12,7 +12,6 @@ use std::{
 use indicatif::{MultiProgress, ProgressBar, ProgressStyle};
 use minisign_verify::{PublicKey, Signature};
 use nu_ansi_term::Color::{Blue, Cyan, Green, Magenta, Red, Yellow};
-use rand::{distr::Alphanumeric, Rng};
 use soar_config::{config::get_config, utils::default_install_patterns};
 use soar_core::{
     database::{connection::DieselDatabase, models::Package},
@@ -31,7 +30,7 @@ use soar_db::repository::{
 };
 use soar_dl::types::Progress;
 use soar_package::integrate_package;
-use soar_utils::{hash::calculate_checksum, pattern::apply_sig_variants};
+use soar_utils::{hash::{calculate_checksum, hash_string}, pattern::apply_sig_variants};
 use tabled::{
     builder::Builder,
     settings::{themes::BorderCorrection, Panel, Style},
@@ -1066,11 +1065,11 @@ pub async fn install_single_package(
         .filter(|s| s.len() >= 12)
         .map(|s| s[..12].to_string())
         .unwrap_or_else(|| {
-            rand::rng()
-                .sample_iter(&Alphanumeric)
-                .take(12)
-                .map(char::from)
-                .collect()
+            let input = format!(
+                "{}:{}:{}",
+                target.package.pkg_id, target.package.pkg_name, target.package.version
+            );
+            hash_string(&input)[..12].to_string()
         });
 
     let install_dir = get_config()

crates/soar-cli/src/install.rs

@@ -101,6 +101,7 @@ pub fn handle_progress(state: Progress, progress_bar: &ProgressBar) {
         } => {
             progress_bar.set_length(total);
             progress_bar.set_position(current);
+            progress_bar.reset_elapsed();
         }
         Progress::Chunk {
             current, ..
@@ -143,6 +144,7 @@ pub fn handle_install_progress(
             if let Some(pb) = progress_bar {
                 pb.set_length(total);
                 pb.set_position(current);
+                pb.reset_elapsed();
             }
         }
         Progress::Chunk {

crates/soar-cli/src/progress.rs

@@ -1,6 +1,6 @@
 use std::{
     fs::{self, File, OpenOptions, Permissions},
-    io::{Read as _, Write as _},
+    io::{Read as _, Seek as _, SeekFrom, Write as _},
     os::unix::fs::PermissionsExt as _,
     path::{Path, PathBuf},
     sync::Arc,
@@ -254,8 +254,13 @@ impl Download {
         if output_path.is_file() {
             match self.overwrite {
                 OverwriteMode::Skip => {
-                    debug!(path = %output_path.display(), "file exists, skipping download");
-                    return Ok(output_path);
+                    // Only skip if there's no resume info (complete download)
+                    // If resume info exists, it's a partial download that should continue
+                    if resume_info.is_none() {
+                        debug!(path = %output_path.display(), "file exists, skipping download");
+                        return Ok(output_path);
+                    }
+                    debug!(path = %output_path.display(), "file exists but is partial, resuming download");
                 }
                 OverwriteMode::Force => {
                     debug!(path = %output_path.display(), "file exists, forcing overwrite");
@@ -273,7 +278,7 @@ impl Download {
                     }
                 }
             }
-        };
+        }
 
         if let Some(parent) = output_path.parent() {
             trace!(path = %parent.display(), "creating parent directories");
@@ -395,8 +400,14 @@ impl Download {
         }
 
         let mut file = if is_resuming {
-            trace!(path = %path.display(), "opening file for append (resume)");
-            OpenOptions::new().append(true).open(path)?
+            let resume_pos = resume_from.unwrap();
+            trace!(path = %path.display(), resume_pos = resume_pos, "opening file for resume");
+            // Truncate file to resume position to avoid duplicated bytes
+            // (file may have more bytes than last checkpoint due to writes between checkpoints)
+            let mut file = OpenOptions::new().write(true).open(path)?;
+            file.set_len(resume_pos)?;
+            file.seek(SeekFrom::End(0))?;
+            file
         } else {
             trace!(path = %path.display(), "creating new file");
             File::create(path)?

crates/soar-dl/src/download.rs

@@ -72,6 +72,28 @@ pub fn verify_checksum<P: AsRef<Path>>(file_path: P, expected: &str) -> HashResu
     Ok(actual.eq_ignore_ascii_case(expected))
 }
 
+/// Calculates a hash from a string input.
+///
+/// Returns a hex-encoded blake3 hash string.
+///
+/// # Arguments
+///
+/// * `input` - The string to hash.
+///
+/// # Example
+///
+/// ```
+/// use soar_utils::hash::hash_string;
+///
+/// let hash = hash_string("hello world");
+/// assert_eq!(hash.len(), 64);
+/// ```
+pub fn hash_string(input: &str) -> String {
+    let mut hasher = blake3::Hasher::new();
+    hasher.update(input.as_bytes());
+    hasher.finalize().to_hex().to_string()
+}
+
 #[cfg(test)]
 mod tests {
     use std::io::Write;