svg_conform 0.1.8 → 0.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 28ff5f6535023bd5817354101e080198c1f5e8b6a9cf0694555a45734e653283
-  data.tar.gz: ef1f87272855005ac53e4bc2f06467298c3e58dd3a332a5207d9f27aabf3b9a4
+  metadata.gz: 8f59b89abd58959b19078c28ce450f6ee102a007176d102031a04233f924465e
+  data.tar.gz: 61e68396baf1673fcd8d764a2d141cd62006d1167d2cea0b9d8695a7bb7ea6c2
 SHA512:
-  metadata.gz: d4a56e3f70abddf80258a6774300c99db15d996f892ba8c8fd6a7d9e78d7c19432a3264d8958312c5e7cff5006628a648775dd3cbc4d06d9de0fde5ef179e9d3
-  data.tar.gz: 93d8c6ef9a74d628fa6b48aa4ab92ba41c4abc804d29d8b2670656e6114e869b614c0fef80363d393e7d72311e903a113fcf74cb060e8d41ea54f528a0442a77
+  metadata.gz: ffb22ae1a52cbf687b054dbe17ccf8aa44b17bbeeaad5122b57300e9562e73e9cbf005d0c330b2ce13c28a7e6f349ddd191b3d8e8918c6e8745d663debcbec7c
+  data.tar.gz: dedcb6619d41fc142b0bb6432f15a693b38699ca52b80ba0e255d38a7feca6a86fca6a52daa15f5e694b8f4ca1da323cf67289129305b99a3f15616df336415a

data/.gitignore

@@ -10,3 +10,7 @@
 # rspec failure tracking
 .rspec_status
 svgcheck-reference/
+
+.kilocode
+.claude
+

data/.rubocop_todo.yml

@@ -1,80 +1,26 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2026-01-21 09:23:23 UTC using RuboCop version 1.82.1.
+# on 2026-01-23 01:37:10 UTC using RuboCop version 1.82.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 4
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: with_first_argument, with_fixed_indentation
-Layout/ArgumentAlignment:
-  Exclude:
-    - 'lib/svg_conform/validation_context.rb'
-
-# Offense count: 2
+# Offense count: 1
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyleAlignWith.
 # SupportedStylesAlignWith: either, start_of_block, start_of_line
 Layout/BlockAlignment:
   Exclude:
-    - 'lib/svg_conform/sax_validation_handler.rb'
     - 'spec/svg_conform/profiles/svg_1_2_rfc_profile_spec.rb'
 
-# Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
-Layout/BlockEndNewline:
-  Exclude:
-    - 'lib/svg_conform/sax_validation_handler.rb'
-
-# Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
-Layout/ElseAlignment:
-  Exclude:
-    - 'lib/svg_conform/requirements/no_external_css_requirement.rb'
-
-# Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyleAlignWith.
-# SupportedStylesAlignWith: keyword, variable, start_of_line
-Layout/EndAlignment:
-  Exclude:
-    - 'lib/svg_conform/requirements/no_external_css_requirement.rb'
-
-# Offense count: 4
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: AllowMultipleStyles, EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle.
-# SupportedHashRocketStyles: key, separator, table
-# SupportedColonStyles: key, separator, table
-# SupportedLastArgumentHashStyles: always_inspect, always_ignore, ignore_implicit, ignore_explicit
-Layout/HashAlignment:
-  Exclude:
-    - 'lib/svg_conform/validation_context.rb'
-
-# Offense count: 4
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: Width, AllowedPatterns.
-Layout/IndentationWidth:
-  Exclude:
-    - 'lib/svg_conform/requirements/no_external_css_requirement.rb'
-    - 'lib/svg_conform/sax_validation_handler.rb'
-
-# Offense count: 643
+# Offense count: 620
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: Max, AllowHeredoc, AllowURI, AllowQualifiedName, URISchemes, AllowRBSInlineAnnotation, AllowCopDirectives, AllowedPatterns, SplitStrings.
 # URISchemes: http, https
 Layout/LineLength:
   Enabled: false
 
