cargo-rail: cleaning cli artifacts post-refactor.

Author: loadingalias

.gitignore

@@ -29,6 +29,7 @@ demos/
 test.md
 examples/RECORDING_TEMPLATE.md
 examples/issues.md
+commands_check.md
 
 # Cargo-Rail (Testing)
 *rail.toml

docs/commands.md

@@ -21,7 +21,7 @@ Commands:
   split     Split a crate to a standalone repository with git history
   sync      Sync changes between monorepo and split repos
   release   Publish releases (version bump, changelog, tag, publish)
-  check     Validate release readiness
+  check     Reserved for future use
   clean     Clean generated artifacts (cache, backups, reports)
   config    Configuration management
   help      Print this message or the help of the given subcommand(s)
@@ -211,7 +211,7 @@ Examples:
   cargo rail unify --check                # Preview changes (CI mode)
   cargo rail unify                        # Apply changes
   cargo rail unify --backup               # Apply with backup
-  cargo rail unify --pin-transitives      # Pin fragmented deps (hakari replacement)
+  cargo rail unify --show-diff            # Show manifest changes
   cargo rail unify undo                   # Restore from backup
   cargo rail unify undo --list            # List available backups
 ```
@@ -258,9 +258,6 @@ Options:
       --json
           Output in JSON format (shorthand for -f json)
 
-      --non-interactive
-          Skip interactive prompts
-
   -c, --check
           Dry-run mode: preview generated config without writing
 
@@ -271,8 +268,7 @@ Options:
           Print version
 
 Examples:
-  cargo rail init                       # Generate .config/rail.toml (interactive)
-  cargo rail init --non-interactive     # Generate with defaults (CI mode)
+  cargo rail init                       # Generate .config/rail.toml
   cargo rail init --check               # Preview generated config
   cargo rail init -o rail.toml          # Custom output path
   cargo rail init --force               # Overwrite existing config
@@ -425,9 +421,6 @@ Options:
           
           [default: manual]
 
-      --no-protected-branches
-          Allow direct commits to protected branches
-
   -c, --check
           Dry-run mode: preview changes without executing
 
@@ -467,9 +460,10 @@ Publish releases (version bump, changelog, tag, publish)
 Usage: cargo rail release [OPTIONS] <COMMAND>
 
 Commands:
-  init  Configure release settings
-  run   Execute release (plan or publish)
-  help  Print this message or the help of the given subcommand(s)
+  init   Configure release settings
+  run    Execute release (plan or publish)
+  check  Validate release readiness
+  help   Print this message or the help of the given subcommand(s)
 
 Options:
   -q, --quiet
@@ -486,6 +480,8 @@ Options:
 
 Examples:
   cargo rail release init my-crate              # Configure release for my-crate
+  cargo rail release check my-crate             # Validate release readiness
+  cargo rail release check --all                # Check all workspace crates
   cargo rail release run my-crate --check       # Preview release plan
   cargo rail release run my-crate               # Release (patch bump)
   cargo rail release run my-crate --bump minor
@@ -572,12 +568,12 @@ Options:
 
 ---
 
-## cargo rail check
+### cargo rail release check
 
 ```
 Validate release readiness
 
-Usage: cargo rail check [OPTIONS] [CRATE_NAMES]...
+Usage: cargo rail release check [OPTIONS] [CRATE_NAMES]...
 
 Arguments:
   [CRATE_NAMES]...
@@ -611,12 +607,33 @@ Options:
 
   -V, --version
           Print version
+```
 
-Examples:
-  cargo rail check my-crate             # Check single crate
-  cargo rail check crate-a crate-b      # Check multiple crates
-  cargo rail check --all                # Check all workspace crates
-  cargo rail check --all -f json        # JSON output for CI
+---
+
+## cargo rail check
+
+```
+Reserved for future use
+
+Usage: cargo rail check [OPTIONS]
+
+Options:
+  -q, --quiet
+          Suppress progress messages (for CI/automation)
+
+      --json
+          Output in JSON format (shorthand for -f json)
+
+  -h, --help
+          Print help (see a summary with '-h')
+
+  -V, --version
+          Print version
+
+Reserved for future use.
+
+This command is currently a placeholder. Use 'cargo rail release check' instead.
 ```
 
 ---
@@ -675,10 +692,21 @@ Commands:
   help      Print this message or the help of the given subcommand(s)
 
 Options:
-  -q, --quiet    Suppress progress messages (for CI/automation)
-      --json     Output in JSON format (shorthand for -f json)
-  -h, --help     Print help
-  -V, --version  Print version
+  -q, --quiet
+          Suppress progress messages (for CI/automation)
+
+      --json
+          Output in JSON format (shorthand for -f json)
+
+  -h, --help
+          Print help (see a summary with '-h')
+
+  -V, --version
+          Print version
+
+Examples:
+  cargo rail config validate            # Validate rail.toml
+  cargo rail config validate -f json    # JSON output for CI
 ```
 
 ---

