feat: complete Phase 3 structured logging - remote action instrumentation

- Add tracing instrumentation to CloudInitValidator::execute()
- Add tracing instrumentation to DockerValidator::execute()
- Add tracing instrumentation to DockerComposeValidator::execute()
- Implement three-level hierarchical tracing spans (Commands → Steps → Remote Actions)
- Add span fields: action_type='validation', component, server_ip
- Fix clippy warnings for underscore-prefixed parameters
- All 218 tests passing, linters passing, no unused dependencies
- Update progress tracking: Phase 3 complete (19/25 tasks, 76% overall progress)

This completes the leaf-level remote action instrumentation, establishing
complete traceability through the three-level architecture.

Author: josecelano

docs/structured-logging-implementation-plan.md

@@ -1,11 +1,20 @@
 # Structured Logging Implementation Plan
 
-**Status**: 🟡 **Planning Phase**  
+**Status**: 🟡 **In Progress** - Ready for Phase 4  
 **Created**: September 16, 2025  
 **Last Updated**: September 16, 2025
 
 ## 📋 Overview
 
+**Overall Progress**: **19/25 tasks completed (76%)**
+
+| Phase                   | Status         | Tasks Complete | Progress |
+| ----------------------- | -------------- | -------------- | -------- |
+| Phase 1: Commands       | 🟢 Completed   | 4/4            | 100%     |
+| Phase 2: Steps          | 🟢 Completed   | 11/11          | 100%     |
+| Phase 3: Remote Actions | 🟢 Completed   | 4/4            | 100%     |
+| Phase 4: Optimization   | 🔴 Not Started | 0/6            | 0%       |
+
 This document outlines the implementation plan for introducing hierarchical structured logging using tracing spans to align with our three-level architecture (Commands → Steps → Remote Actions).
 
 ### 🎯 Goals
@@ -169,42 +178,43 @@ Level 1: Commands (Top-level orchestration)
 
 ### Phase 3: Remote Actions (Level 3) - Leaf Operations
 
-**Status**: 🔴 **Not Started**  
+**Status**: ✅ **Complete**  
 **Priority**: Medium  
-**Estimated Effort**: 2-3 days
+**Estimated Effort**: 2-3 days  
+**Completed**: September 2025
 
 #### Remote Action Trait
 
-- [ ] **3.1** Update `RemoteAction` trait definition
+- [x] **3.1** Update `RemoteAction` trait definition
   - File: `src/remote_actions/mod.rs`
   - Add `#[instrument]` guidance to trait documentation
   - Consider adding span fields to trait methods
 
 #### Validation Actions
 
-- [ ] **3.2** `CloudInitValidator::execute()`
+- [x] **3.2** `CloudInitValidator::execute()`
 
   - File: `src/remote_actions/cloud_init.rs`
   - Span: `cloud_init_validation`
   - Fields: `action_type="validation"`, `component="cloud_init"`, `server_ip`
 
-- [ ] **3.3** `DockerValidator::execute()`
+- [x] **3.3** `DockerValidator::execute()`
 
   - File: `src/remote_actions/docker.rs`
   - Span: `docker_validation`
   - Fields: `action_type="validation"`, `component="docker"`, `server_ip`
 
-- [ ] **3.4** `DockerComposeValidator::execute()`
+- [x] **3.4** `DockerComposeValidator::execute()`
   - File: `src/remote_actions/docker_compose.rs`
   - Span: `docker_compose_validation`
   - Fields: `action_type="validation"`, `component="docker_compose"`, `server_ip`
 
 #### Acceptance Criteria
 
-- All remote actions are wrapped in spans
-- Remote actions nest properly under step spans
-- Complete three-level hierarchy visible in logs
-- Server IP and component information tracked in spans
+- ✅ All remote actions are wrapped in spans
+- ✅ Remote actions nest properly under step spans
+- ✅ Complete three-level hierarchy visible in logs
+- ✅ Server IP and component information tracked in spans
 
 #### Files to Modify
 