-# Offense count: 2
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: AllowInHeredoc.
-Layout/TrailingWhitespace:
-  Exclude:
-    - 'lib/svg_conform/validation_context.rb'
-
 # Offense count: 2
 # Configuration parameters: AllowedMethods.
 # AllowedMethods: enums
@@ -124,28 +70,28 @@ Lint/UnreachableCode:
   Exclude:
     - 'lib/svg_conform/commands/check.rb'
 
-# Offense count: 150
+# Offense count: 146
 # Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes, Max.
 Metrics/AbcSize:
   Enabled: false
 
-# Offense count: 23
+# Offense count: 24
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns, inherit_mode.
 # AllowedMethods: refine
 Metrics/BlockLength:
   Max: 253
 
-# Offense count: 6
+# Offense count: 2
 # Configuration parameters: CountBlocks, CountModifierForms.
 Metrics/BlockNesting:
   Max: 4
 
-# Offense count: 126
+# Offense count: 125
 # Configuration parameters: AllowedMethods, AllowedPatterns, Max.
 Metrics/CyclomaticComplexity:
   Enabled: false
 
-# Offense count: 262
+# Offense count: 258
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 Metrics/MethodLength:
   Max: 154
@@ -155,18 +101,11 @@ Metrics/MethodLength:
 Metrics/ParameterLists:
   Max: 9
 
-# Offense count: 100
+# Offense count: 99
 # Configuration parameters: AllowedMethods, AllowedPatterns, Max.
 Metrics/PerceivedComplexity:
   Enabled: false
 
-# Offense count: 4
-# Configuration parameters: EnforcedStyle, AllowedIdentifiers, AllowedPatterns, ForbiddenIdentifiers, ForbiddenPatterns.
-# SupportedStyles: snake_case, camelCase
-Naming/VariableName:
-  Exclude:
-    - 'check_clippath_font.rb'
-
 # Offense count: 2
 # Configuration parameters: MinSize.
 Performance/CollectionLiteralInLoop:
@@ -193,11 +132,19 @@ RSpec/DescribeClass:
     - 'spec/svg_conform/references/integration_spec.rb'
     - 'spec/svgcheck_compatibility_spec.rb'
 
-# Offense count: 136
+# Offense count: 139
 # Configuration parameters: CountAsOne.
 RSpec/ExampleLength:
   Max: 53
 
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: implicit, each, example
+RSpec/HookArgument:
+  Exclude:
+    - 'spec/spec_helper.rb'
+
 # Offense count: 2
 RSpec/LeakyConstantDeclaration:
   Exclude:
@@ -258,11 +205,3 @@ Style/OptionalBooleanParameter:
 Style/RedundantCondition:
   Exclude:
     - 'lib/svg_conform/external_checkers/svgcheck/parser.rb'
-
-# Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyleForMultiline.
-# SupportedStylesForMultiline: comma, consistent_comma, diff_comma, no_comma
-Style/TrailingCommaInArguments:
-  Exclude:
-    - 'lib/svg_conform/sax_validation_handler.rb'

data/benchmark/performance_comparison.rb

