refactor: move CLI modules under presentation/cli/ for SDK separation

Move all CLI-specific presentation modules (controllers, dispatch, input,
views, error, errors, tests) into an explicit presentation/cli/ sub-tree.
This establishes a clean physical boundary between the CLI and SDK delivery
mechanisms in the presentation layer.

Changes:
- Create src/presentation/cli/ with its own mod.rs
- Move controllers/, dispatch/, input/, views/, error.rs, errors.rs, tests/
  into presentation/cli/
- Update presentation/mod.rs to declare only cli and sdk modules
  with backward-compatible re-exports
- Update all import paths across 160 files (crate::presentation::X →
  crate::presentation::cli::X)
- Update all doc comment examples to use new paths
- Verify SDK has zero imports from presentation::cli

This is the first step in the 4-part SDK extraction plan:
1. ✅ CLI/SDK separation (this commit)
2. Extract SDK workspace package
3. Extract shared types package
4. Fix DDD layer boundary violations

Author: josecelano

docs/refactors/plans/presentation-cli-sdk-separation.md

@@ -41,13 +41,13 @@ This is the first step in a 4-part incremental plan to extract the SDK into its
 **Total Active Proposals**: 1
 **Total Postponed**: 0
 **Total Discarded**: 0
-**Completed**: 0
+**Completed**: 1
 **In Progress**: 0
-**Not Started**: 1
+**Not Started**: 0
 
 ### Phase Summary
 
-- **Phase 0 - Directory Reorganization (High Impact, Medium Effort)**: ⏳ 0/1 completed (0%)
+- **Phase 0 - Directory Reorganization (High Impact, Medium Effort)**: ✅ 1/1 completed (100%)
 
 ### Discarded Proposals
 