@@ -274,17 +284,17 @@ Level 1: Commands (Top-level orchestration)
 ### Overall Progress
 
 - **Total Tasks**: 25
-- **Completed**: 4 (16%)
+- **Completed**: 19 (76%)
 - **In Progress**: 0 (0%)
-- **Not Started**: 21 (84%)
+- **Not Started**: 6 (24%)
 
 ### Phase Progress
 
 | Phase                   | Status         | Tasks Complete | Progress |
 | ----------------------- | -------------- | -------------- | -------- |
-| Phase 1: Commands       | � Completed    | 4/4            | 100%     |
-| Phase 2: Steps          | 🔴 Not Started | 0/11           | 0%       |
-| Phase 3: Remote Actions | 🔴 Not Started | 0/4            | 0%       |
+| Phase 1: Commands       | 🟢 Completed   | 4/4            | 100%     |
+| Phase 2: Steps          | � Completed    | 11/11          | 100%     |
+| Phase 3: Remote Actions | � Completed    | 4/4            | 100%     |
 | Phase 4: Optimization   | 🔴 Not Started | 0/6            | 0%       |
 
 ## 🛠️ Technical Implementation Details

src/remote_actions/cloud_init.rs

@@ -1,5 +1,5 @@
 use std::net::IpAddr;
-use tracing::info;
+use tracing::{info, instrument};
 
 use crate::command_wrappers::ssh::SshClient;
 use crate::command_wrappers::ssh::SshConnection;
@@ -27,7 +27,16 @@ impl RemoteAction for CloudInitValidator {
         "cloud-init-validation"
     }
 
-    async fn execute(&self, _server_ip: &IpAddr) -> Result<(), RemoteActionError> {
+    #[instrument(
+        name = "cloud_init_validation",
+        skip(self),
+        fields(
+            action_type = "validation",
+            component = "cloud_init",
+            server_ip = %server_ip
+        )
+    )]
+    async fn execute(&self, server_ip: &IpAddr) -> Result<(), RemoteActionError> {
         info!(
             action = "cloud_init_validation",
             "Validating cloud-init completion"

src/remote_actions/docker.rs

@@ -1,5 +1,5 @@
 use std::net::IpAddr;
-use tracing::{info, warn};
+use tracing::{info, instrument, warn};
 
 use crate::command_wrappers::ssh::SshClient;
 use crate::command_wrappers::ssh::SshConnection;
@@ -27,7 +27,16 @@ impl RemoteAction for DockerValidator {
         "docker-validation"
     }
 
-    async fn execute(&self, _server_ip: &IpAddr) -> Result<(), RemoteActionError> {
+    #[instrument(
+        name = "docker_validation",
+        skip(self),
+        fields(
+            action_type = "validation",
+            component = "docker",
+            server_ip = %server_ip
+        )
+    )]
+    async fn execute(&self, server_ip: &IpAddr) -> Result<(), RemoteActionError> {
         info!(
             action = "docker_validation",
             "Validating Docker installation"

src/remote_actions/docker_compose.rs

@@ -1,5 +1,5 @@
 use std::net::IpAddr;
-use tracing::{info, warn};
+use tracing::{info, instrument, warn};
 
 use crate::command_wrappers::ssh::SshClient;
 use crate::command_wrappers::ssh::SshConnection;
@@ -28,7 +28,16 @@ impl RemoteAction for DockerComposeValidator {
     }
 
     #[allow(clippy::too_many_lines)]
-    async fn execute(&self, _server_ip: &IpAddr) -> Result<(), RemoteActionError> {
+    #[instrument(
+        name = "docker_compose_validation",
+        skip(self),
+        fields(
+            action_type = "validation",
+            component = "docker_compose",
+            server_ip = %server_ip
+        )
+    )]
+    async fn execute(&self, server_ip: &IpAddr) -> Result<(), RemoteActionError> {
         info!(
             action = "docker_compose_validation",
             "Validating Docker Compose installation"