cli: wire MultiError in add/rm; add typo suggestions

Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I98240ec0f9e3932a46e79f82f32cd5d36a6a6964

src/cli/commands/add.rs

@@ -1,7 +1,7 @@
 use std::collections::HashMap;
 
 use crate::{
-  error::{PakkerError, Result},
+  error::{MultiError, PakkerError, Result},
   model::{Config, LockFile, Project},
   platform::create_platform,
   resolver::DependencyResolver,
@@ -139,10 +139,19 @@ pub async fn execute(
   let platforms = create_all_platforms()?;
 
   let mut new_projects = Vec::new();
+  let mut errors = MultiError::new();
 
   // Resolve each input
   for input in &args.inputs {
-    let project = resolve_input(input, &platforms, &lockfile).await?;
+    let project = match resolve_input(input, &platforms, &lockfile).await {
+      Ok(p) => p,
+      Err(e) => {
+        // Collect error but continue with other inputs
+        log::warn!("Failed to resolve '{input}': {e}");
+        errors.push(e);
+        continue;
+      },
+    };
 
     // Check if already exists by matching platform IDs (not pakku_id which is
     // random)
@@ -174,6 +183,15 @@ pub async fn execute(
       continue;
     }
 
+    // Prompt for confirmation unless --yes flag is set
+    if !args.yes {
+      let prompt_msg = format!("Add project '{}'?", project.get_name());
+      if !crate::ui_utils::prompt_yes_no(&prompt_msg, true)? {
+        log::info!("Skipping project: {}", project.get_name());
+        continue;
+      }
+    }
+
     new_projects.push(project);
   }
 
@@ -213,6 +231,9 @@ pub async fn execute(
     new_projects = all_new_projects;
   }
 
+  // Track count before moving
+  let added_count = new_projects.len();
+
   // Add projects to lockfile (updates already handled above)
   for project in new_projects {
     lockfile.add_project(project);
@@ -221,7 +242,20 @@ pub async fn execute(
   // Save lockfile
   lockfile.save(lockfile_dir)?;
 
-  log::info!("Successfully added {} project(s)", args.inputs.len());
+  log::info!("Successfully added {added_count} project(s)");
+
+  // Return aggregated errors if any occurred
+  if !errors.is_empty() {
+    let error_count = errors.len();
+    log::warn!(
+      "{error_count} project(s) failed to resolve (see warnings above)"
+    );
+    // Return success if at least some projects were added, otherwise return
+    // errors
+    if added_count == 0 && args.inputs.len() == error_count {
+      return errors.into_result(());
+    }
+  }
 
   Ok(())
 }