feat(deployments): streamline deployment process and enhance user experience

- Updated deployment commands to remove confirmation prompts, allowing for automatic deployments when the `--yes` flag is used.
- Added documentation for deployment procedures, emphasizing the use of Bun over Node.js.
- Introduced a new section in the setup command to manage the downloading of the GeoLite2-City.mmdb database, including progress feedback and error handling.
- Enhanced the onboarding experience by allowing users to skip Git provider connection during setup, improving flexibility for project creation.

This commit significantly improves the deployment workflow by simplifying user interactions and providing clearer guidance during setup and deployment processes.

Author: dviejokfs

apps/temps-cli/CLAUDE.md

@@ -4,6 +4,12 @@ globs: "*.ts, *.tsx, *.html, *.css, *.js, *.jsx, package.json"
 alwaysApply: false
 ---
 
+## Deployment
+
+When deploying this application, do not ask for confirmation. Proceed with deployments automatically.
+
+## Bun Usage
+
 Default to using Bun instead of Node.js.
 
 - Use `bun <file>` instead of `node <file>` or `ts-node <file>`

apps/temps-cli/src/commands/deploy/deploy-image.ts

@@ -3,7 +3,7 @@ import { setupClient, client } from '../../lib/api-client.js'
 import { watchDeployment } from '../../lib/deployment-watcher.jsx'
 import { getProjectBySlug, getProject, getEnvironments } from '../../api/sdk.gen.js'
 import type { EnvironmentResponse } from '../../api/types.gen.js'
