justfile

@@ -14,6 +14,3 @@ run mode="quiet":
 
 build:
     cargo build --release
-
-clean:
-    cargo clean

src/main.rs

@@ -136,45 +136,16 @@ fn handle_package_command(
 
 fn handle_packages_command(
     managers: &[Box<dyn PackageManager>], 
-    source_filter: Option<String>  // Removed the underscore!
+    _source_filter: Option<String>
 ) -> Result<(), Box<dyn std::error::Error>> {
-    if let Some(source_name) = source_filter {
+    if let Some(dnf) = managers.iter().find(|m| m.name() == "dnf") {
-        // Filter by specific source
+        let packages = dnf.get_installed_packages()?;
-        let source_type = match source_name.to_lowercase().as_str() {
+        println!("Installed packages ({} total):", packages.len());
-            "dnf" => PackageSource::Dnf,
-            "flatpak" => PackageSource::Flatpak,
-            "apt" => PackageSource::Apt,
-            "snap" => PackageSource::Snap,
-            _ => {
-                println!("Unknown source: {}. Available: dnf, flatpak, apt, snap", source_name);
-                return Ok(());
-            }
-        };
-        
-        // Find manager for this source
-        if let Some(manager) = managers.iter().find(|m| m.source_type() == source_type) {
-            let packages = manager.get_installed_packages()?;
-            println!("{} packages ({} total):", source_name, packages.len());
         for package in packages.iter() {
-                println!("  {} {}", package.name, package.version);
+            println!("  {} {} ({})", 
-            }
+                    package.name, 
-        } else {
+                    package.version,
-            println!("No {} package manager found or available", source_name);
+                    package.source.name());
-        }
-    } else {
-        // Show all packages from all managers
-        for manager in managers {
-            match manager.get_installed_packages() {
-                Ok(packages) => {
-                    println!("\n{} packages ({} total):", manager.name(), packages.len());
-                    for package in packages.iter() {
-                        println!("  {} {}", package.name, package.version);
-                    }
-                }
-                Err(e) => {
-                    println!("Error getting packages from {}: {}", manager.name(), e);
-                }
-            }
         }
     }
     Ok(())