refactor: move error context types to their respective state modules

Move provision-related error types (ProvisionFailureContext, ProvisionStep,
ProvisionErrorKind) from error_context.rs to provision_failed.rs.

Move configure-related error types (ConfigureFailureContext, ConfigureStep,
ConfigureErrorKind) from error_context.rs to configure_failed.rs.

This improves cohesion by keeping error context types together with the
state types they describe, making the codebase more scalable and easier
to maintain.

Changes:
- Move ProvisionFailureContext and related types to provision_failed.rs
- Move ConfigureFailureContext and related types to configure_failed.rs
- Update mod.rs to re-export types from new locations
- Remove error_context.rs module
- Move serialization tests to appropriate modules
- All 735 tests passing

Author: josecelano

src/domain/environment/state/configure_failed.rs

@@ -11,12 +11,67 @@
 //! - Manual configuration correction (advanced users)
 //! - Review trace file for detailed error information
 
+use std::path::PathBuf;
+use std::time::Duration;
+
+use chrono::{DateTime, Utc};
 use serde::{Deserialize, Serialize};
 
-use crate::domain::environment::state::{
-    AnyEnvironmentState, ConfigureFailureContext, StateTypeError,
-};
-use crate::domain::environment::Environment;
+use crate::domain::environment::state::{AnyEnvironmentState, StateTypeError};
+use crate::domain::environment::{Environment, TraceId};
+
+// ============================================================================
+// Configure Command Error Context
+// ============================================================================
+
+/// Error context for configure command failures
+///
+/// Captures comprehensive information about configuration failures including
+/// the specific step that failed, error classification, timing, and trace details.
+#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
+pub struct ConfigureFailureContext {
+    /// Which step failed during configuration
+    pub failed_step: ConfigureStep,
+
+    /// Error category for type-safe handling
+    pub error_kind: ConfigureErrorKind,
+
+    /// Human-readable error summary
+    pub error_summary: String,
+
+    /// When the failure occurred
+    pub failed_at: DateTime<Utc>,
+
+    /// When execution started
+    pub execution_started_at: DateTime<Utc>,
+
+    /// How long execution ran before failing
+    pub execution_duration: Duration,
+
+    /// Unique trace identifier
+    pub trace_id: TraceId,
+
+    /// Path to the detailed trace file (if generated)
+    pub trace_file_path: Option<PathBuf>,
+}
+
+/// Steps in the configure workflow
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize)]
+pub enum ConfigureStep {
+    /// Installing Docker
+    InstallDocker,
+    /// Installing Docker Compose
+    InstallDockerCompose,
+}
+
+/// Error categories for configure failures
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize)]
+pub enum ConfigureErrorKind {
+    /// Software installation failed
+    InstallationFailed,
+    /// Command execution failed
+    CommandExecutionFailed,
+}
 
 /// Error state - Application configuration failed
 ///
@@ -64,12 +119,6 @@ impl AnyEnvironmentState {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::domain::environment::state::{
-        ConfigureErrorKind, ConfigureFailureContext, ConfigureStep,
-    };
-    use crate::domain::environment::TraceId;
-    use chrono::Utc;
-    use std::time::Duration;
 
     fn create_test_context() -> ConfigureFailureContext {
         ConfigureFailureContext {
@@ -161,4 +210,50 @@ mod tests {
             assert!(result.is_ok());
         }
     }
+
+    mod context_tests {
+        use super::*;
+
+        #[test]
+        fn it_should_serialize_configure_failure_context() {
+            let context = ConfigureFailureContext {
+                failed_step: ConfigureStep::InstallDocker,
+                error_kind: ConfigureErrorKind::InstallationFailed,
+                error_summary: "Docker installation failed".to_string(),
+                failed_at: Utc::now(),
+                execution_started_at: Utc::now(),
+                execution_duration: Duration::from_secs(15),
+                trace_id: TraceId::new(),
+                trace_file_path: None,
+            };
+
+            let json = serde_json::to_string(&context).unwrap();
+            assert!(json.contains("InstallDocker"));
+            assert!(json.contains("InstallationFailed"));
+        }
+
+        #[test]
+        fn it_should_deserialize_configure_failure_context() {
+            let trace_id = TraceId::new();
+            let json = format!(
+                r#"{{
+                    "failed_step": "InstallDockerCompose",
+                    "error_kind": "CommandExecutionFailed",
+                    "error_summary": "Command execution failed",
+                    "failed_at": "2025-10-06T10:00:00Z",
+                    "execution_started_at": "2025-10-06T09:59:30Z",
+                    "execution_duration": {{"secs": 30, "nanos": 0}},
+                    "trace_id": "{trace_id}",
+                    "trace_file_path": null
+                }}"#
+            );
+
+            let context: ConfigureFailureContext = serde_json::from_str(&json).unwrap();
+            assert_eq!(context.failed_step, ConfigureStep::InstallDockerCompose);
+            assert_eq!(
+                context.error_kind,
+                ConfigureErrorKind::CommandExecutionFailed
+            );
+        }
+    }
 }

src/domain/environment/state/error_context.rs

@@ -49,7 +49,6 @@ mod configured;
 mod configuring;
 mod created;
 mod destroyed;
-mod error_context;
 mod provision_failed;
 mod provisioned;
 mod provisioning;
@@ -60,16 +59,12 @@ mod run_failed;
 mod running;
 
 // Re-export state types
-pub use configure_failed::ConfigureFailed;
+pub use configure_failed::{ConfigureFailed, ConfigureErrorKind, ConfigureFailureContext, ConfigureStep};
 pub use configured::Configured;
 pub use configuring::Configuring;
 pub use created::Created;
 pub use destroyed::Destroyed;
-pub use error_context::{
-    ConfigureErrorKind, ConfigureFailureContext, ConfigureStep, ProvisionErrorKind,
-    ProvisionFailureContext, ProvisionStep,
-};
-pub use provision_failed::ProvisionFailed;
+pub use provision_failed::{ProvisionFailed, ProvisionErrorKind, ProvisionFailureContext, ProvisionStep};
 pub use provisioned::Provisioned;
 pub use provisioning::Provisioning;
 pub use release_failed::ReleaseFailed;