Consolidate command execution through runner abstraction

The changes migrate various command executions (nargo, garaga, foundry)
to use a common runner abstraction, removing direct backend calls in
favor of a more unified approach to command execution.

Author: 0xpantera

crates/bargo-core/src/commands/build.rs

@@ -1,7 +1,6 @@
 use color_eyre::Result;
 
 use crate::{
-    backends,
     commands::common::run_nargo_command,
     config::Config,
     util::{self, Flavour, Timer, format_operation_result, success},
@@ -15,13 +14,6 @@ pub fn should_rebuild(pkg: &str, cfg: &Config) -> Result<bool> {
     util::needs_rebuild(pkg)
 }
 
-/// Run `nargo execute` with the provided arguments.
-///
-/// The slice is typically produced by [`build_nargo_args`].
-pub fn run_nargo_execute(args: &[&str]) -> Result<()> {
-    backends::nargo::run(args)
-}
-
 /// Execute the build workflow
 pub fn run(cfg: &Config) -> Result<()> {
     if cfg.dry_run {

crates/bargo-core/src/commands/common.rs

@@ -38,7 +38,7 @@ pub fn build_nargo_args(cfg: &Config, base_args: &[&str]) -> Result<Vec<String>>
 /// - Building arguments with global flags via `build_nargo_args`
 /// - Verbose logging (when enabled and not quiet)
 /// - Dry-run mode (prints command without executing)
-/// - Backend execution via `backends::nargo::run`
+/// - Command execution via the configured runner
 ///
 /// # Arguments
 /// * `cfg` - The global configuration containing all flags
@@ -66,10 +66,83 @@ pub fn run_nargo_command(cfg: &Config, base_args: &[&str]) -> Result<()> {
     cfg.runner.run(&spec)
 
     // TODO: Migrate remaining shell-outs to use runner abstraction:
-    // - backends::nargo::run calls in other modules
-    // - scarb command executions
-    // - garaga command executions
+    // - scarb command executions (currently empty module)
     // - starknet CLI integrations
+    //
+    // Completed migrations:
+    // ✅ bb command executions
+    // ✅ garaga command executions
+    // ✅ foundry command executions
+    // ✅ nargo command executions
+}
+
+/// Run a garaga command with consolidated argument building, logging, and dry-run handling
+///
+/// This is the primary helper for executing garaga commands consistently across all
+/// command modules. It handles:
+/// - Verbose logging (when enabled and not quiet)
+/// - Dry-run mode (prints command without executing)
+/// - Command execution via the configured runner
+///
+/// # Arguments
+/// * `cfg` - The global configuration containing all flags and runner
+/// * `args` - Arguments to pass to garaga
+///
+/// # Returns
+/// * `Result<()>` - Success or error from command execution
+///
+/// # Example
+/// ```ignore
+/// // Execute "garaga gen --system ultra_starknet_zk_honk --vk ./target/starknet/vk"
+/// run_garaga_command(&config, &["gen", "--system", "ultra_starknet_zk_honk", "--vk", "./target/starknet/vk"])?;
+/// ```
+pub fn run_garaga_command(cfg: &Config, args: &[&str]) -> Result<()> {
+    let args_vec: Vec<String> = args.iter().map(|s| s.to_string()).collect();
+
+    if cfg.verbose && !cfg.quiet {
+        info!("Running: garaga {}", args_vec.join(" "));
+    }
+
+    // Create command specification for garaga
+    let spec = CmdSpec::new("garaga".to_string(), args_vec);
+
+    // Use the runner to execute the command (handles dry-run automatically)
+    cfg.runner.run(&spec)
+}
+
+/// Run a foundry command with consolidated argument building, logging, and dry-run handling
+///
+/// This is the primary helper for executing foundry commands consistently across all
+/// command modules. It handles:
+/// - Verbose logging (when enabled and not quiet)
+/// - Dry-run mode (prints command without executing)
+/// - Command execution via the configured runner
+///
+/// # Arguments
+/// * `cfg` - The global configuration containing all flags and runner
+/// * `command` - The foundry command to run (forge, cast, anvil)
+/// * `args` - Arguments to pass to the foundry command
+///
+/// # Returns
+/// * `Result<()>` - Success or error from command execution
+///
+/// # Example
+/// ```ignore
+/// // Execute "forge init --force contracts/evm"
+/// run_foundry_command(&config, "forge", &["init", "--force", "contracts/evm"])?;
+/// ```
+pub fn run_foundry_command(cfg: &Config, command: &str, args: &[&str]) -> Result<()> {
+    let args_vec: Vec<String> = args.iter().map(|s| s.to_string()).collect();
+
+    if cfg.verbose && !cfg.quiet {
+        info!("Running: {} {}", command, args_vec.join(" "));
+    }
+
+    // Create command specification for foundry command
+    let spec = CmdSpec::new(command.to_string(), args_vec);
+
+    // Use the runner to execute the command (handles dry-run automatically)
+    cfg.runner.run(&spec)
 }
 
 /// Run a bb command with consolidated argument building, logging, and dry-run handling