chore: remove rand dependency

Author: Taxrosdev

Cargo.lock

@@ -11,7 +11,6 @@ readme = "README.md"
 edition = "2024"
 
 [dependencies]
-rand = "0.9.1"
 serde = { version = "1.0.219", features = ["derive"] }
 serde_json = "1.0.140"
 walkdir = { version = "2.5.0", optional = true }

Cargo.toml

@@ -5,6 +5,7 @@ pub fn hash(input: &Vec<u8>) -> String {
     xxh3_64(&input).to_string()
 }
 
+// Converts the manifest to a string, and then hashes it
 pub fn hash_manifest(input: &Vec<(String, String, bool)>) -> String {
     // Not a true hash. Just a temporary place to dump data, which can then be hashed
     let mut current_hash = "".to_string();
@@ -28,4 +29,34 @@ mod tests {
         let result = hash(&vec![1, 2, 3]);
         assert_eq!(result, "16991689376074199867");
     }
+
+    #[test]
+    fn hash_empty_vec() {
+        let result = hash(&vec![]);
+        assert_eq!(result, "3244421341483603138");
+    }
+
+    #[test]
+    fn hash_manifest_stable() {
+        let manifest = vec![
+            ("file1.txt".to_string(), "hash1".to_string(), false),
+            ("file2.txt".to_string(), "hash2".to_string(), true),
+        ];
+        let result = hash_manifest(&manifest);
+        assert_eq!(result, "1259801786371591190");
+    }
+
+    #[test]
+    fn hash_manifest_empty() {
+        let manifest: Vec<(String, String, bool)> = vec![];
+        let result = hash_manifest(&manifest);
+        assert_eq!(result, "3244421341483603138");
+    }
+
+    #[test]
+    fn hash_manifest_single_entry() {
+        let manifest = vec![("main.rs".to_string(), "abc123".to_string(), true)];
+        let result = hash_manifest(&manifest);
+        assert_eq!(result, "9815591975043689442");
+    }
 }

src/hash.rs

@@ -1,11 +1,11 @@
 use std::{
+    env::temp_dir,
     fs::{self, create_dir_all, rename},
+    option,
     os::unix::fs::{PermissionsExt, symlink},
     path::Path,
 };
 
-use rand::RngCore;
-
 use crate::{artifacts::get_artifact, compression::decompress_file};
 
 mod artifacts;
@@ -155,7 +155,7 @@ pub fn install_artifact(artifact_name: &String, store_path: &Path, repo_cache_pa
     }
 
     // Create a temporary symlink for atomic update
-    let tmp_file_name = format!(".tmp_{}", rand::rng().next_u64());
+    let tmp_file_name = get_temp_file(None, &store_artifacts_path);
 
     let tmp_symlink = store_artifacts_path.join(&tmp_file_name);
     let final_symlink = store_artifacts_path.join(artifact_name);
@@ -164,6 +164,18 @@ pub fn install_artifact(artifact_name: &String, store_path: &Path, repo_cache_pa
     rename(&tmp_symlink, &final_symlink).unwrap();
 }
 
+fn get_temp_file(potential: Option<u8>, dir: &Path) -> String {
+    let potential = potential.unwrap_or_default();
+
+    let file_name = format!(".tmp_{}", &potential);
+
+    return if dir.join(&file_name).exists() {
+        get_temp_file(Some(potential + 1), dir)
+    } else {
+        file_name
+    };
+}
+
 fn make_chunk_executable(chunk_hash: &String, store_path: &Path) {
     let chunk_path = store_path.join("chunks").join(chunk_hash);