refactor: remove redundant LXD instance name module

Remove src/infrastructure/adapters/lxd/instance/name.rs that was only
re-exporting the domain InstanceName type. Update all imports to use
the original InstanceName from the domain layer directly.

This eliminates unnecessary abstraction and improves code clarity by
using domain types consistently throughout the codebase.

Author: josecelano

src/application/commands/provision.rs

@@ -24,10 +24,8 @@ use crate::application::steps::{
     WaitForSSHConnectivityStep,
 };
 #[allow(unused_imports)]
-use crate::domain::ProfileName;
+use crate::domain::{InstanceName, ProfileName};
 use crate::infrastructure::adapters::ansible::AnsibleClient;
-#[allow(unused_imports)]
-use crate::infrastructure::adapters::lxd::InstanceName;
 use crate::infrastructure::adapters::opentofu::client::{InstanceInfo, OpenTofuError};
 use crate::infrastructure::ansible::AnsibleTemplateRenderer;
 use crate::infrastructure::tofu::{ProvisionTemplateError, TofuTemplateRenderer};

src/infrastructure/adapters/lxd/instance/info.rs

@@ -15,7 +15,7 @@
 
 use std::net::IpAddr;
 
-use super::name::InstanceName;
+use crate::domain::InstanceName;
 
 /// Instance information from LXD
 #[derive(Debug, Clone, PartialEq)]

src/infrastructure/adapters/lxd/instance/mod.rs

@@ -1,18 +1,19 @@
 //! LXD instance data types and utilities
 //!
 //! This module provides data types and utilities for working with LXD instances
-//! (containers and virtual machines), including instance information and naming.
+//! (containers and virtual machines), including instance information.
 //!
 //! ## Components
 //!
 //! - `info` - Instance information structures containing runtime data
-//! - `name` - Instance naming utilities and validation
+//!
+//! Instance naming utilities are provided by the domain layer's `InstanceName` type.
 //!
 //! These types are used throughout the LXD integration to represent and
 //! manipulate instance data in a type-safe manner.
 
 pub mod info;
-pub mod name;
 
 pub use info::InstanceInfo;
-pub use name::InstanceName;
+// Re-export InstanceName from domain layer for convenience
+pub use crate::domain::InstanceName;

src/infrastructure/adapters/lxd/instance/name.rs

@@ -27,9 +27,7 @@
 use serde::Serialize;
 use thiserror::Error;
 
-use crate::domain::ProfileName;
-#[allow(unused_imports)]
-use crate::infrastructure::adapters::lxd::instance::InstanceName;
+use crate::domain::{InstanceName, ProfileName};
 
 /// Errors that can occur when building the variables context
 #[derive(Error, Debug)]

src/infrastructure/template/wrappers/tofu/lxd/variables/context.rs

@@ -109,7 +109,7 @@ impl VariablesTemplate {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::infrastructure::adapters::lxd::instance::InstanceName;
+    use crate::domain::InstanceName;
     use tempfile::NamedTempFile;
 
     fn create_test_context() -> VariablesContext {

src/infrastructure/template/wrappers/tofu/lxd/variables/mod.rs

@@ -45,9 +45,7 @@ use std::sync::Arc;
 use thiserror::Error;
 
 use crate::domain::template::{TemplateManager, TemplateManagerError};
-use crate::domain::ProfileName;
-#[allow(unused_imports)]
-use crate::infrastructure::adapters::lxd::instance::InstanceName;
+use crate::domain::{InstanceName, ProfileName};
 use crate::infrastructure::template::wrappers::tofu::lxd::variables::{
     VariablesContextBuilder, VariablesTemplate, VariablesTemplateError,
 };