@@ -86,13 +86,13 @@ The SDK and CLI are independent today (the SDK has zero imports from CLI modules
 
 ### Proposal #0: Move CLI Modules into an Explicit `cli/` Sub-Tree
 
-**Status**: ⏳ Not Started
+**Status**: ✅ Complete
 **Impact**: 🟢🟢🟢 High
 **Effort**: 🔵🔵 Medium
 **Priority**: P0
 **Depends On**: None
-**Completed**: -
-**Commit**: -
+**Completed**: 2026-02-25
+**Commit**: (this commit)
 
 #### Problem
 
@@ -138,18 +138,18 @@ The top-level `presentation/mod.rs` becomes minimal — it declares the two sub-
 
 #### Implementation Checklist
 
-- [ ] Create `src/presentation/cli/` directory
-- [ ] Move `controllers/`, `dispatch/`, `input/`, `views/`, `error.rs`, `errors.rs`, `tests/` into `presentation/cli/`
-- [ ] Create `src/presentation/cli/mod.rs` with the same module declarations and re-exports currently in `presentation/mod.rs` (minus `sdk`)
-- [ ] Update `src/presentation/mod.rs` to declare only `pub mod cli;` and `pub mod sdk;` (plus backward-compat re-exports if needed)
-- [ ] Update all `use crate::presentation::controllers::...` imports across the codebase to `use crate::presentation::cli::controllers::...`
-- [ ] Update all `use crate::presentation::views::...` imports to `use crate::presentation::cli::views::...`
-- [ ] Update all `use crate::presentation::dispatch::...`, `use crate::presentation::input::...`, `use crate::presentation::error*` imports similarly
-- [ ] Update `src/main.rs` and `src/bootstrap/` imports
-- [ ] Verify the SDK module has zero imports from `crate::presentation::cli`
-- [ ] Update the `presentation/mod.rs` module documentation
-- [ ] Verify all tests pass (`cargo test`)
-- [ ] Run linter and fix any issues
+- [x] Create `src/presentation/cli/` directory
+- [x] Move `controllers/`, `dispatch/`, `input/`, `views/`, `error.rs`, `errors.rs`, `tests/` into `presentation/cli/`
+- [x] Create `src/presentation/cli/mod.rs` with the same module declarations and re-exports currently in `presentation/mod.rs` (minus `sdk`)
+- [x] Update `src/presentation/mod.rs` to declare only `pub mod cli;` and `pub mod sdk;` (plus backward-compat re-exports)
+- [x] Update all `use crate::presentation::controllers::...` imports across the codebase to `use crate::presentation::cli::controllers::...`
+- [x] Update all `use crate::presentation::views::...` imports to `use crate::presentation::cli::views::...`
+- [x] Update all `use crate::presentation::dispatch::...`, `use crate::presentation::input::...`, `use crate::presentation::error*` imports similarly
+- [x] Update `src/main.rs` and `src/bootstrap/` imports
+- [x] Verify the SDK module has zero imports from `crate::presentation::cli`
+- [x] Update the `presentation/mod.rs` module documentation
+- [x] Verify all tests pass (`cargo test`)
+- [x] Run linter and fix any issues
 
 #### Testing Strategy
 
@@ -195,5 +195,5 @@ The top-level `presentation/mod.rs` becomes minimal — it declares the two sub-
 ---
 
 **Created**: 2026-02-24
-**Last Updated**: 2026-02-24
-**Status**: 📋 Planning
+**Last Updated**: 2026-02-25
+**Status**: ✅ Complete

src/bootstrap/app.rs

@@ -22,7 +22,11 @@ use std::sync::Arc;
 use clap::Parser;
 use tracing::info;
 
-use crate::{bootstrap, presentation};
+use crate::bootstrap;
+use crate::presentation::cli::dispatch::route_command;
+use crate::presentation::cli::dispatch::ExecutionContext;
+use crate::presentation::cli::error::handle_error;
+use crate::presentation::cli::Cli;
 
 /// Main application entry point
 ///
@@ -41,7 +45,7 @@ use crate::{bootstrap, presentation};
 ///
 /// Both panics are intentional as logging is critical for observability.
 pub async fn run() {
-    let cli = presentation::Cli::parse();
+    let cli = Cli::parse();
 
     let logging_config = cli.global.logging_config();
 
@@ -62,15 +66,12 @@ pub async fn run() {
         cli.global.verbosity_level(),
         &cli.global.working_dir,
     ));
-    let context = presentation::dispatch::ExecutionContext::new(container, cli.global.clone());
+    let context = ExecutionContext::new(container, cli.global.clone());
 
     match cli.command {
         Some(command) => {
-            if let Err(e) =
-                presentation::dispatch::route_command(command, &cli.global.working_dir, &context)
-                    .await
-            {
-                presentation::error::handle_error(&e, &context.user_output());
+            if let Err(e) = route_command(command, &cli.global.working_dir, &context).await {
+                handle_error(&e, &context.user_output());
                 std::process::exit(1);
             }
         }

src/bootstrap/container.rs

@@ -12,24 +12,24 @@ use parking_lot::ReentrantMutex;
 use crate::application::command_handlers::PurgeCommandHandler;
 use crate::domain::environment::repository::EnvironmentRepository;
 use crate::infrastructure::persistence::repository_factory::RepositoryFactory;
-use crate::presentation::controllers::configure::ConfigureCommandController;
-use crate::presentation::controllers::constants::DEFAULT_LOCK_TIMEOUT;
-use crate::presentation::controllers::create::subcommands::environment::CreateEnvironmentCommandController;
-use crate::presentation::controllers::create::subcommands::schema::CreateSchemaCommandController;
-use crate::presentation::controllers::create::subcommands::template::CreateTemplateCommandController;
-use crate::presentation::controllers::destroy::DestroyCommandController;
-use crate::presentation::controllers::docs::DocsCommandController;
-use crate::presentation::controllers::list::ListCommandController;
-use crate::presentation::controllers::provision::ProvisionCommandController;
-use crate::presentation::controllers::purge::PurgeCommandController;
-use crate::presentation::controllers::register::RegisterCommandController;
-use crate::presentation::controllers::release::ReleaseCommandController;
-use crate::presentation::controllers::render::RenderCommandController;
-use crate::presentation::controllers::run::RunCommandController;
-use crate::presentation::controllers::show::ShowCommandController;
-use crate::presentation::controllers::test::handler::TestCommandController;
-use crate::presentation::controllers::validate::ValidateCommandController;
-use crate::presentation::views::{UserOutput, VerbosityLevel};
+use crate::presentation::cli::controllers::configure::ConfigureCommandController;
+use crate::presentation::cli::controllers::constants::DEFAULT_LOCK_TIMEOUT;
+use crate::presentation::cli::controllers::create::subcommands::environment::CreateEnvironmentCommandController;
+use crate::presentation::cli::controllers::create::subcommands::schema::CreateSchemaCommandController;
+use crate::presentation::cli::controllers::create::subcommands::template::CreateTemplateCommandController;
+use crate::presentation::cli::controllers::destroy::DestroyCommandController;
+use crate::presentation::cli::controllers::docs::DocsCommandController;
+use crate::presentation::cli::controllers::list::ListCommandController;
+use crate::presentation::cli::controllers::provision::ProvisionCommandController;
+use crate::presentation::cli::controllers::purge::PurgeCommandController;
+use crate::presentation::cli::controllers::register::RegisterCommandController;
+use crate::presentation::cli::controllers::release::ReleaseCommandController;
+use crate::presentation::cli::controllers::render::RenderCommandController;
+use crate::presentation::cli::controllers::run::RunCommandController;
+use crate::presentation::cli::controllers::show::ShowCommandController;
+use crate::presentation::cli::controllers::test::handler::TestCommandController;
+use crate::presentation::cli::controllers::validate::ValidateCommandController;
+use crate::presentation::cli::views::{UserOutput, VerbosityLevel};
 use crate::shared::clock::Clock;
 use crate::shared::SystemClock;
 
@@ -44,7 +44,7 @@ use crate::shared::SystemClock;
 /// ```rust
 /// use std::path::Path;
 /// use torrust_tracker_deployer_lib::bootstrap::container::Container;
-/// use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+/// use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
 ///
 /// let working_dir = Path::new(".");
 /// let container = Container::new(VerbosityLevel::Normal, working_dir);
@@ -79,7 +79,7 @@ impl Container {
     /// ```rust
     /// use std::path::Path;
     /// use torrust_tracker_deployer_lib::bootstrap::container::Container;
-    /// use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+    /// use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
     ///
     /// // For normal application use
     /// let container = Container::new(VerbosityLevel::Normal, Path::new("."));
@@ -122,7 +122,7 @@ impl Container {
     /// ```rust
     /// use std::path::Path;
     /// use torrust_tracker_deployer_lib::bootstrap::container::Container;
-    /// use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+    /// use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
     ///
     /// let container = Container::new(VerbosityLevel::Normal, Path::new("."));
     /// let user_output = container.user_output();
@@ -143,7 +143,7 @@ impl Container {
     /// ```rust
     /// use std::path::Path;
     /// use torrust_tracker_deployer_lib::bootstrap::container::Container;
-    /// use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+    /// use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
     ///
     /// let container = Container::new(VerbosityLevel::Normal, Path::new("."));
     /// let repository_factory = container.repository_factory();
@@ -165,7 +165,7 @@ impl Container {
     /// ```rust
     /// use std::path::Path;
     /// use torrust_tracker_deployer_lib::bootstrap::container::Container;
-    /// use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+    /// use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
     ///
     /// let container = Container::new(VerbosityLevel::Normal, Path::new("."));
     /// let repository = container.repository();
@@ -186,7 +186,7 @@ impl Container {
     /// ```rust
     /// use std::path::Path;
     /// use torrust_tracker_deployer_lib::bootstrap::container::Container;
-    /// use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+    /// use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
     ///
     /// let container = Container::new(VerbosityLevel::Normal, Path::new("."));
     /// let clock = container.clock();

…entation/controllers/configure/errors.rs …tion/cli/controllers/configure/errors.rssrc/presentation/controllers/configure/errors.rs renamed to src/presentation/cli/controllers/configure/errors.rs src/presentation/controllers/configure/errors.rs renamed to src/presentation/cli/controllers/configure/errors.rs

@@ -8,7 +8,7 @@ use thiserror::Error;
 
 use crate::application::command_handlers::configure::errors::ConfigureCommandHandlerError;
 use crate::domain::environment::name::EnvironmentNameError;
-use crate::presentation::views::progress::ProgressReporterError;
+use crate::presentation::cli::views::progress::ProgressReporterError;
 
 /// Configure command specific errors
 ///
@@ -115,9 +115,9 @@ impl ConfigureSubcommandError {
     /// use std::path::Path;
     /// use std::sync::Arc;
     /// use torrust_tracker_deployer_lib::bootstrap::Container;
-    /// use torrust_tracker_deployer_lib::presentation::dispatch::ExecutionContext;
-    /// use torrust_tracker_deployer_lib::presentation::controllers::configure;
-    /// use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+    /// use torrust_tracker_deployer_lib::presentation::cli::dispatch::ExecutionContext;
+    /// use torrust_tracker_deployer_lib::presentation::cli::controllers::configure;
+    /// use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
     ///
     /// let container = Container::new(VerbosityLevel::Normal, Path::new("."));
     /// let context = ExecutionContext::new(Arc::new(container), global_args);
@@ -140,8 +140,8 @@ impl ConfigureSubcommandError {
     /// use std::time::Duration;
     /// use parking_lot::ReentrantMutex;
     /// use std::cell::RefCell;
-    /// use torrust_tracker_deployer_lib::presentation::controllers::configure::handler::ConfigureCommandController;
-    /// use torrust_tracker_deployer_lib::presentation::views::{UserOutput, VerbosityLevel};
+    /// use torrust_tracker_deployer_lib::presentation::cli::controllers::configure::handler::ConfigureCommandController;
+    /// use torrust_tracker_deployer_lib::presentation::cli::views::{UserOutput, VerbosityLevel};
     /// use torrust_tracker_deployer_lib::infrastructure::persistence::repository_factory::RepositoryFactory;
     /// use torrust_tracker_deployer_lib::shared::clock::SystemClock;
     ///

…ntation/controllers/configure/handler.rs …ion/cli/controllers/configure/handler.rssrc/presentation/controllers/configure/handler.rs renamed to src/presentation/cli/controllers/configure/handler.rs src/presentation/controllers/configure/handler.rs renamed to src/presentation/cli/controllers/configure/handler.rs

@@ -13,11 +13,13 @@ use crate::domain::environment::name::EnvironmentName;
 use crate::domain::environment::repository::EnvironmentRepository;
 use crate::domain::environment::state::Configured;
 use crate::domain::environment::Environment;
-use crate::presentation::input::cli::OutputFormat;
-use crate::presentation::views::commands::configure::{ConfigureDetailsData, JsonView, TextView};
-use crate::presentation::views::progress::ProgressReporter;
-use crate::presentation::views::progress::VerboseProgressListener;
-use crate::presentation::views::UserOutput;
+use crate::presentation::cli::input::cli::OutputFormat;
+use crate::presentation::cli::views::commands::configure::{
+    ConfigureDetailsData, JsonView, TextView,
+};
+use crate::presentation::cli::views::progress::ProgressReporter;
+use crate::presentation::cli::views::progress::VerboseProgressListener;
+use crate::presentation::cli::views::UserOutput;
 use crate::shared::clock::Clock;
 
 use super::errors::ConfigureSubcommandError;
@@ -272,9 +274,9 @@ impl ConfigureCommandController {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::presentation::controllers::constants::DEFAULT_LOCK_TIMEOUT;
-    use crate::presentation::views::testing::TestUserOutput;
-    use crate::presentation::views::VerbosityLevel;
+    use crate::presentation::cli::controllers::constants::DEFAULT_LOCK_TIMEOUT;
+    use crate::presentation::cli::views::testing::TestUserOutput;
+    use crate::presentation::cli::views::VerbosityLevel;
     use crate::shared::SystemClock;
 
     /// Create test dependencies for configure command handler tests

…resentation/controllers/configure/mod.rs …ntation/cli/controllers/configure/mod.rssrc/presentation/controllers/configure/mod.rs renamed to src/presentation/cli/controllers/configure/mod.rs src/presentation/controllers/configure/mod.rs renamed to src/presentation/cli/controllers/configure/mod.rs

@@ -22,9 +22,9 @@
 //! use std::path::Path;
 //! use std::sync::Arc;
 //! use torrust_tracker_deployer_lib::bootstrap::Container;
-//! use torrust_tracker_deployer_lib::presentation::dispatch::ExecutionContext;
-//! use torrust_tracker_deployer_lib::presentation::controllers::configure;
-//! use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+//! use torrust_tracker_deployer_lib::presentation::cli::dispatch::ExecutionContext;
+//! use torrust_tracker_deployer_lib::presentation::cli::controllers::configure;
+//! use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
 //!
 //! let container = Container::new(VerbosityLevel::Normal, Path::new("."));
 //! let context = ExecutionContext::new(Arc::new(container), global_args);
@@ -42,9 +42,9 @@
 //! use std::path::Path;
 //! use std::sync::Arc;
 //! use torrust_tracker_deployer_lib::bootstrap::Container;
-//! use torrust_tracker_deployer_lib::presentation::dispatch::ExecutionContext;
-//! use torrust_tracker_deployer_lib::presentation::controllers::configure;
-//! use torrust_tracker_deployer_lib::presentation::views::VerbosityLevel;
+//! use torrust_tracker_deployer_lib::presentation::cli::dispatch::ExecutionContext;
+//! use torrust_tracker_deployer_lib::presentation::cli::controllers::configure;
+//! use torrust_tracker_deployer_lib::presentation::cli::views::VerbosityLevel;
 //!
 //! let container = Container::new(VerbosityLevel::Normal, Path::new("."));
 //! let context = ExecutionContext::new(Arc::new(container), global_args);
@@ -67,8 +67,8 @@
 //! use std::time::Duration;
 //! use parking_lot::ReentrantMutex;
 //! use std::cell::RefCell;
-//! use torrust_tracker_deployer_lib::presentation::controllers::configure::handler::ConfigureCommandController;
-//! use torrust_tracker_deployer_lib::presentation::views::{UserOutput, VerbosityLevel};
+//! use torrust_tracker_deployer_lib::presentation::cli::controllers::configure::handler::ConfigureCommandController;
+//! use torrust_tracker_deployer_lib::presentation::cli::views::{UserOutput, VerbosityLevel};
 //! use torrust_tracker_deployer_lib::infrastructure::persistence::repository_factory::RepositoryFactory;
 //! use torrust_tracker_deployer_lib::shared::clock::SystemClock;
 //!

…ation/controllers/configure/tests/mod.rs …n/cli/controllers/configure/tests/mod.rssrc/presentation/controllers/configure/tests/mod.rs renamed to src/presentation/cli/controllers/configure/tests/mod.rs src/presentation/controllers/configure/tests/mod.rs renamed to src/presentation/cli/controllers/configure/tests/mod.rs

@@ -12,12 +12,12 @@ mod integration_tests {
 
     use crate::domain::environment::repository::EnvironmentRepository;
     use crate::infrastructure::persistence::repository_factory::RepositoryFactory;
-    use crate::presentation::controllers::configure;
-    use crate::presentation::controllers::configure::handler::ConfigureCommandController;
-    use crate::presentation::controllers::constants::DEFAULT_LOCK_TIMEOUT;
-    use crate::presentation::input::cli::OutputFormat;
-    use crate::presentation::views::testing::TestUserOutput;
-    use crate::presentation::views::{UserOutput, VerbosityLevel};
+    use crate::presentation::cli::controllers::configure;
+    use crate::presentation::cli::controllers::configure::handler::ConfigureCommandController;
+    use crate::presentation::cli::controllers::constants::DEFAULT_LOCK_TIMEOUT;
+    use crate::presentation::cli::input::cli::OutputFormat;
+    use crate::presentation::cli::views::testing::TestUserOutput;
+    use crate::presentation::cli::views::{UserOutput, VerbosityLevel};
     use crate::shared::clock::Clock;
     use crate::shared::SystemClock;
 

src/presentation/controllers/constants.rs …resentation/cli/controllers/constants.rssrc/presentation/controllers/constants.rs renamed to src/presentation/cli/controllers/constants.rs src/presentation/controllers/constants.rs renamed to src/presentation/cli/controllers/constants.rs

@@ -18,9 +18,9 @@
 //! ## Usage Example
 //!
 //! ```rust
-//! use torrust_tracker_deployer_lib::presentation::controllers::constants::{DEFAULT_LOCK_TIMEOUT, DEFAULT_VERBOSITY};
+//! use torrust_tracker_deployer_lib::presentation::cli::controllers::constants::{DEFAULT_LOCK_TIMEOUT, DEFAULT_VERBOSITY};
 //! use torrust_tracker_deployer_lib::infrastructure::persistence::repository_factory::RepositoryFactory;
-//! use torrust_tracker_deployer_lib::presentation::views::UserOutput;
+//! use torrust_tracker_deployer_lib::presentation::cli::views::UserOutput;
 //!
 //! // Use default lock timeout for repository operations
 //! let repository_factory = RepositoryFactory::new(DEFAULT_LOCK_TIMEOUT);
@@ -31,7 +31,7 @@
 
 use std::time::Duration;
 
-use crate::presentation::views::VerbosityLevel;
+use crate::presentation::cli::views::VerbosityLevel;
 
 /// Default timeout for file lock operations in repository
 ///

…resentation/controllers/create/errors.rs …ntation/cli/controllers/create/errors.rssrc/presentation/controllers/create/errors.rs renamed to src/presentation/cli/controllers/create/errors.rs src/presentation/controllers/create/errors.rs renamed to src/presentation/cli/controllers/create/errors.rs

@@ -23,9 +23,9 @@
 //! ```ignore
 //! use std::path::{Path, PathBuf};
 //! use std::sync::{Arc, Mutex};
-//! use torrust_tracker_deployer_lib::presentation::input::cli::commands::CreateAction;
-//! use torrust_tracker_deployer_lib::presentation::controllers::create;
-//! use torrust_tracker_deployer_lib::presentation::dispatch::ExecutionContext;
+//! use torrust_tracker_deployer_lib::presentation::cli::input::cli::commands::CreateAction;
+//! use torrust_tracker_deployer_lib::presentation::cli::controllers::create;
+//! use torrust_tracker_deployer_lib::presentation::cli::dispatch::ExecutionContext;
 //!
 //! # #[tokio::main]
 //! # async fn main() {