@@ -0,0 +1,120 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+# Performance benchmark for svg_conform optimizations
+# Measures validation speed for single and batch operations
+
+require "bundler/setup"
+require "benchmark"
+require_relative "../lib/svg_conform"
+
+# Helper method to create test SVG content
+def create_test_svg(element_count: 10, attribute_count: 5)
+  elements = (1..element_count).map do |i|
+    attrs = (1..attribute_count).map { |j| "attr#{j}='value#{j}'" }.join(" ")
+    "<rect id='rect#{i}' x='#{i * 10}' y='#{i * 10}' width='10' height='10' #{attrs}/>"
+  end
+
+  <<~SVG
+    <?xml version="1.0"?>
+    <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 1000 1000">
+      #{elements.join("\n      ")}
+    </svg>
+  SVG
+end
+
+puts "=" * 80
+puts "svg_conform Performance Benchmark"
+puts "=" * 80
+puts
+
+# Test configurations
+test_cases = [
+  { name: "Small document", elements: 10, attributes: 3 },
+  { name: "Medium document", elements: 50, attributes: 5 },
+  { name: "Large document", elements: 100, attributes: 8 },
+]
+
+validator = SvgConform::Validator.new
+
+test_cases.each do |test_case|
+  puts "Test: #{test_case[:name]}"
+  puts "  Elements: #{test_case[:elements]}, Attributes per element: #{test_case[:attributes]}"
+  puts
+
+  svg_content = create_test_svg(element_count: test_case[:elements],
+                                attribute_count: test_case[:attributes])
+
+  # Warmup run
+  validator.validate(svg_content, profile: :svg_1_2_rfc)
+
+  # Benchmark single validation
+  single_time = Benchmark.measure do
+    100.times do
+      validator.validate(svg_content, profile: :svg_1_2_rfc)
+    end
+  end
+
+  puts "  Single validation (100 runs): #{(single_time.real * 1000).round(2)}ms"
+  puts "    Average per validation: #{(single_time.real * 10).round(2)}ms"
+
+  # Benchmark batch validation
+  contents = Array.new(10) do
+    create_test_svg(element_count: test_case[:elements],
+                    attribute_count: test_case[:attributes])
+  end
+
+  batch_time = Benchmark.measure do
+    10.times do
+      contents.each do |content|
+        validator.validate(content, profile: :svg_1_2_rfc)
+      end
+    end
+  end
+
+  puts "  Batch validation (10 files × 10 runs): #{(batch_time.real * 1000).round(2)}ms"
+  puts "    Average per file: #{(batch_time.real * 100).round(2)}ms"
+  puts
+end
+
+puts "=" * 80
+puts "Memory Profiling"
+puts "=" * 80
+puts
+
+if Object.const_defined?(:GC)
+  GC.start
+  before_mem = `ps -o rss= -p #{Process.pid}`.to_i
+
+  # Create and validate 100 documents
+  100.times do
+    svg = create_test_svg(element_count: 50, attribute_count: 5)
+    validator.validate(svg, profile: :svg_1_2_rfc)
+  end
+
+  GC.start
+  after_mem = `ps -o rss= -p #{Process.pid}`.to_i
+
+  mem_increase = after_mem - before_mem
+  puts "  RSS memory increase: #{mem_increase} KB"
+  puts "  Per validation: #{(mem_increase / 100.0).round(2)} KB"
+else
+  puts "  Memory profiling not available on this platform"
+end
+
+puts
+puts "=" * 80
+puts "Benchmark Complete"
+puts "=" * 80
+puts
+puts "Optimizations tested:"
+puts "  ✓ Phase 1.1: ElementProxy#path_id memoization"
+puts "  ✓ Phase 1.2: Element configuration index"
+puts "  ✓ Phase 2.1: Requirement classification cache"
+puts "  ✓ Phase 2.2: Global properties constant"
+puts "  ✓ Phase 2.3: ElementProxy attributes cache"
+puts "  ✓ Phase 3.1: Configuration validation cache"
+puts "  ✓ Phase 3.3: ProfileCompiler class"
+puts "  ✓ Phase 4.1: TrackerFactory extraction"
+puts "  ✓ Phase 5.1: Batch validation optimization"
+puts

data/config/profiles/metanorma.yml

@@ -16,6 +16,8 @@ requirements:
     allowed_namespaces:
       - "http://www.w3.org/2000/svg"
       - ""  # Default namespace (no prefix)
+    allowed_attribute_patterns:
+      - "on*"  # Allow event handler attributes (onmouseover, onmouseout, etc.) per issue #57
     element_configs:
       # Direct encoding from svgcheck word_properties.py elements dictionary
       - tag: "svg"
@@ -156,6 +158,7 @@ requirements:
     description: "Validates ID references point to existing elements"
 
   # Forbidden content - no multimedia, scripting, etc.
