refactor: move TraceId from state/ to environment/ directory

Move trace_id.rs from src/domain/environment/state/ to src/domain/environment/ to keep the state/ folder clean with only state types.

TraceId is a domain value object that provides unique identifiers for error traces, not a state type itself. This change improves code organization by separating value objects from state machine types.

Changes:
- Move src/domain/environment/state/trace_id.rs to src/domain/environment/trace_id.rs
- Update all imports across 12 files to use new path
- Update module exports in environment/mod.rs and environment/state/mod.rs

Author: josecelano

src/application/commands/configure.rs

@@ -5,10 +5,10 @@ use tracing::{info, instrument, warn};
 use crate::application::steps::{InstallDockerComposeStep, InstallDockerStep};
 use crate::domain::environment::repository::EnvironmentRepository;
 use crate::domain::environment::state::{
-    ConfigureErrorKind, ConfigureFailureContext, ConfigureStep, TraceId,
+    ConfigureErrorKind, ConfigureFailureContext, ConfigureStep,
 };
 use crate::domain::environment::{
-    ConfigureFailed, Configured, Configuring, Environment, Provisioned,
+    ConfigureFailed, Configured, Configuring, Environment, Provisioned, TraceId,
 };
 use crate::infrastructure::external_tools::ansible::adapter::AnsibleClient;
 use crate::infrastructure::trace::ConfigureTraceWriter;

src/application/commands/provision.rs

@@ -25,9 +25,9 @@ use crate::application::steps::{
 };
 use crate::domain::environment::repository::EnvironmentRepository;
 use crate::domain::environment::state::{
-    ProvisionErrorKind, ProvisionFailureContext, ProvisionStep, TraceId,
+    ProvisionErrorKind, ProvisionFailureContext, ProvisionStep,
 };
-use crate::domain::environment::{
+use crate::domain::environment::{TraceId,
     Created, Environment, ProvisionFailed, Provisioned, Provisioning,
 };
 #[allow(unused_imports)]

src/domain/environment/mod.rs

@@ -48,11 +48,15 @@
 pub mod name;
 pub mod repository;
 pub mod state;
+mod trace_id;
 
 // Test utilities (only available in test configuration)
 #[cfg(test)]
 pub mod testing;
 
+// Re-export TraceId for use by state module
+pub use trace_id::TraceId;
+
 // Re-export commonly used types for convenience
 pub use name::{EnvironmentName, EnvironmentNameError};
 pub use state::{

src/domain/environment/state/configure_failed.rs

@@ -65,8 +65,9 @@ impl AnyEnvironmentState {
 mod tests {
     use super::*;
     use crate::domain::environment::state::{
-        ConfigureErrorKind, ConfigureFailureContext, ConfigureStep, TraceId,
+        ConfigureErrorKind, ConfigureFailureContext, ConfigureStep,
     };
+    use crate::domain::environment::TraceId;
     use chrono::Utc;
     use std::time::Duration;
 

src/domain/environment/state/configuring.rs

@@ -163,8 +163,9 @@ mod tests {
         #[test]
         fn it_should_transition_from_configuring_to_configure_failed() {
             use crate::domain::environment::state::{
-                ConfigureErrorKind, ConfigureFailureContext, ConfigureStep, TraceId,
+                ConfigureErrorKind, ConfigureFailureContext, ConfigureStep,
             };
+            use crate::domain::environment::TraceId;
             use chrono::Utc;
             use std::time::Duration;
 

src/domain/environment/state/error_context.rs

@@ -9,7 +9,7 @@ use std::time::Duration;
 use chrono::{DateTime, Utc};
 use serde::{Deserialize, Serialize};
 
-use super::TraceId;
+use crate::domain::environment::TraceId;
 
 // ============================================================================
 // Provision Command Error Context

src/domain/environment/state/mod.rs

@@ -58,7 +58,6 @@ mod released;
 mod releasing;
 mod run_failed;
 mod running;
-mod trace_id;
 
 // Re-export state types
 pub use configure_failed::ConfigureFailed;
@@ -78,7 +77,6 @@ pub use released::Released;
 pub use releasing::Releasing;
 pub use run_failed::RunFailed;
 pub use running::Running;
-pub use trace_id::TraceId;
 
 /// Error type for invalid type conversions when working with type-erased environments
 ///
@@ -390,6 +388,7 @@ mod tests {
 
     /// Helper to create a test `ProvisionFailureContext` with custom error message
     fn create_test_provision_context(error_message: &str) -> ProvisionFailureContext {
+        use crate::domain::environment::TraceId;
         use chrono::Utc;
         use std::time::Duration;
 
@@ -407,6 +406,7 @@ mod tests {
 
     /// Helper to create a test `ConfigureFailureContext` with custom error message
     fn create_test_configure_context(error_message: &str) -> ConfigureFailureContext {
+        use crate::domain::environment::TraceId;
         use chrono::Utc;
         use std::time::Duration;
 

src/domain/environment/state/provision_failed.rs

@@ -65,8 +65,9 @@ impl AnyEnvironmentState {
 mod tests {
     use super::*;
     use crate::domain::environment::state::{
-        ProvisionErrorKind, ProvisionFailureContext, ProvisionStep, TraceId,
+        ProvisionErrorKind, ProvisionFailureContext, ProvisionStep,
     };
+    use crate::domain::environment::TraceId;
     use chrono::Utc;
     use std::time::Duration;
 

src/domain/environment/state/provisioned.rs

@@ -97,7 +97,8 @@ mod tests {
         }
 
         fn create_test_provision_context() -> ProvisionFailureContext {
-            use crate::domain::environment::state::{ProvisionErrorKind, ProvisionStep, TraceId};
+            use crate::domain::environment::state::{ProvisionErrorKind, ProvisionStep};
+            use crate::domain::environment::TraceId;
             use chrono::Utc;
             use std::time::Duration;
 

src/domain/environment/state/provisioning.rs

@@ -174,8 +174,9 @@ mod tests {
         #[test]
         fn it_should_transition_from_provisioning_to_provision_failed() {
             use crate::domain::environment::state::{
-                ProvisionErrorKind, ProvisionFailureContext, ProvisionStep, TraceId,
+                ProvisionErrorKind, ProvisionFailureContext, ProvisionStep,
             };
+            use crate::domain::environment::TraceId;
             use chrono::Utc;
             use std::time::Duration;