Leftover cleanup from prior PR

app/models/demo/generator.rb

@@ -29,35 +29,6 @@ class Demo::Generator
   # Expose the seed so callers can reproduce a run if necessary.
   attr_reader :seed
 
-  # ---------------------------------------------------------------------------
-  #                            Performance helpers
-  # ---------------------------------------------------------------------------
-
-  private
-
-    # Simple timing helper. Pass a descriptive label and a block; the runtime
-    # will be printed automatically when the block completes.
-    # If max_seconds is provided, raise RuntimeError when the block exceeds that
-    # duration.  Useful to keep CI/dev machines honest about demo-data perf.
-    def with_timing(label, max_seconds: nil)
-      start = Process.clock_gettime(Process::CLOCK_MONOTONIC)
-      result = yield
-      duration = Process.clock_gettime(Process::CLOCK_MONOTONIC) - start
-      puts "⏱️  #{label} completed in #{duration.round(2)}s"
-
-      if max_seconds && duration > max_seconds
-        raise "Demo::Generator ##{label} exceeded #{max_seconds}s (#{duration.round(2)}s)"
-      end
-
-      result
-    end
-
-    # Override Kernel#rand so *all* `rand` calls inside this instance (including
-    # those already present in the file) are routed through the seeded PRNG.
-    def rand(*args)
-      @rng.rand(*args)
-    end
-
   # Generate empty family - no financial data
   def generate_empty_data!(skip_clear: false)
     with_timing(__method__) do
@@ -112,13 +83,33 @@ class Demo::Generator
     end
   end
 
-    # Multi-currency support (keeping existing functionality)
+  private
-    def generate_multi_currency_data!(family_names)
+
-      with_timing(__method__) do
+    # Simple timing helper. Pass a descriptive label and a block; the runtime
-        generate_for_scenario(:multi_currency, family_names)
+    # will be printed automatically when the block completes.
+    # If max_seconds is provided, raise RuntimeError when the block exceeds that
+    # duration.  Useful to keep CI/dev machines honest about demo-data perf.
+    def with_timing(label, max_seconds: nil)
+      start = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+      result = yield
+      duration = Process.clock_gettime(Process::CLOCK_MONOTONIC) - start
+      puts "⏱️  #{label} completed in #{duration.round(2)}s"
+
+      if max_seconds && duration > max_seconds
+        raise "Demo::Generator ##{label} exceeded #{max_seconds}s (#{duration.round(2)}s)"
       end
+
+      result
     end
 
+    # Override Kernel#rand so *all* `rand` calls inside this instance (including
+    # those already present in the file) are routed through the seeded PRNG.
+    def rand(*args)
+      @rng.rand(*args)
+    end
+
+
+
     def clear_all_data!
       family_count = Family.count
       if family_count > 50
@@ -1226,8 +1217,3 @@ class Demo::Generator
       puts "   ✅ Set property and vehicle valuations"
     end
 end
-
-# Expose public API after full class definition
-Demo::Generator.public_instance_methods.include?(:generate_default_data!) or Demo::Generator.class_eval do
-  public :generate_empty_data!, :generate_new_user_data!, :generate_default_data!, :generate_multi_currency_data!
-end