feat(cli): Redesign CLI with noun-first subcommands

Replaces the verb-first pattern ('lore list issues', 'lore show
issue 42') with noun-first subcommands that feel more natural:

  lore issues          # list issues
  lore issues 42       # show issue #42
  lore mrs             # list merge requests
  lore mrs 99          # show MR #99
  lore ingest          # ingest everything
  lore ingest issues   # ingest only issues
  lore count issues    # count issues
  lore status          # sync status
  lore auth            # verify auth
  lore doctor          # health check

Key changes:
- New IssuesArgs, MrsArgs, IngestArgs, CountArgs structs with
  short flags (-n, -s, -p, -a, -l, -o, -f, -J, etc.)
- Global -J/--json flag as shorthand for --robot
- 'lore ingest' with no argument ingests both issues and MRs,
  emitting combined JSON summary in robot mode
- --asc flag replaces --order=asc/desc for brevity
- Renamed flags: --has-due-date -> --has-due, --type -> --for,
  --confirm -> --yes, target_branch -> --target, etc.

Old commands (list, show, auth-test, sync-status) are preserved
as hidden backward-compat aliases that emit deprecation warnings
to stderr before delegating to the new handlers.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

src/main.rs

@@ -18,7 +18,7 @@ use lore::cli::commands::{
     run_doctor, run_ingest, run_init, run_list_issues, run_list_mrs, run_show_issue, run_show_mr,
     run_sync_status,
 };
-use lore::cli::{Cli, Commands};
+use lore::cli::{Cli, Commands, CountArgs, IngestArgs, IssuesArgs, MrsArgs};
 use lore::core::db::{create_connection, get_schema_version, run_migrations};
 use lore::core::error::{GiError, RobotErrorOutput};
 use lore::core::paths::get_config_path;
@@ -47,33 +47,25 @@ async fn main() {
     let robot_mode = cli.is_robot_mode();
 
     let result = match cli.command {
+        Commands::Issues(args) => handle_issues(cli.config.as_deref(), args, robot_mode).await,
+        Commands::Mrs(args) => handle_mrs(cli.config.as_deref(), args, robot_mode).await,
+        Commands::Ingest(args) => handle_ingest(cli.config.as_deref(), args, robot_mode).await,
+        Commands::Count(args) => {
+            handle_count(cli.config.as_deref(), args, robot_mode).await
+        }
+        Commands::Status => handle_sync_status_cmd(cli.config.as_deref(), robot_mode).await,
+        Commands::Auth => handle_auth_test(cli.config.as_deref(), robot_mode).await,
+        Commands::Doctor => handle_doctor(cli.config.as_deref(), robot_mode).await,
+        Commands::Version => handle_version(robot_mode),
         Commands::Init {
             force,
             non_interactive,
         } => handle_init(cli.config.as_deref(), force, non_interactive, robot_mode).await,
-        Commands::AuthTest => handle_auth_test(cli.config.as_deref(), robot_mode).await,
-        Commands::Doctor { json } => handle_doctor(cli.config.as_deref(), json || robot_mode).await,
-        Commands::Version => handle_version(robot_mode),
         Commands::Backup => handle_backup(robot_mode),
-        Commands::Reset { confirm: _ } => handle_reset(robot_mode),
+        Commands::Reset { yes: _ } => handle_reset(robot_mode),
         Commands::Migrate => handle_migrate(cli.config.as_deref(), robot_mode).await,
-        Commands::SyncStatus => handle_sync_status(cli.config.as_deref(), robot_mode).await,
-        Commands::Ingest {
-            r#type,
-            project,
-            force,
-            full,
-        } => {
-            handle_ingest(
-                cli.config.as_deref(),
-                &r#type,
-                project.as_deref(),
-                force,
-                full,
-                robot_mode,
-            )
-            .await
-        }
+
+        // --- Backward-compat: deprecated aliases ---
         Commands::List {
             entity,
             limit,
@@ -89,14 +81,17 @@ async fn main() {
             sort,
             order,
             open,
-            json,
             draft,
             no_draft,
             reviewer,
             target_branch,
             source_branch,
         } => {
-            handle_list(
+            eprintln!(
+                "{}",
+                style("warning: 'lore list' is deprecated, use 'lore issues' or 'lore mrs'").yellow()
+            );
+            handle_list_compat(
                 cli.config.as_deref(),
                 &entity,
                 limit,
@@ -112,7 +107,7 @@ async fn main() {
                 &sort,
                 &order,
                 open,
-                json || robot_mode,
+                robot_mode,
                 draft,
                 no_draft,
                 reviewer.as_deref(),
@@ -121,24 +116,42 @@ async fn main() {
             )
             .await
         }
-        Commands::Count { entity, r#type } => {
-            handle_count(cli.config.as_deref(), &entity, r#type.as_deref(), robot_mode).await
-        }
         Commands::Show {
             entity,
             iid,
             project,
-            json,
         } => {
-            handle_show(
+            eprintln!(
+                "{}",
+                style(format!(
+                    "warning: 'lore show' is deprecated, use 'lore {}s {}'",
+                    entity, iid
+                ))
+                .yellow()
+            );
+            handle_show_compat(
                 cli.config.as_deref(),
                 &entity,
                 iid,
                 project.as_deref(),
-                json || robot_mode,
+                robot_mode,
             )
             .await
         }
+        Commands::AuthTest => {
+            eprintln!(
+                "{}",
+                style("warning: 'lore auth-test' is deprecated, use 'lore auth'").yellow()
+            );
+            handle_auth_test(cli.config.as_deref(), robot_mode).await
+        }
+        Commands::SyncStatus => {
+            eprintln!(
+                "{}",
+                style("warning: 'lore sync-status' is deprecated, use 'lore status'").yellow()
+            );
+            handle_sync_status_cmd(cli.config.as_deref(), robot_mode).await
+        }
     };
 
     if let Err(e) = result {
@@ -207,6 +220,259 @@ fn handle_error(e: Box<dyn std::error::Error>, robot_mode: bool) -> ! {
     std::process::exit(1);
 }
 
+// ============================================================================
+// Primary command handlers
+// ============================================================================
+
+async fn handle_issues(
+    config_override: Option<&str>,
+    args: IssuesArgs,
+    robot_mode: bool,
+) -> Result<(), Box<dyn std::error::Error>> {
+    let config = Config::load(config_override)?;
+    let order = if args.asc { "asc" } else { "desc" };
+
+    if let Some(iid) = args.iid {
+        // Show mode
+        let result = run_show_issue(&config, iid, args.project.as_deref())?;
+        if robot_mode {
+            print_show_issue_json(&result);
+        } else {
+            print_show_issue(&result);
+        }
+    } else {
+        // List mode
+        let filters = ListFilters {
+            limit: args.limit,
+            project: args.project.as_deref(),
+            state: args.state.as_deref(),
+            author: args.author.as_deref(),
+            assignee: args.assignee.as_deref(),
+            labels: args.label.as_deref(),
+            milestone: args.milestone.as_deref(),
+            since: args.since.as_deref(),
+            due_before: args.due_before.as_deref(),
+            has_due_date: args.has_due,
+            sort: &args.sort,
+            order,
+        };
+
+        let result = run_list_issues(&config, filters)?;
+
+        if args.open {
+            open_issue_in_browser(&result);
+        } else if robot_mode {
+            print_list_issues_json(&result);
+        } else {
+            print_list_issues(&result);
+        }
+    }
+
+    Ok(())
+}
+
+async fn handle_mrs(
+    config_override: Option<&str>,
+    args: MrsArgs,
+    robot_mode: bool,
+) -> Result<(), Box<dyn std::error::Error>> {
+    let config = Config::load(config_override)?;
+    let order = if args.asc { "asc" } else { "desc" };
+
+    if let Some(iid) = args.iid {
+        // Show mode
+        let result = run_show_mr(&config, iid, args.project.as_deref())?;
+        if robot_mode {
+            print_show_mr_json(&result);
+        } else {
+            print_show_mr(&result);
+        }
+    } else {
+        // List mode
+        let filters = MrListFilters {
+            limit: args.limit,
+            project: args.project.as_deref(),
+            state: args.state.as_deref(),
+            author: args.author.as_deref(),
+            assignee: args.assignee.as_deref(),
+            reviewer: args.reviewer.as_deref(),
+            labels: args.label.as_deref(),
+            since: args.since.as_deref(),
+            draft: args.draft,
+            no_draft: args.no_draft,
+            target_branch: args.target.as_deref(),
+            source_branch: args.source.as_deref(),
+            sort: &args.sort,
+            order,
+        };
+
+        let result = run_list_mrs(&config, filters)?;
+
+        if args.open {
+            open_mr_in_browser(&result);
+        } else if robot_mode {
+            print_list_mrs_json(&result);
+        } else {
+            print_list_mrs(&result);
+        }
+    }
+
+    Ok(())
+}
+
+async fn handle_ingest(
+    config_override: Option<&str>,
+    args: IngestArgs,
+    robot_mode: bool,
+) -> Result<(), Box<dyn std::error::Error>> {
+    let config = Config::load(config_override)?;
+
+    match args.entity.as_deref() {
+        Some(resource_type) => {
+            // Single entity ingest
+            let result = run_ingest(
+                &config,
+                resource_type,
+                args.project.as_deref(),
+                args.force,
+                args.full,
+                robot_mode,
+            )
+            .await?;
+
+            if robot_mode {
+                print_ingest_summary_json(&result);
+            } else {
+                print_ingest_summary(&result);
+            }
+        }
+        None => {
+            // Ingest everything: issues then MRs
+            if !robot_mode {
+                println!(
+                    "{}",
+                    style("Ingesting all content (issues + merge requests)...").blue()
+                );
+                println!();
+            }
+
+            let issues_result = run_ingest(
+                &config,
+                "issues",
+                args.project.as_deref(),
+                args.force,
+                args.full,
+                robot_mode,
+            )
+            .await?;
+
+            let mrs_result = run_ingest(
+                &config,
+                "mrs",
+                args.project.as_deref(),
+                args.force,
+                args.full,
+                robot_mode,
+            )
+            .await?;
+
+            if robot_mode {
+                print_combined_ingest_json(&issues_result, &mrs_result);
+            } else {
+                print_ingest_summary(&issues_result);
+                print_ingest_summary(&mrs_result);
+            }
+        }
+    }
+
+    Ok(())
+}
+
+/// JSON output for combined ingest (issues + mrs).
+#[derive(Serialize)]
+struct CombinedIngestOutput {
+    ok: bool,
+    data: CombinedIngestData,
+}
+
+#[derive(Serialize)]
+struct CombinedIngestData {
+    resource_type: String,
+    issues: CombinedIngestEntityStats,
+    merge_requests: CombinedIngestEntityStats,
+}
+
+#[derive(Serialize)]
+struct CombinedIngestEntityStats {
+    projects_synced: usize,
+    fetched: usize,
+    upserted: usize,
+    labels_created: usize,
+    discussions_fetched: usize,
+    notes_upserted: usize,
+}
+
+fn print_combined_ingest_json(
+    issues: &lore::cli::commands::ingest::IngestResult,
+    mrs: &lore::cli::commands::ingest::IngestResult,
+) {
+    let output = CombinedIngestOutput {
+        ok: true,
+        data: CombinedIngestData {
+            resource_type: "all".to_string(),
+            issues: CombinedIngestEntityStats {
+                projects_synced: issues.projects_synced,
+                fetched: issues.issues_fetched,
+                upserted: issues.issues_upserted,
+                labels_created: issues.labels_created,
+                discussions_fetched: issues.discussions_fetched,
+                notes_upserted: issues.notes_upserted,
+            },
+            merge_requests: CombinedIngestEntityStats {
+                projects_synced: mrs.projects_synced,
+                fetched: mrs.mrs_fetched,
+                upserted: mrs.mrs_upserted,
+                labels_created: mrs.labels_created,
+                discussions_fetched: mrs.discussions_fetched,
+                notes_upserted: mrs.notes_upserted,
+            },
+        },
+    };
+
+    println!("{}", serde_json::to_string(&output).unwrap());
+}
+
+async fn handle_count(
+    config_override: Option<&str>,
+    args: CountArgs,
+    robot_mode: bool,
+) -> Result<(), Box<dyn std::error::Error>> {
+    let config = Config::load(config_override)?;
+
+    let result = run_count(&config, &args.entity, args.for_entity.as_deref())?;
+    if robot_mode {
+        print_count_json(&result);
+    } else {
+        print_count(&result);
+    }
+    Ok(())
+}
+
+async fn handle_sync_status_cmd(
+    config_override: Option<&str>,
+    robot_mode: bool,
+) -> Result<(), Box<dyn std::error::Error>> {
+    let config = Config::load(config_override)?;
+
+    let result = run_sync_status(&config)?;
+    if robot_mode {
+        print_sync_status_json(&result);
+    } else {
+        print_sync_status(&result);
+    }
+    Ok(())
+}
+
 async fn handle_init(
     config_override: Option<&str>,
     force: bool,
@@ -389,11 +655,11 @@ async fn handle_auth_test(
 
 async fn handle_doctor(
     config_override: Option<&str>,
-    json: bool,
+    robot_mode: bool,
 ) -> Result<(), Box<dyn std::error::Error>> {
     let result = run_doctor(config_override).await;
 
-    if json {
+    if robot_mode {
         println!("{}", serde_json::to_string_pretty(&result)?);
     } else {
         print_doctor_results(&result);
@@ -406,191 +672,6 @@ async fn handle_doctor(
     Ok(())
 }
 
-async fn handle_ingest(
-    config_override: Option<&str>,
-    resource_type: &str,
-    project_filter: Option<&str>,
-    force: bool,
-    full: bool,
-    robot_mode: bool,
-) -> Result<(), Box<dyn std::error::Error>> {
-    let config = Config::load(config_override)?;
-
-    match run_ingest(&config, resource_type, project_filter, force, full, robot_mode).await {
-        Ok(result) => {
-            if robot_mode {
-                print_ingest_summary_json(&result);
-            } else {
-                print_ingest_summary(&result);
-            }
-            Ok(())
-        }
-        Err(e) => {
-            eprintln!("{}", style(format!("Error: {e}")).red());
-            std::process::exit(1);
-        }
-    }
-}
-
-#[allow(clippy::too_many_arguments)]
-async fn handle_list(
-    config_override: Option<&str>,
-    entity: &str,
-    limit: usize,
-    project_filter: Option<&str>,
-    state_filter: Option<&str>,
-    author_filter: Option<&str>,
-    assignee_filter: Option<&str>,
-    label_filter: Option<&[String]>,
-    milestone_filter: Option<&str>,
-    since_filter: Option<&str>,
-    due_before_filter: Option<&str>,
-    has_due_date: bool,
-    sort: &str,
-    order: &str,
-    open_browser: bool,
-    json_output: bool,
-    draft: bool,
-    no_draft: bool,
-    reviewer_filter: Option<&str>,
-    target_branch_filter: Option<&str>,
-    source_branch_filter: Option<&str>,
-) -> Result<(), Box<dyn std::error::Error>> {
-    let config = Config::load(config_override)?;
-
-    match entity {
-        "issues" => {
-            let filters = ListFilters {
-                limit,
-                project: project_filter,
-                state: state_filter,
-                author: author_filter,
-                assignee: assignee_filter,
-                labels: label_filter,
-                milestone: milestone_filter,
-                since: since_filter,
-                due_before: due_before_filter,
-                has_due_date,
-                sort,
-                order,
-            };
-
-            let result = run_list_issues(&config, filters)?;
-
-            if open_browser {
-                open_issue_in_browser(&result);
-            } else if json_output {
-                print_list_issues_json(&result);
-            } else {
-                print_list_issues(&result);
-            }
-
-            Ok(())
-        }
-        "mrs" => {
-            let filters = MrListFilters {
-                limit,
-                project: project_filter,
-                state: state_filter,
-                author: author_filter,
-                assignee: assignee_filter,
-                reviewer: reviewer_filter,
-                labels: label_filter,
-                since: since_filter,
-                draft,
-                no_draft,
-                target_branch: target_branch_filter,
-                source_branch: source_branch_filter,
-                sort,
-                order,
-            };
-
-            let result = run_list_mrs(&config, filters)?;
-
-            if open_browser {
-                open_mr_in_browser(&result);
-            } else if json_output {
-                print_list_mrs_json(&result);
-            } else {
-                print_list_mrs(&result);
-            }
-
-            Ok(())
-        }
-        _ => {
-            eprintln!("{}", style(format!("Unknown entity: {entity}")).red());
-            std::process::exit(1);
-        }
-    }
-}
-
-async fn handle_count(
-    config_override: Option<&str>,
-    entity: &str,
-    type_filter: Option<&str>,
-    robot_mode: bool,
-) -> Result<(), Box<dyn std::error::Error>> {
-    let config = Config::load(config_override)?;
-
-    let result = run_count(&config, entity, type_filter)?;
-    if robot_mode {
-        print_count_json(&result);
-    } else {
-        print_count(&result);
-    }
-    Ok(())
-}
-
-async fn handle_sync_status(
-    config_override: Option<&str>,
-    robot_mode: bool,
-) -> Result<(), Box<dyn std::error::Error>> {
-    let config = Config::load(config_override)?;
-
-    let result = run_sync_status(&config)?;
-    if robot_mode {
-        print_sync_status_json(&result);
-    } else {
-        print_sync_status(&result);
-    }
-    Ok(())
-}
-
-async fn handle_show(
-    config_override: Option<&str>,
-    entity: &str,
-    iid: i64,
-    project_filter: Option<&str>,
-    json: bool,
-) -> Result<(), Box<dyn std::error::Error>> {
-    let config = Config::load(config_override)?;
-
-    match entity {
-        "issue" => {
-            let result = run_show_issue(&config, iid, project_filter)?;
-            if json {
-                print_show_issue_json(&result);
-            } else {
-                print_show_issue(&result);
-            }
-            Ok(())
-        }
-        "mr" => {
-            let result = run_show_mr(&config, iid, project_filter)?;
-            if json {
-                print_show_mr_json(&result);
-            } else {
-                print_show_mr(&result);
-            }
-            Ok(())
-        }
-        _ => {
-            eprintln!("{}", style(format!("Unknown entity: {entity}")).red());
-            std::process::exit(1);
-        }
-    }
-}
-
 /// JSON output for version command.
 #[derive(Serialize)]
 struct VersionOutput {
@@ -758,3 +839,134 @@ async fn handle_migrate(
 
     Ok(())
 }
+
+// ============================================================================
+// Backward-compat handlers (deprecated, delegate to new handlers)
+// ============================================================================
+
+#[allow(clippy::too_many_arguments)]
+async fn handle_list_compat(
+    config_override: Option<&str>,
+    entity: &str,
+    limit: usize,
+    project_filter: Option<&str>,
+    state_filter: Option<&str>,
+    author_filter: Option<&str>,
+    assignee_filter: Option<&str>,
+    label_filter: Option<&[String]>,
+    milestone_filter: Option<&str>,
+    since_filter: Option<&str>,
+    due_before_filter: Option<&str>,
+    has_due_date: bool,
+    sort: &str,
+    order: &str,
+    open_browser: bool,
+    json_output: bool,
+    draft: bool,
+    no_draft: bool,
+    reviewer_filter: Option<&str>,
+    target_branch_filter: Option<&str>,
+    source_branch_filter: Option<&str>,
+) -> Result<(), Box<dyn std::error::Error>> {
+    let config = Config::load(config_override)?;
+
+    match entity {
+        "issues" => {
+            let filters = ListFilters {
+                limit,
+                project: project_filter,
+                state: state_filter,
+                author: author_filter,
+                assignee: assignee_filter,
+                labels: label_filter,
+                milestone: milestone_filter,
+                since: since_filter,
+                due_before: due_before_filter,
+                has_due_date,
+                sort,
+                order,
+            };
+
+            let result = run_list_issues(&config, filters)?;
+
+            if open_browser {
+                open_issue_in_browser(&result);
+            } else if json_output {
+                print_list_issues_json(&result);
+            } else {
+                print_list_issues(&result);
+            }
+
+            Ok(())
+        }
+        "mrs" => {
+            let filters = MrListFilters {
+                limit,
+                project: project_filter,
+                state: state_filter,
+                author: author_filter,
+                assignee: assignee_filter,
+                reviewer: reviewer_filter,
+                labels: label_filter,
+                since: since_filter,
+                draft,
+                no_draft,
+                target_branch: target_branch_filter,
+                source_branch: source_branch_filter,
+                sort,
+                order,
+            };
+
+            let result = run_list_mrs(&config, filters)?;
+
+            if open_browser {
+                open_mr_in_browser(&result);
+            } else if json_output {
+                print_list_mrs_json(&result);
+            } else {
+                print_list_mrs(&result);
+            }
+
+            Ok(())
+        }
+        _ => {
+            eprintln!("{}", style(format!("Unknown entity: {entity}")).red());
+            std::process::exit(1);
+        }
+    }
+}
+
+async fn handle_show_compat(
+    config_override: Option<&str>,
+    entity: &str,
+    iid: i64,
+    project_filter: Option<&str>,
+    json: bool,
+) -> Result<(), Box<dyn std::error::Error>> {
+    let config = Config::load(config_override)?;
+
+    match entity {
+        "issue" => {
+            let result = run_show_issue(&config, iid, project_filter)?;
+            if json {
+                print_show_issue_json(&result);
+            } else {
+                print_show_issue(&result);
+            }
+            Ok(())
+        }
+        "mr" => {
+            let result = run_show_mr(&config, iid, project_filter)?;
+            if json {
+                print_show_mr_json(&result);
+            } else {
+                print_show_mr(&result);
+            }
+            Ok(())
+        }
+        _ => {
+            eprintln!("{}", style(format!("Unknown entity: {entity}")).red());
+            std::process::exit(1);
+        }
+    }
+}