clean dependencies on update, clean remappings at update

README.md

@@ -51,3 +51,5 @@ The full list of dependencies is available [here](./all_dependencies.toml).
 - Error handling.
 - Tests. (I know, I know...)
 - Refactor the code. (lots of duplicated code).
+- enable dependencies in the foundry toml file.
+- skip downloading if dependency already downloaded

remappings.txt

@@ -1,4 +1,3 @@
-openzeppelin=dependencies/openzeppelin-v4.9.2
-solady=dependencies/solady-v0.0.107
-solady=dependencies/solady-v0.0.49
-uniswap-v3-periphery=dependencies/uniswap-v3-periphery-v1.0.0
+@openzeppelin=dependencies/@openzeppelin-v1.0.5
+@openzeppelin=dependencies/@openzeppelin-v4.9.2
+@solady=dependencies/@solady-v0.0.41

soldeer.toml

@@ -1,9 +1,9 @@
 
 
 [dependencies]
-"openzeppelin~v4.9.2" = "https://github.com/OpenZeppelin/openzeppelin-contracts/archive/refs/tags/v4.9.2.zip"
-"openzeppelin~v1.0.5" = "https://github.com/OpenZeppelin/openzeppelin-contracts/archive/refs/tags/v1.0.5.zip"
-"openzeppelin~v2.0.0-rc.4" = "https://github.com/OpenZeppelin/openzeppelin-contracts/archive/refs/tags/v2.0.0-rc.4.zip"
+"@openzeppelin~v4.9.2" = "https://github.com/OpenZeppelin/openzeppelin-contracts/archive/refs/tags/v4.9.2.zip"
+"@openzeppelin~v1.0.5" = "https://github.com/OpenZeppelin/openzeppelin-contracts/archive/refs/tags/v1.0.5.zip"
+"@solady~v0.0.41" = "https://github.com/Vectorized/solady/archive/refs/tags/v0.0.41.zip"
 
 [remappings]
 enabled = true

src/config.rs

@@ -114,34 +114,19 @@ fn update_foundry() {
         File::create("remappings.txt").unwrap();
     }
     println!("Updating foundry...");
-    let contents = read_file_to_string(String::from("remappings.txt"));
-
-    let existing_remappings: Vec<String> = contents
-        .split("\n")
-        .map(|s| s.to_string())
-        .collect();
     let mut new_remappings: String = String::new();
     let dependencies: Vec<Dependency> = read_config(String::new());
 
-    let mut existing_remap: Vec<String> = Vec::new();
-    existing_remappings.iter().for_each(|remapping| {
-        let split: Vec<&str> = remapping.split("=").collect::<Vec<&str>>();
-        existing_remap.push(String::from(split[0]));
-    });
-
     dependencies.iter().for_each(|dependency| {
-        let index = existing_remap.iter().position(|r| r == &dependency.name);
-        if index.is_none() {
-            println!("Adding a new remap {}", &dependency.name);
-            new_remappings.push_str(
-                &format!(
-                    "{}=dependencies/{}-{}\n",
-                    &dependency.name,
-                    &dependency.name,
-                    &dependency.version
-                )
-            );
-        }
+        println!("Adding a new remap {}", &dependency.name);
+        new_remappings.push_str(
+            &format!(
+                "{}=dependencies/{}-{}\n",
+                &dependency.name,
+                &dependency.name,
+                &dependency.version
+            )
+        );
     });
 
     if new_remappings.len() == 0 {
@@ -152,10 +137,11 @@ fn update_foundry() {
     let mut file: std::fs::File = fs::OpenOptions
         ::new()
         .write(true)
-        .append(true)
+        .truncate(true)
+        .append(false)
         .open(Path::new("remappings.txt"))
         .unwrap();
-
+    println!("New remappings: {}", &new_remappings);
     match write!(file, "{}", &new_remappings) {
         Ok(_) => {}
         Err(e) => {

src/dependency_downloader.rs

@@ -11,7 +11,15 @@ use crate::utils::get_current_working_dir;
 
 // TODOs:
 // - needs to be downloaded in parallel
-pub async fn download_dependencies(dependencies: &Vec<Dependency>) -> Result<(), DownloadError> {
+pub async fn download_dependencies(
+    dependencies: &Vec<Dependency>,
+    clean: bool
+) -> Result<(), DownloadError> {
+    if clean {
+        let dep_path = get_current_working_dir().unwrap().join("dependencies");
+        fs::remove_dir_all(&dep_path).unwrap();
+        fs::create_dir(&dep_path).unwrap();
+    }
     for dependency in dependencies.iter() {
         let file_name: String = format!("{}-{}.zip", dependency.name, dependency.version);
         match download_dependency(&file_name, &dependency.url).await {
@@ -54,6 +62,7 @@ pub async fn download_dependency_remote(
     let dependencies: Vec<Dependency> = read_config((&tmp_path).to_str().unwrap().to_string());
     for dependency in dependencies.iter() {
         if dependency.name == *dependency_name && dependency.version == *dependency_version {
+            println!("dependency url: {}", dependency.url);
             match
                 download_dependency(
                     &format!("{}-{}.zip", &dependency_name, &dependency_version),

src/main.rs

@@ -67,7 +67,7 @@ async fn main() {
         remappings();
     } else if command.0 == "update" || (command.0 == "install" && command.1 == "") {
         let dependencies: Vec<Dependency> = read_config(String::new());
-        if download_dependencies(&dependencies).await.is_err() {
+        if download_dependencies(&dependencies, true).await.is_err() {
             eprintln!("Error downloading dependencies");
             exit(500);
         }