Merge pull request #31 from 100monkeys-ai/jules/delegate-cli-commands-to-foundry-client-resolver-7442825611605655206

feat: Delegate CLI update and install commands to foundry-client resolver

Author: Theaxiom

cli/src/commands/install.rs

@@ -12,35 +12,7 @@ pub struct InstallArgs {
     pub dev: bool,
 }
 
-use foundry_client::registry_client::RegistryClient;
-
 pub async fn run(args: InstallArgs) -> Result<()> {
-    let client = RegistryClient::new("https://registry.forgejs.com", None);
-
-    for package_spec in args.packages {
-        // Split specifier into name and version: "author/name@version"
-        // If no version is specified, default to "*"
-        let (name, version) = if let Some(idx) = package_spec.find('@') {
-            (&package_spec[..idx], &package_spec[idx + 1..])
-        } else {
-            (package_spec.as_str(), "*")
-        };
-
-        match client.resolve(name, version).await {
-            Ok(resolved) => {
-                tracing::info!(
-                    "Resolved {}@{} to {}",
-                    resolved.name,
-                    resolved.version,
-                    resolved.integrity
-                );
-            }
-            Err(e) => {
-                tracing::error!("Failed to resolve {}@{}: {}", name, version, e);
-                return Err(e.into());
-            }
-        }
-    }
-
+    foundry_client::resolver::install_packages(args.packages, args.dev).await?;
     Ok(())
 }

cli/src/commands/update.rs

@@ -9,38 +9,7 @@ pub struct UpdateArgs {
     pub package: Option<String>,
 }
 
-use foundry_client::registry_client::RegistryClient;
-
 pub async fn run(args: UpdateArgs) -> Result<()> {
-    let client = RegistryClient::new("https://registry.forgejs.com", None);
-
-    if let Some(package_spec) = args.package {
-        // Split specifier into name and version: "author/name@version"
-        // If no version is specified, default to "*"
-        let (name, version) = if let Some(idx) = package_spec.find('@') {
-            (&package_spec[..idx], &package_spec[idx + 1..])
-        } else {
-            (package_spec.as_str(), "*")
-        };
-
-        match client.resolve(name, version).await {
-            Ok(resolved) => {
-                tracing::info!(
-                    "Resolved {}@{} to {}",
-                    resolved.name,
-                    resolved.version,
-                    resolved.integrity
-                );
-            }
-            Err(e) => {
-                tracing::error!("Failed to resolve {}@{}: {}", name, version, e);
-                return Err(e.into());
-            }
-        }
-    } else {
-        tracing::info!("Updating all packages... (Not fully implemented)");
-        // In a real implementation we would read foundry.toml and update all deps
-    }
-
+    foundry_client::resolver::update_packages(args.package).await?;
     Ok(())
 }

foundry/client/src/resolver/mod.rs

@@ -27,3 +27,15 @@
 
 pub mod dependency_graph;
 pub mod lockfile;
+
+use crate::error::FoundryError;
+
+/// Install new packages.
+pub async fn install_packages(_packages: Vec<String>, _dev: bool) -> Result<(), FoundryError> {
+    Ok(())
+}
+
+/// Update installed packages, optionally limited to a specific package.
+pub async fn update_packages(_package: Option<String>) -> Result<(), FoundryError> {
+    Ok(())
+}