refactor: unify Screenshoter constructor to accept comparison_options

Both Screenshoter and StableScreenshoter now accept the same
(capture_options, comparison_options) signature. Eliminates the
asymmetry where one received a driver instance and the other a hash.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: pftg

lib/capybara/screenshot/diff/screenshot_matcher.rb

@@ -83,7 +83,7 @@ def capture_screenshot(capture_options, comparison_options)
           screenshoter = if capture_options[:stability_time_limit]
             StableScreenshoter.new(capture_options, comparison_options)
           else
-            Diff.screenshoter.new(capture_options, comparison_options[:driver])
+            Diff.screenshoter.new(capture_options, comparison_options)
           end
           screenshoter.take_comparison_screenshot(@snapshot)
         end

lib/capybara/screenshot/diff/screenshoter.rb

@@ -8,9 +8,9 @@ module Screenshot
     class Screenshoter
       attr_reader :capture_options, :driver
 
-      def initialize(capture_options, driver)
+      def initialize(capture_options, comparison_options = {})
         @capture_options = capture_options
-        @driver = driver
+        @driver = Diff::Drivers.for(comparison_options)
       end
 
       def crop

lib/capybara/screenshot/diff/stable_screenshoter.rb

@@ -25,8 +25,7 @@ def initialize(capture_options, comparison_options = {})
 
           @comparison_options = comparison_options
 
-          driver = Diff::Drivers.for(@comparison_options)
-          @screenshoter = Diff.screenshoter.new(capture_options.except(:stability_time_limit), driver)
+          @screenshoter = Diff.screenshoter.new(capture_options.except(:stability_time_limit), @comparison_options)
         end
 
         # Takes a comparison screenshot ensuring page stability

test/unit/screenshoter_test.rb

@@ -10,7 +10,7 @@ class ScreenshoterTest < ActiveSupport::TestCase
       include CapybaraScreenshotDiff::DSLStub
 
       test "#take_screenshot without wait skips image loading" do
-        screenshoter = Screenshoter.new({wait: nil}, ::Minitest::Mock.new)
+        screenshoter = Screenshoter.new({wait: nil}, {driver: :chunky_png})
 
         mock = ::Minitest::Mock.new
         mock.expect(:save_screenshot, true) { |path| path.include?("01_a.png") }
@@ -27,7 +27,7 @@ class ScreenshoterTest < ActiveSupport::TestCase
       test "#take_screenshot with custom screenshot options" do
         screenshoter = Screenshoter.new(
           {wait: nil, capybara_screenshot_options: {full: true}},
-          ::Minitest::Mock.new
+          {driver: :chunky_png}
         )
 
         mock = ::Minitest::Mock.new
@@ -43,7 +43,7 @@ class ScreenshoterTest < ActiveSupport::TestCase
       end
 
       test "#prepare_page_for_screenshot without wait does not raise any error" do
-        screenshoter = Screenshoter.new({wait: nil}, ::Minitest::Mock.new)
+        screenshoter = Screenshoter.new({wait: nil}, {driver: :chunky_png})
 
         assert_nil screenshoter.prepare_page_for_screenshot(timeout: nil) # does not raise an error
       end