fix: catch all exceptions including those from Ruby::Box isolation

Use 'rescue Exception' with explicit re-raise of system exceptions
(SystemExit, Interrupt, SignalException) to ensure all errors are
caught and displayed properly, including exceptions from Ruby::Box
whose classes don't match the outer world's StandardError hierarchy.

Author: tobi

lib/liquid/spec/cli/eval.rb

@@ -228,7 +228,9 @@ def self.run_eval(options, adapter_file)
             show_contribution_message(has_difference)
 
             exit(1) unless test_passed
-          rescue => e
+          rescue SystemExit, Interrupt, SignalException
+            raise
+          rescue Exception => e
             print_error(e, hint, verbose, spec_data)
 
             if compare_mode
@@ -395,7 +397,9 @@ def self.run_reference_implementation(template_source, assigns, verbose)
           output = template.render(context)
 
           [output, nil]
-        rescue => e
+        rescue SystemExit, Interrupt, SignalException
+          raise
+        rescue Exception => e
           [nil, e]
         end
 

lib/liquid/spec/cli/runner.rb

@@ -233,7 +233,14 @@ def self.run_specs_frozen(config, options)
             errors = 0
 
             suite_specs.each do |spec|
-              result = run_single_spec(spec, config)
+              begin
+                result = run_single_spec(spec, config)
+              rescue SystemExit, Interrupt, SignalException
+                raise
+              rescue Exception => e
+                # Catch all exceptions including those from Ruby::Box isolation
+                result = { status: :error, error: e }
+              end
 
               case result[:status]
               when :pass
@@ -380,7 +387,10 @@ def self.run_specs_compare(config, options)
             suite_specs.each do |spec|
               begin
                 result = compare_single_spec(spec, config)
-              rescue StandardError, Exception => e
+              rescue SystemExit, Interrupt, SignalException
+                raise
+              rescue Exception => e
+                # Catch all exceptions including those from Ruby::Box isolation
                 result = { status: :error, error: e }
               end
 
@@ -555,7 +565,9 @@ def self.run_reference_spec(spec, _compile_options, assigns, render_options)
 
           ref_result = template.render(context)
           [ref_result, nil]
-        rescue => e
+        rescue SystemExit, Interrupt, SignalException
+          raise
+        rescue Exception => e
           [nil, e]
         end
 
@@ -566,7 +578,9 @@ def self.run_adapter_spec(spec, compile_options, assigns, render_options)
             template = LiquidSpec.do_compile(spec.template, compile_options)
             template.name = spec.template_name if spec.template_name && template.respond_to?(:name=)
             adapter_result = LiquidSpec.do_render(template, assigns, render_options)
-          rescue StandardError => e
+          rescue SystemExit, Interrupt, SignalException
+            raise
+          rescue Exception => e
             adapter_error = e
           end
           [adapter_result, adapter_error]