+  # Note: onmouseover, onmouseout, onfocus, onblur are intentionally allowed per issue #57
   - id: "forbidden_content"
     type: "ForbiddenContentRequirement"
     description: "Prohibits multimedia, scripting content"

data/docs/performance.adoc

@@ -0,0 +1,239 @@
+= Performance Optimization Guide
+
+This guide documents the performance optimization patterns used in svg_conform.
+
+== Goals
+
+The performance optimization strategy focuses on:
+
+* Reducing validation time for repeated operations
+* Minimizing memory allocations during SAX parsing
+* Caching expensive operations without introducing complexity
+* Maintaining clean separation of concerns
+
+== Optimization Patterns
+
+=== Memoization
+
+Cache computed values to avoid repeated calculations.
+
+*Example:* `ElementProxy#path_id` uses `@cached_path_id` to store the computed path:
+
+[source,ruby]
+----
+def path_id
+  @cached_path_id ||= begin
+    parts = @path + ["#{@name}[#{@position]}"]
+    "/#{parts.join('/')}"
+  end
+end
+----
+
+*When to use:* When a method is called multiple times with the same result and the computation is expensive.
+
+*When to avoid:* When the result changes between calls, or the computation is trivial.
+
+=== Class-Level Caching with Thread Safety
+
+Use class-level caches with Mutex for thread-safe shared state.
+
+*Example:* Requirement classification cache in `SaxValidationHandler`:
+
+[source,ruby]
+----
+@classification_cache = {}
+@classification_cache_mutex = Mutex.new
+
+def classify_requirements_with_cache
+  profile_key = @profile.class.name
+  classified = self.class.classification_cache_mutex.synchronize do
+    self.class.classification_cache[profile_key]
+  end
+
+  # ... compute and cache if not found
+end
+----
+
+*When to use:* When the cached value is shared across all instances and doesn't change.
+
+*When to avoid:* When the cached value is instance-specific or changes frequently.
+
+=== Frozen Constants
+
+Define immutable data as frozen constants to avoid reallocations.
+
+*Example:* `GLOBAL_PROPERTIES` constant in `AllowedElementsRequirement`:
+
+[source,ruby]
+----
+GLOBAL_PROPERTIES = %w[
+  about base baseprofile d break class content cx cy
+  # ... full list of properties
+].freeze
+----
+
+*When to use:* For lookup tables, configuration data, and other immutable collections.
+
+*When to avoid:* When the data needs to be modified at runtime.
+
+=== Hash Indexes for O(1) Lookups
+
+Pre-build hash indexes for fast lookups instead of linear array searches.
+
+*Example:* Element configuration index in `AllowedElementsRequirement`:
+
+[source,ruby]
+----
+def after_initialize
+  build_element_config_index if element_configs&.any?
+end
+
+def build_element_config_index
+  @element_config_index = {}
+  element_configs.each do |config|
+    @element_config_index[config.tag] = config
+  end
+end
+
+# Usage - O(1) instead of O(n)
+element_config = @element_config_index&.dig(element_name)
+----
+
+*When to use:* When frequently looking up items by key in a collection.
+
+*When to avoid:* When the collection is small or accessed sequentially.
+
+=== Lazy Initialization with `||=`
+
+Use the "or-equals" operator for lazy initialization.
+
+*Example:* Cached attributes array in `ElementProxy`:
+
+[source,ruby]
+----
+def attributes
+  @cached_attributes ||= @raw_attributes.map { |name, value| SaxAttribute.new(name, value) }
+end
+----
+
+*When to use:* When a value is expensive to compute and may not be needed.
+
+*When to avoid:* When `nil` or `false` are valid cached values (use explicit nil check instead).
+
+=== Batch Processing Optimization
+
+Amortize fixed costs across multiple operations.
+
+*Example:* Batch file validation in `Validator`:
+
+[source,ruby]
+----
+def validate_files(file_paths, profile: :svg_1_2_rfc, **options)
+  # Load profile once
+  profile_obj = resolve_profile(profile)
+
+  file_paths.each do |file_path|
+    validate_file_with_profile(file_path, profile_obj, **options)
+  end
+end
+----
+
+*When to use:* When processing multiple items with shared setup costs.
+
+*When to avoid:* When processing single items or when items have different configurations.
+
+=== Factory Pattern for Complex Initialization
+
+Centralize object creation for cleaner initialization.
+
+*Example:* `TrackerFactory` for creating validation trackers:
+
+[source,ruby]
+----
+module Validation
+  module TrackerFactory
+    def self.create_all_trackers(document)
+      node_id_manager = create_node_id_manager(document)
+
+      {
+        error_tracker: create_error_tracker,
+        node_id_manager: node_id_manager,
+        # ... other trackers
+      }
+    end
+  end
+end
+----
+
+*When to use:* When initialization involves multiple related objects or complex logic.
+
+*When to avoid:* For simple object creation that doesn't benefit from centralization.
+
+== Anti-Patterns to Avoid
+
+=== Premature Optimization
+
+Don't optimize without benchmarks. Measure first, then optimize.
+
+=== Global Mutable State
+
+Avoid mutable global state. Use class-level caches with thread-safe access.
+
+=== Over-Abstraction
+
+Don't create abstractions just for the sake of it. Each abstraction should pull its weight.
+
+=== Optimizing Uncommon Paths
+
+Focus on hot paths. Don't optimize code that rarely runs.
+
+=== Sacrificing Readability
+
+Performance gains should not come at the cost of code clarity.
+
+== Performance Metrics
+
+The `benchmark/performance_comparison.rb` script measures:
+
+* Single validation speed (average time per document)
+* Batch validation speed (average time per file)
+* Memory usage per validation
+
+Run benchmarks before and after optimizations to measure impact:
+
+[source,shell]
+----
+ruby benchmark/performance_comparison.rb
+----
+
+== Architecture Considerations
+
+=== SAX vs DOM Validation
+
+svg_conform uses SAX validation by default for better performance:
+
+* *SAX mode*: Constant memory, streaming parser, handles large files
+* *DOM mode*: Loads full document, uses more memory, slower for large files
+
+Always prefer SAX for validation. Only use DOM when applying remediations.
+
+=== Caching Strategy
+
+* *Instance-level cache*: Use for data specific to a validation run
+* *Class-level cache*: Use for data shared across all instances
+* *Thread safety*: Always use Mutex for class-level shared state
+
+== Future Optimization Opportunities
+
+Areas that may benefit from future optimization:
+
+* Parallel batch validation for multi-core systems
+* Compiled profiles with pre-computed lookup tables
+* Streaming remediation to avoid full DOM load
+* Attribute name interning for reduced memory
+
+== Further Reading
+
+* `PERFORMANCE_OPTIMIZATION_PLAN.md` - Detailed optimization plan
+* `CLAUDE.md` - Architecture overview
+* `benchmark/performance_comparison.rb` - Performance benchmarks

