fix(cli): implement --share flag to display share URL

Fixes bounty issue #1248

The --share flag was defined but not implemented, only printing a message
that sharing was not yet implemented when verbose mode was enabled.

This change implements the session sharing functionality by:
- Adding cortex-share as a dependency
- Using ShareManager to share the session after successful completion
- Printing the share URL to stdout for the user

Author: factorydroid

Cargo.lock

@@ -131,6 +131,7 @@ cortex-storage = { path = "cortex-storage" }
 cortex-core = { path = "cortex-core" }
 cortex-tui = { path = "cortex-tui" }
 cortex-update = { path = "cortex-update" }
+cortex-share = { path = "cortex-share" }
 
 # OpenTUI crates
 opentui = { path = "opentui-rs" }

Cargo.toml

@@ -30,6 +30,7 @@ cortex-common = { workspace = true, features = ["cli"] }
 cortex-login = { workspace = true }
 cortex-process-hardening = { workspace = true }
 cortex-app-server = { workspace = true }
+cortex-share = { workspace = true }
 
 clap = { workspace = true }
 clap_complete = { workspace = true }

cortex-cli/Cargo.toml

@@ -241,15 +241,17 @@ impl RunCli {
     pub async fn run(self) -> Result<()> {
         // Validate temperature if provided
         if let Some(temp) = self.temperature
-            && !(0.0..=2.0).contains(&temp) {
-                bail!("Temperature must be between 0.0 and 2.0, got {temp}");
-            }
+            && !(0.0..=2.0).contains(&temp)
+        {
+            bail!("Temperature must be between 0.0 and 2.0, got {temp}");
+        }
 
         // Validate top_p if provided
         if let Some(top_p) = self.top_p
-            && !(0.0..=1.0).contains(&top_p) {
-                bail!("top-p must be between 0.0 and 1.0, got {top_p}");
-            }
+            && !(0.0..=1.0).contains(&top_p)
+        {
+            bail!("top-p must be between 0.0 and 1.0, got {top_p}");
+        }
 
         // Check if original message args are all whitespace (before quote-wrapping)
         // This catches cases like `cortex run "   "` which would otherwise pass validation
@@ -460,9 +462,10 @@ impl RunCli {
             SessionMode::New { title } => {
                 let id = uuid::Uuid::new_v4().to_string();
                 if let Some(ref t) = title
-                    && self.verbose {
-                        eprintln!("New session: {id} (title: {t})");
-                    }
+                    && self.verbose
+                {
+                    eprintln!("New session: {id} (title: {t})");
+                }
                 id
             }
         };
@@ -558,11 +561,12 @@ impl RunCli {
         while let Ok(event) = handle.event_rx.recv().await {
             // Check timeout
             if let Some(timeout) = timeout_duration
-                && start_time.elapsed() > timeout {
-                    eprintln!("Timeout reached after {} seconds", self.timeout);
-                    error_occurred = true;
-                    break;
-                }
+                && start_time.elapsed() > timeout
+            {
+                eprintln!("Timeout reached after {} seconds", self.timeout);
+                error_occurred = true;
+                break;
+            }
 
             event_count += 1;
 
@@ -745,11 +749,22 @@ impl RunCli {
             send_notification(&session_id, !error_occurred)?;
         }
 
-        // Share session if requested
-        if self.share {
-            // TODO: Implement session sharing when API is available
-            if self.verbose {
-                eprintln!("Session sharing requested but not yet implemented");
+        // Share session if requested and no error occurred
+        if self.share && !error_occurred {
+            let share_manager = cortex_share::ShareManager::new();
+            match share_manager.share(&session_id).await {
+                Ok(shared_session) => {
+                    // Print the share URL to stdout for the user
+                    println!("Share URL: {}", shared_session.url);
+                }
+                Err(e) => {
+                    eprintln!(
+                        "{}Warning:{} Failed to share session: {}",
+                        TermColor::Yellow.ansi_code(),
+                        TermColor::Default.ansi_code(),
+                        e
+                    );
+                }
             }
         }