src/commands/cli.rs

@@ -98,6 +98,8 @@ Examples:
 const RELEASE_HELP: &str = "\
 Examples:
   cargo rail release init my-crate              # Configure release for my-crate
+  cargo rail release check my-crate             # Validate release readiness
+  cargo rail release check --all                # Check all workspace crates
   cargo rail release run my-crate --check       # Preview release plan
   cargo rail release run my-crate               # Release (patch bump)
   cargo rail release run my-crate --bump minor
@@ -106,18 +108,15 @@ Examples:
 
 const INIT_HELP: &str = "\
 Examples:
-  cargo rail init                       # Generate .config/rail.toml (interactive)
-  cargo rail init --non-interactive     # Generate with defaults (CI mode)
+  cargo rail init                       # Generate .config/rail.toml
   cargo rail init --check               # Preview generated config
   cargo rail init -o rail.toml          # Custom output path
   cargo rail init --force               # Overwrite existing config";
 
 const CHECK_HELP: &str = "\
-Examples:
-  cargo rail check my-crate             # Check single crate
-  cargo rail check crate-a crate-b      # Check multiple crates
-  cargo rail check --all                # Check all workspace crates
-  cargo rail check --all -f json        # JSON output for CI";
+Reserved for future use.
+
+This command is currently a placeholder. Use 'cargo rail release check' instead.";
 
 const CLEAN_HELP: &str = "\
 Examples:
@@ -219,9 +218,6 @@ pub enum Commands {
     /// Overwrite existing configuration
     #[arg(long)]
     force: bool,
-    /// Skip interactive prompts
-    #[arg(long)]
-    non_interactive: bool,
     /// Dry-run mode: preview generated config without writing
     #[arg(long, short = 'c')]
     check: bool,
@@ -256,9 +252,6 @@ pub enum Commands {
     /// Conflict resolution strategy
     #[arg(long, default_value_t, value_enum)]
     strategy: ConflictStrategy,
-    /// Allow direct commits to protected branches
-    #[arg(long)]
-    no_protected_branches: bool,
     /// Dry-run mode: preview changes without executing
     #[arg(long, short = 'c')]
     check: bool,
@@ -275,19 +268,9 @@ pub enum Commands {
     command: ReleaseCommand,
   },
 
-  /// Validate release readiness
+  /// Reserved for future use
   #[command(after_long_help = CHECK_HELP)]
-  Check {
-    /// Crate name(s) to check (mutually exclusive with --all)
-    #[arg(conflicts_with = "all")]
-    crate_names: Vec<String>,
-    /// Check all workspace crates (mutually exclusive with crate names)
-    #[arg(short, long)]
-    all: bool,
-    /// Output format
-    #[arg(long, short = 'f', default_value_t, value_enum)]
-    format: OutputFormat,
-  },
+  Check {},
 
   /// Clean generated artifacts (cache, backups, reports)
   #[command(after_long_help = CLEAN_HELP)]
@@ -407,6 +390,18 @@ pub enum ReleaseCommand {
     #[arg(long, short = 'f', default_value_t, value_enum)]
     format: OutputFormat,
   },
+  /// Validate release readiness
+  Check {
+    /// Crate name(s) to check (mutually exclusive with --all)
+    #[arg(conflicts_with = "all")]
+    crate_names: Vec<String>,
+    /// Check all workspace crates (mutually exclusive with crate names)
+    #[arg(short, long)]
+    all: bool,
+    /// Output format
+    #[arg(long, short = 'f', default_value_t, value_enum)]
+    format: OutputFormat,
+  },
 }
 
 fn get_styles() -> clap::builder::Styles {
@@ -423,15 +418,14 @@ impl Commands {
       Commands::Affected { format, .. }
       | Commands::Test { format, .. }
       | Commands::Unify { format, .. }
-      | Commands::Sync { format, .. }
-      | Commands::Check { format, .. } => format.is_json_like(),
+      | Commands::Sync { format, .. } => format.is_json_like(),
       Commands::Split { command } => match command {
         SplitCommand::Init { .. } => false,
         SplitCommand::Run { format, .. } => format.is_json_like(),
       },
       Commands::Release { command } => match command {
         ReleaseCommand::Init { .. } => false,
-        ReleaseCommand::Run { format, .. } => format.is_json_like(),
+        ReleaseCommand::Run { format, .. } | ReleaseCommand::Check { format, .. } => format.is_json_like(),
       },
       Commands::Config { command } => match command {
         ConfigCommand::Validate { format } => format.is_json_like(),
@@ -446,14 +440,13 @@ impl Commands {
       Commands::Affected { format, .. }
       | Commands::Test { format, .. }
       | Commands::Unify { format, .. }
-      | Commands::Sync { format, .. }
-      | Commands::Check { format, .. } => *format = OutputFormat::Json,
+      | Commands::Sync { format, .. } => *format = OutputFormat::Json,
       Commands::Split {
         command: SplitCommand::Run { format, .. },
       } => *format = OutputFormat::Json,
       Commands::Split { .. } => {}
       Commands::Release {
-        command: ReleaseCommand::Run { format, .. },
+        command: ReleaseCommand::Run { format, .. } | ReleaseCommand::Check { format, .. },
       } => *format = OutputFormat::Json,
       Commands::Release { .. } => {}
       Commands::Config { command } => match command {

src/commands/init.rs

@@ -9,18 +9,11 @@ use crate::progress;
 use crate::toml::builder::RailConfigBuilder;
 use crate::workspace::WorkspaceContext;
 use std::fs;
-use std::io::{self, Write};
 use std::path::{Path, PathBuf};
 
 /// Generate rail.toml configuration
-pub fn run_init(
-  ctx: &WorkspaceContext,
-  output_path: &str,
-  force: bool,
-  non_interactive: bool,
-  check: bool,
-) -> RailResult<()> {
-  run_init_impl(ctx.workspace_root(), output_path, force, non_interactive, check)
+pub fn run_init(ctx: &WorkspaceContext, output_path: &str, force: bool, check: bool) -> RailResult<()> {
+  run_init_impl(ctx.workspace_root(), output_path, force, check)
 }
 
 /// Auto-detect reasonable unify defaults
@@ -83,13 +76,7 @@ fn write_config_file(config_path: &Path, content: &str) -> RailResult<()> {
 }
 
 /// Shared implementation for both run_init and run_init_standalone
-fn run_init_impl(
-  workspace_root: &Path,
-  output_path: &str,
-  force: bool,
-  non_interactive: bool,
-  check: bool,
-) -> RailResult<()> {
+fn run_init_impl(workspace_root: &Path, output_path: &str, force: bool, check: bool) -> RailResult<()> {
   let config_path = workspace_root.join(output_path);
 
   // Check if target path already exists
@@ -111,30 +98,6 @@ fn run_init_impl(
   }
 
   let detected_targets = detect_targets(workspace_root);
-
-  if !non_interactive && !check {
-    // Show preview before confirmation
-    eprintln!("output: {}", config_path.display());
-    if detected_targets.is_empty() {
-      eprintln!("targets: (none detected, will use host default)");
-    } else {
-      eprintln!("targets: {}", detected_targets.join(", "));
-    }
-    eprintln!();
-
-    eprint!("generate? [Y/n]: ");
-    io::stderr().flush()?;
-
-    let mut input = String::new();
-    io::stdin().read_line(&mut input)?;
-    let input = input.trim().to_lowercase();
-
-    if input == "n" || input == "no" {
-      eprintln!("cancelled");
-      return Ok(());
-    }
-  }
-
   let config = build_rail_config(detected_targets);
 
   let toml_content = RailConfigBuilder::new()
@@ -159,14 +122,8 @@ fn run_init_impl(
 }
 
 /// Standalone init (without WorkspaceContext)
-pub fn run_init_standalone(
-  workspace_root: &Path,
-  output_path: &str,
-  force: bool,
-  non_interactive: bool,
-  check: bool,
-) -> RailResult<()> {
-  run_init_impl(workspace_root, output_path, force, non_interactive, check)
+pub fn run_init_standalone(workspace_root: &Path, output_path: &str, force: bool, check: bool) -> RailResult<()> {
+  run_init_impl(workspace_root, output_path, force, check)
 }
 
 #[cfg(test)]