-import { promptSelect, promptConfirm, promptText } from '../../ui/prompts.js'
+import { promptSelect, promptText } from '../../ui/prompts.js'
 import {
   startSpinner,
   succeedSpinner,
@@ -212,19 +212,6 @@ export async function deployImage(options: DeployImageOptions): Promise<void> {
   )
   newline()
 
-  // Confirm deployment (skip if --yes flag)
-  if (!options.yes) {
-    const confirmed = await promptConfirm({
-      message: 'Start deployment?',
-      default: true,
-    })
-
-    if (!confirmed) {
-      info('Deployment cancelled')
-      return
-    }
-  }
-
   // Trigger deployment
   startSpinner('Starting deployment...')
 

apps/temps-cli/src/commands/deploy/deploy-static.ts

@@ -3,7 +3,7 @@ import { setupClient, client } from '../../lib/api-client.js'
 import { watchDeployment } from '../../lib/deployment-watcher.jsx'
 import { getProjectBySlug, getProject, getEnvironments } from '../../api/sdk.gen.js'
 import type { EnvironmentResponse } from '../../api/types.gen.js'
-import { promptSelect, promptConfirm } from '../../ui/prompts.js'
+import { promptSelect } from '../../ui/prompts.js'
 import {
   startSpinner,
   succeedSpinner,
@@ -221,19 +221,6 @@ export async function deployStatic(options: DeployStaticOptions): Promise<void>
   )
   newline()
 
-  // Confirm deployment (skip if --yes flag)
-  if (!options.yes) {
-    const confirmed = await promptConfirm({
-      message: 'Start deployment?',
-      default: true,
-    })
-
-    if (!confirmed) {
-      info('Deployment cancelled')
-      return
-    }
-  }
-
   // Prepare the file for upload
   startSpinner('Preparing static bundle...')
 

apps/temps-cli/src/commands/deploy/deploy.ts

@@ -7,7 +7,7 @@ import {
   getLastDeployment,
 } from '../../api/sdk.gen.js'
 import type { EnvironmentResponse, DeploymentResponse } from '../../api/types.gen.js'
-import { promptSelect, promptText, promptConfirm } from '../../ui/prompts.js'
+import { promptSelect, promptText } from '../../ui/prompts.js'
 import { startSpinner, succeedSpinner, failSpinner, updateSpinner } from '../../ui/spinner.js'
 import { success, info, warning, newline, icons, colors, header, keyValue, box } from '../../ui/output.js'
 
@@ -134,18 +134,6 @@ export async function deploy(options: DeployOptions): Promise<void> {
   )
   newline()
 
-  if (!options.yes) {
-    const confirmed = await promptConfirm({
-      message: 'Start deployment?',
-      default: true,
-    })
-
-    if (!confirmed) {
-      info('Deployment cancelled')
-      return
-    }
-  }
-
   // Start deployment
   startSpinner('Starting deployment...')
 

crates/temps-auth/src/temps_middleware.rs

@@ -118,7 +118,7 @@ impl AuthMiddleware {
         let host_without_port = host.split(':').next().unwrap_or(host);
 
         // Log for debugging
-        tracing::info!(
+        tracing::debug!(
             "Auth middleware: host={}, host_without_port={}, demo_mode_header={}, path={}",
             host,
             host_without_port,
@@ -135,7 +135,7 @@ impl AuthMiddleware {
                 .check_demo_mode(host_without_port, demo_mode_header, demo_user_id)
                 .await
             {
-                tracing::info!(
+                tracing::debug!(
                     "Demo mode: authenticated as user {} (id={})",
                     demo_user.email,
                     demo_user.id
@@ -409,7 +409,7 @@ impl AuthMiddleware {
 
         // Check if demo mode header is set by proxy (proxy already validated the host)
         if demo_mode_header {
-            tracing::info!(
+            tracing::debug!(
                 "Demo mode detected via X-Temps-Demo-Mode header (host: {})",
                 host_without_port
             );
@@ -432,7 +432,7 @@ impl AuthMiddleware {
                 return None;
             }
 
-            tracing::info!(
+            tracing::debug!(
                 "Demo mode detected: host {} matches expected demo host {}",
                 host_without_port,
                 expected_demo_host
@@ -449,7 +449,7 @@ impl AuthMiddleware {
                     } else {
                         crate::permissions::Role::User
                     };
-                    tracing::info!(
+                    tracing::debug!(
                         "Demo mode: authenticated as selected user (id={}, email={}, role={:?})",
                         user.id,
                         user.email,
@@ -473,7 +473,7 @@ impl AuthMiddleware {
         // Default: Find or create demo user
         match self.user_service.find_or_create_demo_user().await {
             Ok(demo_user) => {
-                tracing::info!(
+                tracing::debug!(
                     "Demo mode: auto-authenticated as demo user (id={}, email={})",
                     demo_user.id,
                     demo_user.email

crates/temps-cli/src/commands/setup.rs

@@ -10,6 +10,7 @@ use argon2::password_hash::{rand_core::OsRng, SaltString};
 use argon2::{Argon2, PasswordHasher};
 use clap::Args;
 use colored::Colorize;
+use indicatif::{ProgressBar, ProgressStyle};
 use rand::Rng;
 use rustls::crypto::CryptoProvider;
 use sea_orm::{ActiveModelTrait, ColumnTrait, EntityTrait, QueryFilter, Set};
@@ -211,6 +212,11 @@ pub struct SetupCommand {
     /// Used when running behind a reverse proxy or load balancer
     #[arg(long, env = "TEMPS_EXTERNAL_URL")]
     pub external_url: Option<String>,
+
+    /// Skip downloading GeoLite2-City.mmdb database from GitHub
+    /// By default, the database is downloaded automatically if not present
+    #[arg(long, default_value = "false")]
+    pub skip_geolite2_download: bool,
 }
 
 fn generate_secure_password() -> String {
@@ -680,20 +686,151 @@ fn ask_confirmation(prompt: &str) -> anyhow::Result<bool> {
     Ok(response == "y" || response == "yes")
 }
 
-/// Check if GeoLite2-City.mmdb exists and warn if missing
-/// This database is required to run the application but not for setup
-fn check_geolite2_database(data_dir: &PathBuf) {
+/// URL for downloading GeoLite2-City.mmdb from GitHub
+const GEOLITE2_DOWNLOAD_URL: &str =
+    "https://raw.githubusercontent.com/gotempsh/temps/refs/heads/main/crates/temps-cli/GeoLite2-City.mmdb";
+
+/// Download GeoLite2-City.mmdb from GitHub with progress bar
+async fn download_geolite2_database(data_dir: &PathBuf) -> anyhow::Result<()> {
+    use futures::StreamExt;
+
+    let geo_db_path = data_dir.join("GeoLite2-City.mmdb");
+
+    println!();
+    println!(
+        "   {} {}",
+        "📥".bright_cyan(),
+        "Downloading GeoLite2-City.mmdb...".bright_white()
+    );
+    println!(
+        "   {} {}",
+        "🔗".bright_blue(),
+        GEOLITE2_DOWNLOAD_URL.bright_cyan()
+    );
+    println!();
+
+    // Create HTTP client
+    let client = reqwest::Client::new();
+    let response = client
+        .get(GEOLITE2_DOWNLOAD_URL)
+        .send()
+        .await
+        .map_err(|e| anyhow::anyhow!("Failed to start download: {}", e))?;
+
+    // Check response status
+    if !response.status().is_success() {
+        return Err(anyhow::anyhow!(
+            "Failed to download GeoLite2 database: HTTP {}",
+            response.status()
+        ));
+    }
+
+    // Get content length for progress bar
+    let total_size = response.content_length().unwrap_or(0);
+
+    // Create progress bar with beautiful styling
+    let pb = if total_size > 0 {
+        let pb = ProgressBar::new(total_size);
+        pb.set_style(
+            ProgressStyle::default_bar()
+                .template("   {spinner:.green} [{elapsed_precise}] [{bar:40.cyan/blue}] {bytes}/{total_bytes} ({bytes_per_sec}) {msg}")
+                .unwrap()
+                .progress_chars("█▓▒░  "),
+        );
+        pb.set_message("downloading...");
+        pb
+    } else {
+        // Unknown size - use spinner
+        let pb = ProgressBar::new_spinner();
+        pb.set_style(
+            ProgressStyle::default_spinner()
+                .template("   {spinner:.green} [{elapsed_precise}] {bytes} ({bytes_per_sec}) {msg}")
+                .unwrap(),
+        );
+        pb.set_message("downloading...");
+        pb
+    };
+
+    // Create temporary file for download
+    let temp_path = geo_db_path.with_extension("mmdb.tmp");
+    let mut file = fs::File::create(&temp_path)
+        .map_err(|e| anyhow::anyhow!("Failed to create temporary file: {}", e))?;
+
+    // Stream the download with progress updates
+    let mut stream = response.bytes_stream();
+    let mut downloaded: u64 = 0;
+
+    while let Some(chunk_result) = stream.next().await {
+        let chunk = chunk_result.map_err(|e| anyhow::anyhow!("Download error: {}", e))?;
+
+        use std::io::Write;
+        file.write_all(&chunk)
+            .map_err(|e| anyhow::anyhow!("Failed to write to file: {}", e))?;
+
+        downloaded += chunk.len() as u64;
+        pb.set_position(downloaded);
+    }
+
+    // Finish progress bar
+    pb.finish_with_message("complete!");
+
+    // Rename temp file to final destination
+    fs::rename(&temp_path, &geo_db_path)
+        .map_err(|e| anyhow::anyhow!("Failed to finalize download: {}", e))?;
+
+    println!();
+    println!(
+        "   {} {} {}",
+        "✅".bright_green(),
+        "Downloaded to:".bright_white(),
+        geo_db_path.display().to_string().bright_cyan()
+    );
+
+    // Verify file size is reasonable (> 1MB, typical size is ~60MB)
+    let metadata = fs::metadata(&geo_db_path)?;
+    let size_mb = metadata.len() as f64 / 1024.0 / 1024.0;
+    println!(
+        "   {} {} {:.1} MB",
+        "📊".bright_blue(),
+        "File size:".bright_white(),
+        size_mb
+    );
+
+    if metadata.len() < 1_000_000 {
+        warn!(
+            "Downloaded file seems too small ({} bytes), may be corrupted",
+            metadata.len()
+        );
+        println!(
+            "   {} {}",
+            "⚠️ ".bright_yellow(),
+            "Warning: File seems small, may be corrupted".bright_yellow()
+        );
+    }
+
+    println!();
+
+    Ok(())
+}
+
+/// Check if GeoLite2-City.mmdb exists
+/// Returns true if database exists, false if it needs to be downloaded
+fn check_geolite2_database(data_dir: &PathBuf) -> bool {
     let geo_db_path = data_dir.join("GeoLite2-City.mmdb");
     let current_dir_path = PathBuf::from("./GeoLite2-City.mmdb");
 
     // Check both locations
     if geo_db_path.exists() || current_dir_path.exists() {
         print_success("GeoLite2 database found");
-        return;
+        return true;
     }
 
-    // Database not found - show warning with download instructions
-    print_warning("GeoLite2 database not found");
+    false
+}
+
+/// Show warning when GeoLite2 database is missing and download was skipped
+fn show_geolite2_missing_warning(data_dir: &PathBuf) {
+    print_warning("GeoLite2 database not found (download skipped)");
     println!();
     println!(
         "{}",
@@ -709,10 +846,17 @@ fn check_geolite2_database(data_dir: &PathBuf) {
         "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━".bright_yellow()
     );
     println!();
+    println!(
+        "   {} {} {}",
+        "💡".bright_yellow(),
+        "Quick fix:".bright_white().bold(),
+        "Run setup without --skip-geolite2-download".bright_cyan()
+    );
+    println!();
     println!(
         "   {} {}",
         "📥".bright_cyan(),
-        "Download instructions:".bright_white().bold()
+        "Or download manually:".bright_white().bold()
     );
     println!(
         "   {}",
@@ -1013,12 +1157,36 @@ impl SetupCommand {
 
         print_success("Encryption key configured");
 
-        // Check for GeoLite2 database (warning only - not required for setup)
-        check_geolite2_database(&data_dir);
+        // Check for GeoLite2 database
+        let geolite2_exists = check_geolite2_database(&data_dir);
 
         // Create tokio runtime for async operations
         let rt = tokio::runtime::Runtime::new()?;
 
+        // Download GeoLite2 database if not present (unless skipped)
+        if !geolite2_exists {
+            if self.skip_geolite2_download {
+                // User explicitly skipped download - show warning
+                show_geolite2_missing_warning(&data_dir);
+            } else {
+                // Download by default
+                print_section("GeoLite2 Database Download");
+                match rt.block_on(download_geolite2_database(&data_dir)) {
+                    Ok(()) => {
+                        print_success("GeoLite2 database downloaded successfully");
+                    }
+                    Err(e) => {
+                        print_error(&format!("Failed to download GeoLite2 database: {}", e));
+                        println!(
+                            "   {} You can manually download the database later.",
+                            "ℹ️ ".bright_blue()
+                        );
+                        println!();
+                    }
+                }
+            }
+        }
+
         // Establish database connection (this also runs migrations)
         print_section("Database Setup");
         print_substep("Checking database connectivity...");