FIX: Allow Date values and YAML aliases in parsing

lib/yaml_janitor/linter.rb

@@ -23,7 +23,7 @@ def lint(yaml_content, fix: false, file: nil)
       violations = []
 
       # Load with comments
-      loaded = Psych::Pure.load(yaml_content, comments: true)
+      loaded = Psych::Pure.load(yaml_content, comments: true, permitted_classes: [Date, Time, DateTime, Symbol, Regexp], aliases: true)
 
       # Format using our custom emitter
       formatted = Emitter.new(loaded, @config).emit
@@ -111,8 +111,9 @@ def generate_diff(original, formatted, path)
     private
 
     def verify_semantics!(original, fixed)
-      original_data = YAML.load(original)
-      fixed_data = YAML.load(fixed)
+      load_opts = { permitted_classes: [Date, Time, DateTime, Symbol, Regexp], aliases: true }
+      original_data = YAML.safe_load(original, **load_opts)
+      fixed_data = YAML.safe_load(fixed, **load_opts)
 
       if original_data != fixed_data
         raise SemanticMismatchError, "Fixed YAML has different semantics than original"

test/integration_test.rb

@@ -49,8 +49,8 @@ def test_comment_preservation_with_indentation_fix
       assert_match(/^  retries:/, fixed_content)
 
       # Verify semantics are preserved
-      original_data = YAML.load(yaml_with_comments)
-      fixed_data = YAML.load(fixed_content)
+      original_data = YAML.safe_load(yaml_with_comments)
+      fixed_data = YAML.safe_load(fixed_content)
       assert_equal original_data, fixed_data, "Semantics should be preserved"
     end
   end
@@ -195,8 +195,8 @@ def test_compact_array_format
       refute_match(/^  -\s*$/, fixed_content, "Should not have dash on its own line")
 
       # Verify semantics are preserved
-      original_data = YAML.load(yaml_with_arrays)
-      fixed_data = YAML.load(fixed_content)
+      original_data = YAML.safe_load(yaml_with_arrays)
+      fixed_data = YAML.safe_load(fixed_content)
       assert_equal original_data, fixed_data, "Semantics should be preserved"
     end
   end
@@ -228,4 +228,92 @@ def test_diff_generation
     # Should not be empty
     refute_empty diff.strip, "Diff should not be empty"
   end
+
+  def test_yaml_with_date_values
+    yaml_with_dates = <<~YAML
+      ---
+      project:
+        name: my-app
+        created: 2025-12-01
+        updated: 2026-01-15
+        deadline: 2026-06-30
+    YAML
+
+    Tempfile.create(["test", ".yml"]) do |file|
+      file.write(yaml_with_dates)
+      file.flush
+
+      config = YamlJanitor::Config.new(overrides: { indentation: 2 })
+      linter = YamlJanitor::Linter.new(config: config)
+      result = linter.lint_file(file.path, fix: true)
+
+      # Should not raise "Tried to load unspecified class: Date"
+      refute result[:error], "Should not error on Date values: #{result[:error]}"
+
+      fixed_content = File.read(file.path)
+      assert_includes fixed_content, "2025-12-01"
+      assert_includes fixed_content, "2026-01-15"
+    end
+  end
+
+  def test_yaml_with_aliases
+    yaml_with_aliases = <<~YAML
+      ---
+      defaults: &defaults
+        adapter: postgres
+        host: localhost
+        port: 5432
+      development:
+        <<: *defaults
+        database: myapp_dev
+      production:
+        <<: *defaults
+        database: myapp_prod
+    YAML
+
+    Tempfile.create(["test", ".yml"]) do |file|
+      file.write(yaml_with_aliases)
+      file.flush
+
+      config = YamlJanitor::Config.new(overrides: { indentation: 2 })
+      linter = YamlJanitor::Linter.new(config: config)
+      result = linter.lint_file(file.path, fix: true)
+
+      # Should not raise "Alias parsing was not enabled"
+      refute result[:error], "Should not error on YAML aliases: #{result[:error]}"
+
+      fixed_content = File.read(file.path)
+      # Aliases are resolved/expanded by the emitter, so verify
+      # the semantic content is present rather than anchor syntax
+      assert_includes fixed_content, "adapter: postgres"
+      assert_includes fixed_content, "database: myapp_dev"
+      assert_includes fixed_content, "database: myapp_prod"
+    end
+  end
+
+  def test_yaml_with_dates_and_aliases_combined
+    yaml_combined = <<~YAML
+      ---
+      shared: &shared
+        created: 2025-06-15
+        timeout: 30
+      service_a:
+        <<: *shared
+        name: alpha
+      service_b:
+        <<: *shared
+        name: beta
+    YAML
+
+    Tempfile.create(["test", ".yml"]) do |file|
+      file.write(yaml_combined)
+      file.flush
+
+      config = YamlJanitor::Config.new(overrides: { indentation: 2 })
+      linter = YamlJanitor::Linter.new(config: config)
+      result = linter.lint_file(file.path, fix: true)
+
+      refute result[:error], "Should handle dates and aliases together: #{result[:error]}"
+    end
+  end
 end