fix: fixed broken update fn

src/functions/install.rs

@@ -117,7 +117,8 @@ pub fn install_package(package_name: &str) -> Result<(), Box<dyn Error>> {
         meta.name.clone(),
         InstalledPackage {
             repo_path: repo_path.to_string_lossy().to_string(),
-            files: installed_files,
+            installed_files: installed_files,
+            copy_files: meta.files.clone(),
             pkg_type: meta.pkg_type.clone(),
             build_command: meta.build.clone(),
         },

src/functions/list.rs

@@ -21,7 +21,7 @@ pub fn list_packages(list_args: ListArgs) -> Result<(), Box<dyn Error>> {
                     package.pkg_type,
                     package.repo_path,
                     package.build_command.clone().unwrap_or_else(|| "None".into()),
-                    package.files.join("\n    ")
+                    package.installed_files.join("\n    ")
                 );
             } else {
                 info!("{}", pkg);
@@ -41,7 +41,7 @@ pub fn list_packages(list_args: ListArgs) -> Result<(), Box<dyn Error>> {
                 package.pkg_type,
                 package.repo_path,
                 package.build_command.clone().unwrap_or_else(|| "None".into()),
-                package.files.join("\n    ")
+                package.installed_files.join("\n    ")
             );
         } else {
             info!("{}", name);

src/functions/mod.rs

@@ -19,7 +19,8 @@ pub struct PackageDB {
 #[derive(Deserialize, Serialize, Debug)]
 pub struct InstalledPackage {
     repo_path: String,
-    files: Vec<String>,
+    installed_files: Vec<String>,
+    copy_files: Vec<String>,
     pkg_type: String,
     build_command: Option<String>,
 }

src/functions/uninstall.rs

@@ -10,7 +10,7 @@ pub fn uninstall_package(package_name: &str) -> Result<(), Box<dyn Error>> {
 
     let mut db = load_db()?;
     if let Some(pkg) = db.packages.remove(package_name) {
-        for file in pkg.files {
+        for file in pkg.installed_files {
             let path = PathBuf::from(file);
             if path.exists() {
                 fs::remove_file(&path)?;

src/functions/update.rs

@@ -1,8 +1,9 @@
 use super::{InstalledPackage, save_db, load_db}; 
-use log::info;
+use log::{info, debug};
 use std::error::Error;
 use std::path::PathBuf;
 use std::fs;
+use std::env;
 use std::process::Command;
 use colored::Colorize;
 
@@ -12,7 +13,7 @@ pub fn update_package(package_name: Option<String>) -> Result<(), Box<dyn Error>
     if let Some(name) = package_name {
         if let Some(pkg) = db.packages.get_mut(&name) {
             info!("> Updating package '{}'", name.yellow().bold());
-            update_file(pkg)?;
+            update_file(pkg, &name)?;
             info!("Successfully updated '{}'", name.yellow().bold());
         } else {
             info!("Package '{}' not found in database", name.yellow());
@@ -21,7 +22,7 @@ pub fn update_package(package_name: Option<String>) -> Result<(), Box<dyn Error>
         info!("> Updating all packages...");
         for (name, pkg) in db.packages.iter_mut() {
             info!("Updating '{}'", name.yellow().bold());
-            update_file(pkg)?;
+            update_file(pkg, name)?;
         }
     }
 
@@ -29,10 +30,11 @@ pub fn update_package(package_name: Option<String>) -> Result<(), Box<dyn Error>
     Ok(())
 }
 
-fn update_file(pkg: &mut InstalledPackage) -> Result<(), Box<dyn Error>> {
+fn update_file(pkg: &mut InstalledPackage, package_name: &str) -> Result<(), Box<dyn Error>> {
     let repo_path = PathBuf::from(&pkg.repo_path);
 
     // Pull latest changes
+    debug!("Pullng latest version of {} using git...", package_name);
     let output = Command::new("git")
         .args(&["-C", repo_path.to_str().unwrap(), "pull"])
         .output()?;
@@ -56,18 +58,30 @@ fn update_file(pkg: &mut InstalledPackage) -> Result<(), Box<dyn Error>> {
         }
     }
 
+    let home_dir = dirs::home_dir().ok_or("Failed to get home directory")?;
+
+    // Determine target directory
+    let target_base_dir = match pkg.pkg_type.as_str() {
+        "binary" => home_dir.join(".eiipm/bin"),
+        "theme" => env::current_dir()?,
+        "library" => home_dir.join(format!(".eiipm/lib/{}", package_name)),
+        other => return Err(format!("Unknown package type '{}'", other).into()),
+    };
+
     // Copy updated files to targets
-    for file in &pkg.files {
-        let source = repo_path.join(
-            PathBuf::from(file)
-                .file_name()
-                .ok_or("Invalid filename")?,
-        );
-        let target = PathBuf::from(file);
-        if source.exists() {
-            fs::copy(&source, &target)?;
+    for file in &pkg.copy_files {
+        let source = repo_path.join(file);
+        let target = target_base_dir.join(file);
+        if !source.exists() {
+            return Err(format!("File '{}' not found in repo", source.display()).into());
         }
+        if let Some(parent) = target.parent() {
+            fs::create_dir_all(parent)?;
+        }
+        fs::copy(&source, &target)?;
+        info!("Copied {} -> {}", source.display(), target.display());
     }
 
+
     Ok(())
 }