Fix validation output to show environment names in errors

- Fixed Rich markup issue by escaping square brackets in error output
- Added test to ensure multi-environment validation shows which env has issues
- Now errors clearly indicate [env] prefix for better debugging

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: ritwik-g

helm_values_manager/validator.py

@@ -34,7 +34,7 @@ def __init__(self, context: str, message: str, env: Optional[str] = None):
 
     def __str__(self):
         if self.env:
-            return f"[{self.env}] {self.context}: {self.message}"
+            return f"\\[{self.env}] {self.context}: {self.message}"
         return f"{self.context}: {self.message}"
 
 
@@ -192,6 +192,7 @@ def _validate_all_values(self):
         for values_file in self.values_base_path.glob(pattern):
             # Extract environment from filename
             env = values_file.stem.replace("values-", "")
+            # Debug: print(f"Validating environment: {env}")
             self._validate_values_for_env(env)
 
     def _validate_value_type(self, value: Any, expected_type: str) -> bool:
@@ -233,7 +234,8 @@ def print_errors(self):
 
         console.print(ErrorMessage("Validation failed:"))
         for error in self.errors:
-            console.print(f"  - {error}")
+            # Debug: print(f"DEBUG: error.env={error.env}, error={error}")
+            console.print(f"  - {str(error)}")
 
 
 def validate_command(

tests/test_validate.py

@@ -342,4 +342,56 @@ def test_validate_unknown_key_in_values(tmp_path):
         "--env", "dev"
     ])
     assert result.exit_code == 1
-    assert "Unknown key: unknown-key" in result.stdout
+    assert "Unknown key: unknown-key" in result.stdout
+
+
+def test_validate_multi_env_shows_environment_names(tmp_path):
+    """Test that validation errors show which environment has issues."""
+    # Create schema
+    schema_file = tmp_path / "schema.json"
+    schema_data = {
+        "version": "1.0",
+        "values": [
+            {
+                "key": "database-host",
+                "path": "db.host",
+                "description": "Database hostname",
+                "type": "string",
+                "required": True
+            }
+        ]
+    }
+    schema_file.write_text(json.dumps(schema_data, indent=2))
+    
+    # Create values for multiple environments with different errors
+    values_dev = tmp_path / "values-dev.json"
+    values_dev.write_text(json.dumps({
+        "dev": {}  # Missing required value
+    }))
+    
+    values_staging = tmp_path / "values-staging.json"
+    values_staging.write_text(json.dumps({
+        "staging": {
+            "database-host": 123  # Wrong type
+        }
+    }))
+    
+    values_prod = tmp_path / "values-prod.json"
+    values_prod.write_text(json.dumps({
+        "prod": {
+            "database-host": "prod-db.example.com"  # Correct
+        }
+    }))
+    
+    # Validate all environments
+    result = runner.invoke(app, [
+        "validate",
+        "--schema", str(schema_file),
+        "--values", str(tmp_path)
+    ])
+    
+    assert result.exit_code == 1
+    assert "[dev] Values: Missing required value: database-host" in result.stdout
+    assert "[staging] Values: Type mismatch for database-host: expected string" in result.stdout
+    # prod should not appear in errors since it's correct
+    assert "[prod]" not in result.stdout