feat(cli/components): improve push subcommand

crates/cli/src/commands/components/build.rs

@@ -1,19 +1,27 @@
+use crate::components::manifest::Manifest;
+
 #[derive(Debug, clap::Parser)]
 pub struct Options {}
 
 pub async fn run(_opts: Options) -> anyhow::Result<()> {
-    use std::process::Command;
-
-    use crate::components::manifest::{self, Manifest};
+    use crate::components::manifest;
 
     let Some(manifest_path) = manifest::find_manifest_path() else {
         anyhow::bail!("Edgee Manifest not found. Please run `edgee component new` and start from a template or `edgee component init` to create a new empty manifest in this folder.");
     };
     let manifest = Manifest::load(&manifest_path).map_err(|err| anyhow::anyhow!(err))?;
 
+    do_build(&manifest).await?;
+
+    Ok(())
+}
+
+pub async fn do_build(manifest: &Manifest) -> anyhow::Result<()> {
+    use std::process::Command;
+
     tracing::info!("Running: {}", manifest.component.build.command);
     let mut cmd = Command::new("sh");
-    cmd.arg("-c").arg(manifest.component.build.command);
+    cmd.arg("-c").arg(&manifest.component.build.command);
     let status = cmd.status()?;
 
     if status.success() {

crates/cli/src/commands/components/push.rs

@@ -1,6 +1,7 @@
-use crate::components::manifest::Manifest;
 use edgee_api_client::types as api_types;
-use slug;
+
+use crate::components::manifest::Manifest;
+
 #[derive(Debug, clap::Parser)]
 pub struct Options {
     /// The organization name used to create or update your component
@@ -41,7 +42,13 @@ pub async fn run(opts: Options) -> anyhow::Result<()> {
             .api_context("Could not get user organization")?
             .into_inner(),
     };
+
+    let component_url = format!(
+        "https://www.edgee.cloud/~/registry/{}/{}",
+        organization.slug, manifest.component.name,
+    );
     let component_slug = slug::slugify(&manifest.component.name);
+
     match client
         .get_component_by_slug()
         .org_slug(&organization.slug)
@@ -100,12 +107,43 @@ pub async fn run(opts: Options) -> anyhow::Result<()> {
                 .await
                 .api_context("Could not create component")?;
             tracing::info!(
-                "Component `{}/{}` created successfully!",
+                "Component `{}/{}` has been created successfully, you check it out here: {component_url}",
                 organization.slug,
                 component_slug
             );
         }
-        Ok(_) | Err(_) => {}
+        Ok(_) | Err(_) => {
+            client
+                .update_component_by_slug()
+                .org_slug(&organization.slug)
+                .component_slug(&manifest.component.name)
+                .body(
+                    api_types::ComponentUpdateParams::builder()
+                        .description(manifest.component.description.clone())
+                        .documentation_link(
+                            manifest
+                                .component
+                                .documentation
+                                .as_ref()
+                                .map(|url| url.to_string()),
+                        )
+                        .repo_link(
+                            manifest
+                                .component
+                                .repository
+                                .as_ref()
+                                .map(|url| url.to_string()),
+                        ),
+                )
+                .send()
+                .await
+                .api_context("Could not update component infos")?;
+            tracing::info!(
+                "Component `{}/{}` has been updated and is accessible here: {component_url}",
+                organization.slug,
+                component_slug,
+            );
+        }
     }
 
     let changelog =
@@ -121,6 +159,18 @@ pub async fn run(opts: Options) -> anyhow::Result<()> {
         return Ok(());
     }
 
+    let output_path = &manifest.component.build.output_path;
+    if !output_path.exists() {
+        let confirm = Confirm::new("Component have not been built yet, do you want to build it?")
+            .with_default(true)
+            .prompt()?;
+        if !confirm {
+            return Ok(());
+        }
+
+        super::build::do_build(&manifest).await?;
+    }
+
     tracing::info!("Uploading WASM file...");
     let asset_url = client
         .upload_file(&manifest.component.build.output_path)