data/docs/profiles.adoc

@@ -268,10 +268,24 @@ See link:remediation.adoc#namespace-attribute-remediation[NamespaceAttributeReme
 * **Flexible colors**: Any colors allowed (unlike RFC 7996 black/white restriction)
 * **Flexible fonts**: Any font families allowed (unlike RFC 7996 generic-only restriction)
 * **Flexible styles**: Any CSS styles allowed
+* **Event handlers allowed**: Supports event attributes (`on*`) for interactivity
 * **Self-contained resources**: All CSS and fonts must be embedded
 * **Structural compliance**: Proper namespaces and viewBox required
 * **No external dependencies**: External CSS and fonts strictly prohibited
 
+**Event Handler Support**:
+
+The metanorma profile allows SVG event handler attributes (e.g., `onmouseover`, `onmouseout`, `onfocus`, `onblur`) for interactive elements. This is configured using wildcard attribute patterns:
+
+[source,yaml]
+----
+requirements:
+  - type: "AllowedElementsRequirement"
+    id: "allowed_elements"
+    allowed_attribute_patterns:
+      - "on*"  # Allow all event handler attributes
+----
+
 **Requirements**:
 [source,yaml]
 ----

data/docs/requirements.adoc

@@ -174,6 +174,62 @@ foreign). Used with `skip_foreign_namespaces`. Default: `[]`.
 namespaces. When `true`, RDF namespace elements are considered valid and
 skipped. Default: `false`.
 
