refactor: [#238] standardize renderer constants to use three-part pattern

All template renderers now consistently use three constants:
- TEMPLATE_FILE: Source template filename (e.g., 'inventory.yml.tera')
- OUTPUT_FILE: Rendered output filename (e.g., 'inventory.yml')
- TEMPLATE_DIR: Template directory path (e.g., 'ansible')

Updated renderers:
- InventoryRenderer: Added ANSIBLE_TEMPLATE_DIR constant
- VariablesRenderer: Added ANSIBLE_TEMPLATE_DIR constant
- DockerComposeRenderer: Already had all three constants
- EnvRenderer: Added DOCKER_COMPOSE_TEMPLATE_DIR constant
- PrometheusConfigRenderer: Split PROMETHEUS_TEMPLATE_PATH into three constants
- TrackerConfigRenderer: Split TRACKER_TEMPLATE_PATH into three constants
- CloudInitRenderer: Already had all three constants

Benefits:
- Consistent pattern across all renderers (same responsibility, same structure)
- Easier to understand template source and destination paths
- More maintainable when adding new renderers
- Clear separation of concerns (filename vs directory path)

All tests passing (1509 tests).

Author: josecelano

src/infrastructure/templating/ansible/template/renderer/inventory.rs

@@ -98,6 +98,9 @@ impl InventoryRenderer {
     /// Output filename for the rendered inventory file
     const INVENTORY_OUTPUT_FILE: &'static str = "inventory.yml";
 
+    /// Directory path for Ansible templates
+    const ANSIBLE_TEMPLATE_DIR: &'static str = "ansible";
+
     /// Creates a new inventory template renderer
     ///
     /// # Arguments
@@ -191,7 +194,11 @@ impl InventoryRenderer {
     ///
     /// * `String` - The complete template path for inventory.yml.tera
     fn build_template_path() -> String {
-        format!("ansible/{}", Self::INVENTORY_TEMPLATE_FILE)
+        format!(
+            "{}/{}",
+            Self::ANSIBLE_TEMPLATE_DIR,
+            Self::INVENTORY_TEMPLATE_FILE
+        )
     }
 }
 

src/infrastructure/templating/ansible/template/renderer/variables.rs

@@ -99,6 +99,9 @@ impl VariablesRenderer {
     /// Output filename for the rendered variables file
     const VARIABLES_OUTPUT_FILE: &'static str = "variables.yml";
 
+    /// Directory path for Ansible templates
+    const ANSIBLE_TEMPLATE_DIR: &'static str = "ansible";
+
     /// Creates a new variables template renderer
     ///
     /// # Arguments
@@ -192,7 +195,7 @@ impl VariablesRenderer {
     ///
     /// * `String` - The complete template path for variables.yml.tera
     fn build_template_path() -> String {
-        format!("ansible/{}", Self::VARIABLES_TEMPLATE_FILE)
+        format!("{}/{}", Self::ANSIBLE_TEMPLATE_DIR, Self::VARIABLES_TEMPLATE_FILE)
     }
 }
 

src/infrastructure/templating/docker_compose/template/renderer/env.rs

@@ -98,6 +98,9 @@ impl EnvRenderer {
     /// Output filename for the rendered .env file
     const ENV_OUTPUT_FILE: &'static str = ".env";
 
+    /// Directory path for Docker Compose templates
+    const DOCKER_COMPOSE_TEMPLATE_DIR: &'static str = "docker-compose";
+
     /// Creates a new .env template renderer
     ///
     /// # Arguments
@@ -191,7 +194,7 @@ impl EnvRenderer {
     ///
     /// * `String` - The complete template path for env.tera
     fn build_template_path() -> String {
-        format!("docker-compose/{}", Self::ENV_TEMPLATE_FILE)
+        format!("{}/{}", Self::DOCKER_COMPOSE_TEMPLATE_DIR, Self::ENV_TEMPLATE_FILE)
     }
 }
 

src/infrastructure/templating/prometheus/template/renderer/prometheus_config.rs

@@ -49,7 +49,14 @@ pub struct PrometheusConfigRenderer {
 }
 
 impl PrometheusConfigRenderer {
-    const PROMETHEUS_TEMPLATE_PATH: &'static str = "prometheus/prometheus.yml.tera";
+    /// Template filename for the Prometheus Tera template
+    const PROMETHEUS_TEMPLATE_FILE: &'static str = "prometheus.yml.tera";
+
+    /// Output filename for the rendered Prometheus config file
+    const PROMETHEUS_OUTPUT_FILE: &'static str = "prometheus.yml";
+
+    /// Directory path for Prometheus templates
+    const PROMETHEUS_TEMPLATE_DIR: &'static str = "prometheus";
 
     /// Creates a new Prometheus config renderer
     ///
@@ -82,9 +89,11 @@ impl PrometheusConfigRenderer {
         output_dir: &Path,
     ) -> Result<(), PrometheusConfigRendererError> {
         // 1. Load template from template manager
-        let template_path = self
-            .template_manager
-            .get_template_path(Self::PROMETHEUS_TEMPLATE_PATH)?;
+        let template_path = self.template_manager.get_template_path(&format!(
+            "{}/{}",
+            Self::PROMETHEUS_TEMPLATE_DIR,
+            Self::PROMETHEUS_TEMPLATE_FILE
+        ))?;
 
         // 2. Read template content
         let template_content = std::fs::read_to_string(&template_path).map_err(|source| {
@@ -98,7 +107,7 @@ impl PrometheusConfigRenderer {
         let template = PrometheusTemplate::new(template_content, context.clone())?;
 
         // 4. Render to output file
-        let output_path = output_dir.join("prometheus.yml");
+        let output_path = output_dir.join(Self::PROMETHEUS_OUTPUT_FILE);
         template.render_to_file(&output_path)?;
 
         Ok(())

src/infrastructure/templating/tracker/template/renderer/tracker_config.rs

@@ -54,7 +54,14 @@ pub struct TrackerConfigRenderer {
 }
 
 impl TrackerConfigRenderer {
-    const TRACKER_TEMPLATE_PATH: &'static str = "tracker/tracker.toml.tera";
+    /// Template filename for the Tracker Tera template
+    const TRACKER_TEMPLATE_FILE: &'static str = "tracker.toml.tera";
+
+    /// Output filename for the rendered Tracker config file
+    const TRACKER_OUTPUT_FILE: &'static str = "tracker.toml";
+
+    /// Directory path for Tracker templates
+    const TRACKER_TEMPLATE_DIR: &'static str = "tracker";
 
     /// Creates a new tracker config renderer
     ///
@@ -93,7 +100,7 @@ impl TrackerConfigRenderer {
         // 1. Load template from template manager
         let template_path = self
             .template_manager
-            .get_template_path(Self::TRACKER_TEMPLATE_PATH)?;
+            .get_template_path(&format!("{}/{}", Self::TRACKER_TEMPLATE_DIR, Self::TRACKER_TEMPLATE_FILE))?;
 
         // 2. Read template content
         let template_content = std::fs::read_to_string(&template_path).map_err(|source| {
@@ -107,7 +114,7 @@ impl TrackerConfigRenderer {
         let template = TrackerTemplate::new(template_content, context.clone())?;
 
         // 4. Render to output file
-        let output_path = output_dir.join("tracker.toml");
+        let output_path = output_dir.join(Self::TRACKER_OUTPUT_FILE);
         template.render_to_file(&output_path)?;
 
         Ok(())