Fix R plot handling by passing additional arguments to .original_print_trellis and update test to ignore plots return value.

Author: vndee

llm_sandbox/language_handlers/artifact_detection/r_plot_detection.py

@@ -194,7 +194,7 @@
         .plot_counter <<- .plot_counter + 1
         png_file <- sprintf('/tmp/sandbox_plots/%06d.png', .plot_counter)
         png(png_file, width = 800, height = 600, res = 100)
-        .original_print_trellis(x)
+        .original_print_trellis(x, ...)
         dev.off()
       }, error = function(e) {
         cat("lattice capture error:", e$message, "\n")

tests/test_r_handler.py

@@ -60,7 +60,7 @@ def test_run_with_artifacts_with_plotting(self) -> None:
         # Mock the extract_plots method
         handler.extract_plots = MagicMock(return_value=[])
 
-        result, plots = handler.run_with_artifacts(
+        result, _ = handler.run_with_artifacts(
             container=mock_container, code="plot(1:10)", libraries=["ggplot2"], enable_plotting=True
         )