+`allowed_attribute_patterns`:: List of wildcard attribute patterns to exempt from
+validation. Attributes matching any pattern will be allowed regardless of whether
+they appear in element `attributes` lists. Patterns support suffix wildcards (`*`).
+Default: `[]`.
+
+* **Common patterns**:
++
+**`"on*"`**:: Allows all event handler attributes (`onclick`, `onmouseover`, `onmouseout`, etc.)
+**`"data-*"`**:: Allows all `data-` custom attributes
+
+* **Precedence**: When an attribute matches both an allowed pattern and an
+element-specific disallowed attribute (prefixed with `!`), the allowed pattern
+takes precedence and a warning is emitted during validation.
+
+==== Wildcard attribute patterns
+
+.Using allowed_attribute_patterns to exempt event handler attributes
+[example]
+====
+[source,yaml]
+----
+- id: "svg_elements_with_events"
+  type: "AllowedElementsRequirement"
+  description: "Restrict to allowed SVG elements but allow event handlers"
+  allowed_attribute_patterns:
+    - "on*"     # Allow all event handler attributes
+    - "data-*"  # Allow all data-* custom attributes
+  check_attributes: true
+  element_configs:
+    - tag: "polygon"
+      attributes: ["points", "id", "fill"]
+      # !onclick is disallowed, but on* pattern takes precedence with warning
+    - tag: "rect"
+      attributes: ["x", "y", "width", "height"]
+----
+====
+
+With this configuration:
+* Event attributes (`onclick`, `onmouseover`, etc.) are allowed on all elements
+* `data-*` custom attributes are allowed on all elements
+* Other attributes not in element `attributes` lists are still rejected
+
+.Conflict detection warning
+[example]
+====
+When an allowed pattern conflicts with an element-specific disallowed attribute
+(prefixed with `!`), a warning is emitted:
+
+[source,text]
+----
+Configuration warning in svg_elements_with_events: Element 'polygon' has
+disallowed attributes [onclick] that match allowed_attribute_patterns [on*].
+Allowed patterns take precedence over element-specific disallowed attributes.
+----
+====
+
 ==== Configuration
 
 .Example configuration of AllowedElementsRequirement

data/lib/svg_conform/element_proxy.rb

@@ -32,15 +32,20 @@ module SvgConform
       @child_counters = {} # Track child element positions
     end
 
-    # Build full path ID for this element
+    # Build full path ID for this element (memoized for performance)
     def path_id
-      parts = @path + ["#{@name}[#{@position}]"]
-      "/#{parts.join('/')}"
+      @path_id ||= begin
+        parts = @path + ["#{@name}[#{@position}]"]
+        "/#{parts.join('/')}"
+      end
     end
 
-    # Return attributes as array of SaxAttribute objects (for compatibility)
+    # Return attributes as array of SaxAttribute objects (memoized for performance)
+    # Cached to avoid repeated object allocations during SAX parsing
     def attributes
-      @raw_attributes.map { |name, value| SaxAttribute.new(name, value) }
+      @attributes ||= @raw_attributes.map do |name, value|
+        SaxAttribute.new(name, value)
+      end
     end
 
     # Check if this element has a specific attribute
@@ -75,8 +80,8 @@ module SvgConform
         # Boolean check
         has_attribute?(method.to_s.chomp("?"))
       else
-        # Attribute access
-        @attributes[method.to_s] || @attributes[method.to_sym]
+        # Attribute access - use raw_attributes hash, not cached array
+        @raw_attributes[method.to_s] || @raw_attributes[method.to_sym]
       end
     end